Skip to content

fix(release): bundle jsonc parser through its ESM entry - #26

Merged
nullStack65 merged 1 commit into
mainfrom
fix/fork-release-jsonc-esm-20261007
Oct 7, 2026
Merged

nullStack65 merged 1 commit into
mainfrom
fix/fork-release-jsonc-esm-20261007

Conversation

@nullStack65

Copy link
Copy Markdown
Owner

Linux SEA archive smoke for candidate37639312564 failed MODULE_NOT_FOUND './impl/format'. Resolve the exact jsonc-parser specifier to its published ESM entry during server packing, preserving native externals, Node26.8.2 and provenance gates.

The regression bundles the installed parser using production resolver options and executes the emitted artifact, checking JSONC comments, trailing commas and malformed-input errors. Baseline failure reproduced; focused regression1/1, targeted type-aware lint/type check, formatting and diff check pass. Distinct Luna review found no blocker. Exact-head normal CI required before ordinary landing. PR20 retains release ownership; no installation or publication.

@coderabbitai

coderabbitai Bot commented Oct 7, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: nullStack65/t3code/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2fa974fe-033b-4d72-83d0-7a196e0b979b
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Oct 7, 2026
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.6 KiB 13.6 KiB +16 B (+0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 7.1 KiB 7.1 KiB −1 B (−0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.4 KiB 6.5 KiB +17 B (+0.3%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 56.2 KiB 56.2 KiB 0 B (0.0%) 66.4 KiB ✅
Codex Live turn messages 9 9 0 (0.0%) 21 ✅
Claude Total thread wire 13.5 KiB 13.6 KiB +32 B (+0.2%) 15.1 KiB ✅
Claude Thread snapshot wire 7.1 KiB 7.1 KiB +2 B (+0.0%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.4 KiB 6.5 KiB +30 B (+0.5%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 57.0 KiB 57.1 KiB +44 B (+0.1%) 66.4 KiB ✅
Claude Live turn messages 9 10 +1 (+11.1%) 21 ✅

Baseline: cb9636b · PR result: c5af1d3 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 114.1 KiB
  • Claude decoded thread snapshot: 114.8 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@nullStack65
nullStack65 merged commit e3ff441 into main Oct 7, 2026
18 checks passed
@github-actions github-actions Bot added the size:S label Oct 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant