Skip to content

fix(parallel-issues): support resumable artifact preparation - #559

Merged
thewrz merged 5 commits into
mainfrom
feat/issue-545
Aug 30, 2026
Merged

fix(parallel-issues): support resumable artifact preparation#559
thewrz merged 5 commits into
mainfrom
feat/issue-545

Conversation

@thewrz

@thewrz thewrz commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

This was written agentically; verify its assertions:

Why

Interrupted issue worktrees could not safely resume once fence artifacts existed, forcing manual deletion around potentially valuable uncommitted implementation work.

What

  • Add an explicit artifact-resume path that archives the prior fenced set and regenerates it.
  • Preserve and report untracked and modified implementation state during resume.
  • Report resumable existing worktrees before refusing duplicate creation.
  • Document the exact resume command and cover the workflow with fixtures.

Decisions

  • Keep duplicate worktree creation fail-closed while surfacing deterministic resumability evidence.
  • Archive only generated fencing artifacts under private evidence storage and leave every other worktree path untouched.
  • Stacked on fix(parallel-issues): operator-named issues with stale In-progress status are dropped as active under --fast-mode #552 — merge that PR first. Agent-driven merges run chain-advance.sh --retarget --pr <this-PR> --base <default> and require its full proof before merging; interactive human merges may merge then delete the predecessor branch for GitHub's automatic retarget.

base=acb15ef8c748a62c0e2499743d95f629952a5aba
files=5
total.insertions=176
total.deletions=12
total.lines=188
operational.files=5
operational.insertions=176
operational.deletions=12
operational.lines=188
generated.files=0
generated.insertions=0
generated.deletions=0
generated.lines=0
lockfile.files=0
lockfile.insertions=0
lockfile.deletions=0
lockfile.lines=0
fixture.files=0
fixture.insertions=0
fixture.deletions=0
fixture.lines=0
non_operational.files=0
non_operational.insertions=0
non_operational.deletions=0
non_operational.lines=0

Testing

  • Focused resume/worktree regressions — PASS
  • tests/run-tests.sh — ALL GREEN
  • Green log: /home/adam/github/agent-kit/.worktrees/feat/issue-545/.agent/logs/20260828T154425Z-test.log

🤖 Co-authored by Codex gpt-5.6-luna high.

Closes #545

Preserve implementation work while regenerating stale issue artifacts and report resumable worktree state.

Co-Authored-By: Codex gpt-5.6-luna <noreply@openai.com>
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e3749459-3dad-42d6-ad0f-72c3b19df48b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

mergetest and others added 2 commits August 28, 2026 11:06
Keep prior-art resume commands runnable and compute resumability after fetching origin.

Co-Authored-By: Codex gpt-5.6-luna <noreply@openai.com>
@thewrz

thewrz commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator Author

This was written agentically; verify its assertions:

Adversarial review receipt

  • Reviewer: provider=anthropic; model=claude-fable-5; effort=xhigh; mode=cross-provider (reason: User-authorized Claude Fable adversarial review)
  • Counts: P1=0; P2=2; total=2
  • Reviewed head: e189893
  • Confirmed finding: Preserve prior-art file for resume command — verdict=fixed; fix commit SHA(s)=7733397d02dcc0a2c7fb2fc82a9e52fa4bb77371
  • Confirmed finding: Fetch before resumability check — verdict=fixed; fix commit SHA(s)=7733397d02dcc0a2c7fb2fc82a9e52fa4bb77371

🤖 Co-authored by Codex.

@thewrz

thewrz commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator Author

This was written agentically; verify its assertions:

Review ledger

Machine-readable record of every review already performed on this PR.

{
  "version": 1,
  "pr": 559,
  "repo": "wrzonance/agent-kit",
  "reviews": [
    {
      "kind": "adversarial",
      "provider": "anthropic",
      "model": "claude-fable-5",
      "effort": "xhigh",
      "mode": "cross-provider",
      "head_sha": "e1898930d6afb9ea06ea82c75cb858c41ade371b",
      "covered_heads": [
        "7733397d02dcc0a2c7fb2fc82a9e52fa4bb77371",
        "e1898930d6afb9ea06ea82c75cb858c41ade371b"
      ],
      "counts": {
        "p1": 0,
        "p2": 2
      },
      "reviewed_at": "2026-08-28T18:37:29Z"
    }
  ]
}

🤖 Co-authored by Codex.

Base automatically changed from feat/issue-544 to main August 30, 2026 04:12
@thewrz
thewrz marked this pull request as ready for review August 30, 2026 04:15
@thewrz
thewrz merged commit 3f3e8ce into main Aug 30, 2026
2 checks passed
@thewrz
thewrz deleted the feat/issue-545 branch August 30, 2026 04:15
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.

fix(parallel-issues): prepare-issue-artifacts.sh has no resume path — refuses when fence files exist, even with preserved work

1 participant