Skip to content

fix(gjc): resume native sessions from their owning store - #7

Merged
devswha merged 2 commits into
mainfrom
codex/fix-gjc-resume-session-root
Jul 22, 2026
Merged

fix(gjc): resume native sessions from their owning store#7
devswha merged 2 commits into
mainfrom
codex/fix-gjc-resume-session-root

Conversation

@devswha

@devswha devswha commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep new ChatMux GJC runs in the isolated scratch session store
  • resume scratch-owned sessions from that store
  • let native GJC TUI sessions fall back to GJC's default session lookup
  • omit ChatMux's default picker sentinel instead of passing it as a GJC model id

Root causes

ChatMux always passed --session-dir /tmp/gjc-live-sessions to gjc -r. Native TUI sessions are stored under GJC's normal home session tree, so history resume could not find them. After correcting lookup, the resume still failed because the UI fallback value default was emitted as --model default; it is a sentinel, not an installed GJC model.

Verification

  • node --import tsx --test server/gjc-cli.test.ts (21/21)
  • direct native-session resume with GJC 0.11.6
  • npm run typecheck
  • npm run lint
  • npm run build

@devswha

devswha commented Jul 22, 2026

Copy link
Copy Markdown
Owner Author

E2E QA evidence (2026-07-22): after this fix, the actual browser history-resume turn completed with CHATMUX_E2E_GJC_T2 value=GJC_T1. Targeted GJC tests and the final integration verify gate passed.

@devswha
devswha merged commit 6e091b3 into main Jul 22, 2026
1 of 2 checks passed
@devswha
devswha deleted the codex/fix-gjc-resume-session-root branch August 13, 2026 13:40
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