From 9bcc00610fad5029f244d5420d7e3f8f5c523a04 Mon Sep 17 00:00:00 2001
From: Martin Traverse <martin.traverse@gmail.com>
Date: Fri, 26 Jul 2024 16:05:43 +0100
Subject: [PATCH] Update CLI options help and README

---
 cli/README.md |  4 ++--
 cli/pbjs.js   | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/cli/README.md b/cli/README.md
index 7fb6da453..912e8bcd7 100644
--- a/cli/README.md
+++ b/cli/README.md
@@ -69,8 +69,8 @@ Translates between file formats and generates static code.
   --force-number   Enforces the use of 'number' for s-/u-/int64 and s-/fixed64 fields.
   --force-message  Enforces the use of message instances instead of plain objects.
   
-  --null-semantics Make nullable fields match protobuf semantics (including the optional keyword)
-  --null-defaults  Default value for optional fields is null instead of zero value (no effect if --null-semantics is specified)
+  --null-defaults  Default value for optional fields is null instead of zero value.
+  --null-semantics Make nullable fields match protobuf semantics (overrides --null-defaults).
 
 usage: pbjs [options] file1.proto file2.json ...  (or pipe)  other | pbjs [options] -
 ```
diff --git a/cli/pbjs.js b/cli/pbjs.js
index 1d238a090..c276ce464 100644
--- a/cli/pbjs.js
+++ b/cli/pbjs.js
@@ -41,7 +41,7 @@ exports.main = function main(args, callback) {
             "force-message": "strict-message"
         },
         string: [ "target", "out", "path", "wrap", "dependency", "root", "lint", "filter" ],
-        boolean: [ "create", "encode", "decode", "verify", "convert", "delimited", "typeurl", "beautify", "comments", "service", "es6", "sparse", "keep-case", "alt-comment", "force-long", "force-number", "force-enum-string", "force-message", "null-semantics", "null-defaults"],
+        boolean: [ "create", "encode", "decode", "verify", "convert", "delimited", "typeurl", "beautify", "comments", "service", "es6", "sparse", "keep-case", "alt-comment", "force-long", "force-number", "force-enum-string", "force-message", "null-defaults", "null-semantics"],
         default: {
             target: "json",
             create: true,
@@ -62,8 +62,8 @@ exports.main = function main(args, callback) {
             "force-number": false,
             "force-enum-string": false,
             "force-message": false,
-            "null-semantics": false,
-            "null-defaults": false
+            "null-defaults": false,
+            "null-semantics": false
         }
     });
 
@@ -148,8 +148,8 @@ exports.main = function main(args, callback) {
                 "  --force-number   Enforces the use of 'number' for s-/u-/int64 and s-/fixed64 fields.",
                 "  --force-message  Enforces the use of message instances instead of plain objects.",
                 "",
-                "  --null-semantics Make nullable fields match protobuf semantics (including the optional keyword)",
-                "  --null-defaults  Default value for optional fields is null instead of zero value (no effect if --null-semantics is specified)",
+                "  --null-defaults  Default value for optional fields is null instead of zero value.",
+                "  --null-semantics Make nullable fields match protobuf semantics (overrides --null-defaults).",
                 "",
                 "usage: " + chalk.bold.green("pbjs") + " [options] file1.proto file2.json ..." + chalk.gray("  (or pipe)  ") + "other | " + chalk.bold.green("pbjs") + " [options] -",
                 ""