Js TypeError when generated with commonjs_strict
Ninguém assumiu esta issue ainda.
- #8696 de @eKazim — fechado sem integrar
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- javascript, node.js
- Domínio
- build-system, tooling
Direção de pesquisa
Comece reproduzindo o problema com os arquivos fornecidos a.proto e b.proto e o comando protoc usando commonjs_strict; em seguida, inspecione as referências JavaScript geradas entre os arquivos. Rastreie como os exports qualificados por pacote são formados e verifique se a referência gerada usa o caminho do pacote exportado e não lança mais um TypeError.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Version: v3.10.0
Language: Javascript
OS: OSX 10.14.6
Node v12.10.0
When js files generated with commonjs_strict the referrences to types defined in other files are not using the package name to access it from the exported object, so TypeError is thrown due to reading property of undefined.
How to reproduce
a.proto
syntax = "proto3";
package foo.v1;
import "b.proto";
message A {
repeated foo.v1.B foo = 1;
}
b.proto
syntax = "proto3";
package foo.v1;
import "b.proto";
message A {
repeated foo.v1.B foo = 1;
}
protoc \
--js_out=import_style=commonjs_strict:./generated \
a.proto b.proto
In the generated js files the protos_b_pb.B should be protos_b_foo.v1.pb.B
- Linguagem predominante
- JavaScript
- Estrelas
- 470
- Forks
- 92
- Merge médio
- 1d 12h
- PRs com merge (30d)
- 6
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Sem guia de contribuição
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 protocolbuffers/protobuf-javascript
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 30/100
protocolbuffers/protobuf-javascript#248 · 1 comentário · 13 reações ·
-
question
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
protocolbuffers/protobuf-javascript#222 · 9 comentários ·
-
Why map.js sort keys?Abertaenhancement port-fix triaged
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
protocolbuffers/protobuf-javascript#185 · 1 comentário ·
-
enhancement port-fix triaged
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
protocolbuffers/protobuf-javascript#182 · 3 comentários · 1 reação ·
Todas as issues de protocolbuffers/protobuf-javascript
Issues semelhantes
-
Add: EUROTV NEWSAbertachannels:add check:passed
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
-
curriculum documentation quality
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
githubnext/gh-aw-workshop#4638 ·
Mantenedores costumam responder em até 1 dia
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P2
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
openclaw/openclaw-enterprise#2571 · 1 comentário · 1 reação ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
fabriziosalmi/certmate#1247 ·
Mantenedores costumam responder em até 1 dia
-
[guide] docs/metrics/index.md names retired 'Import reference architectures' workflow as the metrics refresh pathTalvez já em andamento @hivecommons-hive assumiu hoje. Abertaagent/guide documentation hive/covered-by-pr hive/hosted-available-lke648397-260827-5n31
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 70/100
cncf/endusers#1537 · 2 comentários ·
Mantenedores costumam responder em até 1 dia