Skip to content

✨ feat: add Mori tmux defaults preset#78

Merged
vaayne merged 4 commits intomainfrom
feat/tmux-defaults-preset
Apr 19, 2026
Merged

✨ feat: add Mori tmux defaults preset#78
vaayne merged 4 commits intomainfrom
feat/tmux-defaults-preset

Conversation

@vaayne
Copy link
Copy Markdown
Owner

@vaayne vaayne commented Apr 18, 2026

Summary

  • add a Settings → Tools toggle for applying Mori-managed tmux defaults
  • split tmux configuration into compatibility, preset, and theme applicators
  • keep tool settings backward-compatible, update docs/localizations, and cover the new default with tests

Testing

  • mise run build
  • mise run test

Split tmux configuration into compatibility, preset, and theme applicators.
Add a Settings > Tools toggle so Mori-managed sessions can default to mouse-on/status-off while still allowing users to keep their own tmux.conf behavior.

Assisted-by: pi
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 55420c9094

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/Mori/App/TmuxThemeApplicator.swift Outdated
Move Mori's tmux onboarding defaults into a bundled preset file and parse a small tmux config subset at runtime.
This keeps TmuxBackend generic while making preset edits easier and preserving cleanup for previously shipped preset options.

Assisted-by: pi
@vaayne
Copy link
Copy Markdown
Owner Author

vaayne commented Apr 18, 2026

use preset may help resolve #72

@vaayne
Copy link
Copy Markdown
Owner Author

vaayne commented Apr 18, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: de1300106a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/Mori/App/TmuxThemeApplicator.swift
vaayne added 2 commits April 19, 2026 11:18
Only cache a tmux theme payload after session theme application succeeds.
This keeps remote or temporarily unavailable tmux backends eligible for later non-forced retries instead of falsely treating the theme as already applied.

Assisted-by: pi
@vaayne vaayne merged commit fd32f26 into main Apr 19, 2026
5 checks passed
@vaayne vaayne deleted the feat/tmux-defaults-preset branch April 19, 2026 13:14
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