Skip to content

docs: document the pre-commit mockery hook in AGENTS.md#130

Merged
devantler merged 1 commit into
mainfrom
claude/docs-document-mockery-hook
Jun 26, 2026
Merged

docs: document the pre-commit mockery hook in AGENTS.md#130
devantler merged 1 commit into
mainfrom
claude/docs-document-mockery-hook

Conversation

@devantler

Copy link
Copy Markdown
Contributor

🤖 Generated by the Daily AI Assistant

What & why

AGENTS.md's repository-structure list described .pre-commit-config.yaml as "linting/spell-checking configuration" and omitted .github/scripts/run-mockery.sh entirely. But the pre-commit config's actual hooks are:

  • golangci-lint-fmt — Go formatting
  • mockery — mock generation, via .github/scripts/run-mockery.sh

(The lint/spell tooling is .mega-linter.yml + cspell.json.) So the agent file — read by Copilot code review and any agentic tool — was both inaccurate about pre-commit and silent about a wired-in helper script.

How

Splits the one bullet into accurate per-config bullets and adds a bullet for run-mockery.sh, noting it's a guarded no-op until the project adds a .mockery.yml/.mockery.yaml (so a fresh clone's pre-commit hook stays green while the generation step is already wired).

Docs/prose only — no behaviour change. Verified run-mockery.sh exists and is wired as the mockery hook's entry on live main; the added lines introduce no new cspell findings.

AGENTS.md's repository-structure list described `.pre-commit-config.yaml` as
"linting/spell-checking configuration" and omitted `.github/scripts/run-mockery.sh`
entirely — but the pre-commit config's actual hooks are golangci-lint-fmt (Go
formatting) and mockery mock generation via that script (lint/spell is `.mega-linter.yml`
+ `cspell.json`). Splits the bullet so each config is described accurately and adds a
bullet for run-mockery.sh (a guarded no-op until a .mockery.yml/.yaml exists, so a
fresh clone's hook stays green while the generation step is already wired).

Keeps the agent file — read by Copilot code review and any agentic tool — in sync
with what the repo actually does. Docs/prose only; no behaviour change.

> 🤖 Generated by the Daily AI Assistant
@devantler devantler marked this pull request as ready for review June 26, 2026 06:51
@devantler devantler requested a review from a team as a code owner June 26, 2026 06:51
@devantler devantler merged commit fa26a23 into main Jun 26, 2026
9 checks passed
@devantler devantler deleted the claude/docs-document-mockery-hook branch June 26, 2026 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant