chore(deps): update stranske/workflows digest to a525e6e - #626
chore(deps): update stranske/workflows digest to a525e6e#626renovate[bot] wants to merge 1 commit into
Conversation
📝 WalkthroughWalkthroughThe workflow updates the pinned ChangesWorkflow fallback pin update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Workflow state fingerprint for Agents Gate Followups. Do not edit. |
Workflow source neededPR #626 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. |
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 fallback SHA pin in the synced agents workflow should not be
updated only in this consumer repo. Revert or leave the local `agents-guard.yml`
pin as-is here, and first make the corresponding change in the upstream
`stranske/Workflows` source workflow so the synced `setup-api-client` reference
stays aligned. Check the matching fallback steps in `agents-guard.yml` and keep
the consumer file consistent with the upstream `agents-*.yml` workflow pattern.
🪄 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: 6e8e43c4-0517-4ee5-9c5d-9c4c71ec53b8
📒 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
⏰ Context from checks skipped due to timeout. (5)
- GitHub Check: classify changed paths
- GitHub Check: one-pdf-pilot-golden
- GitHub Check: replay-gate
- GitHub Check: foundation-fixture-e2e
- GitHub Check: postgres-integration
🧰 Additional context used
📓 Path-based instructions (2)
**/.github/workflows/*.yml
📄 CodeRabbit inference engine (CLAUDE.md)
Reference reusable workflows from
stranske/Workflowswith@mainunless intentionally pinning to an exact commit SHA for a controlled reason.Reference reusable workflows with
@mainunless intentionally pinning to an exact commit SHA for a controlled reason.
Files:
.github/workflows/agents-guard.yml
{**/.github/workflows/agents-*.yml,**/.github/workflows/autofix.yml,**/.github/codex/**}
📄 CodeRabbit inference engine (CLAUDE.md)
For synced workflows, prompts, scripts, and consumer docs (e.g.,
agents-*.yml,autofix.yml,.github/codex/prompts) — fix them instranske/Workflowsfirst, not locally in the consumer repo.
Files:
.github/workflows/agents-guard.yml
🔀 Multi-repo context stranske/Workflows
Linked repositories findings
stranske/Workflows [::stranske/Workflows::]
.github/workflows/agents-guard.yml:103and:173— the fallbackuses:pinsstranske/Workflows/.github/actions/setup-api-clientto6c3391d38bbc20a4577ac42b9aa6c9dc4e462c09. This PR only updates that digest; the surrounding step inputs staysecrets: ${{ toJSON(secrets) }}andgithub_token: ${{ github.token }}..github/actions/setup-api-client/action.yml— the shared action contract is broad but unchanged in the inspected file: it still exposes the same inputs (secrets,github_token, PAT/App secrets,install_dir,skip_deps,verbose) and outputs (token_count,available_tokens,setup_contract).tests/workflows/test_setup_api_client_contract.py:1-20— the contract test asserts thesetup_contractoutput wiring and key script markers (workflows-api-client-setup/v1,available_token_names,auth_modes,dependency_state, etc.), indicating the action’s public shape is expected to remain stable across revisions..github/scripts/__tests__/agents-guard.test.js:1-160— guard tests focus on label/approval behavior, not on the pinned fallback action version, so the SHA bump is not covered as a behavior change.docs/WORKFLOW_GUIDE.md:106— docs describeagents-guard.ymlas relying on the shared API client/token balancer, which matches the current workflow structure and suggests this update is an implementation refresh rather than a contract change.
| steps.eligibility.outputs.should-run == 'true' && | ||
| steps.api_client_base.outputs.available != 'true' | ||
| uses: "stranske/Workflows/.github/actions/setup-api-client@6c3391d38bbc20a4577ac42b9aa6c9dc4e462c09" # v1 | ||
| uses: "stranske/Workflows/.github/actions/setup-api-client@a525e6e3e2431d302073de65723c6e022f4b02fa" # v1 |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟠 Major | 🏗️ Heavy lift
Update this pin in stranske/Workflows first.
This file matches the synced agents-*.yml workflow pattern, so bumping the fallback SHA only in the consumer repo creates drift from the source workflow. The linked upstream stranske/Workflows/.github/workflows/agents-guard.yml still carries the old 6c3391d... pin at the corresponding fallback steps, so the next sync can overwrite this change. As per coding guidelines, “For synced workflows, prompts, scripts, and consumer docs (e.g., agents-*.yml, autofix.yml, .github/codex/ prompts) — fix them in stranske/Workflows first, not locally in the consumer repo.”
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 fallback SHA pin in the
synced agents workflow should not be updated only in this consumer repo. Revert
or leave the local `agents-guard.yml` pin as-is here, and first make the
corresponding change in the upstream `stranske/Workflows` source workflow so the
synced `setup-api-client` reference stays aligned. Check the matching fallback
steps in `agents-guard.yml` and keep the consumer file consistent with the
upstream `agents-*.yml` workflow pattern.
Sources: Coding guidelines, Linked repositories
|
Workflow state fingerprint for Keepalive Loop Reporter. Do not edit. |
|
Superseded by merged Workflows source updates and replacement sync PR #627 from wave sync/workflows-57b9e3dbcb23. Keeping the generated sync PR as the source-of-truth propagation path. |
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:
6c3391d→a525e6eConfiguration
📅 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.