fix(templates): start generated docs with H1 headings#1163
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (10)
📝 WalkthroughWalkthroughThis PR adds top-level markdown headings to eight schema template files and their code-generation paths to comply with markdown linting rule MD041. Templates now begin with ChangesTemplate heading standardization
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
Summary
Fixes #1138.
OpenSpec's built-in generated markdown templates currently start with second-level headings such as
## Why,## ADDED Requirements,## Context, and task group headings. Markdownlint's MD041 rule expects generated markdown files to start with a top-level heading.This PR adds first-line H1 headings to generated proposal, specs, design, and tasks templates while preserving the existing section structure that OpenSpec parsers consume.
Changes
# Proposal,# Specifications,# Design, and# Tasksto the built-inspec-driventemplates.workspace-planningtemplates.Verification
pnpm exec vitest run test/core/artifact-graph/instruction-loader.test.ts test/core/parsers/markdown-parser.test.ts test/core/parsers/requirement-blocks.test.tspnpm run buildpnpm run lintpnpm test(89 files, 1656 tests)Summary by CodeRabbit