Skip to content

Commit b7a5cc3

Browse files
fubhyclaude
andcommitted
Format code and enhance CLAUDE.md with mandatory fmt reminders
- Run cargo fmt --all on Rust files (as required by CLAUDE.md) - Add prominent warnings in CLAUDE.md to never forget formatting - Make cargo fmt requirement impossible to miss with emojis and clear language 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent a65485e commit b7a5cc3

File tree

2 files changed

+12
-3
lines changed

2 files changed

+12
-3
lines changed

CLAUDE.md

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,13 @@
22

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

5+
## ⚠️ CRITICAL MANDATORY STEP ⚠️
6+
**ALWAYS run `cargo fmt --all` after editing ANY .rs file - NO EXCEPTIONS!**
7+
- This is REQUIRED after every single Rust code change
8+
- Must be run from the project root directory
9+
- Must be done BEFORE committing any changes
10+
- If you forget this, you have failed to follow instructions
11+
512
This document assumes that your human has already set up all database configuration required for the test suite to run.
613

714
## Project Overview
@@ -24,14 +31,15 @@ THEGRAPH_STORE_POSTGRES_DIESEL_URL="postgresql://graph:graph@127.0.0.1:5432/grap
2431

2532
### Code Quality
2633
```bash
27-
# Format all code
34+
# 🚨 MANDATORY: Format all code IMMEDIATELY after any .rs file edit
2835
cargo fmt --all
2936

3037
# Check code without building
3138
cargo check
3239
```
3340

34-
Always run `cargo fmt --all` from the project root after editing any .rs files.
41+
🚨 **CRITICAL REMINDER**: `cargo fmt --all` is MANDATORY after editing ANY .rs file - NO EXCEPTIONS!
42+
This must be done from the project root BEFORE any commit. Forgetting this means you failed to follow instructions.
3543

3644
## High-Level Architecture
3745

@@ -88,6 +96,7 @@ Use format: `{crate-name}: {description}`
8896
- Integration tests require Docker Compose setup and additional environment dependencies
8997
- Claude cannot run integration tests due to missing environment dependencies
9098
- Claude must set `THEGRAPH_STORE_POSTGRES_DIESEL_URL` before running any tests
99+
- **🚨 MANDATORY**: Always run `cargo fmt --all` after editing .rs files, BEFORE committing
91100

92101
### Environment Variables
93102
- `GRAPH_LOG=debug`: Enable debug logging

graph/src/data_source/common.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1416,7 +1416,7 @@ mod tests {
14161416
.to_string()
14171417
.contains("not yet supported"));
14181418

1419-
// Test extracting by property name "amount" - not yet implemented
1419+
// Test extracting by property name "amount" - not yet implemented
14201420
let result = CallDecl::extract_nested_struct_field(
14211421
&asset_struct,
14221422
&[FieldAccess::Name("amount".into())],

0 commit comments

Comments
 (0)