Skip to content

Add critical instructions and hooks for Claude Code #8715

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
mergify merged 12 commits into from
Feb 18, 2026
+34 −0
Merged

Add critical instructions and hooks for Claude Code #8715

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
55e86ba
Add critical instructions and hooks for Claude Code
dapplion Jan 28, 2026
a19cb0d
fix hook name
dapplion Feb 5, 2026
0f81ae7
update hook config format
dapplion Feb 10, 2026
d17d63f
add cargo-husky pre-commit hook for cargo fmt
dapplion Feb 10, 2026
351d0e8
merge unstable
dapplion Feb 10, 2026
2017fb4
fix: sort Cargo.toml
dapplion Feb 10, 2026
8717b1a
fix: ignore cargo-husky in cargo-udeps
dapplion Feb 10, 2026
4927681
merge unstable, resolve CLAUDE.md conflict v4
dapplion Feb 10, 2026
43f0547
remove cargo-husky, add make install-hooks
dapplion Feb 11, 2026
948fe51
add .githooks/pre-commit script
dapplion Feb 12, 2026
1023d6f
Update CLAUDE.md with git hooks instructions
dapplion Feb 18, 2026
d6069a4
Update reminder command in settings.json
dapplion Feb 18, 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
15 changes: 15 additions & 0 deletions .claude/settings.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
{
"hooks": {
"PostToolUse": [
{
"matcher": "Edit|Write",
"hooks": [
{
"type": "command",
"command": "echo '\n[Reminder] Run: cargo fmt --all && make lint-fix && cargo check'"
Copy link
Member

@jimmygchen jimmygchen Feb 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need cargo check if we already run clippy

Copy link
Member

@jimmygchen jimmygchen Feb 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm removing the cargo check command and merging this - please correct me if i wrong

}
]
}
]
}
}
6 changes: 6 additions & 0 deletions CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,12 @@

This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.

## CRITICAL - Always Follow

After completing ANY code changes:
1. **MUST** run `cargo fmt --all && make lint-fix` to format and fix linting issues
2. **MUST** run `cargo check` to verify compilation before considering task complete

## Development Commands

**Important**: Always branch from `unstable` and target `unstable` when creating pull requests.
Expand Down