From 70b4e6fe0ac9ff2274e818ad13d990a7a488123d Mon Sep 17 00:00:00 2001 From: Lum1104 Date: Tue, 17 Mar 2026 11:26:51 +0800 Subject: [PATCH] fix(graph-reviewer): remove duplicated critical checks from warning section MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Edge weights, node ID prefix conventions, and tag formatting were listed in both Check 1 (Schema Validation — Critical) and Check 7 (Quality — Warning). A deterministic script following check headings would classify these as warnings, potentially approving invalid graphs. Remove the duplicates from Check 7 since Check 1 already covers them as critical. Co-Authored-By: Claude Opus 4.6 (1M context) --- understand-anything-plugin/agents/graph-reviewer.md | 4 ---- 1 file changed, 4 deletions(-) diff --git a/understand-anything-plugin/agents/graph-reviewer.md b/understand-anything-plugin/agents/graph-reviewer.md index b9c1b7d..eeadfc8 100644 --- a/understand-anything-plugin/agents/graph-reviewer.md +++ b/understand-anything-plugin/agents/graph-reviewer.md @@ -87,10 +87,7 @@ Verify every **edge** has ALL required fields with correct types: **Check 7 -- Quality Checks (Warning)** - No summaries that are empty or just restate the filename (e.g., summary equals the node name or just the filename portion of the path) -- Edge weights are within 0.0-1.0 range -- Node IDs follow prefix conventions: must start with `file:`, `func:`, `class:`, `module:`, or `concept:` - No self-referencing edges (where `source` equals `target`) -- All tags are lowercase and hyphenated (no spaces, no uppercase) - No orphan nodes (nodes with zero edges connecting to or from them) -- log as warning, not critical ### Script Output Format @@ -133,7 +130,6 @@ The script must write this exact JSON structure to the output file: - Orphan nodes with no edges - Short or generic summaries - Tour step count outside 5-15 range -- Non-standard tag formatting - Self-referencing edges ### Executing the Script