fi-research-brief: rebase once onto a moved origin instead of stranding briefs
Some checks failed
Governed runtime contract / contract (push) Failing after 33s

Every push from 2026-09-15 to 2026-09-22 was rejected non-fast-forward
because origin had moved; six briefs stayed local on railiance01. Per
the FI publication decision (2026-09-22), fetch, confirm that unpublished
commits touch only briefs/, rebase once and retry. Never merge or force;
a conflict aborts the rebase and fails the day.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Assistant: claude-code
Assistant-Model: opus
Assistant-Process: 51320@bnt-lap001
Assistant-Session: 9d40b4c7-8e3c-42ee-b755-d658d4640d6c
This commit is contained in:
tegwick 2026-09-22 16:25:34 +02:00
parent 364bed78b2
commit 294201a367
2 changed files with 99 additions and 2 deletions

View file

@ -104,3 +104,67 @@ def test_fi_brief_push_failure_is_not_a_completed_day(
assert posted
assert posted[-1]["event_type"] == "executor_run"
assert posted[-1]["detail"]["pushed"] is False
def _advance_origin(tmp_path: Path, name: str, rel: str) -> None:
"""Commit to origin from a second clone so the executor's push is non-fast-forward."""
other = tmp_path / name
subprocess.run(["git", "clone", "-q", str(tmp_path / "origin.git"), str(other)], check=True)
(other / rel).parent.mkdir(parents=True, exist_ok=True)
(other / rel).write_text("moved\n", encoding="utf-8")
_git(other, "add", rel)
_git(other, "-c", "user.email=o@example.invalid", "-c", "user.name=o", "commit", "-qm", "move origin")
_git(other, "push", "-q", "origin", "HEAD:main")
def test_fi_brief_rebases_once_when_origin_moved(tmp_path: Path) -> None:
repo = _repo(tmp_path)
_with_origin(tmp_path, repo)
_advance_origin(tmp_path, "other", "WORK-RECORDS.md")
result = fi_research_brief.run_fi_research_brief(
repo, day=date(2026, 9, 22), report_to_hub=False, complete_fn=_complete
)
assert result.ok is True
assert result.pushed is True
remote_head = subprocess.run(
["git", "-C", str(tmp_path / "origin.git"), "rev-parse", "HEAD"],
check=True, capture_output=True, text=True,
).stdout.strip()
assert remote_head == result.origin_sha
parents = subprocess.run(
["git", "-C", str(repo), "rev-list", "--parents", "-1", "HEAD"],
check=True, capture_output=True, text=True,
).stdout.split()
assert len(parents) == 2 # linear history: rebased, not merged
def test_fi_brief_refuses_rebase_of_non_brief_commits(tmp_path: Path) -> None:
repo = _repo(tmp_path)
_with_origin(tmp_path, repo)
(repo / "notes.md").write_text("local only\n", encoding="utf-8")
_git(repo, "add", "notes.md")
_git(repo, "-c", "user.email=t@example.invalid", "-c", "user.name=t", "commit", "-qm", "stray")
_advance_origin(tmp_path, "other", "WORK-RECORDS.md")
result = fi_research_brief.run_fi_research_brief(
repo, day=date(2026, 9, 22), report_to_hub=False, complete_fn=_complete
)
assert result.ok is False
assert result.pushed is False
assert "outside briefs/" in result.reason
def test_fi_brief_rebase_conflict_fails_closed(tmp_path: Path) -> None:
repo = _repo(tmp_path)
_with_origin(tmp_path, repo)
_advance_origin(tmp_path, "other", "briefs/2026/09/2026-09-22.md")
result = fi_research_brief.run_fi_research_brief(
repo, day=date(2026, 9, 22), report_to_hub=False, complete_fn=_complete
)
assert result.ok is False
assert result.pushed is False
status = subprocess.run(
["git", "-C", str(repo), "status", "--porcelain=v2", "--branch"],
check=True, capture_output=True, text=True,
).stdout
assert "rebase" not in status.lower()
assert not (repo / ".git" / "rebase-merge").exists()