docs(docs): refresh initial 41 after prp 39 40 (#313)#319
Conversation
PRP-41 drift items D1-D7 + fold in issue #311: - D1-D4: KPI strip counters now read the actual emitted step.data keys (completed_items, total_chunks, total_aliases, scenario_id + winner_scenario_id) — never invented names. - D5: GET /ops/model-health takes only ?limit (no grain param). - D6: ops_snapshot derives stale_aliases_count + total_aliases from OpsSummaryResponse.aliases (no flat keys on the response). - D7: replaced stale line citations with symbol refs so the PRP does not re-stale on the next pipeline.py expansion. - Folded issue #311 (phase accordion lock after completion) into scope as polish item 7 + acceptance criterion D10 — load-bearing for the Inspect-Artifacts post-run UX. - Noted issue #312 (phase2 enrichment idempotency) as a dogfood prerequisite (out of PRP-41 scope, must land before the manual checklist runs). Docs only change. No code touched.
There was a problem hiding this comment.
Sorry @w7-mgfcode, you have reached your weekly rate limit of 500000 diff characters.
Please try again later or upgrade to continue using Sourcery
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Patches
PRPs/INITIAL/INITIAL-showcase-41-agent-ops-polish.mdto remove driftbetween INITIAL-41 (authored pre-PRP-39/40) and the actual surfaces PRP-41
consumes on
dev. Docs only — no code changed.Drift items (D1–D7)
batch_preset.step.data.completed_countcompleted_items(PRP-39 emitscompleted_items/total_items/failed_items)rag_index_subset.step.data.chunks_indexedtotal_chunks(chunks emitted this run across the curated 5-file subset;curated_hitsfor files-indexed)step.data.alias_counttotal_aliasesfromlen(OpsSummaryResponse.aliases)inops_snapshot; fall back tostep.data.aliaskeysscenario_save/scenario_comparepayloads"scenario_simulate_and_save(emitsscenario_id) +multi_plan_compare(emitswinner_scenario_id+ranked)GET /ops/model-health?grain=store_product&limit=5?limit=— nograinparamstale_aliaseskey on/ops/summaryOpsSummaryResponse.aliases[].is_stale; documented response shape (system/jobs/runs/aliases/freshness/attention_items/generated_at)pipeline.py,service.py,experiment.py, frontend hooks::step_agent,::approve_action,::tool_save_scenario,::useDemoPipeline, etc.) so the PRP does not re-stale on the next file expansionFolded into PRP-41 scope
fix(ui): unlock showcase phase accordion after completion) — added as polish item 7 in the "Scope" section + acceptance criterion D10. Load-bearing for the Inspect-Artifacts post-run UX: visitors need to re-open any phase post-run, which the current fully-controlled<Accordion value={value}>(noonValueChange) blocks.Out of scope, but added as PRP-41 prerequisite
fix(seeder): make phase2 enrichment idempotent) — must land before PRP-41's manual dogfood (the 16-line checklist re-runsshowcase-richagainst a populated DB; fix(seeder): make phase2 enrichment idempotent #312'sIntegrityError → HTTP 500would block that). Tracked separately.Test plan
git diff --checkcleanPRPs/INITIAL/INITIAL-showcase-41-agent-ops-polish.mdmodified (117 insertions / 46 deletions)docs(docs): ... (#313)per.claude/rules/commit-format.md