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:
📝 WalkthroughWalkthroughAdds two Copilot-driven GitHub Actions workflows and supporting artifacts: a "Mention in Issue by ID" reusable workflow (workflow_call agent, trigger example, README, and example workflow for manual dispatch) that can comment on or create PRs for a targeted issue; and a "PR Labeler" reusable workflow (agent, trigger, README, and examples) that evaluates PRs and manages a single label from a configured set. Also adds a workflow file to trigger the mention-by-ID flow, updates actions lockfile to include actions/github-script v7, and forwards COPILOT_GITHUB_TOKEN and EXTRA_COMMIT_GITHUB_TOKEN where applicable. Possibly related PRs
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.github/workflows/gh-aw-pr-labeler.lock.yml (1)
466-469: Avoidevalforsetup-commands; run viabash -cwith strict flags.Line 469 uses
eval, which adds a second expansion pass and can cause hard-to-debug command behavior.Suggested hardening
- env: SETUP_COMMANDS: ${{ inputs.setup-commands }} if: ${{ inputs.setup-commands != '' }} name: Repo-specific setup - run: eval "$SETUP_COMMANDS" + shell: bash + run: | + set -euo pipefail + bash -euo pipefail -c "$SETUP_COMMANDS"🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/workflows/gh-aw-pr-labeler.lock.yml around lines 466 - 469, Replace the unsafe eval usage for the Repo-specific setup step: instead of running eval "$SETUP_COMMANDS", invoke the shell explicitly and enable strict flags, e.g. run the commands via bash -c with set -euo pipefail so SETUP_COMMANDS executes in a single expansion pass under strict error handling; update the step that references SETUP_COMMANDS and the run line to use an explicit bash invocation (e.g. bash -lc "set -euo pipefail; $SETUP_COMMANDS") to harden execution.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.github/workflows/gh-aw-pr-labeler.lock.yml:
- Around line 466-469: Replace the unsafe eval usage for the Repo-specific setup
step: instead of running eval "$SETUP_COMMANDS", invoke the shell explicitly and
enable strict flags, e.g. run the commands via bash -c with set -euo pipefail so
SETUP_COMMANDS executes in a single expansion pass under strict error handling;
update the step that references SETUP_COMMANDS and the run line to use an
explicit bash invocation (e.g. bash -lc "set -euo pipefail; $SETUP_COMMANDS") to
harden execution.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 9df429c5-a059-4668-8360-b62d7d13ff6a
📒 Files selected for processing (2)
.github/workflows/gh-aw-pr-labeler.lock.yml.github/workflows/gh-aw-pr-labeler.md
Summary
Mention in Issue by ID, to run the agent against a specific issue number.workflow_dispatchexample, and a repository-local trigger workflow for that issue-targeted assistant.PR Labelerreusable workflow, plus trigger/docs/example, for PR classification labels.actions/github-script@v7in.github/aw/actions-lock.jsonfor PR-labeler label-operation pre-sanitization.Mention in Issue by ID workflow
.github/workflows/gh-aw-mention-in-issue-by-id.md.github/workflows/gh-aw-mention-in-issue-by-id.lock.yml.github/workflows/trigger-mention-in-issue-by-id.ymlgh-agent-workflows/mention-in-issue-by-id/README.mdgh-agent-workflows/mention-in-issue-by-id/example.ymlBehavior
target-issue-numberandprompt(plus optionalmodel,additional-instructions,setup-commands,messages-footer, anddraft-prs).PR Labeler
.github/workflows/gh-aw-pr-labeler.md.github/workflows/gh-aw-pr-labeler.lock.yml.github/workflows/trigger-pr-labeler.ymlgh-agent-workflows/pr-labeler/README.mdgh-agent-workflows/pr-labeler/example.ymlclassification-labelsas a configurable input in the reusable workflow.classification-labelsassmall_boom,medium_boom,big_boomand provides a risk rubric viaadditional-instructions.The body of this PR is automatically managed by the Trigger Update PR Body workflow.
The body of this PR is automatically managed by the Trigger Update PR Body workflow.