crates/cli/src/outline/rule.rs (1)

27-27: ⚡ Quick win

Consider deriving Debug for consistency.

The serializable and runtime rule structs do not derive Debug, while other types in this module (OutlineNameSource, OutlineRoleError) and the model module (SymbolType, OutlineRole, OutlineRoles, OutlineItem) all derive Debug. Adding Debug to SerializableOutlineRule, SerializableOutlineRuleSet, and OutlineRule would improve debugging ergonomics and maintain consistency with the codebase conventions.

Suggested Debug derives

-#[derive(Clone, Deserialize)]
+#[derive(Debug, Clone, Deserialize)]
 #[serde(rename_all = "camelCase")]
 pub struct SerializableOutlineRule {

-#[derive(Clone, Deserialize)]
+#[derive(Debug, Clone, Deserialize)]
 #[serde(rename_all = "camelCase")]
 pub struct SerializableOutlineRuleSet {

-#[derive(Clone)]
+#[derive(Debug, Clone)]
 pub struct OutlineRule {

Also applies to: 46-46, 54-54

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@crates/cli/src/outline/rule.rs` at line 27, Add the Debug derive to the three
rule-related types to match other types and improve debugging: update the derive
attributes for SerializableOutlineRule (currently #[derive(Clone, Deserialize)])
to #[derive(Clone, Deserialize, Debug)], for SerializableOutlineRuleSet to
include Debug, and for OutlineRule to include Debug as well; ensure the Debug
derive is added alongside the existing derives for each struct
(SerializableOutlineRule, SerializableOutlineRuleSet, OutlineRule).

crates/cli/src/outline/rule.rs (3)

19-35: ⚡ Quick win

Consider adding Debug derive for easier debugging.

The struct lacks a Debug derive, which would be helpful during development and troubleshooting. Since all field types already implement Debug (String, SgLang, SymbolType, Vec, SerializableRuleCore), adding it is straightforward.

🔍 Proposed enhancement

-#[derive(Clone, Deserialize)]
+#[derive(Clone, Debug, Deserialize)]
 #[serde(rename_all = "camelCase")]
 pub struct SerializableOutlineRule {

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@crates/cli/src/outline/rule.rs` around lines 19 - 35, The
SerializableOutlineRule struct is missing a Debug derive which makes printing
and debugging instances harder; add Debug to the derive list on the
SerializableOutlineRule struct (alongside Clone and Deserialize) so the type
implements std::fmt::Debug—update the attribute to #[derive(Clone, Deserialize,
Debug)] for SerializableOutlineRule.

---

46-60: ⚡ Quick win

Consider adding Debug derive for easier debugging.

The runtime struct lacks a Debug derive. Adding it would improve debuggability during development and when troubleshooting rule normalization issues.

🔍 Proposed enhancement

-#[derive(Clone)]
+#[derive(Clone, Debug)]
 pub struct OutlineRule {

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@crates/cli/src/outline/rule.rs` around lines 46 - 60, The OutlineRule struct
is missing a Debug derive which makes debugging harder; update the struct
definition for OutlineRule to add #[derive(Clone, Debug)] (or include Debug
alongside existing derives) so instances can be formatted for logs and
debugging, ensuring the derive applies to the struct that contains fields id,
language (SgLang), symbol_type (SymbolType), roles (OutlineRoles), name, and
core (SerializableRuleCore).

---

38-43: ⚡ Quick win

Consider adding Debug derive for easier debugging.

Similar to SerializableOutlineRule, this struct would benefit from a Debug derive for development and troubleshooting.

🔍 Proposed enhancement

-#[derive(Clone, Deserialize)]
+#[derive(Clone, Debug, Deserialize)]
 #[serde(rename_all = "camelCase")]
 pub struct SerializableOutlineRuleSet {

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@crates/cli/src/outline/rule.rs` around lines 38 - 43, The struct
SerializableOutlineRuleSet is missing a Debug derive which makes debugging
harder; add Debug to its derive list (i.e., change #[derive(Clone, Deserialize)]
to #[derive(Clone, Debug, Deserialize)]) so SerializableOutlineRuleSet instances
can be formatted in logs and debuggers, mirroring the existing
SerializableOutlineRule derive usage.