Skip to content

fix(ci): ceo-audit.yml — set CEO_AUDIT_OUTPUT to workspace path#26

Merged
Delqhi merged 7 commits into
mainfrom
fix/ceo-audit-output-path
Jun 4, 2026
Merged

fix(ci): ceo-audit.yml — set CEO_AUDIT_OUTPUT to workspace path#26
Delqhi merged 7 commits into
mainfrom
fix/ceo-audit-output-path

Conversation

@Delqhi

@Delqhi Delqhi commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Bug

3 consecutive ceo-audit.yml failures on SIN-Code-Bundle (2026-06-03, runs 26908050630 + 26911171755):

Path does not exist: /home/runner/work/SIN-Code-Bundle/SIN-Code-Bundle/ceo-audit-output/report.sarif

Root Cause

Workflow created ceo-audit-output/ and findd score.json there, but audit.sh defaulted to $HOME/ceo-audits/<repo>-ceo-audit-<id>/. Two different output dirs → score.json never found → SARIF upload fails.

Fix

Override CEO_AUDIT_OUTPUT=${{ github.workspace }}/ceo-audit-output in env block so output lands where the workflow expects it.

Verification

After merge, the next push to main should:

  • ✅ score.json written to ceo-audit-output/score.json
  • ✅ report.sarif uploaded to Code Scanning
  • ✅ Sticky PR comment posted with grade A+ (100.0/100)
  • ✅ Workflow exit code 0

3 consecutive workflow failures on Bundle: 'Path does not exist:
/home/runner/work/.../ceo-audit-output/report.sarif'.

audit.sh defaulted to $HOME/ceo-audits/<repo>-ceo-audit-<id>/, but
workflow expected ./ceo-audit-output/. Fix: override CEO_AUDIT_OUTPUT
env var in the env block to ${{ github.workspace }}/ceo-audit-output
so all output lands in one predictable place.

After this PR, ceo-audit.yml should:
- Write score.json to ceo-audit-output/score.json
- Write report.sarif to ceo-audit-output/report.sarif
- Upload SARIF successfully to Code Scanning
- Post sticky PR comment with grade A+

Also updates the App commenter step to use SIN_GITHUB_INSTALLATION_TOKEN
(org secret, public repos only) — gracefully falls back to Action
comment if not set.
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

🏆 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 6 commits June 4, 2026 13:16
pip install 'sin-code-bundle[ceo-audit,dev]' fails on Bundle runs because
the bundle has not been published to PyPI. Falls back to installing
from GitHub at the v0.4.4 tag (the last 100.0/100 A+ release).

After publishing to PyPI, this can be simplified back to a single
pip install command.
\#egg=sin-code-bundle[ceo-audit,dev] is invalid egg syntax (brackets
not allowed in egg fragment). Newer pip uses 'pkg[extras] @ URL' form
which supports extras with git URLs.

Reference: https://pip.pypa.io/en/stable/topics/vcs-support/
The skill is shipped via pip, but the workflow assumed
~/.config/opencode/skills/ceo-audit/scripts/audit.sh — which doesn't
exist on GitHub-hosted runners. Fix: locate audit.sh in either the
pip site-packages (preferred) or the opencode skill dir (fallback).

After this, audit.sh should run, score.json should be written to
ceo-audit-output/, and the PR comment should post.
sin-code-bundle[ceo-audit] extra does not yet package the skill
scripts (only python lib). Clone the SSOT directly to get audit.sh
+ axis scripts. After bundle v0.5.0 ships the scripts as a
package_data resource, this step can be removed.
@Delqhi Delqhi merged commit 98b5d08 into main Jun 4, 2026
1 of 2 checks passed
@Delqhi Delqhi deleted the fix/ceo-audit-output-path branch June 4, 2026 11:28
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