Skip to content

chore(ci): remove vulnerability-scan, add security-slack-notify#1579

Merged
PMerlet merged 2 commits into
mainfrom
chore/remove-security-workflows
May 13, 2026
Merged

chore(ci): remove vulnerability-scan, add security-slack-notify#1579
PMerlet merged 2 commits into
mainfrom
chore/remove-security-workflows

Conversation

@PMerlet
Copy link
Copy Markdown
Member

@PMerlet PMerlet commented May 12, 2026

Summary

  • Removes `vulnerability-scan.yml` (weekly Trivy scan + Slack post). Security vulnerabilities are now handled by a Claude routine running weekly.
  • Adds `security-slack-notify.yml` — thin caller workflow that posts to #tech_all when a PR is labeled `:lock: security`. Mirrors what is already in place on forestadmin-server.

Test plan

  • End-to-end Slack notification flow verified in forestadmin-server (test PR closed)
  • After merge, label a PR with `:lock: security` to confirm the workflow fires in this repo too

Note: needs the org-level `SLACK_BOT_TOKEN` secret to be accessible by this repo, and the corresponding Slack bot to be a member of #tech_all.

🤖 Generated with Claude Code

Note

Replace vulnerability-scan workflow with security PR Slack notification workflow

  • Removes vulnerability-scan.yml, which ran a weekly scheduled scan every Friday and supported manual dispatch.
  • Adds security-slack-notify.yml, a workflow that triggers when a PR is labeled :lock: security and posts a Slack notification via a reusable notify-slack-security-pr workflow with PR metadata and repository name.
  • Risk: the weekly vulnerability scan will no longer run automatically or send its associated notifications.

Macroscope summarized 728d863.

PMerlet and others added 2 commits May 12, 2026 15:51
Security vulnerability fixes are now handled by a Claude routine on a
weekly basis.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Thin caller — delegates to forestadmin/.github reusable workflow.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@qltysh
Copy link
Copy Markdown

qltysh Bot commented May 12, 2026

Qlty


Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@PMerlet PMerlet merged commit 1a6b60d into main May 13, 2026
29 checks passed
@PMerlet PMerlet deleted the chore/remove-security-workflows branch May 13, 2026 10:08
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.

1 participant