docs(docs): add prp 41 showcase agent ops polish#322
Conversation
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
Docs-only PR adding
PRPs/PRP-41-showcase-agent-ops-polish.md— the one-pass implementation contract for the FINAL slice of the/showcaseupgrade epic (PRP-38..41).Closes: #321 (implementation issue — this PR ships the spec only).
PRP at a glance
PRPs/PRP-41-showcase-agent-ops-polish.md/agents/sessions/*,/ops/*)onValueChange)Task 1 — Contract Probe (the gate)
Before any implementation Task runs, Task 1 produces
PRPs/ai_docs/prp-41-contract-probe-report.mdthat verifies every cited backend field-for-field. INITIAL-41 already flagged drift ondev:ModelHealthEntry.drift_direction(NOTdrift_verdict)action_id(NOTtool_call_id)pending_approval: bool+pending_action: PendingAction(NOT anapproval_requiredevent — that event fires only on the WS/agents/streampath)If any cite drifts, the executor must STOP and patch the PRP before proceeding.
Unresolved contract assumptions (5)
_Client.yield_eventorchestrator-fill-in semantics forstep_index/phase_index/phase_total./approvedouble-fire response shape (200 idempotent vs 4xx "action consumed").SHOWCASE_RICH_STEP_NAMESfilter selector keepsagentondemo_minimalANDagent_hitl_flowonshowcase_rich.OpsSummaryResponse.runs.countsvsruns.histogrampath + per-item key (countvsvalue)._Client.requestlist-body wrapper (already CONFIRMED — no action; documented for future-proofing).Task 1 resolves 1-4 against live
devbefore any implementation Task runs.Scope discipline
app/code, nofrontend/code, no migrations, no test changes.stash@{0}("local qwen3 rag demo changes before prp-35") is untouched and remains stashed.Test plan
git diff --stat origin/dev...HEADshows exactly one file changed:PRPs/PRP-41-showcase-agent-ops-polish.mdgit diff --name-only origin/dev...HEADcontains no entries underapp/,frontend/,alembic/,tests/git stash listshows the qwen3 stash atstash@{0}unchanged🤖 PRP authoring driven by the PRP epic; this PR ships the artifact only.