Skip to content

fix(start.mjs): remove chdir - #974

Open
xav-ie wants to merge 2 commits into
mksglu:nextfrom
xav-ie:fix/no-server-chdir
Open

fix(start.mjs): remove chdir#974
xav-ie wants to merge 2 commits into
mksglu:nextfrom
xav-ie:fix/no-server-chdir

Conversation

@xav-ie

@xav-ie xav-ie commented Jul 19, 2026

Copy link
Copy Markdown

What / Why / How

On startup the server changed its working directory into the plugin install
directory. That directory then leaks into any tool that reads a process's
working directory to find where the agent is working, for example a terminal
multiplexer opening a new split there instead of in the project. Nothing depends
on the change of directory, so remove it.

Affected platforms

Server startup path; client-agnostic. No behavior change to project resolution.

  • All platforms

Test plan

A guard that the server no longer changes directory, and a check that project
resolution still returns the project when the process starts in the plugin
directory.

Checklist

  • Tests added/updated
  • npm test passes
  • npm run typecheck passes
  • Docs updated if needed (README, platform-support.md)
  • No Windows path regressions (forward slashes only)
  • Targets next branch (unless hotfix)

xav-ie added 2 commits July 18, 2026 21:01
Moving cwd into the plugin dir confuses tools that read it. Unused.
@xav-ie
xav-ie marked this pull request as ready for review July 19, 2026 01:04
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