Skip to content

Add Claude Code GitHub Workflow [skip-review]#15

Merged
daaain merged 5 commits intomainfrom
add-claude-github-actions-1755301571586
Aug 16, 2025
Merged

Add Claude Code GitHub Workflow [skip-review]#15
daaain merged 5 commits intomainfrom
add-claude-github-actions-1755301571586

Conversation

@daaain
Copy link
Copy Markdown
Owner

@daaain daaain commented Aug 15, 2025

🤖 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:

  • Bug fixes and improvements
  • Documentation updates
  • Implementing new features
  • Code reviews and suggestions
  • Writing tests
  • And more!

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

  • This workflow won't take effect until this PR is merged
  • @claude mentions won't work until after the merge is complete
  • The workflow runs automatically whenever Claude is mentioned in PR or issue comments
  • Claude gets access to the entire PR or issue context including files, diffs, and previous comments

Security

  • Our Anthropic API key is securely stored as a GitHub Actions secret
  • Only users with write access to the repository can trigger the workflow
  • All Claude runs are stored in the GitHub Actions run history
  • Claude's default tools are limited to reading/writing files and interacting with our repo by creating comments, branches, and commits.
  • We can add more allowed tools by adding them to the workflow file like:
allowed_tools: Bash(npm install),Bash(npm run build),Bash(npm run lint),Bash(npm run test)

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!

@daaain daaain merged commit 992c077 into main Aug 16, 2025
3 checks passed
cboos added a commit that referenced this pull request Mar 5, 2026
CodeRabbit items considered and intentionally not changed:

#8 (dag.md fenced blocks): No language identifiers on non-code blocks
    — project convention, not using markdownlint.

#10 (dag.py:204 recursive → iterative): Max recursion depth is bounded
    by session message count. Python's default limit (1000) is more than
    sufficient for any real conversation.

#11 (converter.py:463 queue ops at tail): Queue operations are metadata
    entries, not user-visible conversation content. Appending at tail is
    the intended behavior.

#12 (renderer.py:2012 current_render_session leak): By design — branch
    context applies to all subsequent messages until the next line start.
    The variable is reset when a new DAG-line starts.

#13 (session_nav.html:28 fork rows in expandable mode): Non-issue.
    Fork/branch nav items only appear in ToC mode (combined transcript
    and individual session pages). Expandable mode is only used in the
    project index page where fork items wouldn't be generated. Verified
    that individual session pages DO correctly include fork/branch nav.

#14 (message_styles.css:901 font quoting): Cosmetic stylelint preference,
    no stylelint configured in this project.

#15 (test_dag.py:44 error handling): Test helper — should fail loudly on
    bad test data, not silently skip.
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.

1 participant