Skip to content

Restore current PR trajectory summaries - #75

Merged
vks-archastro merged 1 commit into
mainfrom
feat/current-pr-summary
Aug 9, 2026
Merged

vks-archastro merged 1 commit into
mainfrom
feat/current-pr-summary

Conversation

@vks-archastro

Copy link
Copy Markdown
Contributor

Review on ArchCode

Problem and author intent

PR evidence still uploaded its prompt, trajectory, and patch artifact, but the Room stopped publishing the compact trajectory summary card that makes that evidence discoverable. Restore one current, author-owned summary per PR without waking Participate routines or accumulating stale cards.

What changed

  • Publish a typed exhaust message with post_type: trajectory after the policy-applied current artifact is attached.
  • Update the same deterministic summary message across later sessions and versions.
  • Recover safely after response loss and cold local state.
  • Preserve capture-mode redaction and keep publication bounded and best-effort.
  • Extend the real CLI/TCP proof across two sessions and a warm unchanged retry.

Scope

Room-kit client and tests only. No platform or UI code.

Risk assessment

Low to medium. This adds one best-effort message create on first publication and one update when PR evidence changes. Deterministic identity, author ownership, current-artifact checks, and retry recovery prevent duplicate or stale summaries.

User impact

Engineers see the compact “How PR was built” card again. Later evidence replaces that card in place, so Rooms retain the latest complete summary rather than stale versions.

Testing

  • tests/test_pr_evidence_publish.py — test_two_agent_sessions_publish_one_current_artifact_over_tcp_without_github is the canonical end-to-end proof. It runs the real CLI against a TCP server, publishes two sessions, verifies one attachment and one summary, then proves the unchanged retry performs no writes.
  • python3 tests/test_pr_evidence_publish.py
  • python3 tests/test_pr_evidence_conflict.py
  • python3 tests/test_trajectory_summary.py — 8/8 passed.
  • python3 tests/test_pr_evidence_performance.py — unchanged CLI p95 0.1841s.
  • Full Python CI-equivalent suite and installer isolation passed before publication.

Follow-ups and known issues

The existing Room UI owns the visual trajectory-card renderer; this PR restores its event contract.

@vks-archastro
vks-archastro merged commit 4a92aba into main Aug 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant