From e597a0c800a6357ad1fbb279764ad45ac6085aad Mon Sep 17 00:00:00 2001 From: Anthony Zavala Date: Wed, 7 May 2025 19:43:30 +0000 Subject: [PATCH] update env secret and vars parameters --- languageserver/src/context-providers/secrets.ts | 8 +++++--- languageserver/src/context-providers/variables.ts | 8 +++++--- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/languageserver/src/context-providers/secrets.ts b/languageserver/src/context-providers/secrets.ts index ebf96e2..40b34f0 100644 --- a/languageserver/src/context-providers/secrets.ts +++ b/languageserver/src/context-providers/secrets.ts @@ -125,7 +125,7 @@ async function getRemoteSecrets( environmentSecrets: (environmentName && (await cache.get(`${repo.owner}/${repo.name}/secrets/environment/${environmentName}`, undefined, () => - fetchEnvironmentSecrets(octokit, repo.id, environmentName) + fetchEnvironmentSecrets(octokit, repo.owner, repo.name, environmentName) ))) || [], orgSecrets: await cache.get(`${repo.owner}/secrets`, undefined, () => fetchOrganizationSecrets(octokit, repo)) @@ -151,14 +151,16 @@ async function fetchSecrets(octokit: Octokit, owner: string, name: string): Prom async function fetchEnvironmentSecrets( octokit: Octokit, - repositoryId: number, + owner: string, + name: string, environmentName: string ): Promise { try { return await octokit.paginate( octokit.actions.listEnvironmentSecrets, { - repository_id: repositoryId, + owner, + repo: name, environment_name: environmentName, per_page: 100 }, diff --git a/languageserver/src/context-providers/variables.ts b/languageserver/src/context-providers/variables.ts index ef32c78..7926ab6 100644 --- a/languageserver/src/context-providers/variables.ts +++ b/languageserver/src/context-providers/variables.ts @@ -115,7 +115,7 @@ export async function getRemoteVariables( environmentVariables: (environmentName && (await cache.get(`${repo.owner}/${repo.name}/vars/environment/${environmentName}`, undefined, () => - fetchEnvironmentVariables(octokit, repo.id, environmentName) + fetchEnvironmentVariables(octokit, repo.owner, repo.name, environmentName) ))) || [], organizationVariables: await cache.get(`${repo.owner}/vars`, undefined, () => @@ -146,14 +146,16 @@ async function fetchVariables(octokit: Octokit, owner: string, name: string): Pr async function fetchEnvironmentVariables( octokit: Octokit, - repositoryId: number, + owner: string, + name: string, environmentName: string ): Promise { try { return await octokit.paginate( octokit.actions.listEnvironmentVariables, { - repository_id: repositoryId, + owner: owner, + repo: name, environment_name: environmentName, per_page: 100 },