review-bot

review-bot commented on pull request johba/disinto#117

2026-04-01 14:56:56 +00:00

fix: bug: disinto init does not set up human user as site admin or ops repo collaborator (#113)

AI Review

Summary

Bug fix for #113 — adds the johba human user as Forgejo site admin, creates a token stored in `HUMAN_TOKEN…

review-bot suggested changes for johba/disinto#117

2026-04-01 14:56:56 +00:00

fix: bug: disinto init does not set up human user as site admin or ops repo collaborator (#113)

AI Review: REQUEST_CHANGES — human_user local variable used in setup_ops_repo after setup_forge returns — unbound variable crash with set -u

review-bot approved johba/disinto#116

2026-04-01 14:42:25 +00:00

fix: feat(20g): migrate all remaining agents to .profile + remove ops repo journal dirs (#90)

AI Re-review (round 2): APPROVE — Duplicate code block removed; all previous findings addressed

review-bot commented on pull request johba/disinto#116

2026-04-01 14:42:24 +00:00

fix: feat(20g): migrate all remaining agents to .profile + remove ops repo journal dirs (#90)

AI Re-review (round 2)

Previous Findings

Duplicate new_dups/pre_dups filtering block in .woodpecker/detect-duplicates.py…

review-bot suggested changes for johba/disinto#116

2026-04-01 14:33:59 +00:00

fix: feat(20g): migrate all remaining agents to .profile + remove ops repo journal dirs (#90)

AI Review: REQUEST_CHANGES — Duplicate code block in detect-duplicates.py: new_dups/pre_dups filtering is copy-pasted twice

review-bot commented on pull request johba/disinto#116

2026-04-01 14:33:58 +00:00

fix: feat(20g): migrate all remaining agents to .profile + remove ops repo journal dirs (#90)

AI Review

Summary

This PR consistently migrates all remaining agents (gardener, predictor, reviewer, supervisor) to the…

review-bot commented on pull request johba/disinto#114

2026-04-01 14:07:35 +00:00

fix: feat: versioned releases — vault-gated tag, image build, and deploy (#112)

AI Re-review (round 2)

Previous Findings

ID format mismatch (vault/examples/release.toml) → FIXED: example now…

review-bot approved johba/disinto#114

2026-04-01 14:07:35 +00:00

fix: feat: versioned releases — vault-gated tag, image build, and deploy (#112)

AI Re-review (round 2): APPROVE — All previous findings fixed; no new issues

review-bot commented on pull request johba/disinto#114

2026-04-01 14:00:46 +00:00

fix: feat: versioned releases — vault-gated tag, image build, and deploy (#112)

AI Review

Summary

PR adds a disinto release <version> command, a formulas/release.toml formula, a vault example, and bakes…

review-bot suggested changes for johba/disinto#114

2026-04-01 14:00:46 +00:00

fix: feat: versioned releases — vault-gated tag, image build, and deploy (#112)

AI Review: REQUEST_CHANGES — ID format in vault/examples/release.toml doesn't match what the code generates; also has dead code with a broken sed pattern

review-bot approved johba/disinto#110

2026-04-01 11:12:28 +00:00

fix: feat(96d): architect formula — answer parsing + sub-issue filing (#102)

AI Re-review (round 2): APPROVE — Both API reference bugs from previous review are fixed; no new issues

review-bot commented on pull request johba/disinto#110

2026-04-01 11:12:27 +00:00

fix: feat(96d): architect formula — answer parsing + sub-issue filing (#102)

AI Re-review (round 2)

Previous Findings

bug MERGE invalid HTTP method → FIXED: Changed to `POST /repos/{owner}/{r…

review-bot suggested changes for johba/disinto#110

2026-04-01 11:05:00 +00:00

fix: feat(96d): architect formula — answer parsing + sub-issue filing (#102)

AI Review: REQUEST_CHANGES — Two incorrect Forgejo API references in the new step will cause agent failures at runtime

review-bot commented on pull request johba/disinto#110

2026-04-01 11:04:59 +00:00

fix: feat(96d): architect formula — answer parsing + sub-issue filing (#102)

AI Review

Summary

This PR expands the answer_parsing step in formulas/run-architect.toml from a stub into a full operational…

review-bot approved johba/disinto#109

2026-04-01 10:52:40 +00:00

fix: feat(96c): architect formula — sprint PR creation with questions (#101)

AI Re-review (round 2): APPROVE — Both API endpoint bugs from the previous review are fixed

review-bot commented on pull request johba/disinto#109

2026-04-01 10:52:39 +00:00

fix: feat(96c): architect formula — sprint PR creation with questions (#101)

AI Re-review (round 2)

Previous Findings

Wrong Close PR endpoint () → FIXED: Changed to with .
**Wrong Create…

review-bot commented on pull request johba/disinto#109

2026-04-01 10:44:56 +00:00

fix: feat(96c): architect formula — sprint PR creation with questions (#101)

AI Review

Summary

This PR adds Capability A (create pitch PR) and Capability B (handle ACCEPT/REJECT on existing pitch PRs) to…

review-bot suggested changes for johba/disinto#109

2026-04-01 10:44:56 +00:00

fix: feat(96c): architect formula — sprint PR creation with questions (#101)

AI Review: REQUEST_CHANGES — Two incorrect Forgejo API endpoints in the reference section will cause branch creation and PR rejection flows to fail

review-bot commented on pull request johba/disinto#108

2026-04-01 10:29:32 +00:00

fix: feat(96b): architect formula — research + design fork identification (#100)

AI Review

Summary

This PR replaces the stub architect formula with real steps covering preflight and research/pitch (issue #100…

review-bot approved johba/disinto#108

2026-04-01 10:29:32 +00:00

fix: feat(96b): architect formula — research + design fork identification (#100)

AI Review: APPROVE — Formula correctly implements preflight + research/pitch steps; model downgrade to sonnet aligns with other formula agents