Skip to content

fix: preserve Markdown command table structure - #32

Merged
rogerchappel merged 3 commits into
mainfrom
agent/oss-07b0e9bbb912-markdown-table-escaping
Aug 27, 2026
Merged

rogerchappel merged 3 commits into
mainfrom
agent/oss-07b0e9bbb912-markdown-table-escaping

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • escape every dynamic Markdown table cell, including pipes, line breaks, and backslashes
  • choose a safe inline-code delimiter for commands containing backticks
  • cover reporter and CLI output with a multiline shell pipeline while asserting exactly six columns and unchanged JSON values
  • document the Markdown escaping contract

Commits

  • 2b0890b test: cover Markdown table metacharacters
  • d87c079 fix: escape dynamic Markdown table cells
  • 3e30e94 docs: define Markdown escaping behavior

Verification

  • npm ci
  • npm run release:check (27/27 tests, typecheck, build, Markdown/JSON smoke, packed-package installed CLI smoke)
  • git diff --check origin/main...HEAD
  • author and committer identity verified for every commit as Roger Chappel miscanalysis@gmail.com

@rogerchappel

Copy link
Copy Markdown
Owner Author

Automated merge note

Triage class: auto-merge

Summary: Escapes every dynamic Markdown command-table cell so pipes, multiline text, backslashes, and backticks preserve the six-column structure while JSON remains unchanged (3 files, +62/-2).

Checks run: GitHub Actions Repository hygiene and release dry-run completed SUCCESS; worker verification recorded npm ci, npm run release:check (27/27 tests, typecheck, build, Markdown/JSON smoke, packed-package installed CLI smoke), and git diff --check origin/main...HEAD. Final diff, mergeability, one-review protection, Roger Chappel author/committer identity, and immutable head were re-verified immediately before merge.

Rebased/CI-repaired: No.

Verified head SHA: 3e30e945638e321e3237483a3c9328a353359d49.

@rogerchappel
rogerchappel merged commit 657e19e into main Aug 27, 2026
2 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.

1 participant