Skip to content

Hoist error prettifier rule construction#211

Merged
yoavbls merged 1 commit into
yoavbls:mainfrom
Sean-Kenneth-Doherty:codex/hoist-prettifier-rules
May 19, 2026
Merged

Hoist error prettifier rule construction#211
yoavbls merged 1 commit into
yoavbls:mainfrom
Sean-Kenneth-Doherty:codex/hoist-prettifier-rules

Conversation

@Sean-Kenneth-Doherty

Copy link
Copy Markdown
Contributor

Summary

  • build the regex/replacer rule list once per prettifier instead of once per diagnostic message
  • keep the formatter output path unchanged while removing repeated Promise/rule/closure allocation during diagnostic refreshes

Refs #139.

Tests

  • npm run -w @pretty-ts-errors/utils build
  • npm run -w @pretty-ts-errors/formatter lint
  • npm run -w @pretty-ts-errors/formatter test
  • npm run -w @pretty-ts-errors/formatter build
  • npx prettier --check packages/formatter/src/errorMessagePrettifier.ts

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@kevinramharak kevinramharak requested a review from yoavbls May 17, 2026 21:11
@yoavbls

yoavbls commented May 19, 2026

Copy link
Copy Markdown
Owner

Thank you!

@yoavbls yoavbls merged commit 4e7c3a3 into yoavbls:main May 19, 2026
1 check passed
@kevinramharak kevinramharak added the AI AI Generated/Assisted label May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI AI Generated/Assisted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants