preview.yml default globs ./packages/** and ./examples/** recurse into node_modules
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 68/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- github-actions
- 領域
- ci-cd
調査の方向性
preview.yml で、デフォルトの publish および template の glob と、それらが pkg-pr-new にどのように渡されるかを確認することから始めます。リンクされている clink preview の実行を再現手順として使用し、その後、publish ステップがネストされた node_modules を走査しなくなり、名前のない package.json ファイルで失敗しなくなったことを確認します。
索引モデルが issue の本文から書いたものです。
説明
Problem
preview.yml defaults its inputs to recursive globs:
publish:
default: "./packages/**"
template:
default: "./examples/**"
and passes them to pkg-pr-new:
pnpx pkg-pr-new publish --pnpm "$PUBLISH_GLOB" --template "$TEMPLATE_GLOB"
The ** matches recursively, including into each package's node_modules. pkg-pr-new then walks nested dependency package.json files and aborts on any that lack a name field — which is common for internal stub manifests such as vite's:
[error] "name" field in .../packages/clink/node_modules/vite/types/package.json should be defined
##[error]Process completed with exit code 1.
(Preceded by many Skipping .../node_modules/.../*.ts: not a directory or .tgz/.tar.gz file lines as it scans the dependency tree.)
Impact
Any repo whose packages depend on something that ships a nameless nested package.json (vite is a frequent one). The preview job fails at the publish step even though install and build succeed. Latent until the build step actually completes and pkg-pr-new runs.
Repro
- Concrete instance: bombshell-dev/clink#43 — the
previewrun that reached the "Publish Preview" step:
https://github.com/bombshell-dev/clink/actions/runs/30575154965
- 主要言語
- TypeScript
- スター
- 4
- フォーク
- 1
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
bombshell-dev/automation のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
bombshell-dev/automation#33 ·
bombshell-dev/automation の issue をすべて見る
似ている issue
-
priority: P2
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
prime-radiant-inc/evener#3291 ·
メンテナーはふだん 1 日以内に返信
-
accessibility bug revealjs
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
quarto-dev/quarto-cli#14961 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
supabase/agent-skills#614 ·
-
Content
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
RunestoneInteractive/rs#1559 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信