Skip to content

ci: run composer audit instead of roave/security-advisories - #23

Merged
roxblnfk merged 1 commit into
1.xfrom
ci/security-audit
Oct 9, 2026
Merged

roxblnfk merged 1 commit into
1.xfrom
ci/security-audit

Conversation

@roxblnfk

@roxblnfk roxblnfk commented Oct 9, 2026

Copy link
Copy Markdown
Member
Q A
Bugfix? ❌
Breaks BC? ❌
New feature? ❌
Issues —
  • The security workflow now runs composer audit on PHP 8.2 with the highest dependencies instead of requiring roave/security-advisories:dev-latest at CI time.
  • The workflow follows the shared layout used in testo: concurrency groups that cancel superseded PR runs, ramsey/composer-install, strict composer validate.
  • The package does not depend on spiral/roadrunner, so no advisory ignore for GHSA-g9pc-8g42-g6vq is needed.

The old job installed roave/security-advisories on every run and relied on the deprecated `set-output` command; `composer audit` checks the installed tree against the same advisories without touching `composer.json`.

Assisted-By: Claude Opus 5.5
@coderabbitai

coderabbitai Bot commented Oct 9, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 730705a5-1e08-4085-b84a-d15c3426e24d

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@roxblnfk
roxblnfk merged commit 9305e52 into 1.x Oct 9, 2026
14 checks passed
@roxblnfk
roxblnfk deleted the ci/security-audit branch October 9, 2026 18:11
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