Merge pull request #292 from netwrix/dev

script now handles the dict-wrapper case

Author: jth-nw

.github/workflows/claude-documentation-reviewer.yml

@@ -112,11 +112,27 @@ jobs:
 
           try:
               with open('/tmp/suggestions.json') as f:
-                  suggestions = json.load(f)
+                  data = json.load(f)
           except (FileNotFoundError, json.JSONDecodeError) as e:
               print(f"Could not read /tmp/suggestions.json: {e}")
               sys.exit(1)
 
+          # Handle case where Claude wraps the array in an object
+          if isinstance(data, dict):
+              for key in data:
+                  if isinstance(data[key], list):
+                      data = data[key]
+                      break
+              else:
+                  print(f"Unexpected dict structure: {list(data.keys())}")
+                  sys.exit(1)
+
+          if not isinstance(data, list):
+              print(f"Expected array, got {type(data)}")
+              sys.exit(1)
+
+          suggestions = data
+
           if not suggestions:
               subprocess.run(
                   ['gh', 'pr', 'comment', pr_number, '--repo', repo,
@@ -128,13 +144,20 @@ jobs:
 
           comments = []
           for s in suggestions:
+              if not isinstance(s, dict):
+                  print(f"Skipping non-object suggestion: {repr(s)[:100]}")
+                  continue
               body = s['body'] + '\n```suggestion\n' + s['suggestion'] + '\n```'
               c = {'path': s['path'], 'line': s['line'], 'side': 'RIGHT', 'body': body}
               if s.get('start_line') and s['start_line'] != s['line']:
                   c['start_line'] = s['start_line']
                   c['start_side'] = 'RIGHT'
               comments.append(c)
 
+          if not comments:
+              print("No valid suggestions to post")
+              sys.exit(1)
+
           review_data = {
               'commit_id': head_sha,
               'body': f'Found {len(comments)} issue(s). To apply all fixes at once, reply with `@claude` followed by your instructions (e.g. `@claude fix all issues`).',