Skip to content

feat: add agent-shield.yml workflow for compliance#98

Merged
don-petry merged 1 commit intomainfrom
claude/issue-40-20260408-2242
Apr 8, 2026
Merged

feat: add agent-shield.yml workflow for compliance#98
don-petry merged 1 commit intomainfrom
claude/issue-40-20260408-2242

Conversation

@don-petry
Copy link
Copy Markdown
Contributor

@don-petry don-petry commented Apr 8, 2026

Summary

  • Adds .github/workflows/agent-shield.yml — a thin caller stub for the org-level reusable agent-shield-reusable.yml@v1
  • File copied verbatim from standards/workflows/agent-shield.yml per the standards-conformance rules
  • Resolves the compliance finding from the weekly audit

Closes #40

Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Added automated workflow integration to improve code quality and security checks on pull requests and pushes to the main branch.

Copilot AI review requested due to automatic review settings April 8, 2026 22:43
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 617ebc26-bb14-4ccd-97a6-809a5afb60e5

📥 Commits

Reviewing files that changed from the base of the PR and between b1e410a and b6ea6b0.

📒 Files selected for processing (1)
  • .github/workflows/agent-shield.yml

📝 Walkthrough

Walkthrough

Added a new GitHub Actions workflow file that implements the AgentShield compliance check by delegating to an organization-level reusable workflow. The workflow triggers on push and pull_request events to the main branch and operates with read-only permissions.

Changes

Cohort / File(s) Summary
AgentShield Workflow
.github/workflows/agent-shield.yml
New workflow file that delegates to the org-level AgentShield reusable workflow (petry-projects/.github/.github/workflows/agent-shield-reusable.yml@v1) with triggers for main branch push and pull_request events.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: add agent-shield.yml workflow for compliance' clearly describes the main change: adding a new workflow file for compliance purposes.
Linked Issues check ✅ Passed The pull request successfully addresses issue #40 by adding the missing agent-shield.yml workflow file as required by the organization's compliance standards.
Out of Scope Changes check ✅ Passed All changes are within scope: only the required agent-shield.yml workflow file was added to resolve the compliance finding with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-40-20260408-2242

Comment @coderabbitai help to get the list of available commands and usage tips.

@don-petry
Copy link
Copy Markdown
Contributor Author

@don-petry — CI is running. This PR adds .github/workflows/agent-shield.yml (verbatim copy of the standards template) to resolve the compliance finding. Please review and merge when CI is green.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds the required AgentShield workflow stub to bring the repo into compliance with the org CI standards and resolve the “missing-agent-shield.yml” audit finding (Closes #40).

Changes:

  • Added .github/workflows/agent-shield.yml thin caller workflow.
  • Workflow delegates to the org-level reusable agent-shield-reusable.yml@v1 and triggers on push/pull_request to main.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Closes #40

Co-authored-by: don-petry <don-petry@users.noreply.github.com>
@don-petry don-petry force-pushed the claude/issue-40-20260408-2242 branch from 40e15fe to b6ea6b0 Compare April 8, 2026 22:56
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 8, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarQube Cloud

@don-petry don-petry merged commit 6ff2a91 into main Apr 8, 2026
18 of 19 checks passed
@don-petry don-petry deleted the claude/issue-40-20260408-2242 branch April 8, 2026 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compliance: missing-agent-shield.yml

2 participants