Skip to content

Pair combined macOS references without promoting Windows parity - #832

Merged
coneilen merged 2 commits into
mainfrom
coneilen-combined-reference-intake
Oct 15, 2026
Merged

coneilen merged 2 commits into
mainfrom
coneilen-combined-reference-intake

Conversation

@coneilen

Copy link
Copy Markdown
Collaborator

Summary

Pair the newly returned combined macOS gap-fill references to the Windows parity ledger without changing any Windows parity status. Record the fourteen user-accepted omissions as macOS reference: accepted unrecorded, preserving dated intake, failed attempts, accounting deviations and the original publication hold.

Refs #822, #826, #764, #768, #767, #774, #778.

Changes

  • Add one bounded combined-return update to investigation\ui-parity-matrix.md: all 65 exact item IDs/outcomes, all 30 qualified rows, metadata locators and actual SHA-256 hashes from the local C:\gc-ref\macos-groundtruth-2ccf303a-gapfill-combined-20261015 copy. Final tables are only those under macos-groundtruth-2ccf303a-completed-v12, source 2ccf303a04f995184aae1d4b738a9eca0f71b79e.
  • Reconcile 46 Recorded / 5 Differs from request / 14 accepted unrecorded / 0 Dropped for the latest 65; 63 / 6 / 14 / 24 for the original 107; 7 recorded / 1 accepted updater limitation for the eight supplements. The 65-item mapping reaches 30 distinct canonical rows, 23 additional to the qualified thirty, 53 distinct across both crosswalks. Three items explicitly have no exact canonical row: update-other-workspaces, select-folder, image-drop.
  • Preserve exactly 98 canonical identities/order and 6 Validated / 89 Partial / 3 Out of scope, including all thirty demoted rows still Partial. No Windows status promotion or whole-issue acceptance follows macOS-only references.
  • Apply scene/zoom, source/copy and routing corrections, including direct-root Composite Open Terminal versus worker aggregation, partly occluded RUNNING menu versus separate unobscured/AX/provider chronology, transient folder startup versus persistent deselection, actual PNG drop and limited tooltip/ENTRY/edge states.
  • Disclose inherited locator defects rather than treating missing references as passes: old v2 paths appended to edge-focus under v6 and overview-folders under v3 are resolved under v2 and excluded as current success proof. Historical prior_metadata_digests and qualified-row fields are not current seals/dispositions. V4/V8 failures and source-only V7 remain qualified; V3's twelve turns exceeded its initial two-turn ceiling, distinct from V11's one permitted response and zero executed tools.
  • Update only directly related reference/authority sections in investigation\windows-parity-lane-kickoff.md and investigation\windows-parity-orchestration-plan.md. Add the requested concise command-lease retrospective: self-added validators/gates are out of lease; reading checker output does not authorize execution. The rp1 ANSI gate/post-conversion parser failures are process deviations, not product defects; its permanent executor stop stands.

Accepted base: baa2f0c6b9388f11219dd97e7d4cfed2176bc17b (#830). Before any push, the coordinator explicitly authorized a mechanical rebase from 4a30b79d259228cbce7d23b16a3a81a9f8ee9446 onto that exact accepted merge. Read-only fetch/diff confirmed only seven disjoint offscreen files; the rebase completed without conflicts or stash. None of those seven files or J5's capture-core work is edited here.

Test plan

RED: pwsh -NoProfile -File C:\gcs\combined-intake\Test-CombinedIntake.ps1 -Phase RED -> exit 1; 2260 checks, 158 failures, including Missing/duplicate combined item: card-main-running and missing combined hashes/qualified-row crosswalk.
GREEN: pwsh -NoProfile -File C:\gcs\combined-intake\Test-CombinedIntake.ps1 -Phase GREEN -> exit 0; 2452 checks, zero failures; 65 items, 30 qualified rows, 98 unchanged canonical rows.
REGRESSION: pwsh -NoProfile -File C:\gcs\combined-intake\Test-CombinedIntake.ps1 -Phase REGRESSION -> exit 0; 2452 checks, zero failures; metadata/text/hash/status/hold/accounting invariants retained after complete ledger diff review.

The single explicitly granted scratch checker stayed unchanged between those three phases. Counts are documentation/reference assertions, not product tests, runtime passes or independent visual review. It checks final source identities, collection hash entries/path confinement, referenced text/JSON hashes and metadata item/capture identity, exact 65 outcomes and fourteen accepted IDs, all thirty Partial rows, the pre-edit 98-row snapshot, preserved historical intake, correction/hold/privacy/accounting wording and the command-lease retrospective.

Local evidence is retained under C:\gcs\combined-intake and is not committed:

Receipt SHA-256
Test-CombinedIntake.ps1 8B8C481D444F3B2737884EE1913F2440F484B18D609B757C3901DD8AF58FFAF9
canonical-baseline.json 13AD4A99EDC0ADAAE44B2609AF9733C13923D0A17FF4A26EA40549571132A4E1
RED.log 058559548B3C5C3EAB84074641CF6EFEC2737190EE3D0825D6CE012810D5F06C
GREEN.log 57F956EE96C9F453965BB37538F8C4427C2B2427208FC6D9DC577ACB50D1499C
REGRESSION.log D3A067737850381E1D3E0D1C4F39C6558CFE5D44EB230A13E7C6E87C1ADDF657
preflight-01-missing-reference.log 13AE65D9071A155BDCA919C7DDD1CCD3F7C9D78D49475D47355627FA7BD6218D

An earlier checker preflight stopped on a missing inherited v2 locator under v6; its original failing output is retained separately, not substituted for the intended RED or erased. The locator-aware final checker was fixed before the three recorded phases. An earlier read-only metadata command was blocked before execution by antivirus and supplied no evidence; no bypass was attempted.

git --no-pager diff --check -> exit 0, no whitespace errors.

pwsh -NoProfile -File Tools\tdd\Test-TddEvidence.ps1 -BodyPath C:\gcs\combined-intake\pr-body.md -> exit 0, TDD evidence: PASS.

Not verified / limits: no image opens, decoding, crops, copies, upload or new privacy verdict; no new screenshots, visual reviewer, product implementation, native/desktop/backend process, toolchain/bootstrap, build, product tests, broad ValidationRunner or CI dispatch. Producer clearance of 442 PNGs is not independent review of all 442; the parent independently opened only three specific new images plus the prior eleven-image v2 sample. Original held issue-722 images remain excluded and forbidden. make test and make check were not run under this docs-only command lease. Automatic exact-head hosted checks are requested by the PR; documentation path skips are configured skips, not runtime passes. Coordinator review is ledger-diff/metadata review only; this draft grants no readiness, merge or capture authority.

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

Pair the final gap-fill item and qualified-row registers to the ledger, preserve accepted omissions and evidence limits, and clarify exact-command execution authority.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: cc23853f-810d-4ef2-b5de-b397605090d7
Signed-off-by: Colin Neilens <coneilen@microsoft.com>
Distinguish the operator's three new-image reviews from the separate earlier eleven-image v2 independent sample without changing reference coverage or clearance.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: cc23853f-810d-4ef2-b5de-b397605090d7
Signed-off-by: Colin Neilens <coneilen@microsoft.com>
@coneilen
coneilen marked this pull request as ready for review October 15, 2026 16:22
@coneilen
coneilen merged commit 661f283 into main Oct 15, 2026
23 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.

1 participant