avoigt1121 Claude Opus 5 commited on
Commit
e9fe2a5
Β·
1 Parent(s): 125b8a5

docs: regression fix deployed as 125b8a5 and verified on prod in both directions

Browse files
Files changed (2) hide show
  1. TODO.md +10 -8
  2. memory.md +7 -0
TODO.md CHANGED
@@ -130,16 +130,18 @@ the design doc's M0–M4 is the intent behind it β€” see that doc's Β§0).
130
  Numbers reflect the profiled-subset denominator fix. The whole path (orchestrator β†’ this
131
  Space β†’ machine-auth gate β†’ curated artifact) is now proven in production.
132
 
133
- ## Open β€” a chart regression is LIVE on prod; the fix is committed but not deployed
134
-
135
- - [ ] **DEPLOY `819da30`.** Prod (`23d37b6`) has a regression shipped 2026-08-05: a query that
136
- NARROWS the gene set leaves the previous answer's bars under the new answer's caution for one
137
- render. Self-corrects on a second click and never shows a wrong *number*, but it does show one
138
- cohort's bars beneath another cohort's warning. `819da30` fixes it (handler now blanks the
139
- chart before yielding the answer) and is verified in BOTH directions on a single click β€” the
140
- check whose absence caused this. 234 tests pass. Not deployed: needs Annie's go-ahead.
141
  **Do not "clean up" `colors_in_legend` back in** β€” it looks cosmetic and is not; see `memory.md`
142
  2026-08-05 (latest) for the three-way measurement.
 
 
 
143
 
144
  ## Open β€” the dev lane cannot be click-through verified while private
145
 
 
130
  Numbers reflect the profiled-subset denominator fix. The whole path (orchestrator β†’ this
131
  Space β†’ machine-auth gate β†’ curated artifact) is now proven in production.
132
 
133
+ ## Done β€” the chart regression, fixed and deployed
134
+
135
+ - [x] **DEPLOYED `125b8a5` (2026-08-06)** β€” closes the regression `23d37b6` shipped. The handler now
136
+ blanks the chart before yielding the answer, so a narrowing query can never leave the previous
137
+ cohort's bars under the new cohort's caution. **Verified on the live prod Space in BOTH
138
+ directions, each on a single click** β€” CCLE wide β†’ MSK+NRG1 narrow (correct NRG1 bar, not
139
+ CCLE's) β†’ full panel wide (KRAS ~94%, all 33 genes). 234 tests, security scan clean.
 
140
  **Do not "clean up" `colors_in_legend` back in** β€” it looks cosmetic and is not; see `memory.md`
141
  2026-08-05 (latest) for the three-way measurement.
142
+ - [ ] **Realign `dev` with prod.** `dev`/`hf-dev/main` is still at `23d37b6`, so the dev Space now
143
+ trails prod and carries the regression prod has shed. Fast-forward `dev` to `main` and push
144
+ `hf-dev` β€” a separate Space deploy, so it needs Annie's go-ahead.
145
 
146
  ## Open β€” the dev lane cannot be click-through verified while private
147
 
memory.md CHANGED
@@ -36,6 +36,13 @@ EMPTY chart, which claims nothing, rather than a stale one, which claims somethi
36
  directions verified repainting on a single click. `test_ui_handler_gating` reads the final frame via
37
  `_final()`; the ADR-0004 assertions are untouched and a denied path still yields exactly one frame.
38
 
 
 
 
 
 
 
 
39
  **Three process lessons, all of them mine:**
40
  1. **I explained away the symptom the first time.** I saw the stale chart locally, guessed "my click
41
  missed the button", and moved on. It had not missed. A surprising observation deserves one cheap
 
36
  directions verified repainting on a single click. `test_ui_handler_gating` reads the final frame via
37
  `_final()`; the ADR-0004 assertions are untouched and a denied path still yields exactly one frame.
38
 
39
+ **Deployed `125b8a5` 2026-08-06** (gates clean: 234 tests, pip-audit, gitleaks; hook mirrored to
40
+ `github` first). **Verified on the live prod Space in both directions, each on a single click** β€”
41
+ CCLE wide β†’ MSK+NRG1 narrow (correct NRG1 bar immediately, not CCLE's) β†’ full panel wide (KRAS ~94%
42
+ across all 33 genes). The OAuth session drops when the Space rebuilds, so sign in again before
43
+ verifying; an anonymous query renders nothing visible because the plot is None and the JSON
44
+ accordion is collapsed.
45
+
46
  **Three process lessons, all of them mine:**
47
  1. **I explained away the symptom the first time.** I saw the stale chart locally, guessed "my click
48
  missed the button", and moved on. It had not missed. A surprising observation deserves one cheap