Skip to content

test: App commenter verification#27

Merged
Delqhi merged 3 commits into
mainfrom
test/app-commenter
Jun 4, 2026
Merged

test: App commenter verification#27
Delqhi merged 3 commits into
mainfrom
test/app-commenter

Conversation

@Delqhi
Copy link
Copy Markdown
Contributor

@Delqhi Delqhi commented Jun 4, 2026

Verifying the new post_audit_pr.py step works on a real PR.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

🏆 CEO Audit — A+ (100.0/100)

Metric Value
Grade A+
Score 100.0/100
Critical findings 0
High findings 0
Profile QUICK
Min grade gate B

📥 Download full report (Markdown)
📊 Download SARIF (for Code Scanning)

Run ~/.config/opencode/skills/ceo-audit/scripts/audit.sh . --profile=QUICK locally to reproduce.

SIN-Agent added 2 commits June 4, 2026 13:54
score.json is written to ~/ceo-audits/<repo>-ceo-audit-<runid>/
not to ceo-audit-output/. Use find to locate it.

If no score.json found, skip App commenter (Action comment
above still works). continue-on-error: true ensures we never
block the workflow on App issues.
post_audit_pr.py needs a token to authenticate with the GitHub API.
The default GITHUB_TOKEN is auto-blocked by GitHub Actions unless
explicitly passed through to subshell env. Fix: pass via env block.

Falls back to SIN_GITHUB_INSTALLATION_TOKEN (org secret) if set.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

🏆 CEO Audit — A+ (100.0/100)

Metric Value
Grade A+
Score 100.0/100
Critical findings 0
High findings 0
Medium findings 0
Profile QUICK
Min grade gate B

📥 Download full report (Markdown)

Run ID: 26950240475 · Commit: ${github.sha}

Run ~/.config/opencode/skills/ceo-audit/scripts/audit.sh . --profile=QUICK locally to reproduce.

@Delqhi Delqhi merged commit 48998d9 into main Jun 4, 2026
1 of 2 checks passed
@Delqhi Delqhi deleted the test/app-commenter branch June 4, 2026 11:58
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