Skip to content

Add .pnpm-store to .gitignore across all packages#1010

Merged
DZakh merged 1 commit intomainfrom
claude/add-pnpm-store-gitignore-KSudC
Mar 3, 2026
Merged

Add .pnpm-store to .gitignore across all packages#1010
DZakh merged 1 commit intomainfrom
claude/add-pnpm-store-gitignore-KSudC

Conversation

@DZakh
Copy link
Member

@DZakh DZakh commented Mar 3, 2026

Summary

This PR adds .pnpm-store to the .gitignore files across the repository to prevent pnpm's local store directory from being committed to version control.

Changes

  • Added .pnpm-store entry to .gitignore in the following locations:
    • packages/envio/
    • packages/cli/templates/static/ (all template variants: blank_template, codegen, erc20_template, factory_template, greeter_template, greeteronfuel_template, svmblock_template)
    • scenarios/fuel_test/ (root and contract subdirectories: all-events, interaction-tools, ts-interaction-tools)
    • scenarios/helpers/
    • scenarios/test_codegen/

Details

The .pnpm-store directory is a local cache directory created by pnpm package manager and should not be tracked in version control. This change ensures consistency across all packages and templates in the monorepo by excluding this directory from git tracking.

https://claude.ai/code/session_01QGFLuHCFdXRoTBgAUMSpxC

Summary by CodeRabbit

  • Chores
    • Updated .gitignore configuration across CLI templates, sample scenarios, and test directories to exclude pnpm package manager store artifacts from version control. Uniform application across blank, codegen, ERC20, factory, greeter templates and various test scenarios ensures repositories remain clean by preventing accidental cache file tracking.

@DZakh DZakh enabled auto-merge (squash) March 3, 2026 20:03
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8bb13b1 and 82bcb01.

📒 Files selected for processing (14)
  • packages/cli/templates/static/blank_template/shared/.gitignore
  • packages/cli/templates/static/codegen/.gitignore
  • packages/cli/templates/static/erc20_template/shared/.gitignore
  • packages/cli/templates/static/factory_template/shared/.gitignore
  • packages/cli/templates/static/greeter_template/shared/.gitignore
  • packages/cli/templates/static/greeteronfuel_template/shared/.gitignore
  • packages/cli/templates/static/svmblock_template/typescript/.gitignore
  • packages/envio/.gitignore
  • scenarios/fuel_test/.gitignore
  • scenarios/fuel_test/contracts/all-events/.gitignore
  • scenarios/fuel_test/contracts/interaction-tools/.gitignore
  • scenarios/fuel_test/contracts/ts-interaction-tools/.gitignore
  • scenarios/helpers/.gitignore
  • scenarios/test_codegen/.gitignore

📝 Walkthrough

Walkthrough

This PR adds .pnpm-store to 14 .gitignore files across CLI templates and test scenario directories, preventing pnpm store directories from being tracked by version control. One file also modifies a ReScript artifact ignore pattern.

Changes

Cohort / File(s) Summary
CLI Templates
packages/cli/templates/static/blank_template/shared/.gitignore, packages/cli/templates/static/codegen/.gitignore, packages/cli/templates/static/erc20_template/shared/.gitignore, packages/cli/templates/static/factory_template/shared/.gitignore, packages/cli/templates/static/greeter_template/shared/.gitignore, packages/cli/templates/static/greeteronfuel_template/shared/.gitignore, packages/cli/templates/static/svmblock_template/typescript/.gitignore
Added .pnpm-store ignore pattern to each template's .gitignore file.
Envio Package
packages/envio/.gitignore
Modified ReScript ignore pattern from negative format (-*.res.mjs) to positive format (*.res.mjs) and added .pnpm-store ignore pattern.
Test Scenarios
scenarios/fuel_test/.gitignore, scenarios/fuel_test/contracts/all-events/.gitignore, scenarios/fuel_test/contracts/interaction-tools/.gitignore, scenarios/fuel_test/contracts/ts-interaction-tools/.gitignore, scenarios/helpers/.gitignore, scenarios/test_codegen/.gitignore
Added .pnpm-store ignore pattern to scenario .gitignore files; one file also removed .env entry and reordered existing patterns.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hoppity-hop through ignore files we go,
Adding .pnpm-store to each one below,
No tracking those packages, they'll stay out of sight,
Cleaner git repos make us all feel so right!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately summarizes the main change: adding .pnpm-store entries to .gitignore files across multiple packages in the repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/add-pnpm-store-gitignore-KSudC

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.

@DZakh DZakh merged commit 9d9f221 into main Mar 3, 2026
7 checks passed
@DZakh DZakh deleted the claude/add-pnpm-store-gitignore-KSudC branch March 3, 2026 20:07
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.

2 participants