-
Notifications
You must be signed in to change notification settings - Fork 0
Add Claude Code GitHub Workflow #20
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
Conversation
PR Validation ✅All checks passed! Ready for review. Checks Performed:✓ Linting This is a complete dry-run of the deployment process, ensuring your changes will deploy successfully when merged. |
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.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: read | ||
| pull-requests: read | ||
| issues: read | ||
| id-token: write |
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.
Grant write permissions so review bot can comment
The workflow instructs Claude to leave feedback via gh pr comment, yet the job permissions restrict pull-requests and issues to read. With only read access the token that anthropics/claude-code-action receives cannot create review or issue comments, so the step will fail with a 403 or produce no visible output. The job needs pull-requests: write (and issues: write if issue comments are expected) for the review bot to post its results.
Useful? React with 👍 / 👎.
| permissions: | ||
| contents: read | ||
| pull-requests: read | ||
| issues: read | ||
| id-token: write | ||
| actions: read # Required for Claude to read CI results on PRs |
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.
Provide write permissions for Claude replies
When @claude is mentioned this workflow runs anthropics/claude-code-action to respond, but the job grants only pull-requests: read and issues: read. Without write permissions the token cannot post a comment or perform other updates, so every invocation will be unable to reply. Granting pull-requests: write and issues: write is required for the bot to function.
Useful? React with 👍 / 👎.
* Content and style updates. (#15) * Switched from hardcoded ports to using trop. * Updated `trop` article. * Ensure list items use article serif font (#13) * Further article adjustments. (#14) * Content updates * Dictionary additions. * Fix HTML spellcheck failures (#16) - Fixed typos in generic-testing blog post: - 'exapmle' → 'example' (3 instances) - 'actuall' → 'actually' (1 instance) - 'Algrebra' → 'Algebra' (1 instance) - Added technical terms to cspell dictionary: - microbenchmarks - Xcodes - metatype/metatypes All HTML spellcheck tests now pass. Co-authored-by: Claude <noreply@anthropic.com> * Fix markdown rendering in article titles (#17) This change ensures that markdown syntax in titles (like *italic* and **bold**) is properly rendered as HTML when displayed on pages, while being stripped for meta tags, browser titles, and Open Graph images. Changes: - Add stripMarkdown() function to remove markdown syntax for plain text contexts - Update page templates to render markdown in displayed titles using renderInlineMarkdown() - Strip markdown from titles in meta tags, Open Graph data, and browser title - Improve semantic HTML by using <h1> instead of <div> for page titles Fixes issue where asterisks and other markdown syntax appeared literally in article titles instead of being rendered as formatted text. Co-authored-by: Claude <noreply@anthropic.com> * The new learn-spelling command. * Justfile cleanup. * Spelling incosistency fix. * Add Claude Code GitHub Workflow (#20) * "Claude PR Assistant workflow" * "Claude Code Review workflow" * Updating gitignore for pending feature. * Fix ESLint quote style in learn-spelling.js Change single quotes to double quotes to comply with project ESLint config. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * Add Claude Code hook for auto-linting on file changes Configures a PostToolUse hook that runs after Write/Edit operations: - ESLint fix for js/jsx/ts/tsx/astro files - Spellcheck for md/mdx files 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
🤖 Installing Claude Code GitHub App
This PR adds a GitHub Actions workflow that enables Claude Code integration in our repository.
What is Claude Code?
Claude Code is an AI coding agent that can help with:
How it works
Once this PR is merged, we'll be able to interact with Claude by mentioning @claude in a pull request or issue comment.
Once the workflow is triggered, Claude will analyze the comment and surrounding context, and execute on the request in a GitHub action.
Important Notes
Security
There's more information in the Claude Code action repo.
After merging this PR, let's try mentioning @claude in a comment on any PR to get started!