Merge next into main: pinned prettier toolchain and canonical lint config #207

Open
clawbot wants to merge 3 commits from next into main
Collaborator

Integration branch next into main for work landed since the previous merge.

On the branch:

  • Pinned prettier formatting toolchain vendored from the prompts scaffold (.prettierrc, package.json/yarn.lock pinning prettier 3.8.1, hash-verified nvm install in script/bootstrap, config-driven script/fmt); existing markdown reflowed to house style. Tooling only, no runtime behavior change.
  • .golangci.yml re-vendored byte-identical from sneak/prompts: drops the deprecated gomodguard, turns on depguard. No source changes were needed.
  • A flaky handlers test now waits for its background deployment instead of sleeping. Test-only.

Nothing on the branch needs action at deploy time. The milestone is complete; next is green at 19619b1c.

Model: opus-5-5

Integration branch `next` into `main` for work landed since the previous merge. On the branch: - Pinned prettier formatting toolchain vendored from the prompts scaffold (`.prettierrc`, `package.json`/`yarn.lock` pinning prettier 3.8.1, hash-verified nvm install in `script/bootstrap`, config-driven `script/fmt`); existing markdown reflowed to house style. Tooling only, no runtime behavior change. - `.golangci.yml` re-vendored byte-identical from `sneak/prompts`: drops the deprecated `gomodguard`, turns on `depguard`. No source changes were needed. - A flaky handlers test now waits for its background deployment instead of sleeping. Test-only. Nothing on the branch needs action at deploy time. The milestone is complete; `next` is green at `19619b1c`. Model: opus-5-5
clawbot added 1 commit 2026-09-22 20:44:42 +02:00
script/fmt ran unpinned npx prettier with an inline --tab-width and no config, so formatting was not reproducible. This vendors the canonical format toolchain from the prompts scaffold: .prettierrc (tabWidth 4, proseWrap always), package.json and yarn.lock pinning prettier 3.8.1, and a script/bootstrap that installs node/yarn from a hash-verified nvm archive. script/fmt now reads the config over static/js and markdown; gofmt/goimports unchanged. .prettierignore keeps the *.min.js rule protecting the vendored alpine.min.js. Existing markdown was reflowed to house style.

Disclosure: make check does not gate prettier; the linter's pre-existing gomodguard deprecation warning is unrelated and left as-is.

Model: opus-4-8 (implementation and review)
clawbot self-assigned this 2026-09-22 20:44:42 +02:00
Author
Collaborator

Gate on next at 1d769834deab0e96eb9f35629c06143aa8769ea1: make check pass.

Model: opus-5-5

Gate on `next` at `1d769834deab0e96eb9f35629c06143aa8769ea1`: `make check` pass. Model: opus-5-5
clawbot added the merge-ready label 2026-09-23 01:28:03 +02:00
clawbot changed title from Post-1.1.0 milestone (next -> main) to Merge next into main: pinned prettier formatting toolchain 2026-09-23 01:35:09 +02:00
clawbot added 1 commit 2026-09-23 02:34:28 +02:00
Re-vendor canonical .golangci.yml from prompts (closes #208)
Check / check (pull_request) Successful in 3m35s
567f982c3b
Replaces the local `.golangci.yml` with the canonical file from `sneak/prompts`, fetched unchanged. This turns off the deprecated `gomodguard` (removing its warning from every lint run), turns on `depguard`, and gives the already-running `gomodguard_v2` its module block list. The new config finds nothing in upaas, so no source changes. The `test-support` deny list stays canonical because upaas has no separate test-support packages.

Model: opus-5-5
Author
Collaborator

Gate on next at 567f982c3b05bc911a4af6e4bea62670e31b310a: make check pass.

Model: opus-5-5

Gate on `next` at `567f982c3b05bc911a4af6e4bea62670e31b310a`: `make check` pass. Model: opus-5-5
clawbot changed title from Merge next into main: pinned prettier formatting toolchain to Merge next into main: pinned prettier toolchain and canonical lint config 2026-09-23 02:35:55 +02:00
clawbot added 1 commit 2026-09-23 03:04:33 +02:00
`TestHandleWebhookProcessesValidWebhook` waited for the background deployment with a 100 ms sleep, so its temp directory could be removed while the deployment was still writing its log there, and the test failed now and then. It now waits with the webhook service's `WaitForDeployments`, the fix already applied to the webhook service tests for #198. The test context keeps the webhook service it already built. No other handlers test can start a deployment. No production code change.

Model: opus-5-5
Author
Collaborator

Gate on next at 19619b1cd29c6585c773e41d48da32683cc4a96f: make check pass.

Model: opus-5-5

Gate on `next` at `19619b1cd29c6585c773e41d48da32683cc4a96f`: `make check` pass. Model: opus-5-5
clawbot removed their assignment 2026-09-23 03:06:13 +02:00
sneak was assigned by clawbot 2026-09-23 03:06:13 +02:00
All checks were successful
Check / check (pull_request) Successful in 3m24s
Required
Details
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin next:next
git checkout next
Sign in to join this conversation.