Skip to content

Phase E — the teammate half of the job#16

Merged
alejandro-nexcade merged 4 commits into
mainfrom
alejandro/phase-e-teammate
Jul 2, 2026
Merged

Phase E — the teammate half of the job#16
alejandro-nexcade merged 4 commits into
mainfrom
alejandro/phase-e-teammate

Conversation

@alejandro-nexcade

Copy link
Copy Markdown
Collaborator

What (ROADMAP Phase E)

  • E1 Conversation ingest: review-requested PRs show the GitHub conversation (reviews with verdict states, line-anchored review comments, issue comments) as a strictly read-only band — three concurrent gh api reads, fixture-tested parsers, ghost-user tolerance, plain-text rendering of all untrusted content. Conversation lives on a separate model from ephemeral comments (§0.4 by design: the gate never clears it, and it can never feed prompts — reviewer-verified zero consumers).
  • E2 Re-review on push: submit_github_review records last_reviewed_sha; when a teammate pushes past it, the interdiff toggle offers "Since your review" via gh compare, with a header note. Authored (dispatch-snapshot) and teammate paths are disjoint by construction.
  • E3 Linear intent: issue descriptions now seed review intent (Review.body) and capped 300-char gists in plan prompts — with the missing plan-issue list built properly (non-blocking, 15s-bounded Linear fetch; goldens byte-identical).

Review round

Reviewer: ship-ready, zero critical/major. Advisory fixed here: Linear API calls bounded with a 15s timeout (was the one un-timed await). Remaining advisories documented (FRAGILE string-match on the benign no-new-commits error; ISO-sort assumption noted in-code).

Verified

cargo fmt/clippy -D warnings/test --all (458+) · tsc strict · vitest 201 · vite build · reviewer pass.

🤖 Generated with Claude Code

LifeLex and others added 4 commits July 2, 2026 05:21
…Linear intent

fetch_conversation merges reviews/review-comments/issue-comments (three
concurrent gh reads, fixture-tested parsers, state-only approvals kept);
submit_github_review records last_reviewed_sha; get_teammate_interdiff
compares it to the pinned head; Linear descriptions seed review intent
and capped plan-issue gists (goldens byte-identical).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Read-only GitHub conversation context for review-requested PRs (state-
aware labels, plain-text bodies, external links); the interdiff toggle
gains a teammate source driven by last_reviewed_sha vs pinned head with
a 'pushed since your review' header note. Authored (dispatch_snapshot)
and teammate paths are mutually exclusive by construction.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
A stalled Linear response was the one un-timed await in plan generation
and kickoff; errors already degrade gracefully, now hangs do too.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@alejandro-nexcade alejandro-nexcade merged commit 224a4e9 into main Jul 2, 2026
2 checks passed
@LifeLex LifeLex deleted the alejandro/phase-e-teammate branch July 2, 2026 04:39
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.

2 participants