Skip to content

XY-174 re-enter retained review-repair lanes#23

Open
yvette-carlisle wants to merge 3 commits intomainfrom
x/maestro-xy-174
Open

XY-174 re-enter retained review-repair lanes#23
yvette-carlisle wants to merge 3 commits intomainfrom
x/maestro-xy-174

Conversation

@yvette-carlisle
Copy link
Member

Summary

  • add a dedicated ReviewRepair dispatch mode for retained post-review lanes
  • reuse the existing workspace and PR lineage with explicit issue_review_repair_complete + review_repair terminal finalize
  • refresh the local review-handoff marker after a repaired head and document the retained-lane contract

Verification

  • cargo make fmt-check
  • cargo make lint
  • cargo make test
  • python3 /Users/xavier/.codex/skills/plan-writing/scripts/validate_plan_contract.py --path docs/plans/2026-03-21_maestro-xy-174-review-repair-reentry.json
  • python3 /Users/xavier/.codex/skills/plan-execution/scripts/read_plan_contract.py --path docs/plans/2026-03-21_maestro-xy-174-review-repair-reentry.json
  • git diff --check

…:"add retained review-repair re-entry","intent":"resume owned in-review lanes without human relay","impact":"maestro can select retained review-repair lanes, run bounded repair in the same workspace, refresh review lineage, and request review again","breaking":false,"risk":"medium","authority":"linear","delivery_mode":"status-only","refs":[{"system":"linear","id":"XY-174","role":"authority"}]}
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@aurexav
Copy link
Member

aurexav commented Mar 21, 2026

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 92cbe4bdd9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

…"tighten retained review-repair handoff boundaries","intent":"close review-repair contract gaps before re-requesting review on PR #23","impact":"retained repair lanes no longer expose issue_transition, only refresh review lineage after authoritative comment writeback succeeds, and continuation prompts honor the configured success state","breaking":false,"risk":"medium","authority":"linear","delivery_mode":"status-only","refs":[{"system":"linear","id":"XY-174","role":"authority"},{"system":"github","repo":"hack-ink/maestro","number":23,"role":"mirror"}]}
@yvette-carlisle
Copy link
Member Author

@codex review

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@aurexav
Copy link
Member

aurexav commented Mar 21, 2026

@codex review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: eaec9c6267

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

…"harden retained review-repair recovery and backoff dispatch","intent":"close the remaining PR #23 review findings by recovering active retained review-repair lanes after restart and honoring retry backoff before redispatch","impact":"daemon recovery now rebuilds active review-repair leases, review-repair failures retain their backoff window, and targeted regressions cover both paths","breaking":false,"risk":"medium","authority":"linear","delivery_mode":"status-only","refs":[{"system":"linear","id":"XY-174","role":"authority"},{"system":"github","repo":"hack-ink/maestro","number":23,"role":"mirror"}]}
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.

2 participants