Skip to content

chore: Remove comitted .pnpm-store files#2952

Merged
bezbac merged 1 commit into
mainfrom
feature/lfe-9792-remove-committed-pnpm-store-entries
May 15, 2026
Merged

chore: Remove comitted .pnpm-store files#2952
bezbac merged 1 commit into
mainfrom
feature/lfe-9792-remove-committed-pnpm-store-entries

Conversation

@bezbac
Copy link
Copy Markdown
Member

@bezbac bezbac commented May 15, 2026

Greptile Summary

This PR removes accidentally-committed .pnpm-store binary cache files from the repository and adds .pnpm-store to .gitignore to prevent recurrence.

  • Adds .pnpm-store to .gitignore, alongside existing entries for .next, node_modules, and .vercel.
  • Deletes 48 binary/cache files under .pnpm-store/v3/files/ that should never have been tracked by git.

Confidence Score: 5/5

Safe to merge — removes binary cache artifacts that were accidentally committed and prevents them from being re-added.

The only substantive change is a one-line addition to .gitignore. All other file deletions are removing pnpm package-manager cache files that should never have been tracked. No application code is touched.

No files require special attention.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[.pnpm-store accidentally committed] --> B[Add .pnpm-store to .gitignore]
    A --> C[Delete all .pnpm-store/v3/files/* from git tracking]
    B --> D[Future pnpm cache writes are ignored by git]
    C --> D
    D --> E[Repository no longer contains binary cache artifacts]
Loading

Reviews (1): Last reviewed commit: "chore: Remove comitted `.pnpm-store` fil..." | Re-trigger Greptile

@vercel
Copy link
Copy Markdown

vercel Bot commented May 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
langfuse-docs Ready Ready Preview, Comment May 15, 2026 9:38am

Request Review

@github-actions
Copy link
Copy Markdown

@claude review

Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

LGTM, straightforward cleanup of accidentally committed .pnpm-store files plus gitignore entry.

Extended reasoning...

Overview

This PR removes 48 accidentally committed .pnpm-store/ files (pnpm content-addressable cache for @next/third-parties and third-party-capital packages) and adds .pnpm-store to .gitignore to prevent future commits.

Security risks

None. The removed files are pnpm package store artifacts (build outputs, type definitions, package.json, README, LICENSE) — pure deletions with no executable code paths affected. The .gitignore addition is a one-line, well-scoped entry.

Level of scrutiny

Very low scrutiny is appropriate. This is a mechanical repo hygiene change: deleting tracked build artifacts and ignoring the directory going forward. No application code, configuration, or documentation content is touched.

Other factors

The bug hunting system found no issues. The diff is purely additive to .gitignore (one line) plus deletions of cache files. There is no risk to runtime behavior or the docs site build, since pnpm regenerates this store locally during install.

@dosubot dosubot Bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label May 15, 2026
@bezbac bezbac added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit ae47585 May 15, 2026
18 checks passed
@bezbac bezbac deleted the feature/lfe-9792-remove-committed-pnpm-store-entries branch May 15, 2026 09:44
@dosubot dosubot Bot added the auto-merge This PR is set to be merged label May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge This PR is set to be merged size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant