Add workflow to route issues to team board#58
Conversation
This workflow routes issues to the team board based on field changes.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughAdds a new GitHub Actions workflow that triggers on issue field changes ( ChangesIssue routing workflow
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Secret Leak CheckOK No secrets detected in this PR's diff. |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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/route-issue.yml:
- Line 13: Replace the blanket "secrets: inherit" usage in the reusable workflow
call with an explicit secrets mapping that only passes OPS_DISPATCH_CLIENT_ID
and OPS_DISPATCH_APP_PRIVATE_KEY from the repository secrets into the reusable
workflow; remove "secrets: inherit" and add a secrets block that maps each
required secret name to the corresponding repository secret to enforce
least-privilege.
- Line 12: Replace the mutable tag reference in the workflow invocation string
"uses: geolonia/.github/.github/workflows/reusable-route-issue.yml@v1" with the
exact commit SHA recommended by static analysis (e.g., change "`@v1`" to
"@<commit-sha>") so the reusable workflow is pinned to an immutable commit;
update the single line that contains that uses: entry and commit the change.
🪄 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: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: eabd4543-c1e5-448f-8224-05c4694f3dc3
📒 Files selected for processing (1)
.github/workflows/route-issue.yml
| jobs: | ||
| route: | ||
| uses: geolonia/.github/.github/workflows/reusable-route-issue.yml@v1 | ||
| secrets: inherit |
There was a problem hiding this comment.
🧹 Nitpick | 🔵 Trivial | 💤 Low value
Consider explicit secret passing for improved security posture.
While secrets: inherit is convenient and works correctly for this workflow, it passes all repository secrets to the reusable workflow. The reusable workflow requires only OPS_DISPATCH_CLIENT_ID and OPS_DISPATCH_APP_PRIVATE_KEY secrets.
For better least-privilege security, consider explicitly listing only the required secrets:
🛡️ Optional improvement for explicit secret passing
- secrets: inherit
+ secrets:
+ OPS_DISPATCH_CLIENT_ID: ${{ secrets.OPS_DISPATCH_CLIENT_ID }}
+ OPS_DISPATCH_APP_PRIVATE_KEY: ${{ secrets.OPS_DISPATCH_APP_PRIVATE_KEY }}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| secrets: inherit | |
| secrets: | |
| OPS_DISPATCH_CLIENT_ID: ${{ secrets.OPS_DISPATCH_CLIENT_ID }} | |
| OPS_DISPATCH_APP_PRIVATE_KEY: ${{ secrets.OPS_DISPATCH_APP_PRIVATE_KEY }} |
🤖 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/route-issue.yml at line 13, Replace the blanket "secrets:
inherit" usage in the reusable workflow call with an explicit secrets mapping
that only passes OPS_DISPATCH_CLIENT_ID and OPS_DISPATCH_APP_PRIVATE_KEY from
the repository secrets into the reusable workflow; remove "secrets: inherit" and
add a secrets block that maps each required secret name to the corresponding
repository secret to enforce least-privilege.
Source: Linters/SAST tools
Updated the reusable workflow reference to a specific commit hash for version 1.16.0.
Secret Leak CheckOK No secrets detected in this PR's diff. |
This workflow routes issues to the team board based on field changes.
Summary by CodeRabbit