Skip to content

Fix Windows Jump destination parity - #967

Merged
coneilen merged 6 commits into
mainfrom
coneilen-jump-destination-parity
Nov 10, 2026
Merged

coneilen merged 6 commits into
mainfrom
coneilen-jump-destination-parity

Conversation

@coneilen

Copy link
Copy Markdown
Collaborator

Summary

Fix Windows Jump destination parity so selecting a result activates the same leaf-loop destination as sidebar navigation, and ensure palette refresh reads the actual edit buffer.

Closes #799.

Changes

  • Route Jump activation through the existing loop/sidebar activation path, including blocked, stale, and cross-project destinations.
  • Preserve the existing destination selection through query refresh and Return handling.
  • Read the cross-process edit buffer with the native WM_SETTEXT path in acceptance coverage.
  • Add focused, headless, compiled, and native UIA regression coverage for the integrated behavior.

Test plan

RED: & $env:GRAPHCODE_ZIG0152 test src\JumpPalette.zig -target x86_64-windows-msvc -lc -luser32 -lgdi32 --test-filter "jump query refresh follows the actual search buffer and preserves the selected destination" -> expected loop-a, found loop-b
GREEN: & $env:GRAPHCODE_ZIG0152 test src\JumpPalette.zig -target x86_64-windows-msvc -lc -luser32 -lgdi32 --test-filter "jump query refresh follows the actual search buffer and preserves the selected destination" -> 1/1 passed
REGRESSION: pwsh -NoLogo -NoProfile -File .\Tools\windows\validate.ps1 -Task windows-shell -SkipTrayLive -ShellValidationRoot D:\a79 -ShellTestManifest D:\a79\current-ed2f4ee6-retry04\manifests\all.json -SwiftExecutable C:\Users\coneilen\AppData\Local\Programs\Swift\Toolchains\6.3.3+Asserts\usr\bin\swift.exe -> exit 0; 66/66 shell sections

  • Standalone native accessibility provider: 280/280 checks.
  • Full unfiltered compiled runner: 689/689 unique units, 0 failures, 2,763 actual executions.
  • Native UIA live Jump path: exact A destination and selection oracle passed; EOF/job/process/DPI cleanup verified.
  • macOS make test and make check were not run because the change is Windows-only.

Checklist

  • I have read the Contributing Guidelines
  • I have signed off my commits (git commit -s) per the DCO
  • Tests pass locally (make test)
  • Code follows the existing style (make check)
  • I added the test/contract before the implementation and observed the intended RED failure

coneilen and others added 6 commits November 9, 2026 11:07
Reuse the production activation path, preserve stale destinations, and apply the macOS blocked-loop admission rule with retained presence exit status.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>
Add headless destination controls to the shared shell and compiled functional adapter. Keep native/runtime qualification explicitly pending.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>
Reuse the existing shell scope and capture pump for live create-new logs, preserve shared cleanup budgets and UNKNOWN retention, and exercise actual dispatch with owned non-GUI children.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>
Use the existing owned-child supervisor for discovery and compile/test under one clock, preserve raw streams and unknown leases, and verify 16 headless safety controls without executing a C++ fixture.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>
Share the EN_CHANGE query reader with the gated refresh message instead of forcing the UIA loop B fixture. Cover A/B/cross-project, empty and unmatched queries plus Return destination behavior.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>
Reuse the existing WM_SETTEXT helper, preserve the destination oracle, and register headless and bounded owned cross-process regressions.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>
@coneilen
coneilen merged commit 029cae3 into main Nov 10, 2026
26 checks passed
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.

Windows parity: Jump palette selection should open the loop like a sidebar tap, not just select it on the canvas

1 participant