Skip to content

docs(xish): add CLAUDE.md + processor flow overview docs#1370

Merged
rafegoldberg merged 4 commits intonextfrom
docs/xish/add-claude-md+processor-flow-overview-docs
Mar 13, 2026
Merged

docs(xish): add CLAUDE.md + processor flow overview docs#1370
rafegoldberg merged 4 commits intonextfrom
docs/xish/add-claude-md+processor-flow-overview-docs

Conversation

@rafegoldberg
Copy link
Contributor

@rafegoldberg rafegoldberg commented Mar 11, 2026

🧰 Changes

🧬 QA & Testing

Run Claude Code in your local clone; does it read from the CLAUDE.md file correctly?

These are all things that can't be handled as proper unified plugins (parser extensions or MDAST transformers) because the problems occur at a level below — or orthogonal to — the tokenizer. The parser would either misparse or reject the input before any transformer gets a chance to act. So the raw markdown string is patched first, and then artifacts of that patching (like the snake_case mapping) are cleaned up by corresponding transformers later in the pipeline.


```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something I've been thinking about lately, and not something you need to address, is that Jest shipped a change recently to reduce their output when it's run by an LLM in jestjs/jest@3f17932. I wonder what the impact of the whitespace and ASCII tables within these Claude files are towards our token usages, and if rewriting them as Mermaid diagrams would cut that down at all.

@eaglethrost
Copy link
Contributor

Thanks for setting this up @rafegoldberg ! Have some suggestions to add on

see #1370 (comment)

Co-authored-by: Dimas Putra Anugerah <63914983+eaglethrost@users.noreply.github.com>
@rafegoldberg rafegoldberg merged commit b8d9e4c into next Mar 13, 2026
11 checks passed
@rafegoldberg rafegoldberg deleted the docs/xish/add-claude-md+processor-flow-overview-docs branch March 13, 2026 14:57
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.

4 participants