chore(deps): update stranske/workflows digest to c2537cc - #349
chore(deps): update stranske/workflows digest to c2537cc#349renovate[bot] wants to merge 1 commit into
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThe pinned commit SHA for the ChangesAction SHA bump
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Workflow state fingerprint for Keepalive Loop Reporter. Do not edit. |
|
Workflow state fingerprint for Agents Gate Followups. Do not edit. |
Workflow source neededPR #349 needs either a linked GitHub issue or one valid non-issue Workflow Source before PR metadata automation can manage it safely. Please do one of:
Once a valid source is present, this warning will not be reposted. |
9e8f891 to
ed2ddb5
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/agents-guard.yml:
- Line 114: The external action reference
stranske/Workflows/.github/actions/setup-api-client is using an exact commit SHA
pin instead of `@main`, which prevents automatic adoption of future patches.
Replace the commit SHA reference (c2537cc959f2ce05926c4639d25b90678abc97bc) with
`@main` for both occurrences of the setup-api-client action usage. This allows the
action to automatically receive backward-compatible updates without requiring
manual updates in the future.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: fd3c0d1d-8eff-42d3-86c5-9fce0e56dfcd
📒 Files selected for processing (1)
.github/workflows/agents-guard.yml
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
stranske/Workflows(auto-detected)
📜 Review details
🧰 Additional context used
📓 Path-based instructions (4)
{pyproject.toml,.github/workflows/*.{yml,yaml}}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
{pyproject.toml,.github/workflows/*.{yml,yaml}}: For Python projects with coverage thresholds, ensure bothpyproject.toml([tool.coverage.report] fail_under) and workflow files have matchingcoverage-minsettings, or the lower one will determine the actual threshold
The Manager-Database repository has a coverage threshold of 75%
Files:
.github/workflows/agents-guard.yml
.github/workflows/*.{yml,yaml}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
For GitHub Actions workflow
startup_failureerrors, check for invalid YAML syntax, invalidpermissions:block inworkflow_callreusable workflows, invalid permission scopes, or circular workflow references
Files:
.github/workflows/agents-guard.yml
.github/workflows/**/*.{yml,yaml}
📄 CodeRabbit inference engine (CLAUDE.md)
Reference reusable workflows in consumer repos with
@mainunless intentionally pinning to an exact commit SHA for a controlled reason.
Files:
.github/workflows/agents-guard.yml
.github/workflows/*.yml
📄 CodeRabbit inference engine (AGENTS.md)
First-party consumers should reference reusable workflows with
@mainunless intentionally pinning to an exact commit SHA for a controlled reason
Files:
.github/workflows/agents-guard.yml
🔀 Multi-repo context stranske/Workflows
Based on the previous comprehensive research already conducted, here are the cross-repository findings for this PR:
Linked Repositories Findings
stranske/Workflows [::stranske/Workflows::]
Action Interface Stability:
The setup-api-client action maintains a stable contract across the update from d68de19 to c2537cc:
- Inputs: Multiple optional parameters (
secrets,github_token, and fallback individual tokens) remain unchanged - Outputs: Three stable outputs (
token_count,available_tokens,setup_contract) are unmodified - No changes to the action interface or contract between commits [::stranske/Workflows::]
Release Content (v1.15.2):
The changelog confirms this is a patch-level update containing only:
- A single bug fix: "guard runtime-ac external merge lanes"
- Release date: 2026-06-19 [::stranske/Workflows::]
No Breaking Changes:
The repository's compatibility policy (documented in COMPATIBILITY.md) requires marking breaking changes with a BREAKING prefix in the CHANGELOG. The v1.15.2 release contains no such entries, confirming backward compatibility [::stranske/Workflows::]
Consistent Usage Pattern:
The invocation in .github/workflows/agents-guard.yml follows the standard pattern used across all 73 workflows in the repository:
uses: stranske/Workflows/.github/actions/setup-api-client@c2537cc
with:
secrets: ${{ toJSON(secrets) }}
github_token: ${{ secrets.GITHUB_TOKEN }}This pattern remains fully compatible with the updated action [::stranske/Workflows::]
Assessment: The update from d68de19 to c2537cc is a safe patch-level change with no API modifications, breaking changes, or interface alterations. All dependent workflows will continue to function without modification.
| steps.eligibility.outputs.should-run == 'true' && | ||
| steps.api_client_base.outputs.available != 'true' | ||
| uses: "stranske/Workflows/.github/actions/setup-api-client@d68de1904bcdbe16bfe2462b73aa18f41f8a0a47" # v1 | ||
| uses: "stranske/Workflows/.github/actions/setup-api-client@c2537cc959f2ce05926c4639d25b90678abc97bc" # v1 |
There was a problem hiding this comment.
Use @main for the external action reference unless there's a documented controlled reason for the commit SHA pinning.
Both lines 114 and 183 reference the external stranske/Workflows/.github/actions/setup-api-client action using an exact commit SHA (c2537cc959...). According to the coding guidelines, first-party consumers should reference reusable workflows with @main unless intentionally pinning to a specific commit for a documented, controlled reason.
The linked repository findings confirm this update (v1.15.2) is backward-compatible and safe, so using @main would allow future patches to be adopted automatically without manual updates.
If commit SHA pinning is intentional (e.g., for reproducibility or a specific Renovate strategy), add an inline comment documenting the controlled reason.
🔧 Recommended fix to use `@main`
- uses: "stranske/Workflows/.github/actions/setup-api-client@c2537cc959f2ce05926c4639d25b90678abc97bc" # v1
+ uses: stranske/Workflows/.github/actions/setup-api-client@mainApply this change to both lines 114 and 183.
Also applies to: 183-183
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.github/workflows/agents-guard.yml at line 114, The external action
reference stranske/Workflows/.github/actions/setup-api-client is using an exact
commit SHA pin instead of `@main`, which prevents automatic adoption of future
patches. Replace the commit SHA reference
(c2537cc959f2ce05926c4639d25b90678abc97bc) with `@main` for both occurrences of
the setup-api-client action usage. This allows the action to automatically
receive backward-compatible updates without requiring manual updates in the
future.
Source: Coding guidelines
fea3354 to
3138f59
Compare
3138f59 to
cfbea1f
Compare
|
Superseded by the Workflows source-of-truth path: stranske/Workflows#2472 and stranske/Workflows#2498 have merged, and replacement sync PR #357 now carries this generated workflow update without the consumer-local synced-file review debt. |
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for the If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
This PR contains the following updates:
d68de19→c2537ccConfiguration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.