Add Zellij pane layout and naming support#190
Open
wuurrd wants to merge 10 commits into
Open
Conversation
Tmux can expand the sidebar pane when a neighboring pane closes. The sidebar resize sync path treated that expanded pane as a user resize, which could store and propagate a very large width across windows. Add a defensive width ceiling based on the existing default maximum. Synced and explicit widths above that ceiling resolve back to the default sidebar width, and live resize detection reflows the affected window instead of preserving the expanded width. This keeps normal explicit widths working while recovering the broken pane geometry automatically. Document the sidebar width ceiling and cover synced and explicit width recovery with unit tests.
Expose the pytest-based integration suite as its own just recipe so the command is discoverable after the default test recipe became unit-test only. Full integration test runs use pytest-xdist workers, while targeted runs pass their arguments through to pytest. Use the explicit integration test recipe in the workmux pre-merge checks so merge validation still covers the pytest suite. Document the unit and integration test commands in the contributor guide.
Author
|
@raine can you take a look at this? :) |
Contributor
|
Stacked works well: 
But i'm having issues with pane names: ❯ wm add zellij-stacked-test
Error: Failed to create worktree environment for branch 'zellij-stacked-test'
Caused by:
0: Failed to setup panes
1: Failed to set pane name 'agent'
2: Failed to rename zellij pane 'terminal_1'
3: Command failed: zellij action rename-pane --pane-id terminal_1 agent
error: Found argument '--pane-id' which wasn't expected, or isn't valid in this context
If you tried to supply `--pane-id` as a value rather than a flag, use `-- --pane-id`
USAGE:
zellij action rename-pane <NAME>
For more information try --helpwith a config like this: panes:
- command: claude
name: agent
focus: true
- command: nvim
name: editor
split: horizontal
- command: echo 123
name: logs
split: stackedMessage is updated as i had the wrong panes config posted |
Contributor
|
Okay, pane names also works: 
But it required me to update (in my case using |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
nameconfig support and apply it withzellij action rename-pane --pane-idnew-pane -- <command>returns empty stdoutTests