Skip to content

Remove TmuxPersistence setting - always persist sessions#153

Merged
andyrewlee merged 1 commit intomainfrom
simplify-config
Feb 3, 2026
Merged

Remove TmuxPersistence setting - always persist sessions#153
andyrewlee merged 1 commit intomainfrom
simplify-config

Conversation

@andyrewlee
Copy link
Copy Markdown
Owner

@andyrewlee andyrewlee commented Feb 3, 2026

Sessions are now always persisted across restarts. This simplifies the codebase by removing an untested code path.

Changes:

  • Remove TmuxPersistence from UISettings and config loading/saving
  • Remove checkbox from settings UI
  • Simplify CleanupTmuxOnExit and gcStaleTerminalSessions to no-ops
  • Remove related tests and helper functions
  • Update e2e tests to remove persistence toggle test

Amp-Thread-ID: https://ampcode.com/threads/T-019c248b-a0d5-774d-8372-93782ca02943


Open with Devin

Sessions are now always persisted across restarts. This simplifies the
codebase by removing an untested code path.

Changes:
- Remove TmuxPersistence from UISettings and config loading/saving
- Remove checkbox from settings UI
- Simplify CleanupTmuxOnExit and gcStaleTerminalSessions to no-ops
- Remove related tests and helper functions
- Update e2e tests to remove persistence toggle test

Amp-Thread-ID: https://ampcode.com/threads/T-019c248b-a0d5-774d-8372-93782ca02943
Co-authored-by: Amp <amp@ampcode.com>
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional flags.

Open in Devin Review

@andyrewlee andyrewlee merged commit 3bd231a into main Feb 3, 2026
1 check passed
@andyrewlee andyrewlee deleted the simplify-config branch February 3, 2026 17:55
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