Skip to content

Close canvas workspace detail parity gaps - #399

Merged
coneilen merged 4 commits into
scgopi:mainfrom
coneilen:coneilen-microsoft-canvas-workspace-detail-parity
Sep 21, 2026
Merged

coneilen merged 4 commits into
scgopi:mainfrom
coneilen:coneilen-microsoft-canvas-workspace-detail-parity

Conversation

@coneilen

@coneilen coneilen commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • enrich Windows graph canvas loop cards with metric pass/change, elapsed age, backend/model, branch, token usage, and reason-specific NEEDS YOU actions
  • add attention rail age text and retain metric samples for workspace sparklines
  • preserve accurate fired N labels for ordinary edges and document guarded retry/cycle wording as blocked until the Windows model decodes cycleGuard
  • add terminal workspace right-panel sparkline/start/usage/collapse UIA children plus live-gate assertions for right-panel controls and mounted tab preservation
  • update the parity ledger honestly, leaving live/protocol-limited gaps Partial instead of over-marking Validated

Validation

  • zig fmt on touched Zig files
  • git diff --check
  • targeted Zig tests were attempted locally but blocked by the known environment mismatch: local Zig is 0.16.0 while the repo is pinned to 0.15.2, plus local Windows libc/header issues. CI should be treated as the authoritative build/live UIA gate.

RED: zig build test on initial commit f9fa7c0 -> GraphCanvas.test.edge presentation distinguishes kind condition and fired state failed because ordinary fire_count edges rendered retry ×2 instead of expected fired 2
GREEN: zig fmt graphcode-windows\src\GraphCanvas.zig after commit fc6ad1b -> exits 0 after restoring fired N for ordinary edges so the focused assertion matches the implementation
REGRESSION: git diff --check -> exits 0; the ledger records macOS retry ×N requires the currently undecoded cycleGuard field, preventing guarded-cycle wording from being applied to ordinary edges

Signed-off-by: Colin Neilens coneilen@microsoft.com
Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

coneilen and others added 4 commits September 21, 2026 11:30
Signed-off-by: Colin Neilens <coneilen@microsoft.com>

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Colin Neilens <coneilen@microsoft.com>

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@coneilen
coneilen force-pushed the coneilen-microsoft-canvas-workspace-detail-parity branch from d35612f to 4ef6c6c Compare September 21, 2026 18:30
@coneilen
coneilen merged commit 445815e into scgopi:main Sep 21, 2026
10 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