Unexpected `.` in generated file names results in `buf` warning

Offen

#148 erstellt am 28.10.2022 2 Kommentare 11 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: javascript
Bereich: tooling

Rechercherichtung

Reproduziere die Warnung mit dem gezeigten buf generate-Befehl, der buf.gen.yaml-Vorlage und dem Plugin v3.21.2 unter Linux und vergleiche den generierten Pfad mit der direkten protoc-Ausgabe. Verfolge die Plugin-Ausgabe anhand der referenzierten Generierungsspezifikation von plugin.proto; abgeschlossen ist die Arbeit, wenn buf einen relativen Dateinamen ohne eine führende "."-Komponente erhält und keine Warnung mehr ausgibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

good first issue P3 triaged

When using the plugin v3.21.2 through buf on Linux (Ubuntu), I'm getting the following warning:

Warning: Generated file name "./some/path/to/file/my_message_pb.js" does not conform to the Protobuf generation specification. Note that the file name must be relative, use "/" instead of "", and not use "." or ".." as part of the file name. Buf will continue without error here, but please raise an issue with the maintainer of the plugin and reference https://github.com/protocolbuffers/protobuf/blob/95e6c5b4746dd7474d540ce4fb375e3f79a086f8/src/google/protobuf/compiler/plugin.proto#L122

...which is rather self-explanatory. I suspect this might be because of the . at the beginning.

The buf command I was using is the following:

buf generate /path/to/protos --template buf.gen.yaml -o /output/location

(i.e. I used full path for both - input and output paths)

buf generation template is

version: v1
plugins:
  - name: js
    out: .
    opt: import_style=commonjs

I tried changing out to not be . to no avail.

There are no warnings when plugin is used through protoc directly, but I think the buf is right that returned file path should not have . components.

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