J ea4ef4c90a docs(attribution): fix accountable NULL semantics + close chat/quick-create evidence boundary (MUL-4302) ...

Addresses Elon's 2nd-round review on PR #5150 (pre-merge doc/evidence items):

- Migration 151 no longer overclaims NULL. accountable_user_id is NULL not only
  on pre-migration rows but on NEW rows whose audit source resolved no human yet
  (run_only autopilot writes originator_source='unattributed' with NULL
  accountable until rule_owner lands). Header + COMMENT ON COLUMN reworded so a
  schema reader does not misjudge the invariant.

- Chat now uses the UNIFORM evidence pair (kind=chat, ref=chat_session_id), like
  autopilot_run/issue_assignment, instead of relying only on the dedicated
  chat_session_id column — new EvidenceChat kind. Added a service test asserting
  chat stamps direct_human + chat evidence.

- Quick-create is documented as the ONE intentional no-antecedent-row path: no
  comment/issue/session/run exists at enqueue time (the run creates the issue), so
  trigger_evidence_kind/ref stay NULL while the human rides originator/accountable
  and source is direct_human — not a NULL-source bypass.

No authorization behavior change. attribution + service + handler suites pass on a
DB migrated through 151.

Co-authored-by: multica-agent <github@multica.ai>

2026-07-09 17:56:09 +08:00

..

attribution_test.go

feat(attribution): split accountable_user_id from originator, close enqueue bypasses (MUL-4302)

2026-07-09 17:38:19 +08:00

attribution.go

docs(attribution): fix accountable NULL semantics + close chat/quick-create evidence boundary (MUL-4302)

2026-07-09 17:56:09 +08:00