Summary
TestReadmeQuickStart_Compiles and TestLLMs_FullTxtIsUpToDate shell out to local paths (./scripts/gen-llms-full.sh) and assume the repo working tree. A consumer vendoring syncmap and running go test ./... hits non-skipped failures.
Scope
Move both tests behind //go:build repo_tools. Update the Makefile's test / check targets to pass -tags repo_tools for the in-repo suite.
Acceptance criteria
- Consumer
go test ./... on a vendored copy is clean.
- Repo
make check still exercises both tests.
makefile-targets-guard updated expected list if needed.
Source: security-reviewer agent.
Summary
TestReadmeQuickStart_CompilesandTestLLMs_FullTxtIsUpToDateshell out to local paths (./scripts/gen-llms-full.sh) and assume the repo working tree. A consumer vendoring syncmap and runninggo test ./...hits non-skipped failures.Scope
Move both tests behind
//go:build repo_tools. Update the Makefile'stest/checktargets to pass-tags repo_toolsfor the in-repo suite.Acceptance criteria
go test ./...on a vendored copy is clean.make checkstill exercises both tests.makefile-targets-guardupdated expected list if needed.Source: security-reviewer agent.