Add CacheMetadata with repository_id to cache v2 requests

[fkorotkov]

The V2 cache supports passing metadata so let's pass it along.

Fixes #2094

[Copilot]

Pull Request Overview

This PR adds support for passing cache metadata (specifically the repositoryId) to Cache V2 API requests.

• Introduced getCacheMetadata() to read GITHUB_REPOSITORY_ID and build a CacheMetadata object.
• Updated saveCacheV2 to include metadata in both create and finalize cache entry requests.
• Extended tests to set GITHUB_REPOSITORY_ID and assert that metadata is included.

Reviewed Changes

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

File	Description
packages/cache/src/cache.ts	Added import of CacheMetadata, implemented getCacheMetadata(), and wired metadata into V2 requests.
packages/cache/tests/saveCacheV2.test.ts	Set GITHUB_REPOSITORY_ID in tests and added expectations for the metadata field.

Comments suppressed due to low confidence (3)

packages/cache/src/cache.ts:55

• Consider adding a JSDoc comment for getCacheMetadata to explain what fields are included (e.g., repositoryId and scope) and under what conditions it returns undefined.

function getCacheMetadata(): CacheMetadata | undefined {

packages/cache/tests/saveCacheV2.test.ts:30

• Add a test case for when GITHUB_REPOSITORY_ID is not set to verify that metadata is omitted (i.e., undefined) in the request.

  process.env['GITHUB_REPOSITORY_ID'] = '123456789'

packages/cache/src/cache.ts:56

• Verify that repositoryId matches the expected type in CacheMetadata (e.g., number vs. string). If the schema requires a numeric ID, parse it with Number() or parseInt() and handle NaN cases.

  const repositoryId = process.env['GITHUB_REPOSITORY_ID']

[Copilot]

[nitpick] Since getCacheMetadata() is called twice in this function, consider invoking it once (e.g., const metadata = getCacheMetadata()) and reusing the same object for both requests to improve readability and consistency.

Suggested change

	const request: CreateCacheEntryRequest = {
	metadata: getCacheMetadata(),
	const metadata = getCacheMetadata();
	const request: CreateCacheEntryRequest = {
	metadata,

[Copilot]

[nitpick] To reduce duplication and improve clarity, extract the expected metadata object into a constant or use expect.objectContaining({ metadata }) so tests remain concise.

Suggested change

	expect(createCacheEntryMock).toHaveBeenCalledWith({
	metadata: {
	repositoryId: '123456789',
	scope: []
	},
	key,
	version: cacheVersion
	})
	const metadata = {
	repositoryId: '123456789',
	scope: []
	};
	expect(createCacheEntryMock).toHaveBeenCalledWith({
	metadata,
	key,
	version: cacheVersion
	});

[fkorotkov]

FYI @GhadimiR @Link-