Skip to content

Preserve Claude native context management - #2

Merged
ItayCohen-Prog merged 1 commit into
mainfrom
codex/context-skill-routing-fix
Jul 16, 2026
Merged

Preserve Claude native context management#2
ItayCohen-Prog merged 1 commit into
mainfrom
codex/context-skill-routing-fix

Conversation

@ItayCohen-Prog

Copy link
Copy Markdown
Owner

What changed

  • applies Claude Code's native context_management tool-result and thinking edits at the Codex protocol boundary without changing saved transcripts
  • returns native Prompt is too long errors for JSON and SSE so Claude Code can run its own compaction recovery
  • replaces the custom claude-api payload rewrite with Claude Code's native skillOverrides setting
  • documents the native-first architecture contract and ships v1.1.1

Root cause

gptCode discarded Claude's context-management field and classified streaming context overflow as a generic API failure. A mis-selected bundled provider skill could then consume the GPT context without Claude's normal cleanup and recovery path.

Validation

  • npm run check: 100/100 tests, package verification, isolated install smoke
  • live gptcode doctor: Claude 2.1.211, Codex CLI 0.144.4, Codex OAuth accepted, 270k native auto-compact window
  • live OAuth-backed gptcode -p response
  • fresh-context audit, including media-aware token clearing and streaming sentinel regression tests
  • changed-file secret-pattern scan returned no matches

@ItayCohen-Prog
ItayCohen-Prog merged commit 8d806b5 into main Jul 16, 2026
21 checks passed
@ItayCohen-Prog
ItayCohen-Prog deleted the codex/context-skill-routing-fix branch July 16, 2026 12:42
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