chore: Test editorial agents workflow #1065

justinegeffen · 2026-01-26T21:32:36Z

No description provided.

Added Topic Structure Agent documentation for auditing compliance with CTRT framework. Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Added voice-tone guidelines for documentation PRs, focusing on consistent voice, active tone, and present tense usage. Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

…nto justine-claude

Add changelog-formatter.skill.md to enable manual formatting and style checking of Seqera changelogs (Platform, Wave, Nextflow, Fusion). The skill applies universal style rules from the Seqera changelog style guide: - Past tense, active voice, sentence case - Product-specific requirements (attribution for Wave/Nextflow) - Quality checks for consistency and formatting Location: .claudeskills/ (standard location for Claude Code skills)

Add automated formatting and style enforcement for Platform Cloud and Enterprise changelogs. Components added: 1. Manual skill (.claudeskills/platform-changelog-formatter.skill.md) - Format individual Platform changelogs on-demand - Applies component-based organization and universal style rules - Platform-only scope (Cloud and Enterprise) 2. Automated PR review agent (.github/.claude/agents/platform-changelog-style.md) - Automatically reviews Platform changelog changes in PRs - Enforces past tense, active voice, Studios capitalization - Requires component organization (Studios, Compute, Pipelines, etc.) - Fails PR if critical style violations found 3. Review configuration update (.github/.claude/agents/review-config.yaml) - Registered platform-changelog-style agent - Triggers on all Cloud/Enterprise changelog changes - Does NOT apply to Wave/Nextflow/Fusion changelogs Scope: ONLY Seqera Platform Cloud and Enterprise changelogs - changelog/seqera-cloud/*.md - changelog/seqera-enterprise/*.md Wave, Nextflow, and Fusion changelogs remain unchanged (keep attribution format).

Removed changelog-formatter.skill.md as it's redundant with the Platform-specific formatter. We only need platform-changelog-formatter.skill.md which handles Cloud and Enterprise changelogs. Wave, Nextflow, and Fusion changelogs will remain in their current format (with attribution) and don't need automated formatting.

…ASK] Added explicit note in platform-changelog-formatter skill that component headers MUST be level 3 headers (###), NOT bold (**Component**). Updated in two places: - Section 4 (component organization): Added CRITICAL note - Section 7 (quality checks): Clarified the check This ensures components are formatted as: ### Studios NOT: **Studios**

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

- Move platform-changelog-style.md to .claude/agents/ - Remove duplicate .github/.claude/ directory - Align with Claude Code standard architecture Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

- Delete ./lint script (linter doesn't exist yet) - Simplify CLAUDE.md to focus on agent-based reviews - Remove all references to ../Claude/Linting/claude-md-linter.js - Keep terminology standards (RNA-Seq rules, product names) - Document /review command usage instead - Agents already handle terminology, capitalization, and structure checks Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

- Delete .claude/agents/topic-structure.md - Delete .claude/agents/inclusive-language.md - Update review-config.yaml to remove agent configs and related severity items - Update review.md to remove agents from comprehensive and new-content profiles - Update editorial-review/SKILL.md to remove from agent lists - Remove topic-structure-review and inclusive-review jobs from docs-review.yml - Update workflow summary to reflect remaining 4 agents - Remove agents from CLAUDE.md agent list Keeping core agents: voice-tone, terminology, clarity, punctuation, page-structure Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

- Delete .claude/agents/platform-changelog-style.md - Agent was not connected to any workflows or configs - Functionality is covered by platform-changelog-formatter skill - Skill does the actual formatting work, agent was redundant Active agents now: voice-tone, terminology, clarity, punctuation, page-structure, docs-fix Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

@claude

- Clarify fix commands: mention both /fix-docs and @claude fix for different workflows - Remove absolute path reference (/Users/justine.geffen/work/docs/) - Remove misplaced 'Review output format' section (74 lines) - Section contained agent instructions, not user documentation - Included typo 'Googl Guide' and 'Non-american' - Was redundant with agent descriptions - File now focused on user-facing documentation workflow Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

- Update docs-review.yml to watch Platform Enterprise and Cloud docs - Update claude-pr-review.yml to watch Platform docs only - Update review-config.yaml agent patterns to match Platform docs Now watches: - platform-enterprise_docs/**/*.md(x) - platform-cloud/docs/**/*.md(x) - platform-enterprise_versioned_docs/**/*.md(x) No longer watches: - fusion_docs, wave_docs, multiqc_docs, platform-api-docs, changelog This focuses automated reviews on Platform documentation only. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

- Clarify which directories get automated PR reviews - Separate directories into two categories: 1. Automated PR reviews (Platform docs only) 2. Manual review only (other products) - Add file counts for context - Matches actual workflow paths in docs-review.yml and claude-pr-review.yml Now accurately documents that only Platform Enterprise and Cloud docs trigger automated agent reviews, while other products need manual /review. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

netlify · 2026-01-26T21:32:43Z

❌ Deploy Preview for seqera-docs failed. Why did it fail? →

Name	Link
🔨 Latest commit	`cf7759f`
🔍 Latest deploy log	https://app.netlify.com/projects/seqera-docs/deploys/6978b02560bf750008e09330

github-actions · 2026-01-26T21:35:02Z

📝 Documentation Review Summary

Check	Status
Page Structure	❌ failure
Voice/Tone	❌ failure
Terminology	❌ failure
Clarity	❌ failure

Review powered by Claude Code SME agents

To apply fixes: Comment /fix-docs on this PR

The claude-code-action requires OIDC token to authenticate. Added id-token: write permission to both workflows: - docs-review.yml - claude-pr-review.yml This fixes the error: 'Could not fetch an OIDC token' Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

github-actions · 2026-01-26T21:36:38Z

📝 Claude Editorial Review

Review completed - check logs for details.

Powered by Claude Code editorial-review skill
To apply suggested fixes, comment: @claude fix editorial issues

The claude-code-action expects 'direct_prompt' not 'prompt'. Updated all workflow steps to use the correct parameter name. This fixes the warning: 'Unexpected input(s) prompt' Changed in both workflows: - docs-review.yml (5 occurrences) - claude-pr-review.yml (2 occurrences) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

github-actions · 2026-01-26T21:37:22Z

📝 Documentation Review Summary

Check	Status
Page Structure	❌ failure
Voice/Tone	✅ success
Terminology	✅ success
Clarity	✅ success

Review powered by Claude Code SME agents

To apply fixes: Comment /fix-docs on this PR

github-actions · 2026-01-26T21:38:14Z

📝 Claude Editorial Review

Review completed - check logs for details.

Powered by Claude Code editorial-review skill
To apply suggested fixes, comment: @claude fix editorial issues

The claude-code-action posts comments directly to PRs. The Post Structure Results step tried to read 'structure-results.md' which doesn't exist, causing: ENOENT: no such file or directory Removed the redundant post-result step. The action handles PR comments automatically. This fixes error: "ENOENT: no such file or directory, open 'structure-results.md'" Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

The claude-code-action posts comments directly to PRs. The Post Review Comment step tried to read CLAUDE_OUTPUT env var which doesn't exist, resulting in unhelpful 'check logs' message. Removed the redundant post-comment step. The action handles PR comments automatically. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

github-actions · 2026-01-26T21:43:13Z

📝 Documentation Review Summary

Check	Status
Page Structure	✅ success
Voice/Tone	✅ success
Terminology	✅ success
Clarity	✅ success

Review powered by Claude Code SME agents

To apply fixes: Comment /fix-docs on this PR

github-actions · 2026-01-26T21:43:43Z

📝 Documentation Review Summary

Check	Status
Page Structure	❌ failure
Voice/Tone	✅ success
Terminology	✅ success
Clarity	✅ success

Review powered by Claude Code SME agents

To apply fixes: Comment /fix-docs on this PR

christopher-hakkaart · 2026-01-26T21:56:07Z

.claude/agents/page-structure.md

+### 1. Heading Analysis
+
+Check for:
+- Exactly one H1 (should be in frontmatter `title:` or first `#`)


Can we just standardize on one or the other? E.g., always use title: and never H1?

christopher-hakkaart · 2026-01-26T21:59:32Z

.claude/agents/page-structure.md

+## Prerequisites
+
+- Nextflow 23.04 or later
+- A Seqera Platform workspace with Developer role or higher
+- Your pipeline repository accessible to Seqera Platform
+


My personal preference here is to use headings as headings, rather than introductions to points or steps. E.g.,

H2 Title

You need the following to get started:

x

y

z

christopher-hakkaart · 2026-01-26T22:01:07Z

.claude/agents/punctuation.md

+- **Hyphens (-)**: For compound words and line breaks
+- **Consistent spacing**: No spaces around em dashes, spaces around en dashes in ranges
+
+### Heading Punctuation


Maybe add a point about sentence case here? I always seem to be fighting AI on this, but maybe I'm an edge case.

christopher-hakkaart · 2026-01-26T22:02:12Z

.claude/agents/punctuation.md

+
+Process files by:
+1. Parse markdown to identify different content types
+2. Skip code blocks and inline code


Should a point about code blocks be added here? Check indentation? Make sure they are typed?

christopher-hakkaart · 2026-01-26T22:03:35Z

.claude/agents/terminology.md

+- Abbreviations: X issues
+```
+
+## Search Patterns


I like how you've done this. I was going to suggest running this alongside vale to detect all of this, but this makes just as much sense.

christopher-hakkaart

This all looks excellent. I've added a few minor comments, but they are extremely minor and more opinion-based. My view is that we should implement and iterate once we have some more data points of what it's doing well or poorly.

github-actions · 2026-01-27T12:33:35Z

📝 Documentation Review Summary

Check	Status
Page Structure	✅ success
Voice/Tone	✅ success
Terminology	✅ success
Clarity	✅ success

Review powered by Claude Code SME agents

To apply fixes: Comment /fix-docs on this PR

justinegeffen and others added 30 commits September 15, 2025 19:53

Linter work

d0950b4

Create topic-structure.md for documentation auditing

613f61d

Added Topic Structure Agent documentation for auditing compliance with CTRT framework. Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Create page-structure.md

43d076e

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Create voice-tone.md for documentation standards

afb5539

Added voice-tone guidelines for documentation PRs, focusing on consistent voice, active tone, and present tense usage. Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Create terminology.md

ece86cd

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Create clarity.md

e05de98

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Create inclusive-language.md

c3f750f

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Create docs-fix.md

122ab83

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Create review-config.yaml

5ecd73a

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Create docs-review.yml

fe66bba

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Merge branch 'master' into justine-claude

2c302fa

Merge branch 'master' into justine-claude

1f347e4

Merge branch 'master' into justine-claude

fbe36bd

Merge branch 'master' into justine-claude

4e60358

Merge branch 'master' into justine-claude

605d27b

Apply suggestion from @github-actions[bot]

e36c66b

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Merge branch 'justine-claude' of https://github.com/seqeralabs/docs i…

aac8b6f

…nto justine-claude

first commits for structural changes

8855a33

capitalization change

a6001da

platform rules

e174088

Merge branch 'master' into justine-claude

99e7646

Update .github/.claude/agents/docs-fix.md

da4f413

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Apply suggestion from @justinegeffen

cda4c7b

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Apply suggestion from @justinegeffen

1b34840

Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>

Merge branch 'master' into justine-claude

f185674

Standardize Claude Code config to .claude/ directory

848eb74

- Move platform-changelog-style.md to .claude/agents/ - Remove duplicate .github/.claude/ directory - Align with Claude Code standard architecture Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

justinegeffen and others added 8 commits January 26, 2026 23:02

Merge branch 'master' into justine-claude

c1b448a

test: verify agent workflows trigger correctly

a601a5d

justinegeffen and others added 2 commits January 26, 2026 23:35

Merge OIDC permission fix into test branch

646b78d

Merge direct_prompt fix into test branch

7dd605b

justinegeffen and others added 3 commits January 26, 2026 23:40

Merge workflow fixes into test branch

283ad2c

christopher-hakkaart reviewed Jan 26, 2026

View reviewed changes

Merge branch 'master' into test-agents-workflow

cf7759f

chore: Test editorial agents workflow #1065

Are you sure you want to change the base?

chore: Test editorial agents workflow #1065

Uh oh!

Conversation

justinegeffen commented Jan 26, 2026

Uh oh!

netlify bot commented Jan 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ Deploy Preview for seqera-docs failed. Why did it fail? →

Uh oh!

github-actions bot commented Jan 26, 2026

📝 Documentation Review Summary

Uh oh!

github-actions bot commented Jan 26, 2026

📝 Claude Editorial Review

Uh oh!

github-actions bot commented Jan 26, 2026

📝 Documentation Review Summary

Uh oh!

github-actions bot commented Jan 26, 2026

📝 Claude Editorial Review

Uh oh!

github-actions bot commented Jan 26, 2026

📝 Documentation Review Summary

Uh oh!

github-actions bot commented Jan 26, 2026

📝 Documentation Review Summary

Uh oh!

christopher-hakkaart Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

christopher-hakkaart Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

christopher-hakkaart Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

christopher-hakkaart Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

christopher-hakkaart Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

christopher-hakkaart left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jan 27, 2026

📝 Documentation Review Summary

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

netlify bot commented Jan 26, 2026 •

edited

Loading