Skip to content

fix(docs): MkDocs 首页路径误贴 + docs/ 契约检查 - #82

Merged
Tanya7z merged 3 commits into
mainfrom
cursor/bc-2a126b1e-d97b-46f0-a37a-f5fbdded3c78-de20
Jul 26, 2026
Merged

fix(docs): MkDocs 首页路径误贴 + docs/ 契约检查#82
Tanya7z merged 3 commits into
mainfrom
cursor/bc-2a126b1e-d97b-46f0-a37a-f5fbdded3c78-de20

Conversation

@cursor

@cursor cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Summary

审查 main@024512d(依赖清理 + 文档首页改写)后的可验证问题,直接补丁。

本轮 push 的 ootb / Docs / changeset-release 均已 success,无新 wf 失败。

BLOCKER · DRY / Demeter(文档契约)

docs/index.md 把 README 的仓根相对路径(./docs/guide/..../docs/dev/npm-publish.md)原样贴进 MkDocs 站。MkDocs docs_dir=docs,这些链接在站点里变成双重 docs/,首页导航与安装/发布指南全部失效。CI 因 unrecognized_links: ignore 未拦截。

修复:

  • 改为 docs 内相对路径(./guide/./dev/…)
  • 仓外目标(LICENSE / catalog)改 GitHub 绝对 URL
  • 恢复「按角色入口」与本地预览说明

MAJOR · OCP(防再贴)

tools/docs-mkdocs.mjs 在 build/serve 前扫描手写 docs/**/*.md,禁止再出现 ](./docs/…) / ](docs/…)mkdocs.ymlunrecognized_links 调为 warn,坏链进入 CI 日志。

本轮 push 其余评价(无新补丁)

Test plan

  • 负向:临时写入 ](./docs/guide/…) → 契约检查命中
  • python3 -m mkdocs build 成功;site/index.htmlhref="guide/"
  • CI Docs on this PR
Open in Web View Automation 

cursoragent and others added 3 commits July 26, 2026 02:57
docs/index.md 从 README 复制了 ./docs/... 链接,在 docs_dir=docs 下站内导航全断;
改为 docs 相对路径,恢复按角色入口,并对 docs/*.md 禁止再出现 ](./docs/) 误贴。

Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>
Updated the documentation structure by adding a new directory table and removing the role-based entry section.
Updated links for online resources and module repository.
@Tanya7z
Tanya7z marked this pull request as ready for review July 26, 2026 03:07
@cursor

cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor Author

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Tanya7z
Tanya7z merged commit a4cf856 into main Jul 26, 2026
1 check 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.

2 participants