Conversation
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
2 similar comments
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
There was a problem hiding this comment.
Pull request overview
Adds initial Claude Code configuration and repository guidance for working in the CDP monorepo.
Changes:
- Introduces
CLAUDE.mdwith an overview of CDP, repo structure, and preferred engineering patterns. - Adds a committed
.claude/settings.jsonenabling TypeScript and Python LSP plugins. - Updates
.gitignoreto trackCLAUDE.mdand 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.
joanagmaia
left a comment
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
Missing frontend tech stack. At least Vue should be added here.
| # claude | ||
| .claude/settings.local.json |
There was a problem hiding this comment.
| # claude | |
| .claude/settings.local.json | |
| # claude | |
| .claude/settings.local.json | |
| .claude/cache/ | |
| .claude/tmp/ | |
| .claude/logs/ | |
| .claude/sessions/ | |
| .claude/todos/ |
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 newCLAUDE.mdwith project and coding guidelines.Updates
.gitignoreto stop ignoring the entire.claudedirectory/CLAUDE.mdand 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.