chore(hooks): pull the aos pre-commit suite to aos-precommit-v0.16.0 #261

Merged
coilysiren merged 1 commit from chore/precommit-bump into main 2026-08-09 05:32:26 +00:00
Member

The hook pin sat at v0.253.0, a tag from the plain release train, while the managed block's own generator tracks the aos-precommit train. Regenerated by calling that generator rather than transcribing it, so the block is still exactly the generator's output.

Latest is aos-precommit-v0.16.0.

What the bump brings

Two new hooks and one new repo:

  • actions-run-one-line
  • source-doc-refs
  • forgejo-runner-validate

Work it demanded

actions-run-one-line found five YAML block scalars in the release workflow. Their bodies move to scripts/ci/ and each step invokes one in a single line:

  • release-impact.sh
  • release-build.sh
  • release-attach-assets.sh
  • release-publish-tap.sh
  • release-publish-scoop.sh

They are now runnable and shellcheckable outside CI, which they were not as embedded scalars.

One local deviation, filed upstream

The new block drops actionlint's -config-file flag, which turns this repo red. Verified against the hook's own actionlint binary rather than inferred:

  • actionlint .forgejo/workflows/release.yml fails with label "docker" is unknown.
  • actionlint -config-file .github/actionlint.yaml .forgejo/workflows/release.yml passes.

actionlint resolves its config relative to a detected project, and detection keys off .github/workflows, which a Forgejo-only repo does not have. So the flag is load-bearing for exactly the repos this hook set targets.

The flag is restored with a comment naming coilyco-flight-deck/agentic-os#984, since the honest fix is in the template. A re-apply will silently drop it again until that lands, which the comment says out loud.

Validation

pre-commit run --all-files is clean with the new suite. go build ./... and go vet ./... pass. The full go test ./... passes except TestLatestScoredResultsMatchCurrentPacks, red on main since #231 and owned by #240.

Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com

🤖 Generated with Claude Code

The hook pin sat at `v0.253.0`, a tag from the plain release train, while the managed block's own generator tracks the `aos-precommit` train. Regenerated by calling that generator rather than transcribing it, so the block is still exactly the generator's output. Latest is `aos-precommit-v0.16.0`. ## What the bump brings Two new hooks and one new repo: * `actions-run-one-line` * `source-doc-refs` * `forgejo-runner-validate` ## Work it demanded `actions-run-one-line` found five YAML block scalars in the release workflow. Their bodies move to `scripts/ci/` and each step invokes one in a single line: * `release-impact.sh` * `release-build.sh` * `release-attach-assets.sh` * `release-publish-tap.sh` * `release-publish-scoop.sh` They are now runnable and shellcheckable outside CI, which they were not as embedded scalars. ## One local deviation, filed upstream The new block drops actionlint's `-config-file` flag, which turns this repo red. Verified against the hook's own actionlint binary rather than inferred: * `actionlint .forgejo/workflows/release.yml` fails with `label "docker" is unknown`. * `actionlint -config-file .github/actionlint.yaml .forgejo/workflows/release.yml` passes. actionlint resolves its config relative to a detected project, and detection keys off `.github/workflows`, which a Forgejo-only repo does not have. So the flag is load-bearing for exactly the repos this hook set targets. The flag is restored with a comment naming `coilyco-flight-deck/agentic-os#984`, since the honest fix is in the template. A re-apply will silently drop it again until that lands, which the comment says out loud. ## Validation `pre-commit run --all-files` is clean with the new suite. `go build ./...` and `go vet ./...` pass. The full `go test ./...` passes except `TestLatestScoredResultsMatchCurrentPacks`, red on `main` since #231 and owned by #240. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> 🤖 Generated with [Claude Code](https://claude.com/claude-code)
The pin sat at v0.253.0, a tag from the plain release train, while the
managed block's own generator tracks the aos-precommit train. Regenerated
through that generator rather than by hand, so the block stays the
generator's output.

The bump brings two new hooks and one new repo:

* actions-run-one-line
* source-doc-refs
* forgejo-runner-validate

actions-run-one-line found five YAML block scalars in the release workflow.
Their bodies move to scripts/ci/ and the steps invoke them in one line, which
also makes them runnable and shellcheckable outside CI.

One local deviation, marked in the file and filed as agentic-os#984: the new
block drops actionlint's -config-file flag. actionlint only auto-loads its
config for a project with .github/workflows, which a Forgejo-only repo does
not have, so without the flag the custom `docker` runner label is unknown and
the workflow fails to lint. Verified both ways against the hook's own binary.
The flag is restored with a comment pointing at the issue, to be removed when
the template is fixed.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: Kai Siren <coilysiren@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
coilyco-flight-deck/agent-compose!261
No description provided.