From a4e99ecdafc9edfd5c1bdbe712993a9d9eeb9df5 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 16 Jan 2026 22:41:49 +0000 Subject: [PATCH] chore: remove patch references --- docs/gateway/troubleshooting.md | 5 ++--- docs/install/bun.md | 18 ++---------------- docs/install/docker.md | 1 - docs/install/installer.md | 6 ------ docs/platforms/hetzner.md | 1 - package.json | 1 - scripts/docker/cleanup-smoke/Dockerfile | 1 - scripts/e2e/Dockerfile | 1 - 8 files changed, 4 insertions(+), 30 deletions(-) diff --git a/docs/gateway/troubleshooting.md b/docs/gateway/troubleshooting.md index 48e658504..f0021bb47 100644 --- a/docs/gateway/troubleshooting.md +++ b/docs/gateway/troubleshooting.md @@ -365,7 +365,7 @@ clawdbot channels login ### npm install fails (allow-build-scripts / missing tar or yargs). What now? If you’re running from source, use the repo’s package manager: **pnpm** (preferred). -The repo declares `packageManager: "pnpm@…"`, and pnpm patches are tracked in `pnpm.patchedDependencies`. +The repo declares `packageManager: "pnpm@…"`. Typical recovery: ```bash @@ -376,8 +376,7 @@ pnpm clawdbot doctor clawdbot daemon restart ``` -Why: pnpm is the configured package manager for this repo, and the dependency -patching workflow relies on it. +Why: pnpm is the configured package manager for this repo. ### How do I switch between git installs and npm installs? diff --git a/docs/install/bun.md b/docs/install/bun.md index e2989e4cb..446fc1431 100644 --- a/docs/install/bun.md +++ b/docs/install/bun.md @@ -1,5 +1,5 @@ --- -summary: "Bun workflow (experimental): installs, patches, and gotchas vs pnpm" +summary: "Bun workflow (experimental): installs and gotchas vs pnpm" read_when: - You want the fastest local dev loop (bun + watch) - You hit Bun install/patch/lifecycle script issues @@ -8,7 +8,7 @@ read_when: # Bun (experimental) Goal: run this repo with **Bun** (optional, not recommended for WhatsApp/Telegram) -without losing pnpm patch behavior. +without diverging from pnpm workflows. ⚠️ **Not recommended for Gateway runtime** (WhatsApp/Telegram bugs). Use Node for production. @@ -39,20 +39,6 @@ bun run build bun run vitest run ``` -## pnpm patchedDependencies under Bun - -pnpm supports `package.json#pnpm.patchedDependencies` and records it in `pnpm-lock.yaml`. -Bun (and npm/yarn) do not support pnpm patches, so we apply them in `postinstall` when pnpm is **not** the installer: - -- [`scripts/postinstall.js`](https://github.com/clawdbot/clawdbot/blob/main/scripts/postinstall.js) detects the package manager via `npm_config_user_agent` and applies every entry from `package.json#pnpm.patchedDependencies` into `node_modules/...` using a built-in JS patcher (no `git`/system `patch` dependency). -- Under pnpm, this fallback is skipped because pnpm already applies `patchedDependencies` itself. - -To add a new patch that works in both pnpm + Bun: - -1. Add an entry to `package.json#pnpm.patchedDependencies` -2. Add the patch file under `patches/` -3. Run `pnpm install` (updates `pnpm-lock.yaml` patch hash) - ## Bun lifecycle scripts (blocked by default) Bun may block dependency lifecycle scripts unless explicitly trusted (`bun pm untrusted` / `bun pm trust`). diff --git a/docs/install/docker.md b/docs/install/docker.md index 821c53b4c..9e08d5386 100644 --- a/docs/install/docker.md +++ b/docs/install/docker.md @@ -152,7 +152,6 @@ WORKDIR /app # Cache dependencies unless package metadata changes COPY package.json pnpm-lock.yaml pnpm-workspace.yaml .npmrc ./ COPY ui/package.json ./ui/package.json -COPY patches ./patches COPY scripts ./scripts RUN pnpm install --frozen-lockfile diff --git a/docs/install/installer.md b/docs/install/installer.md index 1733c9087..12286be64 100644 --- a/docs/install/installer.md +++ b/docs/install/installer.md @@ -71,9 +71,3 @@ Help: ```bash curl -fsSL https://clawd.bot/install-cli.sh | bash -s -- --help ``` - -## Patches (npm / pnpm / bun) - -Clawdbot’s `postinstall` script includes a builtin JS patcher that can apply `pnpm.patchedDependencies` patches even when the package manager doesn’t support them (notably Bun). pnpm itself already applies `pnpm.patchedDependencies`, so the fallback skips pnpm installs to avoid double-applying. - -See: [Bun notes](/install/bun). diff --git a/docs/platforms/hetzner.md b/docs/platforms/hetzner.md index 3360017bb..8f9de7bc0 100644 --- a/docs/platforms/hetzner.md +++ b/docs/platforms/hetzner.md @@ -241,7 +241,6 @@ RUN curl -L https://github.com/steipete/wacli/releases/latest/download/wacli_Lin WORKDIR /app COPY package.json pnpm-lock.yaml pnpm-workspace.yaml .npmrc ./ COPY ui/package.json ./ui/package.json -COPY patches ./patches COPY scripts ./scripts RUN corepack enable diff --git a/package.json b/package.json index 7a60431fb..9edc69d36 100644 --- a/package.json +++ b/package.json @@ -47,7 +47,6 @@ "README-header.png", "CHANGELOG.md", "LICENSE", - "patches/**", "scripts/postinstall.js", "dist/terminal/**", "dist/routing/**", diff --git a/scripts/docker/cleanup-smoke/Dockerfile b/scripts/docker/cleanup-smoke/Dockerfile index 5cf080318..08fa5c09e 100644 --- a/scripts/docker/cleanup-smoke/Dockerfile +++ b/scripts/docker/cleanup-smoke/Dockerfile @@ -9,7 +9,6 @@ RUN apt-get update \ WORKDIR /repo COPY package.json pnpm-lock.yaml pnpm-workspace.yaml ./ -COPY patches ./patches COPY scripts/postinstall.js ./scripts/postinstall.js RUN corepack enable \ && pnpm install --frozen-lockfile diff --git a/scripts/e2e/Dockerfile b/scripts/e2e/Dockerfile index 5873750aa..5092e38d1 100644 --- a/scripts/e2e/Dockerfile +++ b/scripts/e2e/Dockerfile @@ -7,7 +7,6 @@ WORKDIR /app ENV NODE_OPTIONS="--disable-warning=ExperimentalWarning" COPY package.json pnpm-lock.yaml pnpm-workspace.yaml tsconfig.json vitest.config.ts ./ -COPY patches ./patches COPY src ./src COPY scripts ./scripts COPY docs ./docs