docs: fix plan 3B test commands to use bun, not pnpm
The repo uses bun (bun.lock present, no pnpm/npm available). Replaces all pnpm references in the plan with bun equivalents. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
@@ -59,7 +59,7 @@ Run before starting Task 1:
|
||||
|
||||
```bash
|
||||
cargo build && cargo test
|
||||
cd extension && pnpm install && pnpm test
|
||||
cd extension && bun install && bun run test
|
||||
```
|
||||
|
||||
Both must be green. The plan starts from a clean `main` (Plan 3A is already merged).
|
||||
@@ -1585,7 +1585,7 @@ export interface ImportLastPassCommitResponse extends Extract<Response, { ok: tr
|
||||
|
||||
- [ ] **Step 2: Verify type-check**
|
||||
|
||||
Run: `cd extension && pnpm tsc --noEmit`
|
||||
Run: `cd extension && bunx tsc --noEmit`
|
||||
Expected: PASS (the SW handler doesn't exist yet, but type-only changes don't trigger handler errors).
|
||||
|
||||
- [ ] **Step 3: Commit**
|
||||
@@ -1662,7 +1662,7 @@ Edit `extension/src/service-worker/router/popup-only.ts`. After the closing brac
|
||||
|
||||
- [ ] **Step 2: Verify type-check**
|
||||
|
||||
Run: `cd extension && pnpm tsc --noEmit`
|
||||
Run: `cd extension && bunx tsc --noEmit`
|
||||
Expected: PASS.
|
||||
|
||||
- [ ] **Step 3: Commit**
|
||||
@@ -1876,7 +1876,7 @@ describe('import_lastpass_commit handler', () => {
|
||||
|
||||
- [ ] **Step 2: Run the tests**
|
||||
|
||||
Run: `cd extension && pnpm vitest run src/service-worker/__tests__/import.test.ts`
|
||||
Run: `cd extension && bun run vitest run src/service-worker/__tests__/import.test.ts`
|
||||
Expected: PASS — 5 tests green.
|
||||
|
||||
- [ ] **Step 3: Commit**
|
||||
@@ -1954,7 +1954,7 @@ describe('parse_lastpass_csv / import_lastpass_commit sender check', () => {
|
||||
|
||||
- [ ] **Step 2: Run the tests**
|
||||
|
||||
Run: `cd extension && pnpm vitest run src/service-worker/router/__tests__/router.test.ts`
|
||||
Run: `cd extension && bun run vitest run src/service-worker/router/__tests__/router.test.ts`
|
||||
Expected: PASS — all router tests green (existing tests + 4 new).
|
||||
|
||||
- [ ] **Step 3: Commit**
|
||||
@@ -2252,7 +2252,7 @@ Then in `wireHandlers()` around line 197 (next to `open-backup`), add:
|
||||
|
||||
- [ ] **Step 4: Build and type-check**
|
||||
|
||||
Run: `cd extension && pnpm tsc --noEmit && pnpm build`
|
||||
Run: `cd extension && bunx tsc --noEmit && bun run build`
|
||||
Expected: PASS.
|
||||
|
||||
- [ ] **Step 5: Manual smoke test**
|
||||
@@ -2444,12 +2444,12 @@ describe('Import panel', () => {
|
||||
|
||||
- [ ] **Step 2: Run the tests**
|
||||
|
||||
Run: `cd extension && pnpm vitest run src/vault/components/__tests__/import-panel.test.ts`
|
||||
Run: `cd extension && bun run vitest run src/vault/components/__tests__/import-panel.test.ts`
|
||||
Expected: PASS — 5 tests green.
|
||||
|
||||
- [ ] **Step 3: Run the full extension test suite to confirm no regressions**
|
||||
|
||||
Run: `cd extension && pnpm test`
|
||||
Run: `cd extension && bun run test`
|
||||
Expected: PASS.
|
||||
|
||||
- [ ] **Step 4: Commit**
|
||||
@@ -2501,7 +2501,7 @@ Expected: PASS.
|
||||
|
||||
- [ ] **Step 3: Run the full test suite — extension**
|
||||
|
||||
Run: `cd extension && pnpm test`
|
||||
Run: `cd extension && bun run test`
|
||||
Expected: PASS.
|
||||
|
||||
- [ ] **Step 4: Build the WASM target**
|
||||
@@ -2511,7 +2511,7 @@ Expected: PASS.
|
||||
|
||||
- [ ] **Step 5: Build the extension**
|
||||
|
||||
Run: `cd extension && pnpm build`
|
||||
Run: `cd extension && bun run build`
|
||||
Expected: PASS.
|
||||
|
||||
- [ ] **Step 6: Commit**
|
||||
|
||||
Reference in New Issue
Block a user