Skip to content

fix: ignore reaction tags inside fenced code - #112

Open
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:fix/mcp-history-file-type
Open

fix: ignore reaction tags inside fenced code#112
vycdev2 wants to merge 1 commit into
vycdev:mainfrom
vycdev2:fix/mcp-history-file-type

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Keep literal [REACT:emoji] text inside fenced code blocks as user-facing content.
  • Preserve existing reaction directives outside fenced code blocks.

Verification

  • RED: node --test --test-concurrency=1 tests/responseCodeFence.test.mjs failed on untouched main before the implementation.
  • GREEN: focused code-fence regression and tests/response.test.mjs passed after the implementation.
  • npm ci passed.
  • npm run build passed.
  • npm test passed.
  • Direct node-pty import passed.
  • git diff --check and staged diff checks passed.

Risk

The parser now masks fenced CommonMark code blocks before interpreting reaction directives, without changing text output.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

@vycdev2
vycdev2 force-pushed the fix/mcp-history-file-type branch from 2d5b9ac to 1fe3981 Compare August 10, 2026 02:59
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