Skip to content

chore: gitignore tsbuildinfo files#188

Merged
jlia0 merged 1 commit intomainfrom
jlia0/gitignore-tsbuildinfo
Mar 10, 2026
Merged

chore: gitignore tsbuildinfo files#188
jlia0 merged 1 commit intomainfrom
jlia0/gitignore-tsbuildinfo

Conversation

@jlia0
Copy link
Collaborator

@jlia0 jlia0 commented Mar 10, 2026

Description

Add *.tsbuildinfo to .gitignore. These are TypeScript incremental build cache files generated by tsc --build and should not be tracked in version control.

Changes

  • Add *.tsbuildinfo pattern to .gitignore

Testing

  • Verified build still works with npm run build

🤖 Generated with Claude Code

@greptile-apps
Copy link

greptile-apps bot commented Mar 10, 2026

Greptile Summary

This PR adds *.tsbuildinfo to .gitignore to prevent TypeScript incremental build cache files from being tracked in version control — a correct and reasonable housekeeping change. However, it is incomplete: 6 .tsbuildinfo files are already tracked by git across the packages/ directory, and simply adding a .gitignore pattern does not untrack them. Those files will continue to be committed and show up in diffs until git rm --cached is run for each of them.

Key changes:

  • Adds *.tsbuildinfo pattern under the # TypeScript build output section in .gitignore

Issues found:

  • Pre-existing tracked *.tsbuildinfo files (packages/channels, packages/core, packages/main, packages/server, packages/teams, packages/visualizer) need to be explicitly removed from the git index via git rm --cached in a follow-up commit for this change to be fully effective.

Confidence Score: 3/5

  • Safe to merge as-is, but the intent of the change won't be fully realized without also untracking already-committed .tsbuildinfo files.
  • The .gitignore change itself is correct and harmless, but it is incomplete — 6 .tsbuildinfo files remain tracked in the git index, meaning they will still be committed on future changes. The PR needs a git rm --cached step to fully accomplish its stated goal.
  • .gitignore — the pattern is correct but the pre-existing tracked files must be removed from the index separately.

Important Files Changed

Filename Overview
.gitignore Adds *.tsbuildinfo to .gitignore, but 6 .tsbuildinfo files are already tracked in the repo and must be explicitly removed from the git index with git rm --cached for the ignore to take effect.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Developer runs tsc --build] --> B[*.tsbuildinfo files generated]
    B --> C{Already tracked\nin git index?}
    C -- Yes --> D[git still tracks & commits them\n⚠️ .gitignore has no effect]
    C -- No --> E[.gitignore pattern applies\n✅ Files are ignored]
    D --> F[Fix: git rm --cached *.tsbuildinfo\nthen commit removal]
    F --> E
Loading

Last reviewed commit: bf5218e


# TypeScript build output
dist/
*.tsbuildinfo
Copy link

Choose a reason for hiding this comment

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

Already-tracked .tsbuildinfo files won't be ignored

Adding *.tsbuildinfo to .gitignore only prevents future untracked files from being committed. However, there are currently 6 .tsbuildinfo files already tracked by git in this repository:

  • packages/channels/tsconfig.tsbuildinfo
  • packages/core/tsconfig.tsbuildinfo
  • packages/main/tsconfig.tsbuildinfo
  • packages/server/tsconfig.tsbuildinfo
  • packages/teams/tsconfig.tsbuildinfo
  • packages/visualizer/tsconfig.tsbuildinfo

These files will continue to be tracked and show up in diffs even after this change. To fully untrack them, the following command needs to be run and the result committed:

git rm --cached packages/channels/tsconfig.tsbuildinfo \
               packages/core/tsconfig.tsbuildinfo \
               packages/main/tsconfig.tsbuildinfo \
               packages/server/tsconfig.tsbuildinfo \
               packages/teams/tsconfig.tsbuildinfo \
               packages/visualizer/tsconfig.tsbuildinfo

Or more concisely:

git rm --cached **/*.tsbuildinfo

These are TypeScript incremental build cache files that should not be
tracked in version control.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jlia0 jlia0 force-pushed the jlia0/gitignore-tsbuildinfo branch from bf5218e to c1d402e Compare March 10, 2026 21:35
@jlia0 jlia0 merged commit 814ca23 into main Mar 10, 2026
@jlia0 jlia0 deleted the jlia0/gitignore-tsbuildinfo branch March 10, 2026 21:36
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