Skip to content

fix(zellij): remove whitespace on zellij layout file for Windows#313

Open
AmmoniumX wants to merge 1 commit intofolke:mainfrom
AmmoniumX:fix-windows-term
Open

fix(zellij): remove whitespace on zellij layout file for Windows#313
AmmoniumX wants to merge 1 commit intofolke:mainfrom
AmmoniumX:fix-windows-term

Conversation

@AmmoniumX
Copy link
Copy Markdown

Description

Simple fix: On Windows, if session has a space, such as claude c4e5832c2f, then when trying to open up a terminal using zellij, local layout-file inside zellij.lua becomes a string with a space, specifically config/zellij-layout-claude c9867a1555.kdl. Sidekick then seems to call zellij to initiate a terminal doing something like:

zellij --layout C:\Users\user\AppData\Local\nvim-data/sidekick/zellij-layout-claude c9867a1555.kdl

Which just causes the command to treat the last chunk as an extra argument, leading to the new zellij terminal to error with:

error: Found argument 'c9867a1555.kdl' which wasn't expected, or isn't valid in 
this context

The simplest fix (the one applied in this PR) is to just ensure session doesn't have whitespace by changing the format from %s %s to %s-%s

Screenshots

Issue screenshot:
image

@github-actions github-actions Bot added the size/xs Extra small PR (<3 lines changed) label Apr 29, 2026
@AmmoniumX AmmoniumX changed the title fix: zellij terminal on Windows fix(zellij): remove whitespace on zellij layout file for Windows Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs Extra small PR (<3 lines changed)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant