Skip to content

fix: preserve aggregation throttles with sparse work discovery - #542

Merged
guangyu-reflexio merged 2 commits into
mainfrom
team/aggregation-sparse-inventory
Sep 26, 2026
Merged

guangyu-reflexio merged 2 commits into
mainfrom
team/aggregation-sparse-inventory

Conversation

@guangyu-reflexio

Copy link
Copy Markdown
Contributor

Sparse work discovery omits live projects without due work. Treating that result as the complete fleet erases those projects' retry and repair throttles.

Add an optional complete-scope inventory provider for pruning. An unavailable, failed or shutdown-interrupted inventory preserves state; existing full-sweep callers keep their behavior. Document the contract.

Validation: 22 scheduler tests passed, Ruff passed, Pyright zero errors. Independent correctness, resilience and architecture reviews found a shutdown race and stale docstring; both fixed and verified.

@coderabbitai

coderabbitai Bot commented Sep 26, 2026

Copy link
Copy Markdown

Warning

Review paused — included plan limit reached

Keep your review moving with free on-demand reviews.

  • Run this review for free

On-demand reviews are free for the next 13 days.

  • Ask an admin to make reviews automatic

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Promotion and pricing details

On-demand reviews are free for the next 13 days. After that, they cost $0.25 per reviewed file.

Review limit details

Or wait 50 minutes for your next included review.

Check out review usage here.

Limit details: You’ve used all 2 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: ReflexioAI/reflexio/.coderabbit.yaml

Review profile: CHILL

Plan: Essentials

Run ID: f8332f27-86a8-4747-8f46-7e4316c3200d

📥 Commits

Reviewing files that changed from the base of the PR and between 5c178e8 and 020d008.

📒 Files selected for processing (3)
  • developer.md
  • reflexio/server/services/playbook/aggregation_scheduler.py
  • tests/server/services/playbook/test_aggregation_scheduler.py

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

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-26T22:14:19.387281Z 020d008 Manual request
🔒 Security Review ✅ Completed 2026-09-26T22:12:59.511312Z 020d008 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@guangyu-reflexio

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

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

Reviewed commit: 020d0084c2

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

@guangyu-reflexio
guangyu-reflexio merged commit 0209414 into main Sep 26, 2026
5 checks passed
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