Skip to content

docs: help agents unblock golangci-lint cross-worktree cache contamination #6584

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
janniklasrose merged 2 commits into from
Sep 9, 2026
Merged

docs: help agents unblock golangci-lint cross-worktree cache contamination #6584

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7ecce30
docs: tell agents how to unblock golangci-lint cross-worktree cache c…
janniklasrose Sep 9, 2026
643e121
docs: slim the lint-cache note to trigger + fix only
janniklasrose Sep 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,12 @@ GIT_EDITOR=true GIT_SEQUENCE_EDITOR=true VISUAL=true GIT_PAGER=cat git rebase or
- Use `./task test-update` to regenerate acceptance test outputs after changes.
- The CLI binary supports both `databricks` and `pipelines` command modes based on executable name.

**Stale golangci-lint cache across worktrees.** Triggered by `task lint` or a direct `go tool ... golangci-lint` run reporting issues in files under a *different* or deleted worktree, often with `no such file or directory` warnings. Fix — clear the cache and re-run, don't investigate the reported issues:

```sh
go tool -modfile=tools/go.mod golangci-lint cache clean
```

# Common Mistakes

**RULE: When adding a direct Go dependency, annotate its license in `go.mod` and update `NOTICE`.** Before picking the SPDX identifier, read `internal/build/license_test.go` to see the current allowlist (the `spdxLicenses` map). That test is the source of truth and will fail CI if a direct `require` line lacks a matching SPDX suffix comment (e.g. `// MIT`). Also add a corresponding entry to `NOTICE` under the matching license section. If a dep's license isn't on the allowlist, discuss before adding.
Expand Down