Skip to content

ci: report ctx analysis on pull requests#39

Merged
saldestechnology merged 3 commits into
mainfrom
feat/ctx-pr-reporter
Jul 13, 2026
Merged

ci: report ctx analysis on pull requests#39
saldestechnology merged 3 commits into
mainfrom
feat/ctx-pr-reporter

Conversation

@saldestechnology

Copy link
Copy Markdown
Collaborator

Summary

  • run a read-only, fork-safe ctx analysis for every PR revision
  • publish a sanitized, stale-safe sticky ctx report from a trusted workflow_run job
  • include audit, score, duplicates, architecture checks, hotspots, repository stats, and map results
  • retain the complete JSON analysis as a 14-day workflow artifact

Security

The write-enabled commenter never checks out or executes PR code. It validates the PR, head/base SHAs, repository, run ID, schema, ctx version, and command envelopes against trusted workflow_run data before publishing.

Validation

  • node --test .github/scripts/render-ctx-pr-comment.test.cjs (4 passed)
  • real ctx 0.3.4 analysis bundle rendered successfully (5,731-character comment)
  • cargo test -- --skip embeddings:: (all relevant suites passed; existing sandbox-sensitive self-update socket tests cannot bind locally)
  • workflow YAML parse and git diff --check

@saldestechnology
saldestechnology merged commit 591ea0c into main Jul 13, 2026
9 checks passed
@saldestechnology saldestechnology mentioned this pull request Jul 13, 2026
18 tasks
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