Skip to content

docs: fix website link and anchor warnings - #57919

Open
trey0077 wants to merge 1 commit into
NousResearch:mainfrom
trey0077:docs/fix-website-link-warnings
Open

docs: fix website link and anchor warnings#57919
trey0077 wants to merge 1 commit into
NousResearch:mainfrom
trey0077:docs/fix-website-link-warnings

Conversation

@trey0077

@trey0077 trey0077 commented Jul 3, 2026

Copy link
Copy Markdown

Summary

  • Replace stale RL Training links with the existing Trajectory Format docs
  • Fix broken English docs anchors for platform adapters, provider fallback docs, and Telegram local Bot API docs
  • Fix translated zh-Hans links/anchors by adding explicit heading IDs where needed
  • Correct stale zh-Hans optional-skill links

Test Plan

  • source venv/bin/activate && cd website && npm run build
  • Verified Docusaurus builds both en and zh-Hans locales without broken link/anchor warnings

@trey0077
trey0077 force-pushed the docs/fix-website-link-warnings branch from 72f0e49 to 05b67a0 Compare July 3, 2026 20:30
@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have labels Jul 3, 2026

@teknium1 teknium1 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.

Thanks for finding the stale RL references and locale-anchor mismatches. The premise remains valid on current main: website/docs/getting-started/learning-path.md:103-107 still documents the removed Atropos RL pipeline, while website/docs/developer-guide/trajectory-format.md:1-4 is the available replacement. The zh-Hans Telegram fragment mismatch is also present at website/i18n/zh-Hans/docusaurus-plugin-content-docs/current/user-guide/messaging/telegram.md:317,338.

Problems

  • website/docs/user-guide/skills/optional/software-development/software-development-code-wiki.md:24: please keep the /docs/ prefix. website/docusaurus.config.ts:11 sets baseUrl: '/docs/', and lines 58-60 explicitly identify /docs/user-guide/skills/bundled/x as the route. The proposed /user-guide/... links would bypass that base path.

Suggested changes

  • Restore /docs/ on those two related-skill links, then preserve the remaining stale-link and explicit-ID changes.

Automated hermes-sweeper review.

| Platforms | linux, macos, windows |
| Tags | `Documentation`, `Mermaid`, `Architecture`, `Diagrams`, `Wiki`, `Code-Analysis` |
| Related skills | [`codebase-inspection`](/docs/user-guide/skills/bundled/github/github-codebase-inspection), [`github-repo-management`](/docs/user-guide/skills/bundled/github/github-github-repo-management) |
| Related skills | [`codebase-inspection`](/user-guide/skills/bundled/github/github-codebase-inspection), [`github-repo-management`](/user-guide/skills/bundled/github/github-github-repo-management) |

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.

Please retain the /docs/ prefix here. website/docusaurus.config.ts:11 sets baseUrl: '/docs/', and lines 58-60 explicitly identify /docs/user-guide/skills/bundled/x as the deployed route; /user-guide/... would resolve at the site root.

@teknium1 teknium1 added sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform labels Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants