-
Notifications
You must be signed in to change notification settings - Fork 5
[Automation] Claude agents and workflows #955
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
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>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
✅ Deploy Preview for seqera-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pre-commit formatting suggestions
Pre-commit hooks found some formatting changes. You can apply each suggestion directly by clicking the "Commit suggestion" button, or run pre-commit locally and commit again.
| # Maximum files per review batch (prevents timeout on large PRs) | ||
| max_files_per_batch: 20 | ||
|
|
||
| # Output format for PR comments |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # Output format for PR comments | |
| - major_change # Run when >50% of file changed | ||
|
|
||
| # File patterns (glob) | ||
| patterns: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| patterns: | |
.github/workflows/docs-review.yml
Outdated
| Use the inclusive-language agent to review these changed documentation files: | ||
| ${{ needs.changes.outputs.docs_files }} | ||
|
|
||
| Check for: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Check for: | |
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pre-commit formatting suggestions
Pre-commit hooks found some formatting changes. You can apply each suggestion directly by clicking the "Commit suggestion" button, or run pre-commit locally and commit again.
CLAUDE.md
Outdated
| # Focus only on RNA-Seq issues | ||
| ./lint --no-heading-case platform-enterprise_docs/ | ||
|
|
||
| # Focus only on heading issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # Focus only on heading issues | |
| # Focus only on heading issues |
CLAUDE.md
Outdated
| - Reason: [Google style guide principle or grammar rule] | ||
|
|
||
| + Line Y: [original text] | ||
| - Issue: [description of problem] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - Issue: [description of problem] | |
| - Issue: [description of problem] |
CLAUDE.md
Outdated
| - Specific line-by-line corrections | ||
| - Suggestions for improvement | ||
| - Explanations of Google style guide principles | ||
| - A revised version if requested |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - A revised version if requested | |
| - A revised version if requested |
lint
Outdated
|
|
||
| # Documentation Linter Shortcut | ||
| # Usage: ./lint [options] [directories...] | ||
| # |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| # | |
| # |
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pre-commit formatting suggestions
Pre-commit hooks found some formatting changes. You can apply each suggestion directly by clicking the "Commit suggestion" button, or run pre-commit locally and commit again.
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🪄 Pre-commit formatting suggestions
You can apply each suggestion via the GitHub UI, add a comment containing the keyword fix formatting or set up pre-commit locally and commit again.
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
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.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🪄 Pre-commit formatting suggestions
You can apply each suggestion via the GitHub UI, add a comment containing the keyword fix formatting or set up pre-commit locally and commit again.
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🪄 Pre-commit formatting suggestions
You can apply each suggestion via the GitHub UI, add a comment containing the keyword fix formatting or set up pre-commit locally and commit again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🪄 Pre-commit formatting suggestions
You can apply each suggestion via the GitHub UI, add a comment containing the keyword fix formatting or set up pre-commit locally and commit again.
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
1 similar comment
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🪄 Pre-commit formatting suggestions
You can apply each suggestion via the GitHub UI, add a comment containing the keyword fix formatting or set up pre-commit locally and commit again.
lint
Outdated
| else | ||
| # Pass all arguments through to the linter | ||
| node "$LINTER_PATH" "$@" | ||
| fi No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| fi | |
| fi |
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
- 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>
- Move platform-changelog-formatter to .claude/skills/platform-changelog-formatter/SKILL.md - Move editorial-review to .claude/skills/editorial-review/SKILL.md - Remove non-standard .claudeskills/ directory - Each skill now in its own subdirectory with SKILL.md file - Aligns with Claude Code skills architecture standard Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
- 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>
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
- 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>
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
- 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>
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>
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>
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>
Uh oh!
There was an error while loading. Please reload this page.