Skip to content

Commit 9aa65f6

Browse files
dougborgclaude
andcommitted
ci: install pnpm where format-check runs
`uv run poe format-check` now invokes `pnpm exec prettier` for the markdown step, but the CI test job, the Copilot setup-steps job, and the devcontainer onCreate script only had the Python toolchain installed. Add pnpm + Node setup (CI) and corepack enable + `pnpm install --frozen-lockfile` (devcontainer) so prettier is available wherever format-check runs. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 137bf08 commit 9aa65f6

3 files changed

Lines changed: 31 additions & 1 deletion

File tree

.devcontainer/oncreate.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,12 @@ uv --version
1616
echo "📚 Syncing project dependencies (will be cached)..."
1717
uv sync --all-extras
1818

19+
# Install Node dependencies (needed for prettier — used by markdown
20+
# format hook and `uv run poe format-markdown`).
21+
echo "🟢 Enabling corepack and installing Node dependencies..."
22+
corepack enable
23+
pnpm install --frozen-lockfile
24+
1925
# Install pre-commit hooks
2026
echo "🪝 Installing pre-commit hooks..."
2127
uv run pre-commit install

.github/workflows/ci.yml

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,22 @@ jobs:
6262
cache-dependency-glob: "uv.lock"
6363
python-version: ${{ matrix.python-version }}
6464

65+
- name: Install pnpm
66+
if: needs.changes.outputs.code == 'true'
67+
uses: pnpm/action-setup@v6
68+
69+
- name: Install Node.js
70+
if: needs.changes.outputs.code == 'true'
71+
uses: actions/setup-node@v6
72+
with:
73+
node-version: "20"
74+
cache: "pnpm"
75+
6576
- name: Install dependencies
6677
if: needs.changes.outputs.code == 'true'
67-
run: uv sync --all-extras
78+
run: |
79+
uv sync --all-extras
80+
pnpm install --frozen-lockfile
6881
6982
- name: Check code formatting
7083
if: needs.changes.outputs.code == 'true'

.github/workflows/copilot-setup-steps.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,11 +39,22 @@ jobs:
3939
cache-dependency-glob: "uv.lock"
4040
python-version: "3.14"
4141

42+
- name: Install pnpm
43+
uses: pnpm/action-setup@v6
44+
45+
- name: Install Node.js
46+
uses: actions/setup-node@v6
47+
with:
48+
node-version: "20"
49+
cache: "pnpm"
50+
4251
- name: Install dependencies
4352
run: |
4453
echo "🔧 Installing project dependencies with uv..."
4554
echo "⏱️ This should only take a few seconds..."
4655
uv sync --all-extras
56+
echo "📦 Installing Node deps with pnpm (needed for prettier)..."
57+
pnpm install --frozen-lockfile
4758
echo "✅ Dependencies installed successfully"
4859
4960
- name: Install pre-commit hooks

0 commit comments

Comments
 (0)