Repository navigation
Conversation
Apurva3509
left a comment
There was a problem hiding this comment.
Independent link check at e15b9f7: I followed redirects for every unique replacement URL in this diff. The Academy /all page and the Platform pricing, computer-use, agent-skills overview, and CLI quickstart pages all return HTTP 200, and their final destinations match the surrounding source descriptions. The duplicate Academy replacement is consistently updated in both call sites. I found no blocking issue in the scoped URL repair.
AI-assisted review; the links were verified live directly against the PR head.
…l-use-concepts Upstream already fixed the live-sources.md links; this updates the remaining hard-404 links to canonical academy/platform URLs so agents following them no longer hit dead pages. Fixes anthropics#1698
e15b9f7 to
7e5fddf
Compare
|
Rebased onto the latest What conflicted and how it was resolved
Tests: the repo has no CI, linter or test suite for these markdown skills (no CONTRIBUTING file). I only ran the URL checks above and |
|
El jue, 1 de oct de 2026, 11:28 a.m., Longhao Wang ***@***.***>
escribió:
… *GISWLH* left a comment (anthropics/skills#1730)
<#1730 (comment)>
Rebased onto the latest main to resolve the conflict in
skills/claude-api/shared/live-sources.md.
*What conflicted and how it was resolved*
- Since this PR was opened, main already fixed the same dead links in
live-sources.md: Pricing → about-claude/pricing.md, Computer Use →
tool-use/computer-use-tool.md, Skills → agent-skills/overview.md, and
the Anthropic CLI → cli-sdks-libraries/cli/quickstart.md. It also
added new rows and extended the Computer Use and CLI extraction prompts. I
kept main's version of that file entirely, so the PR no longer touches
it.
- The PR now contains only the two fixes main doesn't have yet:
- skills/academy-guide/SKILL.md: academy.claude.com/resources →
academy.claude.com/all (2 places)
- skills/claude-api/shared/tool-use-concepts.md: the
agents-and-tools/computer-use/overview link →
agents-and-tools/tool-use/computer-use-tool
- I re-checked every URL in the three touched files with curl -L: all
return 200 except an unrelated placeholder in a code example.
academy.claude.com/resources is still a 404, and the replacement URLs
return 200. I also updated the PR title and description to match, since the
PR no longer touches live-sources.md.
*Tests:* the repo has no CI, linter or test suite for these markdown
skills (no CONTRIBUTING file). I only ran the URL checks above and git
diff --check.
—
Reply to this email directly, view it on GitHub
<#1730?email_source=notifications&email_token=CPTKEDCEYLC2X2KVCQ2MLVD5R2AZ7A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKOJTGU3DANZTHA22M4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOSWGM33PORSXEX3DNRUWG2Y#issuecomment-5935607385>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/CPTKEDDQO4VDSNMZTBLBX3T5R2AZ7AVCNFSNUABGKJSXA33TNF2G64TZHMYTANRRHE2TGNBRGQ5US43TOVSTWNJTGYZDMOJXGI3TBILWAI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
|
Reviewed at head Scope: Rebased version of the #1698 dead-URL fix — 3 replacements across 2 files: Verified:
Conclusion: LGTM. Replacement links verified live at the head, both replacements are semantically faithful to the surrounding text, and the reduced rebased scope is correct. Safe to merge. |
Summary
academy-guideand the claude-api tool-use concepts with canonical platform/academy URLs (verified HTTP 200 viacurl -sI -L).skills/claude-api/shared/tool-use-concepts.md(1),skills/academy-guide/SKILL.md(2).skills/claude-api/shared/live-sources.mdis no longer touched:mainalready fixed those links, so this PR was rebased onto it and only keeps the remaining fixes.Fixes #1698
Test plan
curl -sI -Lon each replacement URL returns HTTP 200