Unpin @expo/plist and update it (post-merge follow-up to pnpm migration)
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 65/100
- Tipo de issue
- Refatoração
- Clareza
- Claramente especificada
- Status de atividade
- Pouca atividade
- Stack de tecnologia
- typescript
- Domínio
- build-system, testing
Direção de pesquisa
Leia primeiro packages/host/package.json e packages/host/src/node/cli/apple.test.ts; depois, inspecione o caminho readInfoPlist/readAndParsePlist e a asserção existente. Atualize o intervalo da dependência, torne o teste robusto ao protótipo null do parser e execute o teste relevante; a tarefa estará concluída quando o job macOS Unit tests passar com a versão não fixada.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- TypeScript
- Estrelas
- 191
- Forks
- 10
- Merge médio
- 2d 17h
- PRs com merge (30d)
- 3
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de callstackincubator/react-native-node-api
-
Android 🤖
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projectsAbertaAndroid 🤖 good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
callstackincubator/react-native-node-api#163 · 1 comentário ·
-
enhancement good first issue gyp-to-cmake
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
callstackincubator/react-native-node-api#161 · 3 comentários · 1 reação ·
-
enhancement Host 🏡
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
-
Automatable good first issue
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 15/100
callstackincubator/react-native-node-api#424 · 1 comentário ·
Todas as issues de callstackincubator/react-native-node-api
Issues semelhantes
-
Add: Digi World (Romania) SDAbertacheck:passed streams:add
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
Fission-AI/OpenSpec#1986 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
nestjs/docs.nestjs.com#3554 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
publicodes/publicodes#868 ·
-
namespace operations
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 78/100
EclipseFdn/open-vsx.org#13488 ·
Mantenedores costumam responder em até 2 dias