Skip to content

chore: add Claude Code automations and fix CodeRabbit issues#35

Open
chitcommit wants to merge 3 commits intomainfrom
chore/claude-automations
Open

chore: add Claude Code automations and fix CodeRabbit issues#35
chitcommit wants to merge 3 commits intomainfrom
chore/claude-automations

Conversation

@chitcommit
Copy link
Contributor

Summary

  • Add project CLAUDE.md with architecture docs, commands, and conventions
  • Add Claude Code hooks (typecheck on .ts edit, block lock/dist files, block destructive bash)
  • Add Billy Bullshit agent proxy and /billy code review skill
  • Add /new-command, /security-scan skills and project-conventions auto-skill
  • Fix CodeRabbit review issues: SECURITY.md secret exposure checklist, hook-handlers credential handling, parity test error handling patterns

Test plan

  • Verify npx tsc --noEmit passes
  • Verify npm test passes
  • Verify .claude/settings.json hooks work in Claude Code sessions
  • Verify CLAUDE.md renders correctly

🤖 Generated with Claude Code

chitcommit and others added 3 commits March 3, 2026 21:27
… CLAUDE.md

Wire in project-level Claude Code configuration:
- CLAUDE.md with architecture, commands, conventions, and dependency docs
- chittyagent-billy subagent proxying to billy.chitty.cc review/roast/chat API
- /billy skill for user-invoked BS-scored code reviews
- /new-command skill for scaffolding CLI commands with tests
- /security-scan skill for running full security scan suite
- project-conventions skill (Claude-only) with coding patterns
- PreToolUse hooks blocking lock file and dist/ edits, destructive bash
- PostToolUse hook running tsc --noEmit after .ts edits
- Stop hook warning about uncommitted changes

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Replace hardcoded absolute paths in hooks with git rev-parse --show-toplevel
- Remove license reference from CLAUDE.md (MIT/AGPL discrepancy is pre-existing)
- Add text language identifier to architecture fenced block

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- scan-secrets.sh: use mktemp instead of fixed /tmp path, quote $revset
- SECURITY.md: list all fallback token names in rotation guidance
- direct-route.ts: use hasOwnProperty instead of in operator for CLI validation
- hook-handlers.ts: use || with trim() instead of ?? for token fallback
- parity_node.js: fix false-positive error handling test
- parity_py.py: fix false-positive error handling test

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 4, 2026 03:38
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 4, 2026

Warning

Rate limit exceeded

@chitcommit has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 14 minutes and 0 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Run ID: b8fdb5bc-0a5a-4f47-add7-bb8fd21fab3d

📥 Commits

Reviewing files that changed from the base of the PR and between f0cf4b7 and 6f9c541.

📒 Files selected for processing (11)
  • .claude/agents/chittyagent-billy.md
  • .claude/settings.json
  • .claude/skills/billy/SKILL.md
  • .claude/skills/new-command/SKILL.md
  • .claude/skills/project-conventions/SKILL.md
  • .claude/skills/security-scan/SKILL.md
  • CLAUDE.md
  • SECURITY.md
  • src/commands/hook-handlers.ts
  • tests/parity_node.js
  • tests/parity_py.py
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/claude-automations

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds Claude Code project guidance/automation assets and updates hook handler + parity tests to address prior review findings (notably around credential handling and error handling assertions).

Changes:

  • Added Claude Code documentation and configuration (CLAUDE.md, .claude/settings.json) plus several Claude “skills” and an agent definition.
  • Updated context-auth hook handler to avoid hardcoded credential fallbacks and to optionally resolve a local identity.
  • Simplified parity error-handling tests (Node/Python).

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 14 comments.

Show a summary per file
File Description
src/commands/hook-handlers.ts Updates ChittyCanon DB credential resolution and adds local identity scanning for context authentication.
tests/parity_py.py Simplifies the Python parity error-handling test assertion.
tests/parity_node.js Simplifies the Node parity error-handling test assertion.
SECURITY.md Adds a “Secret Exposure Response Checklist”.
CLAUDE.md Adds Claude Code project overview, architecture notes, and commands/conventions.
.claude/settings.json Adds Claude Code hook automation for file edits, destructive bash, and post-edit typechecking.
.claude/skills/security-scan/SKILL.md Adds a skill doc for running security scans/audit/workflow checks.
.claude/skills/project-conventions/SKILL.md Adds a skill doc describing repo conventions for code generation.
.claude/skills/new-command/SKILL.md Adds a skill doc for scaffolding a new CLI command + test.
.claude/skills/billy/SKILL.md Adds a skill doc for sending code to an external review service and a “roast” mode.
.claude/agents/chittyagent-billy.md Adds an agent definition for the external review service.
Comments suppressed due to low confidence (1)

.claude/skills/project-conventions/SKILL.md:44

  • This file-organization note says MCP lives at src/mcp/server.ts, but npm run mcp executes dist/mcp-server.js (built from src/mcp-server.ts). Update this pointer so contributors/agents edit the file that is actually used by the CLI.
- Plugins: `src/plugins/` (integrations by service: ai, chittyos, cloudflare, linear, neon)
- Types: `src/types/index.ts`
- MCP: `src/mcp/server.ts`


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants