Adopt merged zmx Windows startup fix - #455
Merged
Merged
Conversation
Signed-off-by: Colin Neilens <coneilen@microsoft.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Pin the merged provider commit and reject stale or non-merged selections. Preserve historical baseline tags and quick-chat fixture provenance. Signed-off-by: Colin Neilens <coneilen@microsoft.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
coneilen
added a commit
to simra/GraphCode
that referenced
this pull request
Sep 25, 2026
Integrate the node editor snapshot/menu corrections from scgopi#452 and the merged zmx Windows startup pin from scgopi#455 while preserving the Windows DPI boundaries, buffered rendering, menu refresh, and Swift changes. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Signed-off-by: Colin Neilens <coneilen@microsoft.com>
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
1 participant
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
Adopt the merged Windows startup fix from coneilen/zmx#2 at exact commit 11e20c738b4ebd88031c7a01f1a9d938ee123234.
Update both provider manifests and exact-SHA checks, add rejection tests for stale/unmerged pins, and update current-provider documentation. Preserve historical baseline tags and the old quick-chat fixture unchanged.
No GraphCode product-source changes or parity-ledger promotions.
Evidence
RED: Run TerminalGate.Tests.ps1 with the new expected SHA before updating manifests -> fails with "zmx SHA is not exact".
GREEN: Update both manifests and rerun TerminalGate.Tests.ps1 -> passes.
REGRESSION: Run ProviderPins.Tests.ps1, WindowsShell.Tests.ps1 and the supported ReleaseSafe shell build -> all pass after integrating main through #452; 42 executable source suites, including 294 App tests.
Upstream Windows/Linux/macOS CI passed for the provider fix. The merged commit has the same reviewed tree.
Own-build provider validation passed 104 native tests, 12 parser cases, three cleanup failure-path tests, and three real cmd/ConPTY startup/state/detach/reattach scenarios with stable backend identities and zero owned survivors.
Integration limits
Local stub-backed GraphCode validation established a real app-to-provider connection. A subsequent mandatory pre-send check found zero attached clients for the same backend and stopped before state input or a second app launch. GraphCode restart/state continuity is therefore not claimed locally.
Full legacy UIA/integration gates were not run on this shared host. All applicable current-head GraphCode CI remains required; upstream provider tests are not a substitute.
This branch includes main through #452, but not unmerged #453 or #454.
Reviewed head: 5d864c0.
Commits include DCO sign-off and Copilot coauthor.