Skip to content

XY-125: add blocker-aware dispatch claim policy#8

Merged
yvette-carlisle merged 4 commits intomainfrom
x/maestro-xy-125
Mar 16, 2026
Merged

XY-125: add blocker-aware dispatch claim policy#8
yvette-carlisle merged 4 commits intomainfrom
x/maestro-xy-125

Conversation

@yvette-carlisle
Copy link
Member

Summary

  • add blocker-aware Todo gating and project-slot claim checks to candidate selection
  • plumb Linear blocker data into tracker issue mapping and cover the new dispatch rules with tests
  • advance the split self-bootstrap plans and runtime spec to the post-XY-143 structural follow-up state

Verification

  • cargo make lint-fix
  • cargo make fmt
  • cargo make test
  • cargo make lint
  • cargo make fmt-rust-check
  • cargo make fmt-toml-check

…"add blocker-aware dispatch claim policy","intent":"align Maestro dispatch with XY-125 by enforcing blocker-aware Todo gating and project-local claim slot checks while advancing the structural follow-up plans","impact":"makes candidate selection skip blocked Todo issues and already-claimed project slots, and records that structural follow-ups are now active after daemon supervision closed","breaking":false,"risk":"medium","authority":"linear","delivery_mode":"status-only","refs":[{"system":"linear","id":"XY-125","role":"authority"}]}
@yvette-carlisle
Copy link
Member Author

@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: a1fb1b6010

ℹ️ 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".

…"fix review gaps in claim and blocker handling","intent":"address the PR #8 review findings by making project-slot claims atomic and by fetching additional blocker pages before evaluating Todo eligibility","impact":"prevents concurrent dispatch races from starting multiple project lanes and keeps Todo blocker checks accurate when Linear returns more than one blocker page","breaking":false,"risk":"medium","authority":"linear","delivery_mode":"status-only","refs":[{"system":"linear","id":"XY-125","role":"authority"}]}
@yvette-carlisle
Copy link
Member Author

@codex review

1 similar comment
@yvette-carlisle
Copy link
Member Author

@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: 597e602d89

ℹ️ 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".

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: 597e602d89

ℹ️ 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".

…"clear claimed leases on pre-run failures","intent":"address the latest PR #8 review finding by guaranteeing that any error after an atomic lease claim releases the project slot before returning","impact":"prevents transient tracker or workspace-setup failures from leaving stale issue_leases rows that block later dispatch attempts","breaking":false,"risk":"low","authority":"linear","delivery_mode":"status-only","refs":[{"system":"linear","id":"XY-125","role":"authority"}]}
…"separate post-claim eligibility from lease checks","intent":"address the latest PR #8 blocker by preventing claimed issues from failing their own post-refresh eligibility gate","impact":"keeps live dispatch running after an atomic lease claim while preserving pre-claim lease filtering for candidate selection","breaking":false,"risk":"low","authority":"linear","delivery_mode":"status-only","refs":[{"system":"linear","id":"XY-125","role":"authority"}]}
@yvette-carlisle
Copy link
Member Author

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Breezy!

ℹ️ 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".

@yvette-carlisle yvette-carlisle merged commit 7c56f6a into main Mar 16, 2026
7 checks passed
@yvette-carlisle yvette-carlisle deleted the x/maestro-xy-125 branch March 16, 2026 16:34
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.

1 participant