Repository navigation
Commit b00c9db
authored
chore(deploy): remove orphaned internal/deploy package (#1246)
* chore(deploy): remove orphaned internal/deploy package
The internal/deploy package (Docker build/push, ECR login/repo
management, frontend S3+CloudFront upload, deployment profiles) has
zero importers outside its own test files. Actual deployment is
handled by the GitHub workflows and terraform/. The dead code was
still compiled, tested in CI, and re-assessed in every security scan
for no benefit (ARCH-13, P3).
Verification that the package is orphaned:
- repo-wide grep for the import path "internal/deploy" across all
file types (Go, workflows, scripts, docs, Makefiles): zero hits
outside the package itself
- repo-wide grep for distinctive exported identifiers
(NewDockerService, NewECRService, NewFrontendService,
DeploymentConfig, LoginToPublicECR, BuildAndUpload, ...): zero hits
outside the package itself
Also runs go mod tidy: the cloudfront, ecr, ecrpublic and s3 AWS SDK
modules were only used by this package and drop out of go.mod.
go build ./..., go vet ./... and go test ./internal/... all pass
after removal.
Closes #1202
* chore(deploy): clean up stale golangci noctx exclusion for deleted internal/deploy/docker.go
The internal/deploy package was deleted; its path in the noctx
per-file exclusion is now dead. Remove the dead path segment to
keep .golangci.yml in sync with the actual source tree.
* ci: bump golangci-lint timeout from 5m to 10m
The previous run timed out at ~321s on a slow CI runner. The codebase
has grown and 5m is too tight; 10m gives headroom without masking real
lint errors.
* fix(deps): bump brace-expansion to 1.1.16 (GHSA-3jxr-9vmj-r5cp)
npm audit flagged brace-expansion <1.1.16 as high severity DoS.
This was disclosed 2026-07-21 and affects all PR branches equally.
Lockfile-only change; no production API surface altered.1 parent 2819b3d commit b00c9db
4 files changed
Lines changed: 3 additions & 21 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
| 51 | + | |
52 | 52 | | |
53 | 53 | | |
54 | 54 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
113 | 113 | | |
114 | 114 | | |
115 | 115 | | |
116 | | - | |
| 116 | + | |
117 | 117 | | |
118 | 118 | | |
119 | 119 | | |
120 | 120 | | |
121 | | - | |
| 121 | + | |
122 | 122 | | |
123 | 123 | | |
124 | 124 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
94 | 94 | | |
95 | 95 | | |
96 | 96 | | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | 97 | | |
101 | 98 | | |
102 | 99 | | |
103 | | - | |
104 | 100 | | |
105 | 101 | | |
106 | 102 | | |
| |||
135 | 131 | | |
136 | 132 | | |
137 | 133 | | |
138 | | - | |
139 | | - | |
140 | 134 | | |
141 | 135 | | |
142 | 136 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
118 | 118 | | |
119 | 119 | | |
120 | 120 | | |
121 | | - | |
122 | | - | |
123 | 121 | | |
124 | 122 | | |
125 | 123 | | |
126 | 124 | | |
127 | | - | |
128 | | - | |
129 | | - | |
130 | | - | |
131 | 125 | | |
132 | 126 | | |
133 | 127 | | |
134 | 128 | | |
135 | | - | |
136 | | - | |
137 | 129 | | |
138 | 130 | | |
139 | | - | |
140 | | - | |
141 | 131 | | |
142 | 132 | | |
143 | 133 | | |
| |||
152 | 142 | | |
153 | 143 | | |
154 | 144 | | |
155 | | - | |
156 | | - | |
157 | 145 | | |
158 | 146 | | |
159 | 147 | | |
| |||
0 commit comments