Skip to content

Commit a426884

Browse files
committed
docs(plan): Add synchronize deprecation context to T1
why: Deprecation affects import behavior and priority decisions. what: - Note tmuxinator deprecates true/before in favor of after - Clarify that import should still honor original semantics per value
1 parent 48d33be commit a426884

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

notes/plan.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Parity Implementation Plan
22

3-
*Last updated: 2026-03-06*
3+
*Last updated: 2026-03-07*
44
*Based on: parity-tmuxinator.md, parity-teamocil.md, import-tmuxinator.md, import-teamocil.md*
55

66
## libtmux Limitations
@@ -56,7 +56,7 @@ These libtmux APIs already exist and do NOT need changes:
5656
### T1. No `synchronize` Config Key
5757

5858
- **Blocker**: `WorkspaceBuilder` (`builder.py`) does not check for a `synchronize` key on window configs. The key is silently ignored if present.
59-
- **Blocks**: Pane synchronization (tmuxinator `synchronize: true/before/after`).
59+
- **Blocks**: Pane synchronization (tmuxinator `synchronize: true/before/after`). Note: tmuxinator deprecates `true`/`before` in favor of `after` (`project.rb:21-29`), but all three values still function. The import should honor original semantics of each value.
6060
- **Required**: Add `synchronize` handling in `builder.py`. For `before`/`true`: call `window.set_option("synchronize-panes", "on")` before pane commands are sent. For `after`: call it in `config_after_window()`. For `false`/omitted: no action.
6161
- **Insertion point**: In `build()` around line 320 (after `on_window_create` plugin hook, before `iter_create_panes()` loop) for `before`/`true`. In `config_after_window()` around line 565 for `after`. Note: in tmux 3.2+ (tmuxp's minimum), `synchronize-panes` is a dual-scope option (window|pane, `options-table.c:1423`). Setting it at window level via `window.set_option()` makes all panes inherit it, including those created later by split.
6262
- **Non-breaking**: New optional config key. Existing configs are unaffected.

0 commit comments

Comments
 (0)