Skip to content

Optimize parser hot paths#74

Merged
himattm merged 2 commits into
mainfrom
codex-curate-regex-performance
Jun 6, 2026
Merged

Optimize parser hot paths#74
himattm merged 2 commits into
mainfrom
codex-curate-regex-performance

Conversation

@himattm
Copy link
Copy Markdown
Owner

@himattm himattm commented Jun 6, 2026

Summary

  • Replace regex-based hint normalization in HintExtractor with direct string traversal.
  • Replace SchemaParser hex color regex validation with fixed-length character validation.
  • Add focused regression coverage for prefix stripping, separator normalization, ID rejection, and missing # hex colors.
  • Stabilize the compose theme tests by waiting for async theme expansion instead of assuming fixed manual-clock timing.

This intentionally omits the generated .jules/bolt.md metadata from the duplicate generated performance PRs.

Supersedes #61, #62, #63, #64, #65, #66, #67, #68, #69, #70, #71, #72, and #73.

Validation

  • ./gradlew -q --console=plain :halogen-core:allTests :halogen-engine:jvmTest :halogen-engine:wasmJsBrowserTest
  • ./gradlew -q --console=plain apiCheck
  • ./gradlew -q --console=plain :halogen-compose:jvmTest
  • ./gradlew -q --console=plain :halogen-core:allTests :halogen-engine:jvmTest :halogen-compose:jvmTest :halogen-cache-room:jvmTest :halogen-provider-nano:testDebugUnitTest apiCheck
  • git diff --check

@himattm himattm merged commit 74092f5 into main Jun 6, 2026
4 of 6 checks passed
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