[WSLC] Add --workdir / -w option to 'wslc exec' by ptrivedi · Pull Request #40041 · microsoft/WSL

ptrivedi · 2026-03-30T19:37:17Z

Summary of the Pull Request

Adds --workdir / -w option to wslc exec to set the working directory inside the container
Passes the value through SetContainerOptionsFromArgs into ContainerOptions.WorkingDirectory
Forwards the working directory to the container service via ContainerService::Exec
Validates that the --workdir value is non-empty or all-whitespace using std::iswspace (covers all Unicode whitespace characters)

PR Checklist

Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
Tests: Added/updated if needed and all pass
Localization: All end user facing strings can be localized
Dev docs: Added/updated if needed
Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Changes:

ArgumentDefinitions.h: register WorkDir argument with long form --workdir and short alias -w (appended at end to preserve sorted order of X-macro list)
ContainerExecCommand.cpp: include WorkDir in the exec command's argument list
ContainerModel.h: add WorkingDirectory field to ContainerOptions
ContainerTasks.cpp: read WorkDir arg and populate options.WorkingDirectory
ContainerService.cpp: pass WorkingDirectory when calling exec on the container
ArgumentValidation.cpp: validate --workdir is non-empty/non-whitespace using std::all_of with std::iswspace (with explicit wint_t cast) to correctly handle all Unicode whitespace, not just space and tab

Tests:

CommandLineTestCases.h: parsing cases for --workdir and -w (valid with option before positional, missing value, invalid specifier after positional)
WSLCCLIExecutionUnitTests.cpp: unit tests covering:
- SetContainerOptionsFromArgs sets WorkingDirectory when --workdir is provided
- SetContainerOptionsFromArgs leaves WorkingDirectory empty when not provided
- End-to-end parse of --workdir long option sets WorkingDirectory
- End-to-end parse of -w short alias sets WorkingDirectory
- Empty --workdir value throws ArgumentException
WSLCE2EContainerExecTests.cpp: E2E tests covering:
- Help message includes -w,--workdir
- exec --workdir /tmp <container> pwd prints /tmp
- exec -w /tmp <container> pwd prints /tmp

Validation Steps Performed

Build and run unit tests: bin\<platform>\<target>\test.bat /name:*WSLCCLIExecutionUnitTests*
Verify wslc exec --workdir /app <container> pwd prints /app
Verify wslc exec -w /tmp <container> pwd prints /tmp
Verify wslc exec <container> pwd uses the container's default working directory

Adds a --workdir (-w) argument to the exec command that sets the working directory inside the container for the executed process. Wires the value through ContainerOptions into WSLAProcessLauncher::SetWorkingDirectory. Co-authored-by: Pooja Trivedi <[email protected]> Co-Authored-By: Claude Sonnet 4.6

Copilot

Pull request overview

Adds support for specifying a working directory for wslc exec, wiring a new --workdir/-w CLI argument through argument parsing into ContainerOptions and down to the container exec launcher.

Changes:

Add ArgType::WorkDir (--workdir / -w) and include it in ContainerExecCommand’s supported arguments.
Populate ContainerOptions.WorkingDirectory from parsed args and forward it into WSLCProcessLauncher for container exec.
Add/extend unit tests and command-line parsing test cases for the new option.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
test/windows/wslc/WSLCCLIExecutionUnitTests.cpp	Adds unit tests validating `WorkingDirectory` is set/empty appropriately and parsed via `ContainerExecCommand`.
test/windows/wslc/CommandLineTestCases.h	Adds command-line parsing cases covering `--workdir` / `-w` usage and a missing-value case.
src/windows/wslc/tasks/ContainerTasks.cpp	Reads `WorkDir` and sets `ContainerOptions.WorkingDirectory`.
src/windows/wslc/services/ContainerService.cpp	Applies `WorkingDirectory` to the `WSLCProcessLauncher` before launching container exec.
src/windows/wslc/services/ContainerModel.h	Adds `WorkingDirectory` field to `ContainerOptions`.
src/windows/wslc/commands/ContainerExecCommand.cpp	Registers `WorkDir` as an accepted argument for `container exec`.
src/windows/wslc/arguments/ArgumentDefinitions.h	Defines the new `WorkDir` argument (`--workdir` / `-w`).

test/windows/wslc/CommandLineTestCases.h

Co-authored-by: Copilot <[email protected]>

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

test/windows/wslc/CommandLineTestCases.h

Co-authored-by: Copilot <[email protected]>

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

AmelBawa-msft

LGTM! I suspect E2E tests will fail because we need to update the expected E2E test help messages. We should consider adding E2E tests as well but possibly in a separate PR since we still have a lot more E2E tests to write 😊

src/windows/wslc/arguments/ArgumentDefinitions.h

dkbennett

Looks good, like the setting unit tests. An E2E with working directory would be nice but this looks like we are correctly setting the value in the CLI and passing it into the container runtime. Assuming that has tests this should be good, though it doesn't hurt to have a CLI E2E or two for this.

nit: sort order of _WorkDir as Amir mentioned.

test/windows/wslc/WSLCCLIExecutionUnitTests.cpp

src/windows/wslc/arguments/ArgumentDefinitions.h

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

src/windows/wslc/arguments/ArgumentDefinitions.h

src/windows/wslc/tasks/ContainerTasks.cpp

test/windows/wslc/WSLCCLIExecutionUnitTests.cpp

Co-authored-by: Copilot <[email protected]>

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

- Port existing exec E2E tests from feature branch - Add WSLCE2E_Container_Exec_WorkDir and WSLCE2E_Container_Exec_WorkDir_ShortAlias tests - Update help message in GetAvailableOptions to include -w,--workdir Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

- Reject empty or whitespace-only --workdir in Argument::Validate - Add ExecCommand_ParseWorkDirEmptyValue_ThrowsArgumentException unit test - Add empty-workdir failing case to CommandLineTestCases.h Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Co-Authored-By: Claude Sonnet 4.6

…code coverage Agent-Logs-Url: https://github.com/microsoft/WSL/sessions/b21d1a57-bb3f-4a12-84cf-8e414a453890 Co-authored-by: ptrivedi <[email protected]>

Agent-Logs-Url: https://github.com/microsoft/WSL/sessions/b21d1a57-bb3f-4a12-84cf-8e414a453890 Co-authored-by: ptrivedi <[email protected]>

…crosoft/WSL into user/ptrivedi/exec-workingdir

Copilot

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

src/windows/wslc/arguments/ArgumentDefinitions.h

- Revert Version ArgType alias from NO_ALIAS back to L"v" to preserve existing -v short option - Restore WSLCE2EContainerExecTests.cpp lost in merge conflict resolution Co-Authored-By: Pooja Trivedi

Copilot

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

src/windows/wslc/services/ContainerService.cpp

Co-authored-by: Copilot <[email protected]>

Copilot

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated no new comments.

…flag parse tests The -v short alias was removed from --verbose (changed to NO_ALIAS) to resolve a triple alias conflict with --version and --volume. The parser test cases in the Run argument set still used -v expecting it to resolve to --verbose, but since neither Version nor Volume are in the Run set, -v became unresolvable and caused unexpected parse failures. Replace -v with -h (help flag) in the flag parse test cases to preserve the same combined-flag parsing coverage with a valid short alias. Co-authored-by: Copilot <[email protected]>

…ptrivedi/exec-workingdir

After merging feature/wsl-for-apps, the --user argument is now active in ContainerExecCommand (from PR #40101). Update the expected exec help output to include -u,--user. Co-authored-by: Copilot <[email protected]>

Copilot

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.

Pooja Trivedi added 2 commits March 30, 2026 13:41

Merge branch 'feature/wsl-for-apps' into user/ptrivedi/exec-workingdir

4a2218d

ptrivedi requested a review from a team as a code owner March 30, 2026 19:37

Copilot AI review requested due to automatic review settings March 30, 2026 19:37

Copilot started reviewing on behalf of ptrivedi March 30, 2026 19:38 View session

Copilot AI reviewed Mar 30, 2026

View reviewed changes

test/windows/wslc/CommandLineTestCases.h Outdated Show resolved Hide resolved

Update test/windows/wslc/CommandLineTestCases.h

84851fd

Co-authored-by: Copilot <[email protected]>

Copilot AI review requested due to automatic review settings March 30, 2026 19:48

Copilot started reviewing on behalf of ptrivedi March 30, 2026 19:49 View session

Copilot AI reviewed Mar 30, 2026

View reviewed changes

test/windows/wslc/CommandLineTestCases.h Outdated Show resolved Hide resolved

Update test/windows/wslc/CommandLineTestCases.h

9add70a

Co-authored-by: Copilot <[email protected]>

Copilot AI review requested due to automatic review settings March 30, 2026 20:56

Copilot started reviewing on behalf of ptrivedi March 30, 2026 20:57 View session

Copilot AI reviewed Mar 30, 2026

View reviewed changes

AmelBawa-msft reviewed Mar 30, 2026

View reviewed changes

src/windows/wslc/arguments/ArgumentDefinitions.h Outdated Show resolved Hide resolved

dkbennett approved these changes Mar 31, 2026

View reviewed changes

test/windows/wslc/WSLCCLIExecutionUnitTests.cpp Show resolved Hide resolved

src/windows/wslc/arguments/ArgumentDefinitions.h Outdated Show resolved Hide resolved

Merge branch 'feature/wsl-for-apps' into user/ptrivedi/exec-workingdir

53c56a1

Copilot AI review requested due to automatic review settings April 2, 2026 18:02

Copilot started reviewing on behalf of ptrivedi April 2, 2026 18:03 View session

Copilot AI reviewed Apr 2, 2026

View reviewed changes

src/windows/wslc/arguments/ArgumentDefinitions.h Outdated Show resolved Hide resolved

src/windows/wslc/tasks/ContainerTasks.cpp Show resolved Hide resolved

test/windows/wslc/WSLCCLIExecutionUnitTests.cpp Outdated Show resolved Hide resolved

Fix clang formatting issues

ce1e458

ptrivedi force-pushed the user/ptrivedi/exec-workingdir branch from 7924dd9 to ce1e458 Compare April 2, 2026 18:32

Update test/windows/wslc/WSLCCLIExecutionUnitTests.cpp

e6b90dd

Co-authored-by: Copilot <[email protected]>

Copilot AI review requested due to automatic review settings April 2, 2026 18:35

Copilot started reviewing on behalf of ptrivedi April 2, 2026 18:36 View session

Copilot AI reviewed Apr 2, 2026

View reviewed changes

Pooja Trivedi and others added 4 commits April 2, 2026 15:00

Fix clang formatting in WSLCE2EContainerExecTests.cpp

d54748e

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Fix clang formatting in CommandLineTestCases.h

6b8fcbc

Co-Authored-By: Claude Sonnet 4.6

Missed change from merge conflict resolution

6eacc0b

Copilot started work on behalf of ptrivedi April 3, 2026 21:28 View session

Fix --workdir whitespace validation to use std::iswspace for full Uni…

0979c36

…code coverage Agent-Logs-Url: https://github.com/microsoft/WSL/sessions/b21d1a57-bb3f-4a12-84cf-8e414a453890 Co-authored-by: ptrivedi <[email protected]>

Copilot AI review requested due to automatic review settings April 3, 2026 21:29

ptrivedi review requested due to automatic review settings April 3, 2026 21:29

Use lambda with wint_t cast in iswspace call to avoid potential UB

7c16de1

Agent-Logs-Url: https://github.com/microsoft/WSL/sessions/b21d1a57-bb3f-4a12-84cf-8e414a453890 Co-authored-by: ptrivedi <[email protected]>

Copilot AI review requested due to automatic review settings April 3, 2026 21:33

ptrivedi review requested due to automatic review settings April 3, 2026 21:33

Copilot finished work on behalf of ptrivedi April 3, 2026 21:33

Pooja Trivedi added 2 commits April 3, 2026 17:39

Missed change from merge conflict resolution

a923420

Merge branch 'user/ptrivedi/exec-workingdir' of https://github.com/mi…

a1e710e

…crosoft/WSL into user/ptrivedi/exec-workingdir

Copilot AI review requested due to automatic review settings April 3, 2026 21:41

Copilot started reviewing on behalf of ptrivedi April 3, 2026 21:45 View session

Copilot AI reviewed Apr 3, 2026

View reviewed changes

src/windows/wslc/arguments/ArgumentDefinitions.h Outdated Show resolved Hide resolved

Pooja Trivedi and others added 2 commits April 4, 2026 01:04

Address Copilot PR feedback

53b0bb9

- Revert Version ArgType alias from NO_ALIAS back to L"v" to preserve existing -v short option - Restore WSLCE2EContainerExecTests.cpp lost in merge conflict resolution Co-Authored-By: Pooja Trivedi

Merge branch 'feature/wsl-for-apps' into user/ptrivedi/exec-workingdir

edb9743

Copilot AI review requested due to automatic review settings April 4, 2026 05:13

Copilot started reviewing on behalf of ptrivedi April 4, 2026 05:14 View session

Copilot AI reviewed Apr 4, 2026

View reviewed changes

src/windows/wslc/services/ContainerService.cpp Outdated Show resolved Hide resolved

Update src/windows/wslc/services/ContainerService.cpp

2942cb4

Co-authored-by: Copilot <[email protected]>

Copilot AI review requested due to automatic review settings April 4, 2026 05:21

Copilot started reviewing on behalf of ptrivedi April 4, 2026 05:22 View session

Copilot AI reviewed Apr 4, 2026

View reviewed changes

Pooja Trivedi and others added 3 commits April 6, 2026 09:37

Merge remote-tracking branch 'origin/feature/wsl-for-apps' into user/…

00e3420

…ptrivedi/exec-workingdir

Fix E2E exec help test: add --user option after base branch merge

f7cda40

After merging feature/wsl-for-apps, the --user argument is now active in ContainerExecCommand (from PR #40101). Update the expected exec help output to include -u,--user. Co-authored-by: Copilot <[email protected]>

Copilot AI review requested due to automatic review settings April 6, 2026 16:09

Copilot started reviewing on behalf of ptrivedi April 6, 2026 16:10 View session

Copilot AI reviewed Apr 6, 2026

View reviewed changes

ptrivedi merged commit 2d8c8cf into feature/wsl-for-apps Apr 6, 2026
11 checks passed

Conversation

ptrivedi commented Mar 30, 2026 • edited by Copilot AI Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary of the Pull Request

PR Checklist

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

AmelBawa-msft left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

dkbennett left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

ptrivedi commented Mar 30, 2026 •

edited by Copilot AI

Loading