Skip to content

fix(docs): include SATP Hermes docs in mkdocs deploy job#4125

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/fix-doc-publishing-mkdocs
Draft

fix(docs): include SATP Hermes docs in mkdocs deploy job#4125
Copilot wants to merge 2 commits intomainfrom
copilot/fix-doc-publishing-mkdocs

Conversation

Copy link

Copilot AI commented Feb 20, 2026

The deploy_docs.yml mkdocs publish job was not building or including SATP Hermes documentation in the generated site.

Changes

.github/workflows/deploy_docs.yml

  • Add packages/cactus-plugin-satp-hermes/docs/** to push path triggers so SATP Hermes doc changes also fire the workflow
  • Install Chromium system libraries (libnss3, libatk-bridge2.0-0, libgbm1, etc.) required by @mermaid-js/mermaid-cli
  • Run yarn docs:diagrams in the SATP Hermes package to render all 11 Mermaid architecture diagrams to SVG after the main npm run configure build step
  • Copy the architecture markdown and generated SVGs into docs/docs/satp-hermes/ preserving relative image paths (../assets/diagrams/) used in the source markdown

docs/mkdocs.yml

  • Add top-level SATP Hermes nav section pointing to the copied architecture doc at satp-hermes/architecture/satp-hermes.md

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: RafaelAPB <11261236+RafaelAPB@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix documentation publishing via mkdocs for satp hermes docs fix(docs): include SATP Hermes docs in mkdocs deploy job Feb 20, 2026
Copilot AI requested a review from RafaelAPB February 20, 2026 15:51
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.

3 participants