fix: update .gitignore files to include additional ignored patterns by shunkakinoki · Pull Request #1367 · shunkakinoki/dotfiles

shunkakinoki · 2026-04-05T10:03:21Z

Summary by cubic

Expand repo and global .gitignore to ignore common tool caches (e.g., .claude, .cursor, .paperclip, .serena, .aider tags) and JJ’s .jj directory to prevent accidental commits. Also update Worktrunk config to exclude .paperclip in the copy-ignored step.

^{Written for commit c71157d. Summary will update on new commits.}

mesa-dot-dev · 2026-04-05T10:03:26Z

You do not have enough credits to review this pull request. Please purchase more credits to continue.

coderabbitai · 2026-04-05T10:03:36Z

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 155ad806-13b9-4f2d-b6cd-67072b55e4fc

📥 Commits

Reviewing files that changed from the base of the PR and between b448045 and c71157d.

📒 Files selected for processing (3)

.gitignore
config/worktrunk/config.toml
home-manager/programs/git/.gitignore.global

Disabled knowledge base sources:

Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📝 Walkthrough

Summary by CodeRabbit

Chores
- Updated ignore patterns in project and global Git configuration for tool-related directories and cache files.
- Modified worktree configuration to exclude additional directories from synchronization.

Walkthrough

This pull request expands ignore patterns across multiple configuration files to exclude directories and caches from various AI development tools: .claude, .codex, .conductor, .crush, .cursor, .paperclip, .aider.tags.cache.v4, .serena, and .jj are added to .gitignore and Git's global ignore list, while .paperclip/ is also added to the worktree copy-step exclusion list.

Changes

Cohort / File(s)	Summary
Gitignore Patterns `.gitignore`, `home-manager/programs/git/.gitignore.global`	Added ignore patterns for AI tool directories (`.claude`, `.codex`, `.conductor`, `.crush`, `.cursor`, `.paperclip`, `.aider.tags.cache.*`, `.serena`) and JJ version control (`.jj`); `.claude/settings.json` remains unignored in root `.gitignore`.
Worktree Configuration `config/worktrunk/config.toml`	Added `.paperclip/` to the worktree copy-step exclusion list alongside existing cache and environment directories.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

refactor(git): reorganize global ignore entries #1256: Modifies home-manager/programs/git/.gitignore.global to add/reorganize global ignore patterns for local tooling directories (.jj, .conductor, .cursor, etc.).
refactor(git): reorganize .gitignore.global entries for clarity #1255: Updates home-manager/programs/git/.gitignore.global with additional AI/tooling ignore patterns, with overlapping pattern entries.
feat(worktrunk): upgrade to 0.33.0 and add copy-ignored excludes #1283: Modifies config/worktrunk/config.toml's worktree copy-step exclude list to introduce/update additional ignore patterns (.paperclip/, .jj).

Poem

🐰 A rabbit's nest grows cluttered fast,
With .claude caches, .cursor past,
So .gitignore grows long and wide,
To sweep the .paperclip dust aside—
Now chaos sleeps, our burrow's clean! 🌿

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch git-ignore

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.}

mesa-dot-dev · 2026-04-05T10:03:43Z

Mesa Description

TL;DR

Updated .gitignore files to include additional patterns for ignored files.

What changed?

Modified various .gitignore files across the repository to expand the list of ignored patterns, improving repository cleanliness and preventing unwanted files from being committed. Specific file changes are not available in the provided summary.

^{Description generated by Mesa. Update settings}

gemini-code-assist

Code Review

This pull request updates the repository and global gitignore files to include various AI-related directories and the Jujutsu (.jj) directory, while also updating the worktrunk configuration to exclude .paperclip/. Feedback was provided regarding a bug where the modified .claude ignore pattern prevents the tracking of settings.json, as well as suggestions to resolve redundant patterns and avoid global ignores for directories containing shared configurations.

gemini-code-assist · 2026-04-05T10:04:44Z

 # AI
-.claude/*
+.aider.tags.cache.v4
+.claude


Changing .claude/* to .claude will cause Git to ignore the entire directory. When a directory is ignored, Git does not process negation patterns (like !) for files inside that directory. To keep settings.json tracked while ignoring other files in .claude, you must use the .claude/* pattern.

.claude/*

gemini-code-assist · 2026-04-05T10:04:44Z

@@ -1,7 +1,14 @@
 # AI
-.claude/*
+.aider.tags.cache.v4


The pattern .aider.tags.cache.v4 is redundant because .aider* is already defined on line 78 of this file. Additionally, using a version-specific pattern like v4 is brittle; if you wish to keep this entry at the top for organization, consider a version-agnostic pattern like .aider.tags.cache.* to match the style used in the global gitignore.

gemini-code-assist · 2026-04-05T10:04:44Z

@@ -1,7 +1,11 @@
 # AI
+.aider.tags.cache.*/
+.claude/


Adding .claude/ to the global gitignore can be problematic for repositories that intend to track specific configuration files within that directory (such as settings.json in this repository). While repo-level negations can override global ignores, they require explicitly un-ignoring the directory first. It is generally safer to keep tool-specific ignores that might contain shared configuration at the repository level.

Copilot

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates ignore configuration to cover additional AI/editor tooling artifacts and improves worktree copy behavior by excluding more generated directories.

Changes:

Expand ignored patterns in the repo and global Git ignore configs (AI tool dirs/files + JJ .jj).
Update Worktrunk’s copy-ignored step to exclude .paperclip/.

Reviewed changes

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

File	Description
home-manager/programs/git/.gitignore.global	Adds additional AI tool directories to the global ignore list.
config/worktrunk/config.toml	Excludes `.paperclip/` from the `copy-ignored` step.
.gitignore	Expands repo-level ignores for AI tools and adds `.jj`.

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

Copilot · 2026-04-05T10:09:56Z

@@ -1,7 +1,11 @@
 # AI
+.aider.tags.cache.*/
+.claude/


With .claude/ ignored globally, repositories that want to keep a whitelisted file like .claude/settings.json must also unignore the directory itself (e.g., !.claude/) in the repo ignore file; otherwise the nested unignore may not work. Given this PR also introduces a repo-level whitelist for .claude/settings.json, consider changing the global ignore to only ignore contents (e.g., .claude/*) or document/standardize the required !.claude/ + !.claude/settings.json pair in repo .gitignore.

Suggested change

.claude/

.claude/*

Copilot · 2026-04-05T10:09:57Z

@@ -1,7 +1,11 @@
 # AI
+.aider.tags.cache.*/


The global ignore uses a directory-only wildcard pattern (.aider.tags.cache.*/), while the repo .gitignore ignores a specific versioned entry (.aider.tags.cache.v4). This split is easy to drift and may miss future versions or file-vs-directory differences. Consider standardizing on a single pattern that covers all versions consistently across both files (e.g., a wildcard that matches versioned cache names regardless of whether they’re files or directories).

Suggested change

.aider.tags.cache.*/

.aider.tags.cache.*

fix: update .gitignore files to include additional ignored patterns

c71157d

Copilot AI review requested due to automatic review settings April 5, 2026 10:03

shunkakinoki merged commit 6191f23 into main Apr 5, 2026
23 of 29 checks passed

shunkakinoki deleted the git-ignore branch April 5, 2026 10:03

gemini-code-assist Bot reviewed Apr 5, 2026

View reviewed changes

Copilot AI reviewed Apr 5, 2026

View reviewed changes

Copilot started reviewing on behalf of shunkakinoki April 5, 2026 11:01 View session

coderabbitai Bot mentioned this pull request Apr 13, 2026

chore: add .codex to global gitignore #1470

Merged

1 task

coderabbitai Bot mentioned this pull request May 19, 2026

feat: 4-layer secret leak prevention #1811

Merged

6 tasks

Conversation

shunkakinoki commented Apr 5, 2026 • edited by cubic-dev-ai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by cubic

Uh oh!

mesa-dot-dev Bot commented Apr 5, 2026

Uh oh!

Uh oh!

coderabbitai Bot commented Apr 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Summary by CodeRabbit

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

mesa-dot-dev Bot commented Apr 5, 2026

Mesa Description

TL;DR

What changed?

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

shunkakinoki commented Apr 5, 2026 •

edited by cubic-dev-ai Bot

Loading

coderabbitai Bot commented Apr 5, 2026 •

edited

Loading