Skip to content

Commit e873451

Browse files
committed
fix(ci): re-commit go.work + go.work.sum — required by module hygiene job
The 'chore: reorganize repo' commit removed go.work / go.work.sum and added them to .gitignore. CI's 'module hygiene' job was designed around the assumption that go.work is committed: it runs 'go work sync' then 'go build -mod=readonly', and a separate step greps go.work for each ./external/{eyrie,inspect,sight,tok,trace,yaad} path. Without the committed file, both 'go work sync' and the build fail. Also updates AGENTS.md to reflect that go.work IS committed, and replaces the .gitignore comment with a pointer to the CI workflow.
1 parent fa77e54 commit e873451

4 files changed

Lines changed: 222 additions & 4 deletions

File tree

.gitignore

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,8 @@ testdata/compat-test
5353
bin/
5454
.shared-templates/
5555

56-
# Go workspace (local dev only — not committed per AGENTS.md)
57-
go.work
58-
go.work.sum
56+
# Go workspace is committed because CI's `module hygiene` job runs
57+
# `go work sync` + `-mod=readonly` build and asserts no diff against
58+
# the committed files. See .github/workflows/ci.yml.
59+
# (go.work.sum.hawk/ is a per-test artifact and stays ignored.)
60+
go.work.sum.hawk/

AGENTS.md

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,10 @@ test: add coverage for guardian
150150
- Do not import `internal/` from other ecosystem repos — use `shared/types/`
151151
- Do not put API keys in `.env` or shell env for hawk — use `/config` (OS keychain)
152152
- The `external/` directory is for local dev only; CI clones repos separately
153-
- `go.work` is for local multi-repo development; it is not committed
153+
- `go.work` and `go.work.sum` are committed — CI's `module hygiene` job
154+
runs `go work sync` and asserts the result is in sync with the repo. Both
155+
files point at `./external/*` checkouts; the `.github/actions/checkout-eyrie`
156+
action populates `./external/` on CI runners before the build runs.
154157

155158
## Naming Conventions
156159

go.work

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
go 1.26.3
2+
3+
use (
4+
.
5+
./external/eyrie
6+
./external/inspect
7+
./external/sight
8+
./external/tok
9+
./external/trace
10+
./external/yaad
11+
)

0 commit comments

Comments
 (0)