help: Copy the DEFAULT_OPTIONS object rather than directly adding to it

This was causing options from previous calls to printUsage() to stick around and be displayed for subsequent calls.
HeyPuter · Mar 12, 2024 · 57dff3e · 57dff3e
1 parent 03b15d6
commit 57dff3e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/puter-shell/coreutils/coreutil_lib/help.js b/src/puter-shell/coreutils/coreutil_lib/help.js
@@ -29,7 +29,8 @@ export const DEFAULT_OPTIONS = {
 
 export const printUsage = async (command, out, vars) => {
     const { name, usage, description, args } = command;
-    const options = Object.assign(DEFAULT_OPTIONS, args.options);
+    const options = Object.create(DEFAULT_OPTIONS);
+    Object.assign(options, args.options);
 
     const heading = text => {
         out.write(`\x1B[34;1m${text}:\x1B[0m\n`);