Unpin @expo/plist and update it (post-merge follow-up to pnpm migration)
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 65/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 静か
- 技術スタック
- typescript
- 領域
- build-system, testing
調査の方向性
まず packages/host/package.json と packages/host/src/node/cli/apple.test.ts を読み、次に readInfoPlist/readAndParsePlist の経路と既存のアサーションを確認します。依存関係の範囲を更新し、パーサーの null プロトタイプに対してテストが堅牢になるようにして、関連するテストを実行します。固定されていないバージョンで macOS Unit tests ジョブがパスすれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Context
As part of the pnpm migration (#381), regenerating the lockfile floated @expo/plist from 0.4.7 (the version main's package-lock.json happened to hold) up to 0.4.9. In 0.4.9, plist.parse() returns a null-prototype object, which makes the strict assert.deepEqual in packages/host/src/node/cli/apple.test.ts ("should read Info.plist contents, plus extra keys not in schema") fail on the prototype mismatch.
To keep the migration behaviour-neutral, @expo/plist was pinned to an exact 0.4.7 in packages/host/package.json:
- "@expo/plist": "^0.4.7",
+ "@expo/plist": "0.4.7",
The null prototype only affects the test's strict equality — runtime property access on the parsed object is unaffected — so this is a test-only concern, not a functional regression.
Follow-up
Once the pnpm migration is merged, revisit this pin:
- Update
@expo/plistto the latest0.4.x(or newer) and restore a caret range (^) inpackages/host/package.json. - Make
readInfoPlist/apple.test.tsrobust to the null-prototype return so the test no longer depends on the parser's object prototype. Options:- normalise the parsed value before asserting (e.g.
{ ...contents }orstructuredClone), or - have
readAndParsePlistreturn a plain object, or - use a non-strict deep-equal for this assertion.
- normalise the parsed value before asserting (e.g.
- Confirm the macOS
Unit testsjob passes with the unpinned version.
References
- PR: #381
- File:
packages/host/package.json(@expo/plistpin) - Test:
packages/host/src/node/cli/apple.test.ts
- 主要言語
- TypeScript
- スター
- 188
- フォーク
- 10
- 平均マージ
- 2日 17時間
- マージ済み PR（30日）
- 3
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
callstackincubator/react-native-node-api のほかの issue
-
Android 🤖
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projects オープンAndroid 🤖 good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
callstackincubator/react-native-node-api#163 · コメント 1 件 ·
-
enhancement good first issue gyp-to-cmake
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
callstackincubator/react-native-node-api#161 · コメント 3 件 · リアクション 1 件 ·
-
enhancement Host 🏡
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
Automatable good first issue
難易度 3/5 1〜2日 初心者へのやさしさ 15/100
callstackincubator/react-native-node-api#424 · コメント 1 件 ·
callstackincubator/react-native-node-api の issue をすべて見る
似ている issue
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
Mend: dependency security vulnerability untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100