Skip to content

Commit 79d7f89

Browse files
itaybreclaude
andcommitted
fix(github): Log the artifact name being looked up
The legacy lookup now requests a single artifact by name, so make the debug, trace, and info messages say which artifact name is being fetched instead of implying all artifacts of the repository are listed. Refs GH-879 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
1 parent ab9cd51 commit 79d7f89

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

‎src/artifact_providers/github.ts‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ export class GitHubArtifactProvider extends BaseArtifactProvider {
131131
const per_page = 100;
132132

133133
this.logger.debug(
134-
`Searching GitHub artifacts for ${owner}/${repo}, revision ${revision}`,
134+
`Searching GitHub artifacts of ${owner}/${repo} for artifact named "${revision}"`,
135135
);
136136

137137
let checkNextPage = true;
@@ -149,7 +149,10 @@ export class GitHubArtifactProvider extends BaseArtifactProvider {
149149
});
150150

151151
const { artifacts, total_count } = artifactResponse.data;
152-
this.logger.trace(`All available artifacts on page ${page}:`, artifacts);
152+
this.logger.trace(
153+
`Artifacts named "${revision}" on page ${page}:`,
154+
artifacts,
155+
);
153156

154157
// We need to find the most recent archive where name matches the revision.
155158
// XXX(BYK): we assume the artifacts are listed in descending date order on
@@ -206,7 +209,7 @@ export class GitHubArtifactProvider extends BaseArtifactProvider {
206209

207210
for (let tries = 0; tries < MAX_TRIES; tries++) {
208211
this.logger.info(
209-
`Fetching GitHub artifacts for ${owner}/${repo}, revision ${revision} (attempt ${
212+
`Fetching GitHub artifact named "${revision}" from ${owner}/${repo} (attempt ${
210213
tries + 1
211214
} of ${MAX_TRIES})`,
212215
);

0 commit comments

Comments
 (0)