Skip to content

GitHub Actions がコケる問題を修正#277

Merged
kotonefami merged 3 commits into
mainfrom
fix/actions
May 8, 2026
Merged

GitHub Actions がコケる問題を修正#277
kotonefami merged 3 commits into
mainfrom
fix/actions

Conversation

@kotonefami

Copy link
Copy Markdown
Member

pnpm install の時点でビルドスクリプトの実行が拒否されているのが問題

pnpm install の時点でビルドスクリプトの実行が拒否されているのが問題
@kotonefami kotonefami self-assigned this May 8, 2026
@kotonefami kotonefami added the bug/バグ報告 Something isn't working label May 8, 2026
@kotonefami

Copy link
Copy Markdown
Member Author

これだから GitHub Actions 嫌いなんだよな
Gemini 3.1 Flash に聞いたのも間違いだったか

@Hibi-10000

Hibi-10000 commented May 8, 2026

Copy link
Copy Markdown
Collaborator

pnpm 11 でなんか変わったっぽい
一旦 pnpm 10に固定するべきか

@Hibi-10000

Copy link
Copy Markdown
Collaborator

pnpm-workspace.yamlに strictDepBuilds: falseを追加で直るはず

@kotonefami

Copy link
Copy Markdown
Member Author

よく見たら GHA で動いてるの pnpm 11 じゃねーか! Gemini 3.1 Pro もどん詰まりだったので助かります(そもそも 9 が最新だと思い込んでいる)

最近 npm 関連怖いので strictDepBuilds = false にするのは避けたい気持ちがあるんですが仕方ないですかね……

@Hibi-10000

Hibi-10000 commented May 8, 2026

Copy link
Copy Markdown
Collaborator

pnpm 11 的にはこうしてほしいっぽいですがpnpm 10と違うのでどうだろう

allowBuilds:
  esbuild: false
  spawn-sync: false
  unrs-resolver: false

いずれにせよpostinstallスクリプトが勝手に実行されることはないはずので危険なことはないはず

@Hibi-10000

Hibi-10000 commented May 8, 2026

Copy link
Copy Markdown
Collaborator

allowBuilds設定してpackage.jsonで定義しているpnpmのバージョンを>=11(あるいは^11)にしたっていい
どうせ後でそうすることになるし

pnpm-workspace.yaml に記述する pnpm v11 の新しい形式にした
仮に pnpm v10 以下だったとしてもこれが読み込まれずとも落ちることはないので、目的は達成していそう
@kotonefami

Copy link
Copy Markdown
Member Author

あっ見てなくて全部 true にしちゃった

@kotonefami

Copy link
Copy Markdown
Member Author

これって false だと esbuild など正常に動かないということになりませんかね?
今 pnpm>=11 にするのはアリかも

@Hibi-10000

Copy link
Copy Markdown
Collaborator

今までずっとそうで特に問題は無かったので大丈夫かと...?

@Hibi-10000

Hibi-10000 commented May 8, 2026

Copy link
Copy Markdown
Collaborator

https://esbuild.github.io/getting-started/#additional-npm-flags
どうしてそうなるのかはよくわからんけど CLI で直接 esbuild を呼び出すとき以外は影響ないっぽい...?

追記: 理解した、わざわざjavascriptからネイティブ実行ファイルを呼び出す分少し遅くなるってことね

@kotonefami

Copy link
Copy Markdown
Member Author

今調べてみたんですが、デフォルトでは実行環境に合ったバイナリをダウンロードしてくる、allowBuilds はそれでもビルドする必要が出てきた場合の最終手段っぽいですね
Runner の環境のバイナリはあると思うので、デフォルト false にしてもよさげですね。やってみます

@kotonefami

Copy link
Copy Markdown
Member Author

動いたので大丈夫だと思います(適当)

@kotonefami kotonefami marked this pull request as ready for review May 8, 2026 15:48
@kotonefami kotonefami merged commit 873b45f into main May 8, 2026
1 check passed
@kotonefami kotonefami deleted the fix/actions branch May 8, 2026 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug/バグ報告 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants