Skip to content

Pass issue_number to listLabelsOnIssue in the repair script - #523

Merged
jarstelfox merged 1 commit into
masterfrom
claude/repair-issue-number
Oct 11, 2026
Merged

jarstelfox merged 1 commit into
masterfrom
claude/repair-issue-number

Conversation

@jarstelfox

Copy link
Copy Markdown
Member

🤖

The repair script from #522 asked GitHub for the wrong URL, so its dry run on dev found no new name for any stale label. Octokit's listLabelsOnIssue takes issue_number, not number:

old: https://api.github.com/repos/iFixit/ops/issues//labels?number=1254
new: https://api.github.com/repos/iFixit/ops/issues/1254/labels

Every call came back 404, and the guard that skips deleted issues hid it. The webhook path in #522 doesn't call GitHub, so it isn't affected.

QA

  • On dev, run bin/repair-renamed-labels with no flag. ops P-strapi-shared-fields now shows -> "P-strapi-cli".

🤖 Generated with Claude Code

Octokit's route takes issue_number, so `number` built
/repos/{repo}/issues//labels?number=N, a 404 for every item. The 404
guard for deleted issues hid it, and the dry run on dev found no new
name for any stale label.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jarstelfox
jarstelfox merged commit 9f095c4 into master Oct 11, 2026
1 check passed
@jarstelfox
jarstelfox deleted the claude/repair-issue-number branch October 11, 2026 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant