Skip to content

Conversation

@domdomegg
Copy link
Member

Adds the Claude Code GitHub Actions workflow to enable Claude assistance on issues and pull requests when mentioned with @claude.

@domdomegg domdomegg enabled auto-merge (squash) August 15, 2025 15:17
# This should be safe given the repo is already public
allowed_tools: "Bash"

custom_instructions: |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it makes sense to add this as-is so we can do a general pass at these issues and PRs, but ultimately it seems like we need to give Claude a reference to MCP docs and/or specs to augment instructions and existing MCP knowledge.

For example, locally I am using in this server I created for referencing official docs: https://github.com/olaservo/mcp-advisor

Perhaps we should formalize a simple docs/specs server for this and other use cases?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, having a good docs server would be good! Although I was planning to get something basic up and running with this PR, and we can add extra tools / context to Claude as we discover where it falls short.

Also to be clear: this workflow only runs when someone explicitly tags Claude with @claude, not on all issues and PRs by default.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK! I approved but not sure if you meant you were adding something else here before making this active, so I held off on merging just in case.

@domdomegg domdomegg requested a review from olaservo August 15, 2025 17:40
@domdomegg domdomegg merged commit 4f953be into main Aug 15, 2025
29 of 30 checks passed
@domdomegg domdomegg deleted the adamj/add-claude-workflow branch August 15, 2025 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants