Skip to content

fix(claude-api): replace dead URLs in academy-guide and tool-use-concepts - #1730

Open
GISWLH wants to merge 1 commit into
anthropics:mainfrom
GISWLH:fix/claude-api-dead-urls-1698
Open

GISWLH wants to merge 1 commit into
anthropics:mainfrom
GISWLH:fix/claude-api-dead-urls-1698

Conversation

@GISWLH

@GISWLH GISWLH commented Sep 6, 2026 •

Copy link
Copy Markdown

Summary

  • Replace 3 hard-404 documentation URLs in academy-guide and the claude-api tool-use concepts with canonical platform/academy URLs (verified HTTP 200 via curl -sI -L).
  • Files: skills/claude-api/shared/tool-use-concepts.md (1), skills/academy-guide/SKILL.md (2).
  • skills/claude-api/shared/live-sources.md is no longer touched: main already fixed those links, so this PR was rebased onto it and only keeps the remaining fixes.

Fixes #1698

Test plan

  • curl -sI -L on each replacement URL returns HTTP 200
  • Confirmed original URLs still return 404

@Apurva3509 Apurva3509 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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
@GISWLH
GISWLH force-pushed the fix/claude-api-dead-urls-1698 branch from e15b9f7 to 7e5fddf Compare October 1, 2026 16:18
@GISWLH GISWLH changed the title fix(claude-api): replace dead URLs in live-sources and related skills fix(claude-api): replace dead URLs in academy-guide and tool-use-concepts Oct 1, 2026
@GISWLH

GISWLH commented Oct 1, 2026

Copy link
Copy Markdown
Author

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.

@aandresfelipe767-create

aandresfelipe767-create commented Oct 1, 2026 via email

Copy link
Copy Markdown

@aandresfelipe767-create

aandresfelipe767-create commented Oct 2, 2026 via email

Copy link
Copy Markdown

@aandresfelipe767-create

aandresfelipe767-create commented Oct 2, 2026 via email

Copy link
Copy Markdown

@aandresfelipe767-create

aandresfelipe767-create commented Oct 2, 2026 via email

Copy link
Copy Markdown

@98zc5g5jyw-arch

Copy link
Copy Markdown

Reviewed at head 7e5fddffbe.

Scope: Rebased version of the #1698 dead-URL fix — 3 replacements across 2 files: academy.claude.com/resources → academy.claude.com/all (both call sites in skills/academy-guide/SKILL.md) and agents-and-tools/computer-use/overview → agents-and-tools/tool-use/computer-use-tool in skills/claude-api/shared/tool-use-concepts.md. shared/live-sources.md is no longer touched; the diff contains no other changes.

Verified:

  • Replacement URLs are live: https://academy.claude.com/all returns 200 ("All resources · Claude Academy", includes a search UI — consistent with the surrounding "searchable library" pointer); https://platform.claude.com/docs/en/agents-and-tools/tool-use/computer-use-tool returns 200 with canonical + h1 "Computer use tool" and contains the batch-actions / computer_20251124 material the section promises.
  • Original URLs are dead: academy.claude.com/resources → hard 404; agents-and-tools/computer-use/overview → client-side "Page not found" (soft 404 that still answers HTTP 200 — confirmed by rendering it in a real browser, so a plain status check alone can be misleading here).
  • Rebase resolution is clean: skills/claude-api/shared/live-sources.md at this head is byte-identical to current origin/main (empty diff), so nothing from main's rewrite was lost or altered.
  • No leftover references: the two academy-guide call sites are the only /resources links in that file, and no other computer-use/overview reference remains in the repo. git diff --check is clean.

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.

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.

claude-api skill: 4 dead URLs in shared/live-sources.md registry (agents instructed to WebFetch them get 404s)

4 participants