Skip to content

Add Claude Code Review GitHub Workflow#7

Merged
oskarszoon merged 4 commits into
mainfrom
add-claude-github-actions-1760528175638
Oct 16, 2025
Merged

Add Claude Code Review GitHub Workflow#7
oskarszoon merged 4 commits into
mainfrom
add-claude-github-actions-1760528175638

Conversation

@oskarszoon

Copy link
Copy Markdown
Contributor

🤖 Installing Claude Code GitHub App

This PR adds a GitHub Actions workflow that enables Claude Code integration in our repository.

Important Notes

  • This workflow won't take effect until this PR is merged
  • Claude gets access to the entire PR or issue context including files, diffs, and previous comments

There's more information in the Claude Code action repo.

@oskarszoon oskarszoon force-pushed the add-claude-github-actions-1760528175638 branch from 5c593cb to 21915b3 Compare October 15, 2025 11:44
Comment thread .github/workflows/claude-code-review.yml
@oskarszoon oskarszoon force-pushed the add-claude-github-actions-1760528175638 branch from 59c29c8 to 08b20bc Compare October 15, 2025 13:04
@oskarszoon oskarszoon marked this pull request as ready for review October 15, 2025 15:33
Copilot AI review requested due to automatic review settings October 15, 2025 15:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds Claude Code integration to the repository by introducing a GitHub Actions workflow for automated code review along with comprehensive documentation and agent configurations.

  • Adds GitHub Actions workflow for automated Claude-powered code reviews with different triggers for internal and fork PRs
  • Provides comprehensive project documentation in CLAUDE.md covering architecture, development commands, and Bitcoin-specific context
  • Introduces specialized Claude agents for test automation, Bitcoin expertise, backend architecture, and API testing

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.github/workflows/claude-code-review.yml GitHub Actions workflow for automated code reviews with Claude
CLAUDE.md Comprehensive project documentation for Claude Code integration
.claude/agents/test-writer-fixer.md Agent for automated test writing and test suite maintenance
.claude/agents/bitcoin-expert.md Bitcoin protocol expert agent for BSV-specific guidance
.claude/agents/backend-architect.md Backend architecture and system design specialist agent
.claude/agents/api-tester.md API testing and performance validation specialist agent

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/claude-code-review.yml
Comment thread .claude/agents/test-writer-fixer.md
Comment thread .claude/agents/api-tester.md
@oskarszoon oskarszoon merged commit 2f6663c into main Oct 16, 2025
10 of 11 checks passed
@oskarszoon oskarszoon deleted the add-claude-github-actions-1760528175638 branch October 16, 2025 10:40
oskarszoon added a commit to oskarszoon/teranode that referenced this pull request May 22, 2026
Adds a deep-clone redactor that walks the Settings struct via reflection
and replaces any field tagged redact:"true" with a placeholder. Tags 10
known secret fields: GRPCAdminAPIKey, Coinbase.{UserPwd, P2PPrivateKey,
WalletPrivateKey, SlackToken}, P2P.PrivateKey, Alert.P2PPrivateKey,
RPC.{RPCPass, RPCLimitPass}, BlockAssembly.MinerWalletPrivateKeys.

Prepares for CodeQL alert bsv-blockchain#7 fix.
oskarszoon added a commit to oskarszoon/teranode that referenced this pull request May 22, 2026
Closes CodeQL alert bsv-blockchain#7. PrintSettings was marshaling the full Settings
struct to Info logs, exposing GRPCAdminAPIKey and 9 other tagged secret
fields. PrintSettings now calls settings.Redact before marshal so the
log dump shows only non-secret configuration.
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.

4 participants