Skip to content

Commit ba711c4

Browse files
melvincarvalhomelvincarvalho
committed
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
1 parent fdc219f commit ba711c4

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

‎bin/jss.js‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,8 @@ program
8787
.option('-c, --config <file>', 'Config file path')
8888
.option('--ssl-key <path>', 'Path to SSL private key (PEM)')
8989
.option('--ssl-cert <path>', 'Path to SSL certificate (PEM)')
90-
.option('--multiuser', 'Enable multi-user mode')
91-
.option('--no-multiuser', 'Disable multi-user mode')
90+
.option('--multi-user', 'Enable multi-user mode')
91+
.option('--no-multi-user', 'Disable multi-user mode')
9292
.option('--conneg', 'Enable content negotiation (Turtle support)')
9393
.option('--no-conneg', 'Disable content negotiation')
9494
.option('--notifications', 'Enable WebSocket notifications')
@@ -159,6 +159,12 @@ program
159159
.option('--print-config', 'Print configuration and exit')
160160
.action(async (options) => {
161161
try {
162+
// Normalize --multi-user (Commander camelCase: multiUser) to internal key
163+
if (options.multiUser !== undefined) {
164+
options.multiuser = options.multiUser;
165+
delete options.multiUser;
166+
}
167+
162168
const config = await loadConfig(options, options.config);
163169

164170
// Set DATA_ROOT env var so all modules use the same data directory

0 commit comments

Comments
 (0)