Skip to content

fix(admin): link Skills at their /gh address - #461

Merged
harlan-zw merged 1 commit into
mainfrom
fix/admin-skill-links
Oct 6, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
fix/admin-skill-links

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Related to #453

❓ Type of change

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

📚 Description

The admin page built legacy /skills/owner/repo/name links for integrity issues and recent regenerations. They reached a Skill only through a 301 to its /gh address. Both lists now use repoSkillPath, as every other Skill link does. An issue that carries only a slug links to /gh/<slug>, the same place the redirect sent it.

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

The integrity issues and the regeneration list built legacy /skills/owner/repo/name links, which reach a Skill only through a 301. They now use repoSkillPath, as every other Skill link does.
@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 6, 2026
@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 6, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 6, 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-06 13:28 UTC.

GitHub merged this pull request.

No material findings were recorded.

The pull request closed.

@harlan-github-agent harlan-github-agent Bot removed 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 6, 2026
@harlan-zw
harlan-zw merged commit ce7c679 into main Oct 6, 2026
7 checks passed
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