Skip to content

research spike - use monogram for parser#744

Draft
theoephraim wants to merge 3 commits into
mainfrom
codex/monogram-default-parser
Draft

research spike - use monogram for parser#744
theoephraim wants to merge 3 commits into
mainfrom
codex/monogram-default-parser

Conversation

@theoephraim

@theoephraim theoephraim commented Jun 3, 2026

Copy link
Copy Markdown
Member

Experimenting with migrating from Peggy to monogram

Reasoning is that Monogram would generate textmate, tree-sitter, and monarch tooling automatically, meaning it could never be out of sync.

Summary

  • make the Monogram parser the default path in @env-spec/parser
  • remove old Peggy parser wiring and generation files
  • keep the vendored Monogram runtime + strict parser implementation from the spike
  • add a parser benchmark script and simplify scripts/devDeps accordingly

Validation

  • bun run lint:fix
  • bun run --filter @env-spec/parser typecheck
  • bun run --filter @env-spec/parser test:ci
  • bun run --filter @env-spec/parser bench:parsers
  • bun run test:ci (currently fails in existing packages/varlock/src/lib/test/exec.test.ts due to ANSI colorized stdout assertion, unrelated to parser changes)

@theoephraim theoephraim changed the title feat(env-spec-parser): default to monogram parser research spike - use monogram for parser Jun 4, 2026
@theoephraim theoephraim force-pushed the codex/monogram-default-parser branch from ce8acec to 0a482a9 Compare June 6, 2026 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant