chore: activate lefthook hooks and add fmt-all, mobile-fmt recipes#751
Merged
Conversation
wesbillman
approved these changes
May 26, 2026
wpfleger96
force-pushed
the
fix/activate-lefthook
branch
2 times, most recently
from
b6c036d to
b27b212
Compare
lefthook.yml has existed since the initial desktop app but was never wired up. Add just hooks recipe that sets core.hooksPath and runs lefthook install, add mobile-fmt and fmt-all recipes, enable parallel pre-commit execution, and wire hook installation into dev-setup.sh.
wpfleger96
force-pushed
the
fix/activate-lefthook
branch
from
b27b212 to
35bffbc
Compare
CLI-first changes: - Redirect sprout-mcp Key Patterns section to sprout-cli (MCP being phased out) - Add SPROUT_AUTH_TAG to auto-injected auth env vars - Complete exit code table (2=network, 4=other, 5=conflict were missing) - Document --format compact as a global flag (must precede subcommand) - Add gotcha: messages search requires --kinds to avoid 403 p-gate - Add gotcha: desktop crate excluded from root workspace cargo test - Update See Also: MCP tools → CLI subcommands Lefthook hook changes: - Document pre-commit/pre-push hooks in Quality Gates (installed by just setup) - Add just fmt-all as one-shot formatter before committing - Add just hooks as repair command - Upgrade worktree fmt gotcha from CI note to commit blocker - Add just mobile-fmt auto-fix alongside existing mobile check commands
wpfleger96
force-pushed
the
fix/activate-lefthook
branch
from
61ed091 to
47d9097
Compare
wpfleger96
changed the title
This was referenced May 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
lefthook.ymlsince the initial desktop app but were never wired up.just hooksrecipe: setscore.hooksPath = .hooksand runslefthook install --forceto generate hook scripts.scripts/dev-setup.shso hooks activate automatically onjust setup.parallel: truefor pre-commit hooks (pre-push already had it) — all 5 format/lint checks run simultaneously.just mobile-fmtrecipe (dart format .) andjust fmt-allrecipe (Rust root + Tauri Rust + Dart) as one-shot formatters..hooks/is gitignored since lefthook generates machine-specific scripts.AGENTS.md: document pre-commit/pre-push hooks in Quality Gates, addjust fmt-allandjust hooksusage, upgrade worktree fmt gotcha from CI note to commit blocker, addjust mobile-fmtto mobile commands; also backfill CLI-first updates (SPROUT_AUTH_TAG, complete exit codes,--format compactflag position, two new gotchas).All hook commands delegate to
justrecipes as the single source of truth.