sneak/upaas
1
0
Fork 1
Code Issues 8 Pull Requests Actions Packages Projects Releases 1 Wiki Activity

script/bootstrap does not install goimports, which script/fmt requires #184

New Issue
Open
opened 2026-08-07 18:47:47 +02:00 by clawbot · 0 comments
clawbot commented 2026-08-07 18:47:47 +02:00
Collaborator
Copy Link

Found while working #175 on a fresh environment: make fmt fails
with script/fmt: goimports: not found after a successful
make bootstrap. script/fmt runs gofmt, goimports, and
npx prettier, but script/bootstrap only installs git, make, go,
and golangci-lint. script/bootstrap's contract is to install ALL
dependencies idempotently, so this is a bootstrap gap.

Remediation: install a pinned goimports in script/bootstrap
(e.g. go install golang.org/x/tools/cmd/goimports@vX.Y.Z with an
exact version constant, integrity covered by the Go module checksum
database). While there, decide whether node/prettier availability
for the static/js formatting step also needs a bootstrap guard,
since script/fmt assumes npx prettier exists.

Definition of done:

  • on a machine without goimports, make bootstrap followed by
    make fmt succeeds
  • the installed version is pinned exactly (no @latest), with a
    version comment per REPO_POLICIES.md
  • lands via PR from a feature branch off main
Found while working #175 on a fresh environment: `make fmt` fails with `script/fmt: goimports: not found` after a successful `make bootstrap`. `script/fmt` runs `gofmt`, `goimports`, and `npx prettier`, but `script/bootstrap` only installs git, make, go, and golangci-lint. `script/bootstrap`'s contract is to install ALL dependencies idempotently, so this is a bootstrap gap. Remediation: install a pinned `goimports` in `script/bootstrap` (e.g. `go install golang.org/x/tools/cmd/goimports@vX.Y.Z` with an exact version constant, integrity covered by the Go module checksum database). While there, decide whether node/prettier availability for the `static/js` formatting step also needs a bootstrap guard, since `script/fmt` assumes `npx prettier` exists. Definition of done: - on a machine without `goimports`, `make bootstrap` followed by `make fmt` succeeds - the installed version is pinned exactly (no `@latest`), with a version comment per `REPO_POLICIES.md` - lands via PR from a feature branch off `main`
clawbot added this to the 1.1.0 milestone 2026-08-07 18:47:47 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 merge-ready
All checks pass, reviewed, rebased, ready for merge
 needs-checks
make check does not pass cleanly
 needs-rebase
PR has merge conflicts or is behind main
 needs-review
Code review not yet done
 needs-rework
Code review found issues to fix
 notplanned
question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone 1.1.0
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/upaas#184
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?