From 212903ecd4e481246c62df55f3645cb2a0fcdf2f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Padez?= Date: Fri, 7 Aug 2026 14:09:37 +0000 Subject: [PATCH] correct a stale untested list: two of its four entries already had tests --- docs/workspace-panel-todo.md | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/docs/workspace-panel-todo.md b/docs/workspace-panel-todo.md index d2cf6a12..d58522fd 100644 --- a/docs/workspace-panel-todo.md +++ b/docs/workspace-panel-todo.md @@ -978,8 +978,15 @@ they are marked below, because a dead-code list that is itself wrong is the wors `onClearApp` → `null`), but `setApp` is exported from the barrel. Fixed in the same commit. Two tests are deliberately written to the _current_ behaviour and marked `KNOWN GAP` so they fail the day it changes: a move drops `zoom` and `fitContent` (§5.3). - **Still untested:** the other eleven files of the framework — `WorkspaceView`, `WorkspaceRenderer`, - `PanelSlot`, `normalizeLayout`. §5.5's debounce lost-updates live there, not in `layout-utils`. + **The "still untested" list this line used to carry was stale by the time it was written, and two of + its four entries were wrong.** `normalizeLayout` is not a separate file — it lives in `layout-utils.ts` + and has nine tests of its own (identity-by-reference on a clean tree, `config` dropped along with the + app it belonged to, idempotence, nested repair, `null` always allowed). `WorkspaceRenderer` got eight + tests in `56ca411`. The Workspace directory is 76 tests across three files and green. + **Genuinely still untested: `WorkspaceView` and `PanelSlot`.** But note §5.5's lost updates are no + longer what makes that urgent — every mutation in `WorkspaceView` now goes through `onLayoutChange` + as an *updater*, never as a computed tree, which is the structural fix; a test there would be + guarding the fix rather than finding the bug. Checked, not assumed — `bun test src/workspaces/officerdev/src/components/Workspace/`. - [x] **A panel is told where it is, instead of being handed a key to reverse-engineer.** _(`dbe585f` + `717580f` + `585f234`, branch `agent-coordination-mvp`)_ — the first three items of §5.9. The