Skip to content

[FIX/Parser] Fix invisible chars, excess newlines, and add MCP inspector script#57

Merged
elydelva merged 2 commits into
mainfrom
fix/parser-invisible-chars-and-inspect-script
Mar 27, 2026
Merged

[FIX/Parser] Fix invisible chars, excess newlines, and add MCP inspector script#57
elydelva merged 2 commits into
mainfrom
fix/parser-invisible-chars-and-inspect-script

Conversation

@elydelva

Copy link
Copy Markdown
Owner

Summary

  • Decode HTML-encoded invisible characters (‌,  , etc.) before the unicode strip pass in stripInvisible so they don't survive as literal strings in markdown output
  • Apply stripInvisible to plainText so excess newlines (\n\n\n\n\n) are collapsed there too
  • Add dev:inspect script in @mailmcp/cli and at the monorepo root to launch the MCP Inspector against the local source (bun run src/index.ts --mcp)

Test plan

  • Run bun run dev:inspect from root — MCP Inspector should open
  • Parse a LinkedIn/Fever-style email and verify no ‌ /   in markdown output
  • Verify plainText no longer has 5+ consecutive newlines

🤖 Generated with Claude Code

elydelva and others added 2 commits March 27, 2026 18:26
strip html-encoded invisible characters (‌,  , etc.) before
the unicode strip pass so they don't survive as literal strings in markdown.
apply stripInvisible to plainText so excess newlines are collapsed there too.

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

coderabbitai Bot commented Mar 27, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a3cfdb01-48f1-4439-88e8-517cef5fa9ed

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/parser-invisible-chars-and-inspect-script

Comment @coderabbitai help to get the list of available commands and usage tips.

@elydelva
elydelva marked this pull request as ready for review March 27, 2026 17:27
@elydelva
elydelva merged commit f9d1294 into main Mar 27, 2026
7 checks passed
@elydelva
elydelva deleted the fix/parser-invisible-chars-and-inspect-script branch March 27, 2026 17:27
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