Skip to content

feat(claude): persist New Session defaults#853

Merged
asheshgoplani merged 1 commit intoasheshgoplani:mainfrom
yaroshevych:fix-new-session-extra-args-persistence
May 4, 2026
Merged

feat(claude): persist New Session defaults#853
asheshgoplani merged 1 commit intoasheshgoplani:mainfrom
yaroshevych:fix-new-session-extra-args-persistence

Conversation

@yaroshevych
Copy link
Copy Markdown
Contributor

@yaroshevych yaroshevych commented May 2, 2026

Summary

I noticed that when starting Claude from agent-deck, the New Session dialog behaved inconsistently. Turns out, it didn't save Claude parameters between restarts. In this PR:

  • persist Claude New Session defaults for permission mode, Chrome, teammate mode, and extra args in config.toml
  • restore those defaults when reopening New Session after agent-deck restarts
  • document the new Claude config keys in the agent-deck skill references

Tests

Manually tested:

  • Open agent-deck, create new Claude session, with custom parameters
  • Close agent-deck
  • Open agent-deck, create new Claude session → custom parameters persist

Unit tests:

  • go test ./internal/ui
  • go test ./internal/session -run 'TestClaudeExtraArgsConfigRoundTrip|TestNewClaudeOptions_WithConfig|TestExtraArgsRestartPersist|TestResumeCommandAppendsExtraArgs'

Note: full go test ./internal/session still has existing tmux bootstrap failures in this environment.

@yaroshevych yaroshevych changed the title Persist Claude New Session defaults feat(claude): persist New Session defaults May 2, 2026
@asheshgoplani asheshgoplani merged commit 57ac2c9 into asheshgoplani:main May 4, 2026
asheshgoplani added a commit that referenced this pull request May 5, 2026
WebUI overhaul Phase 1:
- #804 test(web): web test infra + TUI⇄web parity matrix (PR-A) — 3 rounds of dual-review
- #853 feat(claude): persist New Session defaults (yaroshevych)

Committed by Ashesh Goplani
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.

2 participants