chore: Configure editorial workflow to run once on PR creation

[justinegeffen]

Summary

Configures the editorial workflow to run automatically only once (on PR creation/reopen) instead of on every commit. This prevents excessive token consumption while still providing initial automated feedback.

Changes

Workflow behavior (.github/workflows/docs-review.yml)

• ✅ Runs automatically on PR creation (opened)
• ✅ Runs automatically on PR reopen (reopened)
• ❌ Does NOT run on subsequent commits (removed synchronize trigger)
• ✅ Manual re-runs available via workflow dispatch

Documentation updates

• .claude/README.md: Added manual re-run instructions with 5-step process
• CLAUDE.md: Updated all sections to clarify one-time automatic behavior

Rationale

Before:

• Workflow ran on every commit (synchronize event)
• High token consumption from repeated runs
• Re-ran automatically when applying inline suggestions
• No control over when reviews happened

After:

• Initial automated review on PR creation (feedback when needed most)
• Reviewers control subsequent re-runs (token efficiency)
• Clear documentation for manual re-run process
• Better cost management for large PRs with many fixes

Manual re-run process

After the initial automatic review, reviewers can re-run the workflow:

1. Go to Actions → Documentation Review
2. Click Run workflow
3. Select the PR branch
4. Choose review type (all, voice-tone, terminology, clarity)
5. Click Run workflow

Impact

• Token usage: Significantly reduced (1 automatic run vs N runs per PR)
• User experience: Still get immediate feedback on PR creation
• Control: Teams decide when to re-check after applying fixes
• Flexibility: Manual dispatch available anytime

🤖 Generated with Claude Code

[netlify]

✅ Deploy Preview for seqera-docs ready!

Name	Link
🔨 Latest commit	bf46557
🔍 Latest deploy log	https://app.netlify.com/projects/seqera-docs/deploys/6984aa65b57e2700087e0dd4
😎 Deploy Preview	https://deploy-preview-1099--seqera-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[Copilot]

Pull request overview

Adjusts the Claude-powered documentation editorial workflow to run automatically only on PR open/reopen (instead of on every commit) and updates contributor docs to explain the new one-time automatic behavior and manual re-run process.

Changes:

• Updated docs-review.yml to remove the synchronize pull_request trigger (run only on opened/reopened).
• Updated CLAUDE.md to clarify the workflow runs once automatically and to document the manual re-run steps.
• Updated .claude/README.md to document manual re-runs and the new trigger behavior.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
.github/workflows/docs-review.yml	Removes per-commit (synchronize) auto-runs; positions workflow_dispatch as the re-run mechanism.
CLAUDE.md	Updates workflow trigger/behavior documentation and adds a clearer manual re-run procedure.
.claude/README.md	Documents updated trigger behavior and adds manual re-run instructions for reviewers.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[justinegeffen]

Fixed workflow_dispatch issues

Thanks for the detailed feedback! I've addressed all three issues:

1. ✅ PR number now available for manual runs

Added new setup job that determines the PR number:

• For pull_request events: Extracts from github.event.pull_request.number
• For workflow_dispatch: Uses new required pr_number input
• All jobs now reference needs.setup.outputs.pr_number consistently

2. ✅ Manual dispatch can now post results

Removed event gating:

• Deleted if: github.event_name == 'pull_request' from comment posting
• Both consolidated review and summary jobs now work for manual runs
• Results post back to the PR specified in the input

3. ✅ review_type input now functional

Wired to job conditions:

# voice-tone job runs if:
if: ... && (needs.setup.outputs.review_type == 'all' || needs.setup.outputs.review_type == 'voice-tone')

# terminology job runs if:
if: ... && (needs.setup.outputs.review_type == 'all' || needs.setup.outputs.review_type == 'terminology')

Manual dispatch now respects the selected review type and only runs those agents.

Updated manual re-run instructions

Documentation now includes the PR number requirement:

1. Go to Actions → Documentation Review
2. Click Run workflow
3. Select your PR branch
4. Enter the PR number (new requirement)
5. Choose review type
6. Click Run workflow

The workflow will now correctly post inline suggestions and comments back to the specified PR! 🎉