chore: sync workflow templates - #5578
Conversation
Automated sync from stranske/Workflows Template hash: 94b2d167cb49 Changes synced from sync-manifest.yml
📝 WalkthroughWalkthroughTwo GitHub Actions workflows ( ChangesNode.js version bump to 24
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 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 |
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.github/workflows/agents-auto-pilot.yml (1)
162-166: 🧹 Nitpick | 🔵 TrivialInconsistent Node.js versions across workflows.
Node.js 24 is stable and available as of June 2026 (released May 6, 2025, currently in Active LTS). However,
pr-12-playwright.yml:45still uses Node.js20, whileagents-auto-pilot.yml:166andagents-weekly-metrics.yml:50both use Node.js24. Consider standardizing Node.js versions across all workflows.🤖 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-auto-pilot.yml around lines 162 - 166, Standardize Node.js versions across all workflows to ensure consistency. The "Set up Node" step should use the same node-version value in all workflow files. Currently, pr-12-playwright.yml uses Node.js 20 while agents-auto-pilot.yml and agents-weekly-metrics.yml use Node.js 24. Update the node-version in the "Set up Node" step in pr-12-playwright.yml to use '24' to match the other workflows and maintain consistency across the repository.
🤖 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.
Outside diff comments:
In @.github/workflows/agents-auto-pilot.yml:
- Around line 162-166: Standardize Node.js versions across all workflows to
ensure consistency. The "Set up Node" step should use the same node-version
value in all workflow files. Currently, pr-12-playwright.yml uses Node.js 20
while agents-auto-pilot.yml and agents-weekly-metrics.yml use Node.js 24. Update
the node-version in the "Set up Node" step in pr-12-playwright.yml to use '24'
to match the other workflows and maintain consistency across the repository.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 98322093-87c7-4d58-bf1d-2a31a1e84dd2
📒 Files selected for processing (2)
.github/workflows/agents-auto-pilot.yml.github/workflows/agents-weekly-metrics.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 of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: claude-review
- GitHub Check: Resolve review target
🧰 Additional context used
📓 Path-based instructions (4)
{pyproject.toml,.github/workflows/**/*.{yml,yaml}}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Check both
pyproject.toml([tool.coverage.report] fail_under) AND workflow files forcoverage-minsettings - these must match or the lower one wins
Files:
.github/workflows/agents-weekly-metrics.yml.github/workflows/agents-auto-pilot.yml
.github/workflows/**/*.{yml,yaml}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
For
startup_failureGitHub Actions workflow errors with zero jobs, check for common causes: Invalid YAML syntax, top-levelpermissions:block onworkflow_callreusable workflows (conflicts with caller permissions), invalid permission scopes, or circular workflow references
Files:
.github/workflows/agents-weekly-metrics.yml.github/workflows/agents-auto-pilot.yml
.github/workflows/**/*.yml
📄 CodeRabbit inference engine (CLAUDE.md)
Reference reusable workflows with
@mainby default in first-party consumers, unless intentionally pinning to an exact commit SHA for a controlled reason.
Files:
.github/workflows/agents-weekly-metrics.yml.github/workflows/agents-auto-pilot.yml
.github/workflows/agents-*.yml
📄 CodeRabbit inference engine (CLAUDE.md)
Agent workflow files (
agents-*.yml) should be fixed instranske/Workflows, not in this consumer repository.
Files:
.github/workflows/agents-weekly-metrics.yml.github/workflows/agents-auto-pilot.yml
🔇 Additional comments (1)
.github/workflows/agents-weekly-metrics.yml (1)
47-50: Same Node.js version update as agents-auto-pilot.yml.This workflow also updates from Node.js
20to24. The verification request and consistency concerns raised for.github/workflows/agents-auto-pilot.yml(line 166) apply equally here.
Sync Summary
Files Updated
Files Skipped
Review Checklist
Source: stranske/Workflows
Source SHA:
b4f41c99919965c91d9f4f42460cc21bb35bd24aTemplate hash:
94b2d167cb49Sync branch:
sync/workflows-94b2d167cb49Consumer repo:
stranske/Trend_Model_ProjectManifest:
.github/sync-manifest.ymlSummary by CodeRabbit