src: update std::vector<v8::Local<T>> to use v8::LocalVector<T>

[Aditi-1400]

Continuing the clean up of std::vector<v8::Local> we did before. Discovered in #58497
From pull request: #57578:

According to V8's public API documentation, local handles (i.e., objects of type v8::Local) "should never be allocated on the heap". This replaces the usage of heap-allocated data structures containing instances of v8::Local, specifically the std::vectorv8::Localv8::String with recently introduced v8::LocalVector.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.22%. Comparing base (a4c7c9f) to head (63d5a26).
Report is 28 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #58500      +/-   ##
==========================================
+ Coverage   90.20%   90.22%   +0.01%     
==========================================
  Files         635      635              
  Lines      187616   187595      -21     
  Branches    36847    36842       -5     
==========================================
+ Hits       169234   169249      +15     
+ Misses      11150    11130      -20     
+ Partials     7232     7216      -16     

see 30 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/67182/

[nodejs-github-bot]

Landed in f497881