Conversation
Automated sync from stranske/Workflows Template hash: 76689bc445fd Changes synced from sync-manifest.yml
|
Warning Review limit reached
More reviews will be available in 2 seconds. Learn how PR review limits work. To continue reviewing without waiting, enable usage-based billing in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits. 🚦 How do rate limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e5801c604e
ℹ️ 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".
| raise ValueError(f"orchestrator skill reference pack not found: {plan.pack}") | ||
| checkout_path = workspace_path / matching[0].checkout_path | ||
| with contextlib.suppress(FileNotFoundError): | ||
| shutil.rmtree(checkout_path) |
There was a problem hiding this comment.
Reject dot-only reference packs before deleting
In a repo whose .github/reference_packs.json defines a pack named . or .. and .github/orchestrator_skill.json selects that pack, the existing pack-name regex allows it, so matching[0].checkout_path becomes .reference/. or .reference/... This newly added rmtree then deletes the whole .reference directory, or walks up and removes the workspace contents before suppressing the resulting FileNotFoundError; reject dot-only names or resolve/assert the path stays below a safe .reference/<name> directory before deleting.
Useful? React with 👍 / 👎.
|
Closing as stale: newer replacement sync PR #5617 exists from Workflows sync wave sync/workflows-591316374281 after stranske/Workflows#2498 merged. |
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Source SHA:
0b04de717dcadc23aea9e2eca0b8679d27e90666Template hash:
76689bc445fdSync branch:
sync/workflows-76689bc445fdConsumer repo:
stranske/Trend_Model_ProjectManifest:
.github/sync-manifest.yml