Commonjs pb exports do not carry intellisense

Offen

#18 erstellt am 06.3.2021 4 Kommentare 7 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@lukesandberg arbeitet bereits daran.

Seit 09.6.2021.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Beschreibung

enhancement good first issue javascript triaged

What version of protobuf and what language are you using?
Version: v3.6.1
Language: Javascript

What operating system (Linux, Windows, ...) and version?
Ubuntu 20.04, x86_64

What runtime / compiler are you using (e.g., python version or gcc version)
protoc pre-built compiler

What did you do?
Steps to reproduce the behavior:

Create a sample protoc file, here simple.proto. I took it from this example, modified to "proto3"
Use the compiler to autogen JavaScript code: protoc --js_out=import_style=commonjs,binary:. simple.proto
Use the default export as an object with expected Schema: const Schema = require('./simple_pb');
Attempt to use the expected Class(es) on that object: const test = new Schema.Test1();
See that you do not receive intellisense for the Test1() class, nor for expected operations it's instances

What did you expect to see
Intellisense for the simple_pb operations on the default export.

What did you see instead?
No Intellisense, but the operations themselves do execute (setting values, serialzing to Binary).

I was able to "fix" the issue by changing the last line of the simple_pb.js file. The goog export extension did not work, so I just commented it out and spread the goog and proto objects into module.exports:

module.exports = {...goog, ...proto};
// goog.object.extend(exports, proto);

Vorherrschende Sprache: JavaScript
Sterne: 471
Forks: 91
Ø Merge: 3 Std. 57 Min.
Gemergte PRs (30 T.): 2

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus protocolbuffers/protobuf-javascript

Confusing duplication of libraries - @protocolbuffers/protoc-gen-js and protoc-gen-js Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

protocolbuffers/protobuf-javascript#287 · Erstellt am 10.9.2026
Support for ES Modules (import/export) in protoc --js_out Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 30/100

protocolbuffers/protobuf-javascript#248 · 1 Kommentar · 13 Reaktionen · Erstellt am 23.9.2025
BUG: Timestamp.toObject() returning zeroed value Offen

question

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

protocolbuffers/protobuf-javascript#222 · 9 Kommentare · Erstellt am 13.9.2024
Why map.js sort keys? Offen

enhancement port-fix triaged

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

protocolbuffers/protobuf-javascript#185 · 1 Kommentar · Erstellt am 17.1.2024
Generate alternate accessor for fields with presence Offen

enhancement port-fix triaged

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

protocolbuffers/protobuf-javascript#182 · 3 Kommentare · 1 Reaktion · Erstellt am 30.12.2023

Alle Issues in protocolbuffers/protobuf-javascript