Commit Graph

14 Commits

Author SHA1 Message Date
jheyworth 8376ca0ba2
fix: add CommonMark-compliant markdown formatting rules (#830)
* fix: add CommonMark-compliant markdown formatting rules to workflow

Problem:
--------
BMAD generates markdown that violates CommonMark specification by omitting
required blank lines around lists, tables, and code blocks. While GitHub's
renderer (GFM) handles this gracefully, strict parsers like Mac Markdown.app
break completely, rendering lists as plain text and losing document structure.

Solution:
---------
Add 6 mandatory markdown formatting rules to workflow.xml (line 73) that
enforce proper spacing and consistency:

1. ALWAYS add blank line before and after bullet lists
2. ALWAYS add blank line before and after numbered lists
3. ALWAYS add blank line before and after tables
4. ALWAYS add blank line before and after code blocks
5. Use - for bullets consistently (not * or +)
6. Use language identifier for code fences

Impact:
-------
- Makes BMAD output CommonMark compliant
- Ensures compatibility with ALL markdown parsers, not just GitHub
- Follows industry best practices for professional documentation
- Future-proofs against stricter parser implementations
- Zero content changes - only formatting improved

Testing:
--------
Comprehensive three-phase testing completed:
- Phase 1: Synthetic test validating fix mechanism
- Phase 2: Fresh install end-to-end test (API Gateway project)
- Phase 3: GitHub validation with visual proof

Results: 1,112 lines of formatting improvements, 0 content changes,
100% CommonMark compliance achieved.

Test Evidence:
--------------
Complete test repository with before/after comparison, Mac Markdown.app
screenshot proving the issue, and comprehensive documentation:
https://github.com/jheyworth/bmad-markdown-formatting-test

See TEST.md for full documentation, test methodology, and evidence.

🤖 Generated with Claude Code (https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Remove TEST.md per maintainer feedback

As requested by @bmadcode - the test documentation was useful during review but is not needed in the repository. Testing evidence remains documented in the PR description and external test repository.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>
2025-10-28 22:27:48 -05:00
Brian Madison 54985778f2 minor fixes 2025-10-16 21:50:50 -05:00
Brian Madison 7f0e57e466 bmb updates 2025-10-16 09:50:29 -05:00
Brian Madison b9b219a13b prd cleanup 2025-10-15 21:17:09 -05:00
Brian Madison 5eb410d622 update config re deprecated removed file 2025-10-13 19:29:19 -05:00
Brian Madison e61d58d480 workflow level 0 and 1 aligned with brownfield and quick dev 2025-10-12 15:53:24 -05:00
Brian Madison 1b7a3b396f removed bmad folder 2025-10-12 01:41:09 -05:00
Brian Madison ab05cdcdd2 \split analyze workflow 2025-10-12 01:39:24 -05:00
Brian Madison d23643b53b removed some files 2025-10-04 21:34:37 -05:00
Brian Madison 16984c3d92 fix path bug 2025-10-04 21:33:19 -05:00
Brian Madison 9ea68ab8c3 remove bmad installs 2025-10-04 19:28:49 -05:00
Brian Madison c7d76a3037 agent manifest generation, party mode uses it, and tea persona compression 2025-10-04 19:28:10 -05:00
Brian Madison 46cabf72cd doc status updates 2025-10-04 01:29:40 -05:00
Brian Madison 5ee4cf535c BoMB updates 2025-10-04 00:22:59 -05:00