Skip to content

pages: every step one task, Task from go.mod, and the gates in task check - #17

Merged
aledbf merged 2 commits into
mainfrom
taskgate
Oct 24, 2026
Merged

aledbf merged 2 commits into
mainfrom
taskgate

Conversation

@aledbf

@aledbf aledbf commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Depends on spin-stack/go-tools#50. It takes go-tools at that PR's commit (8cf54b0); once #50 is on main, re-pin with go get github.com/spin-stack/go-tools@<main's commit> and mark this ready.

What

Every run: step of pages.yml is one task, and pnpm through Socket Firewall is held too (this repository installs no packages, so that gate only keeps it so).

step now
go vet, go test, palette -check, mark -check, shots -check, screens -check (six steps) go tool task check, which already ran the same six commands in the same order
unpack the screens' images from their release (main only) go tool task screens:download; its if: and GH_TOKEN stay in the workflow, and the repository comes from GITHUB_REPOSITORY (this one outside CI) instead of a ${{ }} in the shell
  • go-tools is a new dependency: taskgate and sfw are go.mod tools, and task check runs go tool taskgate and go tool sfw check first.
  • Task is a go.mod tool at v3.53.1, the version go-tools pins, so the workflow needs only setup-go. This repository never installed Task in CI before.
  • screens/img/ is ignored, since task screens:download can now write it locally.
  • The README shows the tasks as go tool task ... and says what the gates check.

No exemptions.

Checked

  • go tool task check passes locally: taskgate (every step a task, 0 exempt), sfw check, vet, test, and the palette, mark, shots and screens checks.
  • go tool task screens:download unpacks 332 images from the release screens.json names.
  • actionlint 1.7.12 passes, and every action is still pinned by commit.
  • go tool task --list parses the Taskfile, and both tasks the workflow calls exist.

Not checked: the screens download and the Pages deploy on GitHub, which only run on main.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

aledbf and others added 2 commits October 24, 2026 15:30
…heck

go-tools at 8cf54b0 (spin-stack/go-tools#50). The Gate's six checks were task
check written out again; it runs `go tool task check`, which now also runs
taskgate and `sfw check`. Unpacking the screens' images is task
screens:download, which a developer can run too.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
spin-stack/go-tools#50 and #51 are on main: the pin moves from the PR's commit to main's, which
also finds the Task go.mod pins for refs and holds pnpm run to installing nothing on its own.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@aledbf
aledbf marked this pull request as ready for review October 24, 2026 18:59
@aledbf
aledbf merged commit c882b6f into main Oct 24, 2026
2 checks passed
@aledbf
aledbf deleted the taskgate branch October 24, 2026 23:41
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