From 448970a147681c72e3269a70b84f41cb824c5030 Mon Sep 17 00:00:00 2001
From: Agent <agent@pgflow.dev>
Date: Fri, 4 Sep 2026 20:32:02 +0000
Subject: [PATCH 1/2] fix(cli): remove Windows-breaking postinstall chmod
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The pgflow package's postinstall ran `chmod +x dist/index.js || true`.
`chmod` and `true` do not exist in Windows cmd.exe, so `npm install pgflow`
and `npx pgflow@latest install` failed with "'chmod' is not recognized as an
internal or external command" (#610).

npm sets the executable bit on files declared in the `bin` field during
install, so the script was redundant even on Unix and removal changes nothing
for Unix users.

Adds a permanent windows-latest smoke workflow as the regression test: build
the CLI, pnpm pack, install the tarball into a clean directory with lifecycle
scripts enabled, run the npm-created pgflow.cmd --version shim. The rest of CI
runs on Ubuntu and invokes node dist/index.js directly, so npm installation
and Windows shims were never exercised.

One pgflow patch changeset for the 0.15.1 release group (#666). Recreates the
minimal removal from #612 against current main.

## Checks

- pnpm nx build cli — pass
- pnpm pack in pkgs/cli — tarball contains dist/, no postinstall in its package.json
- cli unit tests — 43/43 pass
- prettier --check on all three touched files — pass

Closes #610
---
 .../remove-windows-breaking-postinstall.md    |  5 ++
 .github/workflows/windows-smoke.yml           | 55 +++++++++++++++++++
 pkgs/cli/package.json                         |  5 +-
 3 files changed, 61 insertions(+), 4 deletions(-)
 create mode 100644 .changeset/remove-windows-breaking-postinstall.md
 create mode 100644 .github/workflows/windows-smoke.yml

diff --git a/.changeset/remove-windows-breaking-postinstall.md b/.changeset/remove-windows-breaking-postinstall.md
new file mode 100644
index 000000000..8eacdccee
--- /dev/null
+++ b/.changeset/remove-windows-breaking-postinstall.md
@@ -0,0 +1,5 @@
+---
+"pgflow": patch
+---
+
+Remove the postinstall `chmod` script that broke `npm install pgflow` on Windows; npm sets the executable bit for `bin` files automatically.
diff --git a/.github/workflows/windows-smoke.yml b/.github/workflows/windows-smoke.yml
new file mode 100644
index 000000000..9d3392fd0
--- /dev/null
+++ b/.github/workflows/windows-smoke.yml
@@ -0,0 +1,55 @@
+name: Windows Smoke
+
+# Regression test for https://github.com/pgflow-dev/pgflow/issues/610:
+# `npm install pgflow` must succeed on Windows with lifecycle scripts enabled,
+# and the npm-created pgflow.cmd shim must run. The rest of CI runs on Ubuntu
+# and invokes `node dist/index.js` directly, so it never exercises this.
+on:
+  workflow_dispatch:
+  pull_request:
+
+permissions:
+  contents: read
+
+jobs:
+  windows-package-smoke:
+    runs-on: windows-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: pnpm/action-setup@v4
+        with:
+          version: '10.20.0'
+          run_install: false
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 24
+          cache: 'pnpm'
+          cache-dependency-path: |
+            **/pnpm-lock.yaml
+
+      - name: Install dependencies
+        run: pnpm install --frozen-lockfile --prefer-offline
+
+      - name: Build CLI
+        run: pnpm nx build cli
+
+      - name: Pack CLI tarball
+        shell: bash
+        working-directory: pkgs/cli
+        run: pnpm pack
+
+      # Install the tarball into a clean directory with lifecycle scripts
+      # enabled (no --ignore-scripts): this is the step that failed in #610.
+      - name: Install tarball into a clean directory
+        shell: bash
+        run: |
+          SMOKE_DIR="$RUNNER_TEMP/pgflow-smoke"
+          mkdir -p "$SMOKE_DIR"
+          cd "$SMOKE_DIR"
+          npm install "$GITHUB_WORKSPACE/pkgs/cli"/pgflow-*.tgz
+
+      - name: Run npm-created pgflow.cmd shim
+        shell: bash
+        run: "$RUNNER_TEMP/pgflow-smoke/node_modules/.bin/pgflow.cmd" --version
diff --git a/pkgs/cli/package.json b/pkgs/cli/package.json
index 374fcafac..433c39fe4 100644
--- a/pkgs/cli/package.json
+++ b/pkgs/cli/package.json
@@ -37,8 +37,5 @@
   },
   "files": [
     "dist"
-  ],
-  "scripts": {
-    "postinstall": "chmod +x dist/index.js || true"
-  }
+  ]
 }

From 54d696f450fb4ad5465f2f49960595448a8743e3 Mon Sep 17 00:00:00 2001
From: Agent <agent@pgflow.dev>
Date: Fri, 4 Sep 2026 20:40:15 +0000
Subject: [PATCH 2/2] fix(ci): unquote pgflow.cmd shim path in windows-smoke
 run step

The double-quoted path followed by --version made YAML parse the value as
a quoted scalar with trailing content; GitHub rejected the whole workflow at
load time (run 33916753681: 0 jobs). The path expands from RUNNER_TEMP which
has no spaces on GitHub-hosted runners, so the bare scalar is safe.
---
 .github/workflows/windows-smoke.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/windows-smoke.yml b/.github/workflows/windows-smoke.yml
index 9d3392fd0..da4096045 100644
--- a/.github/workflows/windows-smoke.yml
+++ b/.github/workflows/windows-smoke.yml
@@ -52,4 +52,4 @@ jobs:
 
       - name: Run npm-created pgflow.cmd shim
         shell: bash
-        run: "$RUNNER_TEMP/pgflow-smoke/node_modules/.bin/pgflow.cmd" --version
+        run: $RUNNER_TEMP/pgflow-smoke/node_modules/.bin/pgflow.cmd --version