Fix dockerRemove flag: use --rm instead of -rm

domdomegg · domdomegg · commit 4038b9286b10 · 2025-12-17T02:35:04.000Z
The dockerRemove parameter was passing an invalid Docker flag (-rm) instead of the correct flag (--rm). This fix ensures containers are properly removed after stopping when the dockerRemove option is enabled. Fixes #8
diff --git a/src/start.ts b/src/start.ts
@@ -121,7 +121,7 @@ export const start = async (customOptions?: Partial<StartOptions>): Promise<void
   const commonArgs = ['-jar', jarFilename, '-port', String(options.port)];
   const dockerArgs = ['run', '-p', `${options.port}:${options.port}`];
   if (options.dockerRemove) {
-    dockerArgs.push('-rm');
+    dockerArgs.push('--rm');
   }
   dockerArgs.push(options.dockerImage);
   if (options.install_path) {

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ export const start = async (customOptions?: Partial<StartOptions>): Promise<void`
`121`	`121`	`const commonArgs = ['-jar', jarFilename, '-port', String(options.port)];`
`122`	`122`	const dockerArgs = ['run', '-p', `${options.port}:${options.port}`];
`123`	`123`	`if (options.dockerRemove) {`
`124`		`- dockerArgs.push('-rm');`
	`124`	`+ dockerArgs.push('--rm');`
`125`	`125`	`}`
`126`	`126`	`dockerArgs.push(options.dockerImage);`
`127`	`127`	`if (options.install_path) {`