From 8f446a3aef0d21d8ec813fe50f6eb14b4b6fe660 Mon Sep 17 00:00:00 2001 From: John Brecht Date: Tue, 16 Jun 2026 00:20:54 -0700 Subject: [PATCH] fix(example-app): drop vestigial tutorial-forge-cli dep (clean fresh install) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit example-app declared `tutorial-forge-cli` but never imports it — `pnpm forge` runs the CLI via `tsx ../cli/src/main.ts` (source). The dep made pnpm try to link the CLI's bin into example-app on install, which fails with a scary `[WARN] Failed to create bin … ENOENT` on a fresh clone because the CLI's dist/ isn't built yet. Harmless (the quick-start uses tsx, and npm consumers never see it), but it's the first line of output a repo cloner sees. Removing the unused dep eliminates it; `pnpm forge` still works (verified via `forge list`). Found via a fresh-clone quick-start smoke test ahead of launch. Co-Authored-By: Claude Opus 4.8 (1M context) --- packages/example-app/package.json | 3 +-- pnpm-lock.yaml | 3 --- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/packages/example-app/package.json b/packages/example-app/package.json index a30eeb6..b73d2e8 100644 --- a/packages/example-app/package.json +++ b/packages/example-app/package.json @@ -13,8 +13,7 @@ }, "dependencies": { "express": "^4.21.0", - "tutorial-forge": "workspace:*", - "tutorial-forge-cli": "workspace:*" + "tutorial-forge": "workspace:*" }, "devDependencies": { "@types/express": "^4.17.21", diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index 9fe0a76..f723249 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -64,9 +64,6 @@ importers: tutorial-forge: specifier: workspace:* version: link:../core - tutorial-forge-cli: - specifier: workspace:* - version: link:../cli devDependencies: '@types/express': specifier: ^4.17.21