refactor: use getScoresDirect to get scores in getVp

[wa0x6e]

Continuing on #1770

---

Since the delegation support dropping from getVp, the core logic to get scores from strategies is the same as getScoresDirect

This PR refactors getVp to re-use getScoresDirect as much as possible to retrieve score, and avoid duplication.

Changes

• re-use getScoresDirect to get score
• move getVp to utils.ts (file is becoming too small to warrant its own file)
• move vp.test.ts to integration/utils.test.ts
• since getScoresDirect do not validate the addresses, we lose this feature also on getVp (temporary)

The next PR will focus on adding back the address validation to getScoresDirect

[Copilot]

Pull Request Overview

This PR refactors the getVp function to use the existing getScoresDirect helper, removes the old standalone implementation, and updates tests and snapshots accordingly.

• Remove legacy getVp in src/utils/vp.ts and replace it with a new implementation in src/utils.ts using getScoresDirect.
• Update import paths and test descriptions in integration tests.
• Adjust snapshots to match the revised describe block naming.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
test/integration/utils.test.ts	Changed import to the new getVp location and updated test title
test/integration/snapshots/utils.test.ts.snap	Renamed snapshot key to match the updated describe block
src/utils/vp.ts	Removed legacy getVp implementation
src/utils.ts	Added new getVp implementation, updated imports and types

Comments suppressed due to low confidence (2)

src/utils.ts:88

• The function getProvider is used here but not imported in this file, which will cause a reference error. Please add an import, for example:

import { getProvider } from './utils';

    getProvider(network),

src/utils.ts:77

• [nitpick] The parameter name snapshot shadows the imported snapshot module from @snapshot-labs/snapshot.js. Consider renaming the parameter (e.g., to snapshotTag) to avoid confusion.

export async function getVp(

[bonustrack]

utACK

[ChaituVR]

tAck