Repository navigation
Split CI into reusable checks, add git hooks and bump automation to v0.4.0 - #51
Merged
Merged
Conversation
Port the template's reusable checks.yml with feature.yml and main.yml callers, add the workflow-lint job and drop node-update.yml: the maintenance dashboard now flags Node.js security releases. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
pre-commit runs format:check, lint and a staged secrets scan; pre-push runs typecheck, test and knip. pnpm install enables them through the root prepare script. The Docker gitleaks fallback now forwards GIT_INDEX_FILE, so --staged scans the commit's temporary index. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
agentic-monorepo-starter) changes since the last sync (48c1ee4..d232d8a). The checks now live once in a reusablechecks.yml, called byfeature.ymlon PRs andmain.ymlon push tomain.all-passedkeeps its name, so the ruleset needs no change.workflow-lintjob (actionlint + zizmor) and movesvaluecodes/automationto v0.4.0, one release past the template's v0.3.1. The new release adds a supply-chain section and alerts to the dashboard, and shows its state in the issue title. CodeQL drops theactionslanguage, since workflow-lint covers the workflow files.node-update.yml. The maintenance dashboard now flags Node.js security releases right away, so a Node bump is done by hand from the dashboard..githooks/. pre-commit runsformat:check,lintandsecrets:scan --staged; pre-push runstypecheck,testandknip. They are enabled by the rootpreparescript onpnpm install. The Docker gitleaks fallback forwardsGIT_INDEX_FILE, and.claude/settings.jsondenies--no-verify.After merging
pnpm installonce in your checkout to turn the hooks on./language:actionsalerts stop updating.How to test
pnpm install git config --get core.hooksPath pnpm format:check pnpm lint pnpm knip pnpm typecheck pnpm test pnpm build pnpm secrets:scan --stagedSecurity review
permissions: {}at the top. Each caller job grants only whatchecks.ymlasks for (actions: read,contents: read,security-events: write).c207ce5(v0.4.0) and need onlycontents: read(workflow-lint) orcontents: read+issues: write(dashboard)..github/: no findings.🤖 Generated with Claude Code