Skip to content

clean claude files#563

Merged
spashii merged 1 commit into
mainfrom
clean-claude-md
May 11, 2026
Merged

clean claude files#563
spashii merged 1 commit into
mainfrom
clean-claude-md

Conversation

@spashii
Copy link
Copy Markdown
Member

@spashii spashii commented May 11, 2026

Summary by CodeRabbit

  • Documentation

    • Added comprehensive React best practices skill guide covering performance optimization, bundle size reduction, and re-render optimization.
    • Added PostHog integration guides for React Router v7 applications, including step-by-step setup instructions and example implementations.
  • Chores

    • Updated project configuration files.

Review Change Stack

@spashii spashii merged commit 4b6d044 into main May 11, 2026
7 of 8 checks passed
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 11, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 455316fc-9e0a-4952-9a80-4d552e241324

📥 Commits

Reviewing files that changed from the base of the PR and between b3df0d4 and c1b2046.

📒 Files selected for processing (11)
  • .gitignore
  • echo/.claude/skills/react-best-practices/AGENTS.md
  • echo/.claude/skills/react-best-practices/SKILL.md
  • echo/frontend/.claude/skills/integration-react-react-router-7-data/SKILL.md
  • echo/frontend/.claude/skills/integration-react-react-router-7-data/references/EXAMPLE.md
  • echo/frontend/.claude/skills/integration-react-react-router-7-data/references/basic-integration-1.0-begin.md
  • echo/frontend/.claude/skills/integration-react-react-router-7-data/references/basic-integration-1.1-edit.md
  • echo/frontend/.claude/skills/integration-react-react-router-7-data/references/basic-integration-1.2-revise.md
  • echo/frontend/.claude/skills/integration-react-react-router-7-data/references/basic-integration-1.3-conclude.md
  • echo/frontend/.claude/skills/integration-react-react-router-7-data/references/identify-users.md
  • echo/frontend/.claude/skills/integration-react-react-router-7-data/references/react-router-v7-data-mode.md

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


Walkthrough

This PR ships comprehensive documentation for PostHog integration into React Router v7 "Data mode" applications, adding a skill definition, complete working example project, four-phase workflow guides (plan/edit/revise/conclude), and detailed reference docs covering user identification patterns and setup specifics. Also includes a React Best Practices skill reference and minor .gitignore cleanup.

Changes

.gitignore Update

Layer / File(s) Summary
Ignore Rules
.gitignore
Removed .claude directory pattern and local doc patterns (CLAUDE.md, IMPLEMENTATION_PLANS/, notes_*.md). Added settings.local.json ignore rule.

React Best Practices Skill

Layer / File(s) Summary
Skill Definition
echo/.claude/skills/react-best-practices/SKILL.md
Defined Vercel React Best Practices skill with metadata, performance optimization use cases, priority-based categories (eliminating waterfalls, bundle optimization, performance, re-renders, advanced patterns), quick reference rules, and guidance to consult per-rule Markdown files.

PostHog Integration for React Router v7

Layer / File(s) Summary
Skill Overview
echo/frontend/.claude/skills/integration-react-react-router-7-data/SKILL.md
Skill metadata, four-step workflow sequence, reference file pointers, key principles (env vars, minimal changes, match example), framework guidelines (event handlers, useEffect timing), user identification via headers, error tracking guidance.
Example Project
echo/frontend/.claude/skills/integration-react-react-router-7-data/references/EXAMPLE.md
Complete working React Router v7 app: PostHog initialization, AuthContext with fake login, protected routes (home/profile/burrito), RootErrorBoundary, custom event capture (burrito_considered, user_logged_out), manual exception handling, route type definitions, Vite config, env example.
Workflow Phase 1: Planning
echo/frontend/.claude/skills/integration-react-react-router-7-data/references/basic-integration-1.0-begin.md
Event tracking plan instructions: scan for posthog.capture() usage, identify user/accounting opportunities, create .posthog-events.json with action events (name/description/files), require server-side event coverage, [STATUS] reporting format.
Workflow Phase 2: Implementation
echo/frontend/.claude/skills/integration-react-react-router-7-data/references/basic-integration-1.1-edit.md
Implementation instructions: use env vars, preserve integrations, place PostHog under existing services, add event properties, match example patterns, client-side identify() on login, match server-side distinct IDs, exception capture, completion checklist.
Workflow Phase 3: Validation
echo/frontend/.claude/skills/integration-react-react-router-7-data/references/basic-integration-1.2-revise.md
Validation instructions: check for project errors, verify components are used, run lint/build/format scripts only on edited files, report errors and linting outcomes.
Workflow Phase 4: Conclusion
echo/frontend/.claude/skills/integration-react-react-router-7-data/references/basic-integration-1.3-conclude.md
Conclusion instructions: create PostHog "Analytics basics" dashboard via MCP using event names from code, read .posthog-events.json for event discovery, generate posthog-setup-report.md (wizard template with events table, dashboard/insight links), remove .posthog-events.json, report dashboard URL and report path.
Reference: User Identification
echo/frontend/.claude/skills/integration-react-react-router-7-data/references/identify-users.md
Cross-platform distinct_id and identify() patterns: Web/Android/iOS/React Native/Dart examples, person profile creation, best practices (early identify, unique IDs, reset on logout), reset() examples per platform, setting person properties, deep links for cross-platform linking.
Reference: React Router v7 Setup
echo/frontend/.claude/skills/integration-react-react-router-7-data/references/react-router-v7-data-mode.md
Complete setup: install posthog-js/@posthog/react, required env vars (VITE_PUBLIC_ prefix), client init and RouterProvider wrapping, TypeError troubleshooting (optional chaining), verify event capture, usePostHog access, identify on login, RootErrorBoundary error capture, PostHogCaptureOnViewed visibility tracking (custom properties, trackAllChildren, IntersectionObserver), server-side tracing via __add_tracing_headers.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~12 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch clean-claude-md

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

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