feat(ci): add Claude workflows and project documentation

[andygruening]

Summary

PR title must follow Conventional Commits, e.g. feat(auth): short description.
See https://www.conventionalcommits.org

Changes

Testing

• yarn lint passes
• yarn typecheck passes
• yarn prepare (library build) succeeds
• Native layer unchanged or manually verified on device/simulator
• API.md updated if public exports changed

Related

[github-actions]

Claude finished @andygruening's task —— View job

---

PR Review Complete ✅

I've thoroughly reviewed this pull request that adds GitHub workflow configurations and comprehensive documentation.

Critical Issues Found

❌ PR Title Format Violation

Issue: The PR title "checkout repository in claude workflows" doesn't follow Conventional Commits format.

Required: According to AGENTS.md:119-121, the title must follow Conventional Commits, e.g. feat(ci): add Claude Code workflows and documentation.

Suggested title: feat(ci): add Claude workflows and project documentation

Review Findings by Category

✅ TypeScript API Surface Changes

• No changes to src/index.tsx - the public API surface remains unchanged
• API.md remains accurate - no updates needed since exports didn't change

✅ Native Layer Consistency

• No native changes - no modifications to android/ or ios/ directories
• Native layer consistency maintained - both platforms remain in sync

✅ GitHub Workflows Review

All new workflow files are well-configured:

1. .github/workflows/claude.yml - Claude AI assistant integration

   • ✅ Proper permissions and trigger conditions
   • ✅ Uses actions/checkout@v4 with minimal fetch depth

2. .github/workflows/claude-code-review.yml - Automated PR reviews

   • ✅ Triggers on PR events (opened, synchronize)
   • ✅ Contains the exact review criteria being used here
   • ✅ Uses sticky comments for review consistency

3. .github/workflows/ci.yml modification

   • ✅ Good change: Added pull_request: trigger
   • This enables CI to run on PRs, not just master pushes
   • Improves CI coverage as mentioned in AGENTS.md:135-141

✅ Documentation Quality

All new documentation files are comprehensive and well-structured:

• AGENTS.md - Excellent behavioral guidelines and project overview
• CLAUDE.md - Simple import of AGENTS.md (good DRY principle)
• CONTRIBUTING.md - Clear setup and workflow instructions
• TESTING.md - Honest about current state (no tests yet) with future plans
• SECURITY.md - Proper vulnerability reporting process
• CHANGELOG.md - Follows Keep a Changelog format
• .github/PULL_REQUEST_TEMPLATE.md - Good checklist template
• .github/dependabot.yml - Proper configuration for all package ecosystems

Configuration Analysis

✅ Dependabot Configuration

• Properly configured for root workspace, Expo example, and GitHub Actions
• Reasonable update schedule (weekly on Mondays)
• Appropriate PR limits to avoid spam

✅ Documentation Consistency

• All files cross-reference correctly (CLAUDE.md → AGENTS.md)
• Maintenance matrix in AGENTS.md shows good awareness of dependencies
• Commands and conventions are consistent across files

Summary

This is a well-executed infrastructure PR that adds essential CI/CD workflows and comprehensive documentation. The only blocking issue is the PR title format violation.

Recommendation: Update the PR title to follow Conventional Commits format, then this PR is ready to merge.

Files that work well together:

• Workflows complement the existing CI setup mentioned in AGENTS.md
• Documentation provides clear guidance for contributors
• Templates enforce the quality standards outlined in the docs

---

[github-actions]

Claude finished @andygruening's task —— View job

---

PR Review Complete ✅

I've thoroughly reviewed this pull request that adds GitHub workflow configurations and comprehensive documentation.

Review Findings by Category

✅ TypeScript API Surface Changes

• No changes to src/index.tsx - the public API surface remains unchanged
• API.md remains accurate - no updates needed since exports didn't change
• All 29 exported functions and types remain consistent

✅ Native Layer Consistency

• No modifications to android/ or ios/ directories
• Native layer consistency maintained - both platforms remain in sync
• No version mismatches or configuration drift introduced

✅ Conventional Commits Format

• PR Title: "feat(ci): add Claude workflows and project documentation"
• Format compliance: ✅ Correctly follows Conventional Commits format
• Type feat with scope (ci) and descriptive summary

✅ GitHub Workflows Analysis

All new workflow files are well-configured:

1. .github/workflows/claude.yml - Claude AI assistant integration

   • ✅ Proper permissions and trigger conditions
   • ✅ Uses actions/checkout@v4 with minimal fetch depth
   • ✅ Correct event handling for comments, issues, and reviews

2. .github/workflows/claude-code-review.yml - Automated PR reviews

   • ✅ Triggers appropriately on PR events (opened, synchronize)
   • ✅ Contains the exact review criteria being used
   • ✅ Uses sticky comments for review consistency
   • ✅ Proper permissions setup

3. .github/workflows/ci.yml modification

   • ✅ Good improvement: Added pull_request: trigger (line 6)
   • This enables CI to run on PRs, not just master pushes
   • Improves CI coverage as mentioned in AGENTS.md:135-141

✅ Documentation Quality Assessment

All new documentation files are comprehensive and well-structured:

• AGENTS.md - Excellent behavioral guidelines adapted from Andrej Karpathy

  • Clear coding principles (Think Before Coding, Simplicity First, Surgical Changes)
  • Comprehensive project overview and tech stack description
  • Useful maintenance matrix showing cross-dependencies

• CLAUDE.md - Clean import of AGENTS.md (good DRY principle)

• CONTRIBUTING.md - Clear setup and workflow instructions

  • Proper prerequisites and setup steps
  • Good repo structure table
  • Clear contribution requirements

• TESTING.md - Honest about current state with future plans

• SECURITY.md - Proper vulnerability reporting process

• CHANGELOG.md - Follows Keep a Changelog format

• .github/PULL_REQUEST_TEMPLATE.md - Comprehensive checklist

  • Enforces Conventional Commits in template
  • Good testing verification checklist
  • Matches the review criteria

✅ Configuration Files

• .github/dependabot.yml - Well-configured for all ecosystems
  • Covers root workspace, Expo example, and GitHub Actions
  • Reasonable update schedule and PR limits
  • Proper labeling strategy

🔍 Code Quality & Edge Cases

• No obvious bugs or type errors - all files are well-formed YAML and Markdown
• Permissions are appropriate - workflows have necessary but not excessive permissions
• No security concerns - no hardcoded secrets or unsafe practices
• Good error handling - workflows use proper conditional logic

📋 Cross-Reference Validation

• Documentation is internally consistent
• Commands referenced in docs match actual project structure
• Workflow triggers align with intended use cases
• Template requirements match AGENTS.md guidelines

Summary

This is a well-executed infrastructure PR that adds essential CI/CD workflows and comprehensive documentation. The code is high quality with no blocking issues identified.

Key strengths:

• Follows all repository conventions and standards
• Workflows complement existing CI setup
• Documentation provides clear guidance for contributors
• Templates enforce quality standards outlined in the docs
• No breaking changes to existing functionality

Recommendation: ✅ Approved - This PR is ready to merge.

---

[github-actions]

Claude finished @andygruening's task —— View job

---

PR Review Complete ✅

I've thoroughly reviewed this PR that adds GitHub workflow configurations and comprehensive documentation.

Review Checklist

• Analyze TypeScript API surface changes in src/index.tsx
• Verify API.md consistency with exports
• Check native layer consistency (android/ and ios/)
• Validate Conventional Commits format on PR title
• Review GitHub workflows for correctness
• Check documentation files for issues
• Look for bugs, type errors, and edge cases
• Provide comprehensive review feedback

Review Findings by Category

✅ TypeScript API Surface Changes

• No changes to src/index.tsx - the public API surface remains unchanged
• API.md remains accurate - no updates needed since exports didn't change
• All 29 exported functions and types remain consistent

✅ Native Layer Consistency

• No modifications to android/ or ios/ directories
• Native layer consistency maintained - both platforms remain in sync
• No version mismatches or configuration drift introduced

✅ Conventional Commits Format

• PR Title: "feat(ci): add Claude workflows and project documentation"
• Format compliance: ✅ Correctly follows Conventional Commits format
• Type feat with scope (ci) and descriptive summary

✅ GitHub Workflows Analysis

All new workflow files are well-configured:

1. .github/workflows/claude.yml - Claude AI assistant integration

   • ✅ Proper permissions and trigger conditions
   • ✅ Uses actions/checkout@v4 with minimal fetch depth
   • ✅ Correct event handling for comments, issues, and reviews

2. .github/workflows/claude-code-review.yml - Automated PR reviews

   • ✅ Triggers appropriately on PR events (opened, synchronize)
   • ✅ Contains the exact review criteria being used
   • ✅ Uses sticky comments for review consistency
   • ✅ Proper permissions setup
   • ✅ Excludes Dependabot PRs appropriately

3. .github/workflows/ci.yml modification

   • ✅ Good improvement: Added pull_request: trigger (line 6)
   • This enables CI to run on PRs, not just master pushes
   • Improves CI coverage as mentioned in AGENTS.md

✅ Documentation Quality Assessment

All new documentation files are comprehensive and well-structured:

• AGENTS.md - Excellent behavioral guidelines adapted from Andrej Karpathy

  • Clear coding principles (Think Before Coding, Simplicity First, Surgical Changes)
  • Comprehensive project overview and tech stack description
  • Useful maintenance matrix showing cross-dependencies

• CLAUDE.md - Clean import of AGENTS.md (good DRY principle)

• CONTRIBUTING.md - Clear setup and workflow instructions

• TESTING.md - Honest about current state with future plans

• SECURITY.md - Proper vulnerability reporting process

• CHANGELOG.md - Follows Keep a Changelog format

• .github/PULL_REQUEST_TEMPLATE.md - Comprehensive checklist

  • Enforces Conventional Commits in template
  • Good testing verification checklist
  • Matches the review criteria

✅ Configuration Files

• .github/dependabot.yml - Well-configured for all ecosystems
  • Covers root workspace, Expo example, and GitHub Actions
  • Reasonable update schedule and PR limits
  • Proper labeling strategy

🔍 Code Quality & Edge Cases

• No obvious bugs or type errors - all files are well-formed YAML and Markdown
• Permissions are appropriate - workflows have necessary but not excessive permissions
• No security concerns - no hardcoded secrets or unsafe practices
• Good error handling - workflows use proper conditional logic

📋 Cross-Reference Validation

• Documentation is internally consistent
• Commands referenced in docs match actual project structure
• Workflow triggers align with intended use cases
• Template requirements match AGENTS.md guidelines

Summary

This is a well-executed infrastructure PR that adds essential CI/CD workflows and comprehensive documentation. The code is high quality with no blocking issues identified.

Key strengths:

• Follows all repository conventions and standards
• Workflows complement existing CI setup
• Documentation provides clear guidance for contributors
• Templates enforce quality standards outlined in the docs
• No breaking changes to existing functionality

Recommendation: ✅ Approved - This PR is ready to merge.

---