Skip to content

fix: prevent duplicate post-compaction packing - #58

Merged
davidrobertson merged 1 commit into
mainfrom
codex/fix-single-pack-after-compaction
Aug 4, 2026
Merged

fix: prevent duplicate post-compaction packing#58
davidrobertson merged 1 commit into
mainfrom
codex/fix-single-pack-after-compaction

Conversation

@davidrobertson

Copy link
Copy Markdown
Contributor

Summary

  • Call lcm_pack_context once after compaction and consume structuredContent.markdown from that same result.
  • Keep recovery pending for failed, malformed, inherited, or unrelated tool results.
  • Make sanitized JSON keys inert and keep the recovery path platform-neutral.
  • Align the hook, MCP instructions, recall skill, and README.

Validation

  • Real Codex CLI 0.146.0 manual compaction: 1 PostCompact, 1 pack start, 1 pack completion, then normal continuation.
  • Clean install and TypeScript check passed.
  • 225 tests passed.
  • Smoke test and package dry-run passed.
  • Five exact-commit review lanes passed with no findings.

@davidrobertson
davidrobertson merged commit f69d4c7 into main Aug 4, 2026
3 checks passed
@davidrobertson
davidrobertson deleted the codex/fix-single-pack-after-compaction branch August 4, 2026 23:54
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