Severity: P3. Confidence: high. Informational.
Affected files:
internal/deploy/docker.go
internal/deploy/ecr.go
internal/deploy/frontend.go
internal/deploy/profiles.go
Evidence:
Repo-wide grep finds zero non-test importers; ~26KB of code plus ~55KB of tests referenced only by its own test files. Deployment is actually handled by GitHub workflows and terraform/.
Impact:
Dead Docker/ECR shelling code compiled, tested in CI, and repeatedly re-assessed in security scans for no benefit.
Recommendation:
Delete internal/deploy (or park the design intent in a note and remove the code until a CLI deploy command is actually planned).
Verifier verdict: unverified - P3, adversarial verification skipped.
Source: docs/reviews/codebase-review-2026-06-10.md (automated multi-dimension code review, adversarially verified for P1/P2)
Severity: P3. Confidence: high. Informational.
Affected files:
internal/deploy/docker.gointernal/deploy/ecr.gointernal/deploy/frontend.gointernal/deploy/profiles.goEvidence:
Repo-wide grep finds zero non-test importers; ~26KB of code plus ~55KB of tests referenced only by its own test files. Deployment is actually handled by GitHub workflows and terraform/.
Impact:
Dead Docker/ECR shelling code compiled, tested in CI, and repeatedly re-assessed in security scans for no benefit.
Recommendation:
Delete internal/deploy (or park the design intent in a note and remove the code until a CLI deploy command is actually planned).
Verifier verdict: unverified - P3, adversarial verification skipped.
Source: docs/reviews/codebase-review-2026-06-10.md (automated multi-dimension code review, adversarially verified for P1/P2)