Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Unpin @expo/plist and update it (post-merge follow-up to pnpm migration)

Ouverte
  • Quelque chose ne va pas ?
#383 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
65/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Calme
Stack technique
typescript
Domaine
build-system, testing

Piste de recherche

Lisez d’abord packages/host/package.json et packages/host/src/node/cli/apple.test.ts, puis examinez le chemin readInfoPlist/readAndParsePlist et l’assertion existante. Mettez à jour la plage de dépendances, rendez le test robuste face au prototype null du parseur et exécutez le test pertinent ; c’est terminé lorsque le job macOS Unit tests passe avec la version non épinglée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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/plist to the latest 0.4.x (or newer) and restore a caret range (^) in packages/host/package.json.
  • Make readInfoPlist / apple.test.ts robust 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 } or structuredClone), or
    • have readAndParsePlist return a plain object, or
    • use a non-strict deep-equal for this assertion.
  • Confirm the macOS Unit tests job passes with the unpinned version.

References

  • PR: #381
  • File: packages/host/package.json (@expo/plist pin)
  • Test: packages/host/src/node/cli/apple.test.ts
Langage dominant
TypeScript
Étoiles
191
Forks
11
Merge moyen
11 h 56 min
PR mergées (30 j)
2

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de callstackincubator/react-native-node-api

Toutes les issues de callstackincubator/react-native-node-api

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.