Repository navigation
feat(dev-lead): agent_ref input for pinned script checkout (#535) - #537
Conversation
…506) Backward-compatible foundation for dev-lead self-host pinning. Parameterises the 'Checkout dev-lead scripts' ref (was hardcoded 'main') via a new workflow_call input agent_ref (default 'main' → unchanged behaviour). A forthcoming trigger stub will pass agent_ref: dev-lead/stable so dev-lead's scripts run at the pinned channel. Part of #535 (mirrors the pr-review #506 fix). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
Warning Review limit reached
More reviews will be available in 23 minutes and 55 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |
There was a problem hiding this comment.
Pull request overview
This PR introduces a new workflow_call input (agent_ref) to dev-lead-reusable.yml so the workflow can check out dev-lead scripts from a caller-specified ref (e.g., dev-lead/stable) instead of always using main. This is the first backward-compatible step toward pinning dev-lead’s self-hosted execution to a stable channel and breaking the circular dependency described in #535 / #495.
Changes:
- Added
inputs.agent_ref(default:main) to the reusable workflow interface. - Parameterized the “Checkout dev-lead scripts” step to use
${{ inputs.agent_ref }}.
…506) (#537) Backward-compatible foundation for dev-lead self-host pinning. Parameterises the 'Checkout dev-lead scripts' ref (was hardcoded 'main') via a new workflow_call input agent_ref (default 'main' → unchanged behaviour). A forthcoming trigger stub will pass agent_ref: dev-lead/stable so dev-lead's scripts run at the pinned channel. Part of #535 (mirrors the pr-review #506 fix). Co-authored-by: donpetry-bot <{}+donpetry-bot@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…506) (#537) Backward-compatible foundation for dev-lead self-host pinning. Parameterises the 'Checkout dev-lead scripts' ref (was hardcoded 'main') via a new workflow_call input agent_ref (default 'main' → unchanged behaviour). A forthcoming trigger stub will pass agent_ref: dev-lead/stable so dev-lead's scripts run at the pinned channel. Part of #535 (mirrors the pr-review #506 fix). Co-authored-by: donpetry-bot <{}+donpetry-bot@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…506) (#537) Backward-compatible foundation for dev-lead self-host pinning. Parameterises the 'Checkout dev-lead scripts' ref (was hardcoded 'main') via a new workflow_call input agent_ref (default 'main' → unchanged behaviour). A forthcoming trigger stub will pass agent_ref: dev-lead/stable so dev-lead's scripts run at the pinned channel. Part of #535 (mirrors the pr-review #506 fix). Co-authored-by: donpetry-bot <{}+donpetry-bot@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…506) (#537) Backward-compatible foundation for dev-lead self-host pinning. Parameterises the 'Checkout dev-lead scripts' ref (was hardcoded 'main') via a new workflow_call input agent_ref (default 'main' → unchanged behaviour). A forthcoming trigger stub will pass agent_ref: dev-lead/stable so dev-lead's scripts run at the pinned channel. Part of #535 (mirrors the pr-review #506 fix). Co-authored-by: donpetry-bot <{}+donpetry-bot@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…506) (#537) Backward-compatible foundation for dev-lead self-host pinning. Parameterises the 'Checkout dev-lead scripts' ref (was hardcoded 'main') via a new workflow_call input agent_ref (default 'main' → unchanged behaviour). A forthcoming trigger stub will pass agent_ref: dev-lead/stable so dev-lead's scripts run at the pinned channel. Part of #535 (mirrors the pr-review #506 fix). Co-authored-by: donpetry-bot <{}+donpetry-bot@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>



Backward-compatible first increment of dev-lead self-host (#535). Parameterises the reusable's 'Checkout dev-lead scripts' ref (hardcoded
main) via a newworkflow_callinputagent_ref(defaultmain→ no behaviour change). Lets the forthcoming trigger stub passagent_ref: dev-lead/stableso dev-lead scripts run pinned.Next increments: cut
dev-lead/v1.1.0+ movedev-lead/stable; then convert inlinedev-lead.yml→ stub callingdev-lead-reusable.yml@dev-lead/stable(pending an inline-vs-reusable equivalence check — they're maintained in parallel per AGENTS.md). Part of #535 · #497 · epic #495.