From 2f01d6dfe8d0ee5142342a5693b4882011f1233e Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 8 Feb 2026 19:32:34 +0000
Subject: [PATCH 1/2] Initial plan


From 7627bcc516c4d071e8d5179b0a4a454c23491d62 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 8 Feb 2026 19:42:47 +0000
Subject: [PATCH 2/2] Add .npmrc and document npm setup to enforce consistent
 package-lock.json usage

- Add .npmrc to configure npm behavior (save-exact=true for new dependencies)
- Update CONTRIBUTING.md with comprehensive npm dependency management section
- Document the use of npm ci for routine development to avoid lockfile churn
- Explain the "peer": true behavior in package-lock.json and why it's normal
- Update devcontainer to use npm ci instead of npm install
- Provide clear guidance on when to use npm install vs npm ci

Co-authored-by: rajbos <6085745+rajbos@users.noreply.github.com>
---
 .devcontainer/devcontainer.json |  2 +-
 .npmrc                          | 19 ++++++++++
 CONTRIBUTING.md                 | 67 +++++++++++++++++++++++++++++++++
 3 files changed, 87 insertions(+), 1 deletion(-)
 create mode 100644 .npmrc

diff --git a/.devcontainer/devcontainer.json b/.devcontainer/devcontainer.json
index ee9361a..76faa33 100644
--- a/.devcontainer/devcontainer.json
+++ b/.devcontainer/devcontainer.json
@@ -23,6 +23,6 @@
       }
     }
   },
-  "postCreateCommand": "npm install",
+  "postCreateCommand": "npm ci",
   "remoteUser": "node"
 }
diff --git a/.npmrc b/.npmrc
new file mode 100644
index 0000000..191a675
--- /dev/null
+++ b/.npmrc
@@ -0,0 +1,19 @@
+# NPM Configuration for Copilot Token Tracker Extension
+# This file documents the npm configuration used by this project
+
+# IMPORTANT: This project uses package-lock.json for reproducible builds
+# - Use `npm ci` for installing dependencies (recommended)
+# - Use `npm install` only when adding/updating dependencies
+# - CI/CD workflows use `npm ci` exclusively
+
+# Use package-lock.json for reproducible builds (default behavior)
+package-lock=true
+
+# Do not use legacy peer dependency resolution (default in npm 7+)
+# Some dependencies have peer dependencies we don't use (e.g., react from @vscode/webview-ui-toolkit)
+# This is expected and does not affect functionality
+legacy-peer-deps=false
+
+# Save exact versions (no ^ or ~ prefixes) when adding new dependencies
+# This ensures maximum reproducibility
+save-exact=true
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index ccb4d0c..0ccfb51 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -140,6 +140,7 @@ If you prefer not to use the devcontainer, you can set up the extension locally:
 ### Prerequisites
 
 - [Node.js](https://nodejs.org/) 18.x or 20.x
+- [npm](https://www.npmjs.com/) (comes with Node.js)
 - [Visual Studio Code](https://code.visualstudio.com/)
 - [Git](https://git-scm.com/)
 
@@ -158,6 +159,8 @@ If you prefer not to use the devcontainer, you can set up the extension locally:
    npm install
    ```
 
+   **Note:** The project uses `package-lock.json` for reproducible builds. The `.npmrc` file ensures consistent behavior across different npm versions and environments. CI/CD workflows use `npm ci` to ensure exact dependency versions are installed.
+
 3. **Build the extension:**
 
    ```bash
@@ -239,6 +242,70 @@ npx vsce package
 - `npm test` - Run tests (requires VS Code)
 - `npm run watch-tests` - Run tests in watch mode
 
+## NPM and Dependency Management
+
+The project uses **`package-lock.json`** for reproducible builds and dependency consistency across all environments.
+
+### Key Files
+
+- **`.npmrc`**: Configures npm behavior (use `save-exact=true`, etc.)
+- **`package-lock.json`**: Lockfile that pins exact dependency versions (committed to the repository)
+- **`package.json`**: Defines project dependencies and scripts
+
+### Installation Commands
+
+**RECOMMENDED for day-to-day development:**
+
+- **`npm ci`**: Clean install from lockfile
+  - Requires `package-lock.json` to exist
+  - Installs exact versions from the lockfile (no modifications)
+  - Deletes `node_modules` before installing
+  - **Use this after pulling changes** to avoid lockfile churn
+  - **Used in all CI/CD workflows** for reproducible builds
+  - Faster and more reliable than `npm install`
+
+**Only when adding/updating dependencies:**
+
+- **`npm install`**: Install/update dependencies
+  - Respects `package-lock.json` and may update it
+  - Use when adding new dependencies: `npm install <package>`
+  - Use when updating dependencies: `npm install <package>@latest`
+  - **After adding dependencies, commit both `package.json` and `package-lock.json`**
+
+### Best Practices
+
+1. **Use `npm ci` for routine development** - This prevents accidental lockfile changes and ensures you have the exact dependency versions
+2. **Only use `npm install` when intentionally changing dependencies** - This makes dependency updates explicit and trackable
+3. **Always commit both files together** - When you modify dependencies, commit both `package.json` and `package-lock.json` in the same commit
+4. **Don't manually edit `package-lock.json`** - Let npm manage it automatically
+
+### Why We Use package-lock.json
+
+1. **Reproducible Builds**: Ensures all developers and CI/CD get identical dependency versions
+2. **Consistency**: Prevents "works on my machine" issues caused by different dependency versions
+3. **Security**: Locks down transitive dependencies to prevent supply chain attacks
+4. **Performance**: CI/CD can cache dependencies more effectively with a lockfile
+
+### About Peer Dependencies and "peer": true
+
+Different npm versions (7+) may add or remove `"peer": true` properties in `package-lock.json` when running `npm install`. This is expected behavior for peer dependencies that aren't satisfied.
+
+Some dependencies (like `@vscode/webview-ui-toolkit`) declare peer dependencies (e.g., `react`) that we don't use directly. You may see entries like this in `package-lock.json`:
+
+```json
+"node_modules/react": {
+  "version": "19.2.3",
+  "peer": true,
+  ...
+}
+```
+
+**This is normal and doesn't affect functionality.** To avoid lockfile churn from these changes:
+
+- **Use `npm ci` for routine development** (doesn't modify the lockfile)
+- Only use `npm install` when you're intentionally adding or updating dependencies
+- If you accidentally modify the lockfile, run `git checkout package-lock.json` to restore it
+
 ## Code Guidelines
 
 ### Project Structure