-
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 processing.
|
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.
|
|
||
| 1. Select **Launchpad** > **Add pipeline**. | ||
| 2. Enter the repository URL. | ||
|
|
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.
| # 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: | |
| Use the terminology 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: | |
| Use the clarity 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: | |
| 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: | |
| const statusEmoji = { | ||
| 'success': '✅', | ||
| 'failure': '❌', | ||
| 'skipped': '⏭️', |
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.
| 'skipped': '⏭️', | |
| prompt: | | ||
| Use the docs-fix agent to apply fixes to all changed documentation files in this PR. | ||
| Apply fixes 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.
| Apply fixes 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.
| # 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 |
| - 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] |
| - 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 |
|
|
||
| # 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.
| # | |
| # |
| 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 |
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.
| 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 |
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.
| 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 |
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**
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.
| 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 |
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.
| 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 |
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.
| 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 |
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.
| 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 |
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.
| 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 |
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.
| 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 |
1 similar comment
📝 Documentation Review Summary
Review powered by Claude Code SME agents To apply fixes: Comment |
Uh oh!
There was an error while loading. Please reload this page.