[issue-3393][slice-4/6] Slice-PR routing to slice.repo + lazy...

[james-in-a-box]

Route each slice's PR to slice.repo and give every participating repo its own work branch + context PR (lazy-per-repo, operator ruling #1). Repos with ≥1 slice get an egg/<id>/work branch and a context PR; submitted repos with no slices get neither. Slice-PR and context-PR descriptions cross-reference sibling PRs in the pipeline.

Base PR: #3418

What's in this PR

Commits (4):

• Persist BRC history for slice-4 (Slice PRs are missing analysis/plan docs and all BRC history; need a 'context' PR for refine+plan phases plus per-slice BRC in each slice PR #2548)
• test(Feature: multi-repo pipelines — coordinated PRs across repositories in one pipeline #3393): ruff format slice-4 tester tests (task-4-3)
• Slice-4 (Feature: multi-repo pipelines — coordinated PRs across repositories in one pipeline #3393): slice-PR routing to slice.repo + lazy per-repo context PR
• test(Feature: multi-repo pipelines — coordinated PRs across repositories in one pipeline #3393): slice-4 slice-PR routing + lazy per-repo context PR (task-4-3)

.egg-state/agent-outputs/coder/brc-memory-issue-3393.md |    71 +
 .egg-state/brc-history/3393-implement-slice-4.json      | 12859 ++++++++++++++++++++++++++++++++++
 .egg-state/brc-history/3393-implement-slice-4.md        | 11486 ++++++++++++++++++++++++++++++
 orchestrator/gateway_client/_pr.py                      |    78 +
 orchestrator/routes/pipelines.py                        |   410 +-
 orchestrator/tests/test_pipelines.py                    |   302 +-
 6 files changed, 25201 insertions(+), 5 deletions(-)

This slice

Slice-PR routing to slice.repo + lazy per-repo work branch & context PR

Files affected:

• orchestrator/gateway_client/_pr.py
• orchestrator/routes/pipelines.py
• orchestrator/tests/test_pipelines.py

Tasks (3) + acceptance criteria

• task-4-1: Route slice-PR creation to the slice's repo. create_slice_pr (orchestrator/gateway_client/_pr.py:139-164) is already repo-parameterized; change the caller(s) in orchestrator/routes/pipelines.py to pass slice.repo (falling back to the pipeline primary when a slice's repo is absent, per the slice-1 migration default) instead of the pipeline singleton. Render sibling cross-references into the slice-PR body: list the other pipeline PRs (repo + number) and, for a dependent slice, note the upstream slice's PR it is ordered behind.
  • Acceptance criteria: - Each slice's PR is opened in slice.repo; absent ⇒ primary. - The slice-PR body cross-references sibling pipeline PRs (repo + number) and names the upstream PR for dependent slices. - No behavioral change for N=1 (single repo) pipelines.
• task-4-2: Make the context-PR opener lazy-per-repo. In orchestrator/routes/pipelines.py _open_context_pr_at_implement_start (~11001-11250), which today opens one context PR on egg/<id>/work → main in the single repo: iterate the set of repos that own ≥1 slice, and for each open its own egg/<id>/work branch (same naming, per repo) and its own context PR; skip submitted repos with no slices. A single-slice repo still gets the standard context PR (uniformity beats special-casing). Cross-reference sibling context PRs in each body. Keep the N=1 path byte-equivalent to today.
  • Acceptance criteria: - Every repo with ≥1 slice gets its own egg/<id>/work branch and context PR; a slice-less submitted repo gets neither. - A single-slice repo gets the standard context PR. - Context-PR bodies cross-reference sibling context PRs. - N=1 behavior is unchanged.
• task-4-3: Add tests in orchestrator/tests: a slice's PR is created in slice.repo (and falls back to primary when absent); the slice-PR and context-PR bodies render sibling cross-references; the lazy-per-repo rule holds (repo with ≥1 slice ⇒ work branch + context PR; slice-less repo ⇒ neither; single-slice repo ⇒ standard context PR); N=1 opens exactly one work branch + context PR as today.
  • Acceptance criteria: - Tests assert slice-PR repo routing (+ primary fallback), sibling cross-references, and the lazy-per-repo work-branch/context-PR rule. - N=1 regression asserted (one work branch + one context PR). - make test (narrowed) and make test-all green.

Stack

• Position: slice 4 of 6 in pipeline issue-3393
• Base PR: Multi-repo pipelines: coordinated PRs across repositories (#3393) #3418
• Stacked on top of egg/issue-3393/slice-3