Skip to content

fix(registry): recover moved repositories using stored IDs - #596

Merged
harlan-zw merged 1 commit into
mainfrom
fix/repository-id-recovery
Oct 9, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
fix/repository-id-recovery

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

Renamed Repositories could remain missing or blocked when GitHub stopped resolving their old names.
Known Repositories now recover by stored numeric ID after a missing name or an identity mismatch.
The recovered name must resolve to the same public Repository before sync, asset refresh, or purpose classification continues.
Normal reads keep their existing request count. Temporary failures remain retryable.

Sync and purpose jobs recover the stored Repository by numeric ID, verify its public canonical name, and retain existing move behavior.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 9, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-10-09 04:41 UTC.

GitHub merged this pull request.

  • Logged (15/100): Material documentation gap: the pull request's embedded evidence screenshot is unavailable (HTTP 400), so the described recovery and move behavior cannot be visually confirmed from the description. Optional follow-up: re-upload the attachme View code Optional follow-up.
    • Ask an agent to verify and fix this finding
  • Logged (10/100): The recovery path drops the ID lookup's rate-limit telemetry, so pacing decisions miss one core-bucket request per recovered read. Optional follow-up: merge the identity outcome's rateLimit into the returned outcome (keep the lower remainin View code Optional follow-up.
    • Ask an agent to verify and fix this finding
  • Logged (10/100): A permanently refused move (target name held) repeats the full 3-request recovery on every sync sweep with no verdict recorded, a small unbounded quota cost. Optional follow-up: record a refused-move verdict with a backoff, or resolve the s View code Optional follow-up.
    • Ask an agent to verify and fix this finding

b5959eb9-6bee-452f-9d44-ef705d175597

Selected findings run after merge and open separate pull requests.

@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 9, 2026
@harlan-github-agent harlan-github-agent Bot added harlan-agent-ready The automated Review passed every gate on this head commit. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 9, 2026

@harlan-github-agent harlan-github-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Harlan Agent Kit posted this automated review. AI open source policy.

Automated Review: READY.

Read the Review results

@harlan-zw
harlan-zw merged commit dce1b91 into main Oct 9, 2026
8 checks passed
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-ready The automated Review passed every gate on this head commit. label Oct 9, 2026
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