Skip to content

Commit 40cf079

Browse files
codewizdaveclaude
andcommitted
fix: resolve ls command 404 and alias circular dependency
- Fix trailing slash in repos.ls() endpoint for root listings (path="" produced "/contents/" instead of "/contents") - Remove repo verification from alias set to break circular dependency (users can now create aliases even if repo is temporarily inaccessible) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 65700cb commit 40cf079

2 files changed

Lines changed: 1 addition & 5 deletions

File tree

packages/repofetch/src/cli.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -254,10 +254,6 @@ async function main() {
254254
const name = filteredArgs[2];
255255
const repo = filteredArgs[3];
256256
if (!name || !repo) { console.error("Error: name and repo required"); process.exit(1); }
257-
// Verify repo exists
258-
const testClient = await createClient({ token: process.env.GITHUB_TOKEN });
259-
const result = await testClient.repos.ls(repo);
260-
if (!result.ok) { console.error("Error: Could not verify repository"); process.exit(1); }
261257
aliases[name] = repo;
262258
saveAliases(aliases);
263259
console.log("Alias '" + name + "' set to '" + repo + "'");

packages/sdk/src/modules/repos.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ export async function reposLs(
2121
path?: string
2222
): Promise<Result<RepositoryContentsResult, SdkError>> {
2323
const clean = path ? cleanPath(path) : "";
24-
const endpoint = "/repos/" + repo + "/contents/" + clean;
24+
const endpoint = clean ? "/repos/" + repo + "/contents/" + clean : "/repos/" + repo + "/contents";
2525

2626
const result = await request<GitHubContentsResponse>(ctx, endpoint);
2727

0 commit comments

Comments
 (0)