Problem
After the split (main replaced by the CLI-only carve, de556ad), some config and doc files still reference monorepo-only paths. None of this affects the build or tests.
Items (from the independent re-review of the carve PR)
.golangci.yml:105,111-114,130-134: exclusions for internal/testutil/, internal/api/ and internal/config/...|pkg/ladder/..., none of which exist. Prune them.
.gitleaksignore:3 is a path, not a fingerprint, so gitleaks warns "Invalid entry". Remove it or replace it with a real fingerprint.
.gitignore:95-102: frontend/* entries.
.gitallowed:39-46: comments cite store_postgres_pgxmock_test.go and handler_*_test.go.
.pre-commit-config.yaml: the comment at 51-55 cites internal/credentials/resolver.go; an exclude has frontend/src/index\.html; the gosec comment at 162-163 lists pkg/, providers/* modules.
Makefile:89 vs scripts/gosec-hook.sh:38: the gosec exclude lists differ (the hook adds G117,G118,G122,G702-G706). Align them.
NOTICE:15,43,47,51,59: aws-lambda-go, pgx, pgxmock, golang-migrate and testcontainers aren't in go.mod.
docs/cli/cloud-setup.md:3,9 says "CUDly server reads them" and links to ../../README.md#deployment-self-hosted-via-terraform (the anchor is gone); docs/cli/README.md:15 says "self-hosted". Point these at LeanerCloud/cloud-commitments-platform.
- The
cmd/configure_azure.go:113 and cmd/configure_gcp.go:82 help text says "CUDly CloudFormation stack name". Check that the wording is right for the CLI. The cmd/multi_service_stats.go:173 comment references "the MCP server's".
ci.yml:61 has || true on gocyclo, which swallows a crash. ci.yml:463: Snyk is continue-on-error, so it's never gating.
CLAUDE.md:53 says the module is "rooted at cmd/". It's rooted at the repo root; cmd/ is the main package.
CONTRIBUTING.md:15: ./cudly --help prints no version. Add a --version flag or drop the line.
Acceptance
No monorepo-only paths remain in config or docs. CI stays green and every gate that ran before still runs.
Problem
After the split (main replaced by the CLI-only carve, de556ad), some config and doc files still reference monorepo-only paths. None of this affects the build or tests.
Items (from the independent re-review of the carve PR)
.golangci.yml:105,111-114,130-134: exclusions forinternal/testutil/,internal/api/andinternal/config/...|pkg/ladder/..., none of which exist. Prune them..gitleaksignore:3is a path, not a fingerprint, so gitleaks warns "Invalid entry". Remove it or replace it with a real fingerprint..gitignore:95-102:frontend/*entries..gitallowed:39-46: comments citestore_postgres_pgxmock_test.goandhandler_*_test.go..pre-commit-config.yaml: the comment at 51-55 citesinternal/credentials/resolver.go; an exclude hasfrontend/src/index\.html; the gosec comment at 162-163 listspkg/, providers/*modules.Makefile:89vsscripts/gosec-hook.sh:38: the gosec exclude lists differ (the hook adds G117,G118,G122,G702-G706). Align them.NOTICE:15,43,47,51,59: aws-lambda-go, pgx, pgxmock, golang-migrate and testcontainers aren't in go.mod.docs/cli/cloud-setup.md:3,9says "CUDly server reads them" and links to../../README.md#deployment-self-hosted-via-terraform(the anchor is gone);docs/cli/README.md:15says "self-hosted". Point these at LeanerCloud/cloud-commitments-platform.cmd/configure_azure.go:113andcmd/configure_gcp.go:82help text says "CUDly CloudFormation stack name". Check that the wording is right for the CLI. Thecmd/multi_service_stats.go:173comment references "the MCP server's".ci.yml:61has|| trueon gocyclo, which swallows a crash.ci.yml:463: Snyk iscontinue-on-error, so it's never gating.CLAUDE.md:53says the module is "rooted atcmd/". It's rooted at the repo root;cmd/is the main package.CONTRIBUTING.md:15:./cudly --helpprints no version. Add a--versionflag or drop the line.Acceptance
No monorepo-only paths remain in config or docs. CI stays green and every gate that ran before still runs.