From ba711c43697425df1957ac40eca2858c4a8d6064 Mon Sep 17 00:00:00 2001 From: Melvin Carvalho Date: Wed, 20 May 2026 04:30:48 +0200 Subject: [PATCH] normalize --multiuser to --multi-user for CLI consistency Aligns with --single-user, --single-user-name, --single-user-password and standard kebab-case CLI conventions. Commander.js maps --multi-user to multiUser (camelCase), so the action handler normalizes it back to the internal `multiuser` property. Fixes #511 --- bin/jss.js | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/bin/jss.js b/bin/jss.js index 25a850db..a8540e65 100755 --- a/bin/jss.js +++ b/bin/jss.js @@ -87,8 +87,8 @@ program .option('-c, --config ', 'Config file path') .option('--ssl-key ', 'Path to SSL private key (PEM)') .option('--ssl-cert ', 'Path to SSL certificate (PEM)') - .option('--multiuser', 'Enable multi-user mode') - .option('--no-multiuser', 'Disable multi-user mode') + .option('--multi-user', 'Enable multi-user mode') + .option('--no-multi-user', 'Disable multi-user mode') .option('--conneg', 'Enable content negotiation (Turtle support)') .option('--no-conneg', 'Disable content negotiation') .option('--notifications', 'Enable WebSocket notifications') @@ -159,6 +159,12 @@ program .option('--print-config', 'Print configuration and exit') .action(async (options) => { try { + // Normalize --multi-user (Commander camelCase: multiUser) to internal key + if (options.multiUser !== undefined) { + options.multiuser = options.multiUser; + delete options.multiUser; + } + const config = await loadConfig(options, options.config); // Set DATA_ROOT env var so all modules use the same data directory