vm/src/function/argument.rs (3)

58-61: Tighten wording and fix grammar; clarify the lint reference.

Small nits to improve clarity and correctness.

-// We currently allows only 6 unnamed positional arguments.
-// Please use `#[derive(FromArgs)]` and a struct for more complex argument parsing.
-// The number of limitation came from:
-//
-// https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
+// We currently allow only 6 unnamed positional arguments.
+// For more complex argument parsing, prefer `#[derive(FromArgs)]` on a struct.
+// This limit follows Clippy's `too_many_arguments` lint:
+// https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments

---

585-596: Comment says “7-tuples” but code implements up to 8 (A..H).

Either update the comment to “8-tuples” or drop the H expansion. Prefer updating the comment for accuracy.

-// Implement `FromArgs` for up to 7-tuples, allowing built-in functions to bind
-// up to 7 top-level parameters (note that `PosArgs`, `KwArgs`, nested tuples, etc.
+// Implement `FromArgs` for up to 8-tuples, allowing built-in functions to bind
+// up to 8 top-level parameters (note that `PosArgs`, `KwArgs`, nested tuples, etc.

---

63-66: Minor typo in doc comment.

“then creating” → “when creating”.

-/// The `FuncArgs` struct is one of the most used structs then creating
+/// The `FuncArgs` struct is one of the most used structs when creating

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.