Skip to content

Commit 46c5717

Browse files
committed
fix(ci): simplify to v1 with built-in slash command support
Based on maintainer feedback: - Move commands directly to .claude/commands/ (no symlinks needed) - Use v1 with minimal configuration (auto mode detection) - Remove unnecessary parameters (mode, use_sticky_comment, etc.) - v1 automatically finds commands in .claude/commands/
1 parent 6cc0be8 commit 46c5717

File tree

10 files changed

+93
-110
lines changed

10 files changed

+93
-110
lines changed

.claude/commands/link-review.md

Lines changed: 0 additions & 1 deletion
This file was deleted.

.claude/commands/link-review.md

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
---
2+
description: Review links in changed files for quality and security issues
3+
---
4+
5+
Review the links in the changed files and check for potential issues:
6+
7+
## Link Quality Checks
8+
1. **Broken Links**: Identify any links that might be broken or malformed
9+
2. **Outdated Links**: Check for links to deprecated resources or old documentation
10+
3. **Security**: Ensure no links to suspicious or potentially harmful sites
11+
4. **Best Practices**:
12+
- Links should use HTTPS where possible
13+
- Internal links should use relative paths
14+
- External links should be to stable, reputable sources
15+
16+
## Specific Checks for Anthropic Content
17+
- Links to Claude documentation should point to the latest versions
18+
- API documentation links should be current
19+
- Model documentation should reference current models, not deprecated ones
20+
- GitHub links should use the correct repository paths
21+
22+
## Report Format
23+
Provide a clear summary with:
24+
- ✅ Valid and well-formed links
25+
- ⚠️ Links that might need attention (e.g., HTTP instead of HTTPS)
26+
- ❌ Broken or problematic links that must be fixed
27+
28+
If all links look good, provide a brief confirmation.
29+
30+
Format your response as a helpful PR review comment.

.claude/commands/model-check.md

Lines changed: 0 additions & 1 deletion
This file was deleted.

.claude/commands/model-check.md

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
---
2+
description: Validate Claude model usage against current public models
3+
---
4+
5+
Review the changed files for Claude model usage.
6+
7+
First, fetch the current list of allowed models from:
8+
https://docs.anthropic.com/en/docs/about-claude/models/overview.md
9+
10+
Then check:
11+
1. All model references are from the current public models list
12+
2. Flag any deprecated models (older Sonnet 3.5, Opus 3 versions)
13+
3. Flag any internal/non-public model names
14+
4. Suggest using aliases ending in -latest for better maintainability
15+
16+
Provide clear, actionable feedback on any issues found.

.claude/commands/notebook-review.md

Lines changed: 0 additions & 1 deletion
This file was deleted.
Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
---
2+
description: Comprehensive review of Jupyter notebooks and Python scripts
3+
---
4+
5+
Review the changes to Jupyter notebooks and Python scripts in this PR. Please check for:
6+
7+
## Model Usage
8+
Verify all Claude model references against the current list at:
9+
https://docs.anthropic.com/en/docs/about-claude/models/overview.md
10+
- Flag any deprecated models (older Sonnet 3.5, Opus 3 versions)
11+
- Flag any internal/non-public model names
12+
- Suggest current alternatives when issues found
13+
- Recommend aliases ending in -latest for stability
14+
15+
## Code Quality
16+
- Python code follows PEP 8 conventions
17+
- Proper error handling
18+
- Clear variable names and documentation
19+
- No hardcoded API keys (use os.getenv("ANTHROPIC_API_KEY"))
20+
21+
## Notebook Structure
22+
- Clear introduction explaining what the notebook demonstrates and why it's useful
23+
- Configuration instructions (how to set up API keys, install dependencies, etc.)
24+
- Connecting explanations between cells that help users understand the flow
25+
- Clear markdown explanations between code cells
26+
- Logical flow from simple to complex
27+
- Outputs preserved for educational value
28+
- Dependencies properly imported
29+
30+
## Security
31+
- Check for any hardcoded API keys or secrets (not just Anthropic keys)
32+
- Ensure all sensitive credentials use environment variables (os.environ, getenv, etc.)
33+
- Flag any potential secret patterns (tokens, passwords, private keys)
34+
- Note: Educational examples showing "what not to do" are acceptable if clearly marked
35+
- Safe handling of user inputs
36+
- Appropriate use of environment variables
37+
38+
Provide a clear summary with:
39+
- ✅ What looks good
40+
- ⚠️ Suggestions for improvement
41+
- ❌ Critical issues that must be fixed
42+
43+
Format your response as a helpful PR review comment.

.github/slash-commands/link-review.md

Lines changed: 0 additions & 30 deletions
This file was deleted.

.github/slash-commands/model-check.md

Lines changed: 0 additions & 16 deletions
This file was deleted.

.github/slash-commands/notebook-review.md

Lines changed: 0 additions & 43 deletions
This file was deleted.

.github/workflows/claude-link-review.yml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,13 +23,8 @@ jobs:
2323
fetch-depth: 0
2424

2525
- name: Run Claude Link Review
26-
uses: anthropics/claude-code-action@beta
26+
uses: anthropics/claude-code-action@v1
2727
with:
28-
mode: agent
29-
use_sticky_comment: true
3028
anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
3129
github_token: ${{ secrets.GITHUB_TOKEN }}
32-
timeout_minutes: "30"
33-
allowed_bots: "*"
34-
experimental_slash_commands_dir: ${{ github.workspace }}/.github/slash-commands
3530
prompt: "/link-review"

0 commit comments

Comments
 (0)