Skip to content

fix(clawpatch): address daily finding#234

Closed
michiosw wants to merge 1 commit into
mainfrom
michelosswald/fix-clawpatch-race-check
Closed

fix(clawpatch): address daily finding#234
michiosw wants to merge 1 commit into
mainfrom
michelosswald/fix-clawpatch-race-check

Conversation

@michiosw
Copy link
Copy Markdown
Contributor

Where We Are

The root workspace test script skipped the documented go test -race ./... check. A change could pass pnpm test while still missing the race detector run that the README tells contributors to use.

Where We Want To Go

pnpm test should enforce the same race detector check the repo documents, so the default local test path catches data races before merge.

How do we get there

Update the root test script in package.json to run go test -race ./... before go vet ./.... Verification: go test ./..., go vet ./..., npm exec --yes --package pnpm@10.0.0 -- pnpm install --frozen-lockfile, npm exec --yes --package pnpm@10.0.0 -- pnpm --dir web/guard-dashboard typecheck, and git diff --check all passed in the temp checkout.

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 30, 2026

Greptile Summary

This PR updates the root workspace test command to include the Go race detector. The main change is:

  • pnpm test now runs go test -race ./... before go vet ./....

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Reviews (1): Last reviewed commit: "fix(clawpatch): address daily finding" | Re-trigger Greptile

@michiosw michiosw marked this pull request as draft May 30, 2026 08:08
@michiosw michiosw closed this May 30, 2026
@michiosw michiosw deleted the michelosswald/fix-clawpatch-race-check branch May 30, 2026 08:18
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