Repository navigation
fix: pnpm のロック再解決で未対応の追加引数を渡さない - #143
Merged
Merged
Conversation
ロック再解決用の pnpm update に機能別の無効化オプションを常時付けており、 その引数を受理しない pnpm では解決前に失敗して更新が取り消されていた。 --no-include-github-actions と --no-changeset を外し、pnpm 10 と 12 で 使える --no-save --lockfile-only --ignore-scripts に揃える。 通常プロジェクトと workspace の実行経路で引数全体を検証し、age の注入と 検証後の frozen install が維持されることを確認する。英語・日本語の利用文書も 実際のコマンドに合わせる。
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
 |
owayo
added a commit
that referenced
this pull request
Nov 12, 2026
Cargo が受理する `= 1.*`、`^ 1.x`、`~ 1.2.X` を読み飛ばしていた問題と、選択的復元で主対象の版が消える問題を修正します。 演算子付き wildcard の解析で半角スペースを許可し、更新後も演算子・空白・wildcard の形を保持します。タブや全角空白を新たに受理しません。マニフェストの解析、候補選択、書き戻し、再解析、再実行まで回帰テストで確認します。 選択的復元では、巻き戻しで追加した別系列の版（discarded）も再解決対象に含めます。広い版要求を持つ依存が旧系列を使い続けると、一時固定を外した際に主対象の復元先が不要として削除されていました。現在の lock に存在する crates.io の ID だけを取得元付きで重複なく指定し、独立した差し戻しと無関係な版を保持します。主対象と補助依存の区別、補助依存の剪定条件、元の場所での `--locked` 検証は維持します。 README、エコシステム、復元の説明は英日を更新し、8言語の公式仕様との監査記録を追加しました。既定の Cargo install のコマンドは変更しません。 検証: - 修正前の wildcard の取りこぼしと、実物 Cargo＋偽 local-registry による主対象の復元失敗を再現 - wildcard の round-trip と復元の回帰テストに成功。独立した差し戻し、重複 ID、既に消えた ID も確認 - 最新 main を取り込んだ後のローカル `make ci` が成功（fmt-check / clippy / 2,998件成功・失敗0件・既定の除外1件） - `astro-sight` の最終差分レビューで API 変更・未使用シンボル・未解決の影響なし - 最終 head の GitHub CI 全7ジョブ成功（Linux・macOS の Quality、配布対象5環境のビルド。Windows はテストを含む） #141 の最終組合せ解決と補助依存剪定（#145）、pnpm の未対応引数の修正（#143）を取り込み済みです。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
--install --refresh-lockのpnpm updateで、機能別の無効化オプションを受理しない版は解決前に失敗し、更新が取り消されます。--no-include-github-actionsと--no-changesetを外し、pnpm 10 と 12 が共通で扱える引数で再解決します。--no-save --lockfile-only --ignore-scripts、workspace 全体の対象指定、age 環境変数、検証後の frozen install は維持します。英語・日本語の利用文書も実際のコマンドに合わせました。検証
mise exec -- cargo fmt --all -- --check: passmise exec -- cargo clippy --locked --all-targets -- -D warnings: passmise exec -- cargo test --locked --quiet: 2948 passed、1 ignoredastro-sight review --git: API 変更・未解決の呼び出し元なし実バイナリの確認は Windows の npm 配布を使用しました。この 12.16.1 は旧引数も受理したため、報告された macOS 環境の引数拒否そのものは再現していません。修正後の共通引数での成功を確認しています。
Closes #142