UNPICK added AGENTS.md

Author: kosiew

AGENTS.md

@@ -17,16 +17,36 @@
 - All modules, classes, and functions should include docstrings.
 - Tests must use descriptive function names, pytest style assertions, and be
   grouped in classes when appropriate.
-- Rust code must pass `cargo fmt`, `cargo clippy`, and `cargo tomlfmt` and
-  should omit unnecessary parentheses around `if` conditions.
+- Rust code must pass `cargo fmt`, `cargo clippy`, and `cargo tomlfmt`.
 - Install the pre-commit hooks with `pre-commit install` and run them with
   `pre-commit run --files <file1> <file2>` or `pre-commit run --all-files`.
+
+Install the required Rust components before running pre-commit:
+
+```bash
+rustup component add rustfmt clippy
+```
+
+If installation is blocked by a proxy, see the [offline installation guide](https://rust-lang.github.io/rustup/installation/other.html).
+
 - The same checks can be run manually using the scripts in `ci/scripts`:
   - `./ci/scripts/python_lint.sh`
   - `./ci/scripts/rust_fmt.sh`
   - `./ci/scripts/rust_clippy.sh`
   - `./ci/scripts/rust_toml_fmt.sh`
 
+## Rust Code Style
+
+When generating Rust code for DataFusion, follow these guidelines:
+
+1. Do not add unnecessary parentheses around `if` conditions:
+   - Correct: `if some_condition`
+   - Incorrect: `if (some_condition)`
+
+2. Do not use `expect()` or `panic!()` except in tests - use proper error handling with `Result` types
+
+3. Follow the standard Rust style conventions from rustfmt
+
 ## Code Organization
 - Keep functions focused and under about 50 lines.
 - Break complex tasks into well-named helper functions and reuse existing