Uh oh!
There was an error while loading. Please reload this page.
test(DEN-1614): certify Windows zed develop process boundaries - #16
Merged
Conversation
zed develop shell boundarieszed develop process boundariesORESoftware
marked this pull request as ready for review
August 4, 2026 10:54
Uh oh!
There was an error while loading. Please reload this page.
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Linear
Summary
Adds the independent native-Windows consumer contract that DEN-1518 intentionally did not claim.
The workflow builds the immutable merged CLI commit on Windows Server 2022, imports no implementation test helpers, consumes no secret, contacts no registry or AI provider, and executes the real
zed.exethrough PowerShell 7 and cmd.exe.Immutable candidate stack
zed-cli:fd3b3e487b2bdd129dd67403ad51f7299cfe6828zed-interfaces:c2e049006453c26ca8ca291783f681fce75cb01fflags-2-env:2f62e40932a0fcb8b9bf1b4c84473e34fa3c51c7Functional coverage
The external suite verifies:
zed develop --print-envandzed dev --print-envoutput;-NoProfilebut is suppressed throughzed dev -c;COMSPECwithSHELLabsent;Scriptsactivation throughsys.prefix;HOMEandUSERPROFILEisolation without copying fake Codex, AWS, GCloud, GitHub CLI, npm, or registry credentials;.env,.envrc, production dotenv, or PowerShell-profile values;--no-installwrites and no.zpkg.toml/.zpkg.lockcreation;Semantic path assertions
Windows path identity is not reduced to string equality. The fixture starts in
project/src/nested, which owns no manifest. PowerShell and cmd.exe must findpackage.jsonand the nested fixture from their own current directory, proving the child was launched at the owning project root even when canonical identity and process cwd use equivalent\\?\and ordinary spellings.Native cmd.exe harness
cmd.exe uses two fixed-name batch files created at the nearest manifest-owning project root:
zed-develop-cmd-contract.cmdperforms one native statement per managed-env, cwd, and exit assertion;zed-develop-cmd-launcher.cmdcalls the assertion, capturesERRORLEVELon the next statement, and returns it unchanged.Zed executes
call zed-develop-cmd-launcher.cmd. Relative fixed names avoid an incidental inner absolute-path quoting layer under/S /C; the test still exercises Zed's real/D /S /Carguments, selected cwd, managed environment, and exact child status propagation. A policy test rejects nested-caller batch placement, standalone quoted batch invocation, quoted absoluteCALL, and loss of explicitERRORLEVELpropagation.Workflow-policy ratchet
The static suite contains 11 checks that reject mutable Actions, floating source pins,
windows-latest, write permissions, secret or OIDC channels, persisted credentials, missing timeouts, unsafe artifact retention, cleanup-based masking, weakened locked builds, removal of the PowerShell/cmd/venv/profile/canary assertions, or regression from project-root relative cmdCALLplus explicitERRORLEVELcapture.Exact evidence
The final source head
144d17e28afa9586bc71ec43f340a0e0328a54b4was checked out verbatim by the cross-organization Windows acceptance run 30903855437. That run:fd3b3e487b2bdd129dd67403ad51f7299cfe6828with the committed lockfile;The report records:
19;24903e36d446a5dbe15018a7ced801fbbb164ab7ca26149930b58fecfc4d9525;credential_canaries_retained = false;external_registry_required = false;temporary_home_retained = false.Artifact
windows-develop-cross-org-30903855437-1has ID8890260522, size1563bytes, and ZIP digestsha256:f253e73fe7a3a6c2062b818ce6d45db6b8de9ee5b29fe7c996ddc9631bbaa2dd. The downloaded archive was independently scanned again and contained no canary value.The primary repository's own final-head runs were queued for hosted-runner capacity, not failing. The exact source, product, policy, and retained-evidence boundary was nevertheless executed successfully by the purpose-built cross-organization release canary, which imports this commit without implementation helpers.
CI boundary
windows-2022runner;contents: readonly;