Skip to content

chore: pnpm を 12.15.0 に上げる - #88

Draft
sasagar wants to merge 1 commit into
mainfrom
chore/pnpm-12.15.0
Draft

sasagar wants to merge 1 commit into
mainfrom
chore/pnpm-12.15.0

Conversation

@sasagar

@sasagar sasagar commented Oct 25, 2026

Copy link
Copy Markdown
Member

変えたこと

  • package.json の packageManager を pnpm@12.15.0 に
  • CONTRIBUTING.md の「pnpm 10.x」を「pnpm 12.x」に
  • pnpm-workspace.yaml の allowBuilds に '@vscode/vsce-sign': false・keytar: false を足した
    • pnpm 10 では許可のない postinstall は警告だけで飛ばされていたが、pnpm 12 では ERR_PNPM_IGNORED_BUILDS で install が止まる。これまでどおり動かさない形で明示した
  • pnpm-lock.yaml の先頭に pnpm 自身の版の文書(packageManagerDependencies)が足された

5 つの workflow の pnpm/action-setup@v5 はどれも version 指定なしで packageManager を読むので、workflow は変えていない。

確かめたこと

  • pnpm -v が 12.15.0
  • node_modules を消してから pnpm install --frozen-lockfile が通る
  • 部品の版は変わっていない:lockfile は lockfileVersion: '9.0' のまま、既存部分の削除・変更は 0 行で、足されたのは先頭の文書だけ
  • CI と同じ手順:pnpm lint・format:check・build・check:tsdoc・typecheck・test(core 393 / cli 29 / rehype 14 / react 9 / vue 8 / astro 4 / editor 22 ほか)すべて通る
  • pnpm --filter kumihimo-vscode exec vsce package --no-dependencies で vsix が作れる(vsce-sign の postinstall を動かさなくても問題ない)
  • pnpm audit:93 件(low 11 / moderate 39 / high 39 / critical 4)。この PR では直していない

気づいたこと(この PR では触っていない)

  • pnpm build すると packages/obsidian/manifest.json の version が 1.1.13 → 1.1.15 に書き換わる(main の manifest.json が package の版と合っていない)

🤖 Generated with Claude Code

https://claude.ai/code/session_01QDZzA8fkAc11hCSdTfZeXh

- packageManager を pnpm@12.15.0 に、CONTRIBUTING の記載を pnpm 12.x に
- pnpm 12 は allowBuilds に無い依存の postinstall があると install を止めるため、
  pnpm 10 でも動いていなかった @vscode/vsce-sign・keytar を allowBuilds で false に
- pnpm-lock.yaml に pnpm 自身の版の文書(packageManagerDependencies)が足された。部品の版は変えていない

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QDZzA8fkAc11hCSdTfZeXh

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant