Skip to content

feat: support Cyclops perf audits#39

Merged
grandizzy merged 1 commit into
mainfrom
centaur/support-cyclops-perf-mode-20260717060708
Jul 17, 2026
Merged

feat: support Cyclops perf audits#39
grandizzy merged 1 commit into
mainfrom
centaur/support-cyclops-perf-mode-20260717060708

Conversation

@decofe

@decofe decofe commented Jul 17, 2026

Copy link
Copy Markdown
Member

Summary

  • accept perf and perf=true in Cyclops audit comments
  • publish perf: true in the pr_audit event payload and acknowledgement
  • document the new mode and add regression coverage

Validation

  • node --test actions/pr-audit-comment/pr_audit_comment.test.js
  • git diff --check

Prompted by: @grandizzy

@mablr mablr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@grandizzy
grandizzy merged commit 6024a14 into main Jul 17, 2026
5 checks passed
@grandizzy
grandizzy deleted the centaur/support-cyclops-perf-mode-20260717060708 branch July 17, 2026 07:54
grandizzy pushed a commit that referenced this pull request Jul 17, 2026
Follow-up to #39.

Restore the previous PR-audit behavior where status-comment failures do
not block or misreport event publication:

- continue publishing if the queued comment cannot be created
- treat final status-update failures as warnings
- fail the action only when event publication fails

Adds regression coverage for queued-comment, final-update, and combined
publication/comment failures.

Tested with:

- `node --test actions/pr-audit-comment/pr_audit_comment.test.js`
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.

3 participants