IW-513: phase-pr tells a forge that is down from a forge that refused #782

Merged

mph merged 20 commits from IW-513 into main 2026-10-11 07:07:58 +00:00

Conversation 0 Commits 20 Files changed 85 +6466 -856

claude commented 2026-10-11 00:20:49 +00:00

Member

Copy link

Summary

phase-pr stopped an unattended batch with exit 1 when the forge did not answer (502, rate limit, network fault) before the CI gate. Inside the gate the same condition already exits 3 and the caller resumes. This branch makes the pre-gate path behave the same way, with a bound.

Changes

• TrackerError (Unreachable / Refused / Malformed) on the GitHub, GitLab and Forgejo clients, ForgeGateway, and the 16 forge methods of TrackerOps. Linear and YouTrack stay Either[String, A]. Every message string is unchanged; only the classification is new.
• Pre-gate budget: phase-pr retries a silent forge in the same run, then exits 3 with a resume report. The --timeout budget is held on the review-state card (forge_wait_started, forge_wait_branch), so it continues across runs. When it is spent, exit 1 with a headline that says how long the forge was silent. --no-wait behaves the same.
• Only the PhasePr pre-gate path acts on isTransient. All other callers print .message and behave as before. The CI gate is unchanged.

Phases: #778 (clients and PR/CI/merge calls), #779 (pre-gate budget), #780 (issue calls).

Release order: iw-cli first, then kanon (ForgeCatchUp.scala, KanonForgejoIssue.scala, test-env fixtures), then the dev-docs skills. Start the dashboard from the same release, because the validator rejects unknown card fields.

Review packet: project-management/issues/IW-513/review-packet.md.

Testing

• Unit: classifier tables (TrackerErrorTest, HttpCallFailureTest), budget (ForgeWaitTest), client cases including a real closed port, ~17 new PhasePrHarnessTest cases, card field set/keep/clear.
• E2E: test/forgejo-phase-pr.bats against a real refused connection (exit 3 with stamp, then exit 1 after the budget).
• Contract: gh/glab/Forgejo wording that the classifier depends on.
• Each phase PR passed the full CI suite.

Follow-ups filed: #781, and comments on #569, #585, #762, #440.

Release notes

Výpadek forge už nezastaví automatické zpracování fází

Úkol: IW-513
Datum: 2026-10-11

Příkaz pro otevření pull requestu fáze nyní rozlišuje, zda mu forge (GitHub, GitLab nebo Forgejo) odpověď odmítl, nebo zda vůbec nebyl dosažitelný. Krátký výpadek serveru, přetížení nebo chvilkový problém se sítí proto už nezastaví celou automatickou dávku práce na úkolu. Dávka počká a pokračuje, jakmile je forge opět k dispozici.

Když forge při hledání nebo zakládání pull requestu neodpoví, příkaz to nejprve několikrát zkusí znovu s krátkou pauzou a na výstup vypíše, že se o spojení pokouší znovu. Pokud forge mlčí i potom, příkaz skončí s výsledkem „spusťte mě znovu“, který automatická dávka zná z čekání na CI a na který reaguje dalším pokusem. Doba, po kterou se takto čeká, je omezená stejným časovým limitem jako čekání na CI. Začátek čekání si nástroj zapisuje do stavu úkolu, takže limit platí napříč opakovanými spuštěními a nezačíná pokaždé znovu. Když limit vyprší, příkaz skončí chybou se zprávou, že forge je zřejmě mimo provoz a je třeba, aby se na něj podíval člověk. Stejně se příkaz chová i tehdy, když se na CI vůbec nečeká.

Odmítnutí zůstávají konečná, jako dosud. Chybějící nástroj příkazové řádky, nepřihlášený účet, neplatný nebo prošlý přístupový token, uzavřený či již sloučený pull request nebo odpověď, která si odporuje, zastaví práci hned a s původní zprávou. Takové chyby opakováním nezmizí, a nástroj je proto zbytečně neopakuje. Zprávy, které uživatel u ostatních příkazů pro práci s úkoly vidí, se nemění.

Při přechodu na tuto verzi není třeba nic nastavovat. Změna se týká jen autorů rozšíření, která volají jádro nástroje přímo: chyby z GitHubu, GitLabu a Forgejo nyní nesou druh chyby místo prostého textu, takže taková rozšíření je třeba upravit, aby text chyby četla z této nové podoby. Rozšíření pro Linear a YouTrack se změna netýká.

Closes #513

🤖 Generated with Claude Code

## Summary `phase-pr` stopped an unattended batch with exit 1 when the forge did not answer (502, rate limit, network fault) before the CI gate. Inside the gate the same condition already exits 3 and the caller resumes. This branch makes the pre-gate path behave the same way, with a bound. ## Changes - **`TrackerError`** (`Unreachable` / `Refused` / `Malformed`) on the GitHub, GitLab and Forgejo clients, `ForgeGateway`, and the 16 forge methods of `TrackerOps`. Linear and YouTrack stay `Either[String, A]`. Every message string is unchanged; only the classification is new. - **Pre-gate budget**: `phase-pr` retries a silent forge in the same run, then exits 3 with a resume report. The `--timeout` budget is held on the review-state card (`forge_wait_started`, `forge_wait_branch`), so it continues across runs. When it is spent, exit 1 with a headline that says how long the forge was silent. `--no-wait` behaves the same. - Only the `PhasePr` pre-gate path acts on `isTransient`. All other callers print `.message` and behave as before. The CI gate is unchanged. Phases: #778 (clients and PR/CI/merge calls), #779 (pre-gate budget), #780 (issue calls). **Release order:** iw-cli first, then kanon (`ForgeCatchUp.scala`, `KanonForgejoIssue.scala`, test-env fixtures), then the dev-docs skills. Start the dashboard from the same release, because the validator rejects unknown card fields. Review packet: `project-management/issues/IW-513/review-packet.md`. ## Testing - Unit: classifier tables (`TrackerErrorTest`, `HttpCallFailureTest`), budget (`ForgeWaitTest`), client cases including a real closed port, ~17 new `PhasePrHarnessTest` cases, card field set/keep/clear. - E2E: `test/forgejo-phase-pr.bats` against a real refused connection (exit 3 with stamp, then exit 1 after the budget). - Contract: gh/glab/Forgejo wording that the classifier depends on. - Each phase PR passed the full CI suite. Follow-ups filed: #781, and comments on #569, #585, #762, #440. ## Release notes # Výpadek forge už nezastaví automatické zpracování fází **Úkol:** IW-513 **Datum:** 2026-10-11 Příkaz pro otevření pull requestu fáze nyní rozlišuje, zda mu forge (GitHub, GitLab nebo Forgejo) odpověď odmítl, nebo zda vůbec nebyl dosažitelný. Krátký výpadek serveru, přetížení nebo chvilkový problém se sítí proto už nezastaví celou automatickou dávku práce na úkolu. Dávka počká a pokračuje, jakmile je forge opět k dispozici. Když forge při hledání nebo zakládání pull requestu neodpoví, příkaz to nejprve několikrát zkusí znovu s krátkou pauzou a na výstup vypíše, že se o spojení pokouší znovu. Pokud forge mlčí i potom, příkaz skončí s výsledkem „spusťte mě znovu“, který automatická dávka zná z čekání na CI a na který reaguje dalším pokusem. Doba, po kterou se takto čeká, je omezená stejným časovým limitem jako čekání na CI. Začátek čekání si nástroj zapisuje do stavu úkolu, takže limit platí napříč opakovanými spuštěními a nezačíná pokaždé znovu. Když limit vyprší, příkaz skončí chybou se zprávou, že forge je zřejmě mimo provoz a je třeba, aby se na něj podíval člověk. Stejně se příkaz chová i tehdy, když se na CI vůbec nečeká. Odmítnutí zůstávají konečná, jako dosud. Chybějící nástroj příkazové řádky, nepřihlášený účet, neplatný nebo prošlý přístupový token, uzavřený či již sloučený pull request nebo odpověď, která si odporuje, zastaví práci hned a s původní zprávou. Takové chyby opakováním nezmizí, a nástroj je proto zbytečně neopakuje. Zprávy, které uživatel u ostatních příkazů pro práci s úkoly vidí, se nemění. Při přechodu na tuto verzi není třeba nic nastavovat. Změna se týká jen autorů rozšíření, která volají jádro nástroje přímo: chyby z GitHubu, GitLabu a Forgejo nyní nesou druh chyby místo prostého textu, takže taková rozšíření je třeba upravit, aby text chyby četla z této nové podoby. Rozšíření pro Linear a YouTrack se změna netýká. Closes #513 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude added 20 commits 2026-10-11 00:20:49 +00:00

workflow(wf): Create analysis for IW-513 ... bf1f2f767c

phase-pr cannot tell a forge that is down from a forge that refused, so both stop a batch

Artifacts:
- analysis.md
- issue.md

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018fPBZaKwU97SaaHzVo3ePq

docs(IW-513): correct the analysis where review found errors ... 1be07e60e7

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018fPBZaKwU97SaaHzVo3ePq

workflow(wf): Create task index for IW-513 ... 9555aaee76

phase-pr cannot tell a forge that is down from a forge that refused, so both stop a batch

Artifacts:
- tasks.md

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018fPBZaKwU97SaaHzVo3ePq

docs(IW-513): intake verdict — agent-ready ... 1ffea35292

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018fPBZaKwU97SaaHzVo3ePq

Merge remote-tracking branch 'origin/main' into IW-513 32eaef3a04

Phase 1: context ready ... 8ccf1894ea

Issue: IW-513
Phase: 1 - TrackerError on the forge clients and the port's PR/CI/merge calls

This commit marks the checkpoint for phase 1 context generation.
Use wf-implement IW-513 to return to this point.

The analysis is corrected where the context review found that gh's
"Could not resolve to a Repository" refusal and a bare 401 in a URL
would be classified as the wrong case.

🤖 Generated with Claude Code

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

workflow(wf): Generate Phase 1 tasks for IW-513 ... 149bdbb711

TrackerError on the forge clients and the port's PR/CI/merge calls

Artifacts:
- phase-01-tasks.md

🤖 Generated with Claude Code

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

Phase 1: TrackerError on the forge clients and the port's PR/CI/merge calls (#778) 8e9027770a

chore(IW-513): mark phase 01 complete in tasks.md 3d1f36d139

Phase 2: context ready ... 22e526f224

Issue: IW-513
Phase: 2 - Pre-gate budget, so a forge that will not answer before the gate resumes

This commit marks the checkpoint for phase 2 context generation.
Use wf-implement IW-513 to return to this point.

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

workflow(wf): Generate Phase 2 tasks for IW-513 ... 7898c0c875

Pre-gate budget, so a forge that will not answer before the gate resumes

Artifacts:
- phase-02-tasks.md

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

Phase 2: Pre-gate budget, so a forge that will not answer before the gate resumes (#779) d79d04cdc9

chore(IW-513): mark phase 02 complete in tasks.md 3d056c6ccd

Phase 3: context ready ... 2374522bf2

Issue: IW-513
Phase: 3 - TrackerError on the port's forge issue calls

This commit marks the checkpoint for phase 3 context generation.
Use wf-implement IW-513 to return to this point.

🤖 Generated with Claude Code

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

workflow(wf): Generate Phase 3 tasks for IW-513 ... b4cde3b33f

TrackerError on the port's forge issue calls

Artifacts:
- phase-03-tasks.md

🤖 Generated with Claude Code

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

Phase 3: TrackerError on the port's forge issue calls (#780) 5ec1ee0a14

chore(IW-513): mark phase 03 complete in tasks.md 6dc53d9914

chore(IW-513): file follow-ups (IW-781; comments on #569, #585, #762, #440) ... 7bd16df83e

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

docs(IW-513): review packet and release notes ... 0f18c73cd8

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

docs(IW-513): point review packet at the filed follow-ups ... 6c4ccb7bcb

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

claude referenced this pull request 2026-10-11 07:02:02 +00:00

Small fixes: core #762

mph merged commit 8ad228597e into main 2026-10-11 07:07:58 +00:00

mph referenced this pull request from a commit 2026-10-11 07:07:59 +00:00

Merge pull request 'IW-513: phase-pr tells a forge that is down from a forge that refused' (#782) from IW-513 into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

agent-ready

  

bug

Something isn't working

  

contract

Run the tool-contract suite on this PR

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

needs-human

  

needs-input

Intake asked the human a question on the issue; waiting for the answer

  

question

Further information is requested

  

wontfix

This will not be worked on

No labels

agent-ready

bug

contract

documentation

duplicate

enhancement

good first issue

help wanted

invalid

needs-human

needs-input

question

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

iterative-works/iw-cli!782

No description provided.