Repository navigation
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Summary
Merge Risk: ⚪ Minimal · up to
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
* codex/extract-code-standards: refactor: expose distinct attributed invocable check and fix commands
* codex/extract-code-standards: fix: load symlinked presets through ECS installation autoload
Closes #11. Depends on #7; this PR targets
codex/extract-code-standardsdeliberately. Review only the channel/provenance increment before integrating the initial layer. Do not merge this upper layer first. After #7 merges, recapture base/head identity and rerun checks/reviews before any normal merge.Consumer examples use GitHub
@v0after an actual published stable0.x exists. The reusable checks out its helper/runtime using the actual executing job's workflow repository/SHA, never resolving the moving channel again. Own CI calls the local reusable before any channel exists.Adds repository-only channel maintenance adapted from Changelog's documented implementation: validate the actual published release and exact SHA, refuse major>0 and foreign targets, reject homonymous tags, compare-and-swap the previous branch OID, avoid regression and preserve idempotence. The existing Changelog lifecycle supplies publication outputs. It never creates a release/tag or advances v0 on an unpublished main commit. Required token capability failures remain explicit gates; no credentials or persistent security settings change. Composer/CPX versions remain distinct from the GitHub channel.
Preflight: base
9666d997c864e4e74b7ca0ca8cbce849091353a7→ head181ac036940811d99909230a00aee95a2276196a. Fresh independent read-only reviewer/root/review_coderabbit_fixesapproved this exact identity. Local PHP8.5.11: 61 tests/194 assertions, syntax/ECS for24 PHP files, Composer strict validation, actionlint and whitespace checks pass. Channel fixtures use disposable Git and MockHttpClient; workflow guards execute extracted Bash under synthetic contexts. No real channel mutation was used as a test.Merge requires passing checks AND completed Codex AND CodeRabbit reviews without pending findings on the current head. Local review, old-head review, quota/skipped results and green status alone do not satisfy that gate. The later generated version PR has the same requirement. No release exists yet and no self-hosted CI was dispatched.
Refreshed through normal merges from the current initial branch; no published history rewritten. Fresh independent read-only review approved this exact updated head after reexecuting the stated tests. Remote reviews/checks must renew. The initial branch still awaits the requested composition in #16 and all resulting gates; no merge or release occurred.