one pre-flight check to rule them all

[codefromthecrypt]

Please explain the motivation behind the feature request.

I work in many different projects and sometimes I forget to run either of clippy or cargo fmt. Since running build checks require approval, this introduces quite a delay.

It is a let down when timezones are involved, when you finally get enough attention to click approve on the PR only for it to fail for some minor reason, then need to go back, invariably rebase and do any merge conflict and then try again.

Describe the solution you'd like

Several projects I work on have a make precommit or make check which let you run all the style things in one go.

Sometimes, that same thing does all the things CI would, though I understand we have maybe too many slow tests to want to make an "uber test"

I would like this in goose. a one-step way to do all the style related things.

Describe alternatives you've considered

Remember to re-read AGENTS.md, for the steps since they are there, also

cargo fmt
./scripts/clippy-lint.sh
cargo clippy --fix

Additional context

Bonus points for some way to run the same bot tests locally (copilot AI assistant is recently enabled), so that you don't get style passing but then the bot finds another style you need to hunt down.

• I have verified this does not duplicate an existing feature request

[DOsinga]

nice. this could be a just check-everything I guess. I do have a history entry for this like

clear && ./scripts/clippy-lint.sh && ...

[codefromthecrypt]

question: would it help or hurt to have AGENTS.md use this? (probably can close out this issue with the answer if no)

[DOsinga]

you mean add this to the AGENTS.md in the goose folder so the LLMs know how to check their work? yeah, that seems like a good idea. closing either way though :)