Skip to content

docs(mofa-doc): refine docs quality checks and versioned docs guidance (#233)#288

Merged
lijingrs merged 5 commits intomofa-org:mainfrom
Rahul-2k4:feature/docs-phase2-refine-233-v2
Feb 23, 2026
Merged

docs(mofa-doc): refine docs quality checks and versioned docs guidance (#233)#288
lijingrs merged 5 commits intomofa-org:mainfrom
Rahul-2k4:feature/docs-phase2-refine-233-v2

Conversation

@Rahul-2k4
Copy link
Copy Markdown
Contributor

📋 Summary

Refines docs/mofa-doc in the existing MoFA RS docs system (mdBook), focusing on documentation quality and maintainability on main.

This PR intentionally excludes Agent Hub implementation and keeps scope on:

  • fixing internal documentation link integrity,
  • improving CI docs quality checks,
  • adding versioned-docs guidance for contributors/users.

🔗 Related Issues

Closes #233

Related to #244


🧠 Context

Maintainer guidance clarified that Agent Hub is not a current MoFA RS deliverable. This PR follows that guidance and keeps work inside the existing docs/mofa-doc structure already present in the repository.

The goal is to reduce docs regressions and improve contributor DX without introducing new ecosystems or infra.


🛠️ Changes

  • Added deterministic internal markdown link checker:
    • docs/mofa-doc/scripts/check-internal-links.sh
  • Integrated docs internal-link validation into CI docs job:
    • .github/workflows/ci.yml
  • Fixed/normalized broken EN/ZH internal doc links and added missing API-reference target pages in:
    • docs/mofa-doc/src/api-reference/...
    • docs/mofa-doc/src/zh/api-reference/...
    • several docs/mofa-doc/src/zh/... pages
  • Updated docs contributor validation guidance:
    • docs/mofa-doc/README.md
  • Added versioned documentation pages and navigation entries:
    • docs/mofa-doc/src/appendix/versioned-docs.md
    • docs/mofa-doc/src/zh/appendix/versioned-docs.md
    • docs/mofa-doc/src/SUMMARY.md
    • docs/mofa-doc/src/zh/SUMMARY.md
    • docs/mofa-doc/src/appendix/README.md
    • docs/mofa-doc/src/zh/appendix/README.md

🧪 How you Tested

  1. Internal links:
    • docs/mofa-doc/scripts/check-internal-links.sh
  2. EN+ZH mdBook build:
    • cd docs/mofa-doc && ./scripts/build-docs.sh
  3. Artifact verification:
    • test -f docs/mofa-doc/book/index.html
    • test -f docs/mofa-doc/book/zh/index.html

📸 Screenshots / Logs (if applicable)

Key local verification output:

  • Internal link check passed
  • Built docs successfully:
    • English: .../docs/mofa-doc/book
    • Chinese: .../docs/mofa-doc/book/zh

⚠️ Breaking Changes

  • No breaking changes
  • Breaking change (describe below)

🧹 Checklist

Code Quality

  • Code follows project conventions
  • cargo fmt run
  • cargo clippy passes without warnings

Testing

  • Tests/validation updated (docs validation script + CI step)
  • cargo test passes locally without any error

Documentation

  • README / docs updated

PR Hygiene

  • PR is focused on one logical docs/CI refinement scope
  • Branch is based on latest main
  • No unrelated commits

🚀 Deployment Notes (if applicable)

No runtime deployment changes.
Docs CI now includes internal-link validation before merge.


🧩 Additional Notes for Reviewers

Agent Hub implementation is intentionally out of scope per maintainer direction. The “Scope Note: Agent Hub” in versioned-docs pages is included to prevent future scope confusion for contributors.

@Rahul-2k4 Rahul-2k4 marked this pull request as ready for review February 23, 2026 09:15
@Rahul-2k4
Copy link
Copy Markdown
Contributor Author

hey @lijingrs this fully implements #233 and let me know if u wanna add something more to it

@lijingrs lijingrs merged commit a32a3a0 into mofa-org:main Feb 23, 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.

Docs: Phase 2 - Build Centralized Documentation Site and Agent Hub

3 participants