refine claude-code-doc-review 2.

Author: muhabalwan-aginx

.github/workflows/claude-code-doc-review.yml

@@ -10,7 +10,7 @@ permissions:
 
 jobs:
   documentation-review:
-    name: Review doc changes
+    name: Review requirements impact
     runs-on: ubuntu-latest
 
     steps:
@@ -25,56 +25,44 @@ jobs:
           github_token: ${{ secrets.GITHUB_TOKEN }}
           anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
           prompt: |
-            You are reviewing a pull request to ensure requirements and specifications documentation stays synchronized with code changes.
+            You are reviewing a PR to check if requirements/specs documentation needs updates.
 
-            **Analysis Process:**
+            **Your task:**
 
-            1. Run `git diff origin/${{ github.base_ref }}...HEAD` to identify all changed files
-            2. For each changed file, determine:
-               - Which module/component it belongs to
-               - What functionality was added, modified, or removed
-            3. Read ALL existing files in:
-               - `requirements/` (SHR-*.md for stakeholder requirements, SWR-*.md for software requirements)
-               - `specifications/` (SPEC-*.md for software item specifications)
-            4. Cross-reference code changes against documentation to identify:
-               - Docs that describe changed functionality (needs update)
-               - New functionality lacking documentation (needs new doc)
-               - Documentation that remains accurate (no action needed)
+            1. Run: git diff origin/${{ github.base_ref }}...HEAD
+            2. Identify changed modules (application, platform, wsi, bucket, dataset, etc.)
+            3. Use Glob to find relevant docs:
+               - Glob pattern: requirements/*{MODULE}*.md
+               - Glob pattern: specifications/SPEC-{MODULE}*.md
+            4. Read ONLY the relevant docs for changed modules
+            5. Analyze if they need updates
 
-            **Requirements:**
-            - Analyze ONLY `requirements/` and `specifications/` directories
-            - Ignore template files (files with "TEMPLATE" in the name)
-            - For new docs, follow existing naming conventions exactly
-            - Be specific about WHY each doc needs updating (cite the code change)
+            **Rules:**
+            - ONLY analyze requirements/ and specifications/ directories
+            - Skip files with "TEMPLATE" in name
+            - Use Glob and Read tools (NOT bash ls/head commands)
+            - Be specific about WHY docs need updating
 
-            **Output Format:**
-
-            Post a single PR comment using this exact structure:
+            **Post PR comment:**
 
             ## 📋 Requirements & Specifications Review
 
-            ### 🔍 Code Changes Summary
-            **Modified files:** [list each file]
-            **Affected modules:** [comma-separated list]
-
-            ### 📝 Documentation Updates Required
-            
-            #### Files Needing Updates:
-            - **`requirements/SHR-XXX.md`** - Reason: [specific code change that invalidates current doc]
-            - **`specifications/SPEC-YYY.md`** - Reason: [specific code change that invalidates current doc]
+            ### Changed Code
+            - Files: [list]
+            - Modules: [list]
 
-            #### New Documentation Needed:
-            - **`requirements/SWR-[MODULE]-[FEATURE].md`** - Coverage: [what new requirement this addresses]
-            - **`specifications/SPEC-[MODULE]-[SERVICE].md`** - Coverage: [what new implementation this describes]
+            ### 📝 Docs to Update
+            - `requirements/XXX.md`: [why]
+            - `specifications/XXX.md`: [why]
 
-            ### ✅ Documentation Still Accurate
-            - `requirements/SHR-ZZZ.md` - Covers [feature], unaffected by changes
-            - `specifications/SPEC-AAA.md` - Covers [component], unaffected by changes
+            ### ➕ New Docs Needed
+            - `requirements/SWR-XXX.md`: [what it should cover]
 
-            ### 💡 Recommendations
-            [If applicable: suggest priority order for updates or note if changes are traceability-critical]
+            ### ✅ Docs Still Accurate
+            - [list docs that don't need changes]
 
-            **Note:** If no documentation changes are needed, state: "✅ All requirements and specifications remain accurate."
+            Keep it concise.
           claude_args: >-
-            --max-turns 20
+            --max-turns 30
+            --allowed-tools Read,Glob,Grep,Bash(git:*)
             --model claude-sonnet-4-5-20250929