Feedback telemetry — granular per-message codes + indexed feedbackCode column (Scope B)

Sub-tickets

Questions

Activity

• wi-cli-venus created · 5w ago
• wi-cli-venus note · 5w ago
  Design-only prep delegated to solo-venus-cc (appEvents/sink seam owner) under Elazar keep-going directive, coder-venus-cc routing. Deliverable: (1) justify indexed feedbackCode column vs existing detail->>'code' jsonb; (2) sink migration sketch (DDL exec = db-venus); (3) result-type 'code' collision map (semantic discriminant not-found/forbidden/db-fault/... can't be overloaded → separate feedbackCode field across ~435 sites/~196-msg catalog); (4) callsite inventory. WI STAYS PARKED — implementation gated behind audit design-ping + PM schedule. PM (pm-venus-cc) flagged, may veto.
• wi-cli-venus note · 5w ago
  Scope B (appEvents/sink) DESIGN complete — prep artifact, WI stays PARKED. Full doc: pm/venus-216-scopeB-design-appevents-sink.md. Summary: (1) Promoted feedbackCode column = MARGINAL, recommend NOT promoting — appEvents is 5.5k rows, detail->>'code' already persisted+membership-validated (ux-event/route.ts:90-127) and already consumed by vFriction7d (schema.md:1008); a partial expression index buys identical perf at 100x growth with zero app change. (2) Migration sketch: Option A (RECOMMENDED) CREATE INDEX CONCURRENTLY appEvents_detail_code_idx ON appEvents((detail->>'code')) WHERE detail?'code' — no insert-path/backfill/trigger touch. Option B (promoted column) only if PM wants first-class — forces logger-core dual-write + immutability-bypass backfill; db-venus executes either. (3) Collision map: TWO code namespaces — result-discriminant code (16 values, ~140 sites, 6 action files, coder-verified 2026-07-12) which is load-bearing routing and CANNOT be overloaded, vs GuardrailCode (guardrail-codes.ts). Granular feedback code needs a NEW feedbackCode?:FeedbackCode field on the 6 result types; client notify() already accepts code?:FeedbackCode so gap is pure server->client transport. Count reconciled (coder retracted 435). Gates before any code: PM unpark + audit pre-impl design-ping.
• wi-cli-venus statusChanged · 5w ago
  Unparked for DDL leg: Option A (partial expression index on appEvents detail->>'code', no app change) accepted over Option B (promoted column, marginal gain). Dispatched to db-venus-cc, Class-A DDL gate (audit pre-impl design-ping). Client granular-code rewire (140 sites/6 files, Class-A/full-team) queues after coder clears VENUS-169/139.
• wi-cli-venus progress · 5w ago
  DDL leg complete: appEvents_detail_code_idx (partial expr index, Option A) applied, audit PASS. Non-concurrent build (CONCURRENTLY rejected inside migration txn wrapper, expected 25001 fallback) — impact confirmed negligible: 183ms SHARE lock on a 5.6k-row/4.7MB table, no lock-wait/deadlock/error in Postgres logs. Client granular-code rewire (Class-A/full-team, 140 sites/6 files) remains queued behind coder's current Class-S bug-triage work.
• wi-cli-venus completed · 5w ago
  Scope B shipped a4c2034/v0.95.12. All 31 inline logActionError sites domain-tagged into detail.code (per-domain friction bucketing). Core+edge (action-form-core.ts/use-action-form.ts) re-vendored to Mars MARS-428 parity (ToastFn widened +feedbackCode?, threaded 3x); venus-local telemetry-actions.ts extension, no Mars break. Class-A audit PASS:a4c2034 (audit-venus-ca): plumbing internally consistent, registry-validated, toast guarantee preserved, no blocking defect. Live-verify: v0.95.12, dpl_DVTrFLDAVx8U8iNorwUdyGBeUxCc, dbOk:true.