docs(workflows): visualize the OpenSpec lifecycle - #1507
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThe change documents the OpenSpec workflow with Mermaid diagrams and adds Mermaid chart processing and rendering to the website MDX pipeline. ChangesMermaid workflow documentation and rendering
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested reviewers: Sequence Diagram(s)sequenceDiagram
participant Markdown
participant MDXPipeline
participant Mermaid
participant beautifulmermaid
Markdown->>MDXPipeline: process Mermaid block
MDXPipeline->>Mermaid: provide chart text
Mermaid->>beautifulmermaid: render SVG
beautifulmermaid-->>Mermaid: return SVG or error
Mermaid-->>MDXPipeline: return SVG or fallback code block
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying openspec-docs with
|
| Latest commit: |
4c74f40
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3d4201f9.openspec-docs.pages.dev |
| Branch Preview URL: | https://codex-docs-workflow-diagrams.openspec-docs.pages.dev |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/workflows.md`:
- Around line 76-79: Update the /opsx:archive workflow sequence to show
synchronization as an optional confirmation branch rather than an unconditional
step. Document that --yes enables synchronization for non-interactive callers,
and keep the archive operation represented separately from the optional sync
flow.
- Around line 44-49: Update the workflow graph around Apply, Verify, Sync, and
Archive to add a direct Apply-to-Archive edge, while preserving the existing
optional Verify and Sync paths.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: fb3b742e-96e5-4e6c-a636-bc5f6831b770
📒 Files selected for processing (1)
docs/workflows.md
alfred-openspec
left a comment
There was a problem hiding this comment.
The diagrams render correctly on GitHub, but the deployed Fumadocs preview shows both mermaid fences as raw syntax-highlighted code because the site has no Mermaid component or renderer: https://12ead95f.openspec-docs.pages.dev/docs/the-workflow. Please add docs-site rendering support or use a portable rendered asset so the lifecycle is actually visualized on both surfaces.
|
Addressed the Fumadocs rendering request in d41f88e. The docs site now pre-renders both Mermaid diagrams as responsive, theme-aware SVG with a readable code fallback; Copy Markdown and LLM exports retain the original Mermaid fences. I verified the deployed preview at https://674b8a88.openspec-docs.pages.dev/docs/the-workflow (2 SVG diagrams, 0 raw Mermaid blocks, no remote font imports or page overflow), and all CI/security/CodeQL/CodeRabbit checks pass. @alfred-openspec, please re-review when convenient. |
|
@coderabbitai review |
✅ Action performedReview finished.
|
342580e to
4c74f40
Compare
alfred-openspec
left a comment
There was a problem hiding this comment.
Exact head 4c74f40 is clean. Both lifecycle diagrams match the current OPSX semantics, render as responsive SVGs on the docs site, fall back safely to source on render failure, and remain Mermaid source in LLM output. A fresh locked install, typecheck, production build, exact static-output checks, Cloudflare preview, and the full Linux/macOS/Windows/security matrix pass.
Status
LGTM.
What was missing / the motivation
The workflow guide explained the OpenSpec lifecycle in prose and text examples, but it did not give readers a visual map of the workflow or show how the human, AI assistant, CLI, and project files interact.
Closes #439.
What it does
llms-full.txt.Proof it works
pnpm run lintpnpm test: 3,474 tests passed across 119 files, including all CLI E2E journeyswebsite pnpm run types:checkwebsite pnpm run build: 87 static pages generated successfully@import url(...)label textgit diff --checkNotes / nits
The behavior change is limited to documentation and the documentation-site build. It adds MIT-licensed
beautiful-mermaid, with EPL-2.0elkjsand BSD-2-Clauseentitiestransitives, to the docs build only. It does not change the OpenSpec CLI/runtime, schemas, generated agent instructions, or workflow behavior. Diagrams are rendered during the static build, so users do not download or run a Mermaid client library.Summary by CodeRabbit
New Features
Documentation