Skip to content

chore(gitignore): ignore generated CLAUDE.md + .mcp.json#5

Merged
wcjord merged 1 commit into
mainfrom
chore/gitignore-mcp-json
Jun 4, 2026
Merged

chore(gitignore): ignore generated CLAUDE.md + .mcp.json#5
wcjord merged 1 commit into
mainfrom
chore/gitignore-mcp-json

Conversation

@wcjord

@wcjord wcjord commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

What

Add the sync-copilot-to-claude-generated files (.mcp.json, .mcp.json.generated_by_sync, and where missing CLAUDE.md) to .gitignore.

Why

Per pangeachat/.github#171, sync-copilot-to-claude.py fans out the canonical .github/.mcp.json into every sibling repo's root on every Claude Code SessionStart. The generated copies should be gitignored — same pattern as the existing CLAUDE.md ignore. For the synapse-module repos where CLAUDE.md was never gitignored, this PR also closes that older gap.

Single source of truth at .github/.mcp.json.

Testing

.gitignore-only. git check-ignore matches each pattern in this repo. No code or runtime surface, so no unit/integration/smoke buckets apply per doc/config-only exemption.

Deploy Notes

None.

Summary by CodeRabbit

  • Chores
    • Updated version control configuration to exclude additional system and generated files from the repository.

@coderabbitai

coderabbitai Bot commented Jun 4, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 9dfe2e84-e5bf-441b-acd8-ff74790d4474

📥 Commits

Reviewing files that changed from the base of the PR and between 0c328bb and e99cc1e.

📒 Files selected for processing (1)
  • .gitignore

📝 Walkthrough

Walkthrough

The pull request updates the repository's .gitignore configuration file to exclude macOS system artifacts and Claude/MCP-related generated outputs from version control, ensuring these temporary and platform-specific files do not clutter the repository.

Changes

Repository Ignore Configuration

Layer / File(s) Summary
Ignore patterns for macOS and generated files
.gitignore
Added .DS_Store entry for macOS system files and new rules for Claude/MCP-generated outputs (CLAUDE.md, .mcp.json, .mcp.json.generated_by_sync) with clarifying comments.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through the git, so clean,
No .DS_Store clutter in sight to be seen,
Claude's traces fade, MCP files too,
The repo stays tidy, all shiny and new! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding entries to .gitignore for generated CLAUDE.md and .mcp.json files, which matches the actual modifications to .gitignore.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/gitignore-mcp-json

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

@wcjord wcjord merged commit 8c2906c into main Jun 4, 2026
0 of 9 checks passed
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