Skip to content

chore: init Claude Code setup#3943

Open
skwowet wants to merge 6 commits intomainfrom
chore/claude-code-setup
Open

chore: init Claude Code setup#3943
skwowet wants to merge 6 commits intomainfrom
chore/claude-code-setup

Conversation

@skwowet
Copy link
Copy Markdown
Collaborator

@skwowet skwowet commented Mar 23, 2026

Note

Low Risk
Only adds developer tooling/config and documentation; no runtime or production code paths are changed.

Overview
Adds Claude Code repository setup by committing .claude/settings.json (enabling TypeScript and Pyright LSP plugins) and a new CLAUDE.md with project and coding guidelines.

Updates .gitignore to stop ignoring the entire .claude directory/CLAUDE.md and instead ignore only the local override file .claude/settings.local.json.

Written by Cursor Bugbot for commit 83ee8b3. This will update automatically on new commits. Configure here.

Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
@skwowet skwowet self-assigned this Mar 23, 2026
@skwowet skwowet marked this pull request as ready for review April 1, 2026 15:31
Copilot AI review requested due to automatic review settings April 1, 2026 15:31
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

2 similar comments
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

Copy link
Copy Markdown
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 initial Claude Code configuration and repository guidance for working in the CDP monorepo.

Changes:

  • Introduces CLAUDE.md with an overview of CDP, repo structure, and preferred engineering patterns.
  • Adds a committed .claude/settings.json enabling TypeScript and Python LSP plugins.
  • Updates .gitignore to track CLAUDE.md and ignore only local Claude settings overrides.

Reviewed changes

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

File Description
CLAUDE.md Adds project/context documentation and preferred patterns for contributors/assistants.
.gitignore Stops ignoring CLAUDE.md and narrows Claude ignores to local-only settings.
.claude/settings.json Adds shared Claude Code plugin configuration.

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

Copy link
Copy Markdown
Contributor

@joanagmaia joanagmaia left a comment

Choose a reason for hiding this comment

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

Given the size of the codebase it might make sense to be more specific on some additional practices on the backend. Also to add more coverage to frontend structure.

But I think this is already a good way to start. Once we merge let's all contribute it 👌


## Tech stack

TypeScript, Node.js, Express, PostgreSQL (pg-promise), Temporal, Kafka, Redis, OpenSearch, Zod, Bunyan, AWS S3.
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.

Missing frontend tech stack. At least Vue should be added here.

Comment on lines +199 to +200
# claude
.claude/settings.local.json
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.

Suggested change
# claude
.claude/settings.local.json
# claude
.claude/settings.local.json
.claude/cache/
.claude/tmp/
.claude/logs/
.claude/sessions/
.claude/todos/

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