From c2b04a3bd44dcae21f5defa72fd878f36e1f51e2 Mon Sep 17 00:00:00 2001 From: james-haytko_nwx Date: Thu, 26 Feb 2026 14:17:01 -0600 Subject: [PATCH 1/3] Use three focused review passes and revert to Sonnet Replace the single open-ended scan with three explicit passes (grammar, style, Netwrix standards), each covering every sentence in the document. This prevents loud PR change issues from crowding out subtle preexisting issues by forcing systematic attention across each category independently. Also reverts model to claude-sonnet-4-5-20250929 since Opus produced the same results as Sonnet. Generated with AI Co-Authored-By: Claude Code --- .../workflows/claude-documentation-reviewer.yml | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/.github/workflows/claude-documentation-reviewer.yml b/.github/workflows/claude-documentation-reviewer.yml index d6c6602d28..44cd329be1 100644 --- a/.github/workflows/claude-documentation-reviewer.yml +++ b/.github/workflows/claude-documentation-reviewer.yml @@ -89,11 +89,19 @@ jobs: 1. Use the Read tool to read each file in full. Do not look at the PR diff yet. - 2. Identify ALL issues throughout the entire document — every sentence, every paragraph, every section. Do not stop after finding a few issues. Do not limit yourself to areas that look like they were recently changed. + 2. Review the document in three focused passes. In each pass, go through every sentence and paragraph in the document: + + Pass 1 — Grammar: Check every sentence for sentence fragments, run-on sentences, ambiguous pronoun references, subject-verb agreement errors, misplaced or dangling modifiers, and any other grammatical errors. + + Pass 2 — Style: Check every sentence for active vs. passive voice, present vs. other tenses, use of second person ("you"), informal or unprofessional language, anthropomorphism, use of first person plural ("we"), nominalizations, redundancy, and any other style issues from your instructions. + + Pass 3 — Netwrix standards: Check every sentence for incorrect product names, undefined acronyms on first use, incorrect or non-standard terminology, and any other Netwrix-specific conventions from your instructions. + + Compile a single list of all issues found across all three passes before moving to the next step. 3. Use `gh pr diff ${{ github.event.pull_request.number }}` to get the list of lines that were added or modified in this PR. - 4. Categorize each issue you found in step 2 as either: + 4. Categorize each issue from step 2 as either: - Issues in PR changes: the issue is on a line that was added or modified in this PR - Preexisting issues: the issue exists on a line that was not changed by this PR @@ -108,7 +116,7 @@ jobs: Then fix ALL issues directly in the files using the Write and Edit tools. Do not post a PR comment. Do not commit or push. claude_args: | - --model claude-opus-4-6 + --model claude-sonnet-4-5-20250929 --allowedTools "Read,Write,Edit,Bash(gh pr view:*),Bash(gh pr diff:*)" --append-system-prompt "${{ steps.read-prompt.outputs.prompt }}" From 8481a6f1186467f6e0bb0283f427c52f104cdcbc Mon Sep 17 00:00:00 2001 From: james-haytko_nwx Date: Thu, 26 Feb 2026 14:19:38 -0600 Subject: [PATCH 2/3] Move multi-pass instructions to agent system prompt The three review passes (grammar, style, Netwrix standards) now live in the agent's system.md rather than the workflow prompt. The workflow prompt is simplified back to "identify ALL issues per your instructions." Generated with AI Co-Authored-By: Claude Code --- .github/workflows/claude-documentation-reviewer.yml | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/.github/workflows/claude-documentation-reviewer.yml b/.github/workflows/claude-documentation-reviewer.yml index 44cd329be1..be88af0d32 100644 --- a/.github/workflows/claude-documentation-reviewer.yml +++ b/.github/workflows/claude-documentation-reviewer.yml @@ -89,15 +89,7 @@ jobs: 1. Use the Read tool to read each file in full. Do not look at the PR diff yet. - 2. Review the document in three focused passes. In each pass, go through every sentence and paragraph in the document: - - Pass 1 — Grammar: Check every sentence for sentence fragments, run-on sentences, ambiguous pronoun references, subject-verb agreement errors, misplaced or dangling modifiers, and any other grammatical errors. - - Pass 2 — Style: Check every sentence for active vs. passive voice, present vs. other tenses, use of second person ("you"), informal or unprofessional language, anthropomorphism, use of first person plural ("we"), nominalizations, redundancy, and any other style issues from your instructions. - - Pass 3 — Netwrix standards: Check every sentence for incorrect product names, undefined acronyms on first use, incorrect or non-standard terminology, and any other Netwrix-specific conventions from your instructions. - - Compile a single list of all issues found across all three passes before moving to the next step. + 2. Identify ALL issues in the document per your instructions. 3. Use `gh pr diff ${{ github.event.pull_request.number }}` to get the list of lines that were added or modified in this PR. From c965c67fee119290d0c05773a36862599d04d687 Mon Sep 17 00:00:00 2001 From: james-haytko_nwx Date: Thu, 26 Feb 2026 14:20:35 -0600 Subject: [PATCH 3/3] Exclude PRs targeting main from documentation reviewer Generated with AI Co-Authored-By: Claude Code --- .github/workflows/claude-documentation-reviewer.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/claude-documentation-reviewer.yml b/.github/workflows/claude-documentation-reviewer.yml index be88af0d32..577f2f9ae1 100644 --- a/.github/workflows/claude-documentation-reviewer.yml +++ b/.github/workflows/claude-documentation-reviewer.yml @@ -3,6 +3,8 @@ name: Documentation Reviewer on: pull_request_target: types: [opened, edited, reopened, synchronize] + branches-ignore: + - main paths: - '**.md'