feat: add --help flag to install/uninstall scripts

Signed-off-by: leocavalcante <leo@cavalcante.dev>

Author: leocavalcante

postinstall.mjs

@@ -29,6 +29,24 @@ const flags = parseCliFlags(process.argv)
 const DRY_RUN = flags.dryRun
 const VERBOSE = flags.verbose
 
+/** Print usage information and exit */
+if (flags.help) {
+	console.log(`Usage: node postinstall.mjs [options]
+
+Install OpenCoder agents to ~/.config/opencode/agents/
+
+Options:
+  --dry-run   Simulate installation without copying files
+  --verbose   Enable verbose output for debugging
+  --help      Show this help message and exit
+
+Examples:
+  node postinstall.mjs              # Install agents
+  node postinstall.mjs --dry-run    # Preview what would be installed
+  node postinstall.mjs --verbose    # Install with detailed logging`)
+	process.exit(0)
+}
+
 /** Create logger with verbose flag */
 const logger = createLogger(VERBOSE)
 const verbose = logger.verbose

preuninstall.mjs

@@ -28,6 +28,24 @@ const flags = parseCliFlags(process.argv)
 const DRY_RUN = flags.dryRun
 const VERBOSE = flags.verbose
 
+/** Print usage information and exit */
+if (flags.help) {
+	console.log(`Usage: node preuninstall.mjs [options]
+
+Remove OpenCoder agents from ~/.config/opencode/agents/
+
+Options:
+  --dry-run   Simulate removal without deleting files
+  --verbose   Enable verbose output for debugging
+  --help      Show this help message and exit
+
+Examples:
+  node preuninstall.mjs              # Remove agents
+  node preuninstall.mjs --dry-run    # Preview what would be removed
+  node preuninstall.mjs --verbose    # Remove with detailed logging`)
+	process.exit(0)
+}
+
 /** Create logger with verbose flag */
 const logger = createLogger(VERBOSE)
 const verbose = logger.verbose

tests/install.test.ts

@@ -1128,6 +1128,58 @@ Adding more text to ensure minimum length requirement is satisfied here.`
 			// Should show validation passed for each file
 			expect(stdout).toContain("Validation passed")
 		})
+
+		it("should run actual postinstall.mjs with --help", async () => {
+			const proc = Bun.spawn(["node", "postinstall.mjs", "--help"], {
+				cwd: process.cwd(),
+				stdout: "pipe",
+				stderr: "pipe",
+			})
+
+			const exitCode = await proc.exited
+			const stdout = await new Response(proc.stdout).text()
+			const stderr = await new Response(proc.stderr).text()
+
+			// Should exit with code 0
+			expect(exitCode).toBe(0)
+
+			// Should show usage information
+			expect(stdout).toContain("Usage:")
+			expect(stdout).toContain("postinstall.mjs")
+			expect(stdout).toContain("Options:")
+			expect(stdout).toContain("--dry-run")
+			expect(stdout).toContain("--verbose")
+			expect(stdout).toContain("--help")
+
+			// Should not have errors
+			expect(stderr).toBe("")
+		})
+
+		it("should run actual preuninstall.mjs with --help", async () => {
+			const proc = Bun.spawn(["node", "preuninstall.mjs", "--help"], {
+				cwd: process.cwd(),
+				stdout: "pipe",
+				stderr: "pipe",
+			})
+
+			const exitCode = await proc.exited
+			const stdout = await new Response(proc.stdout).text()
+			const stderr = await new Response(proc.stderr).text()
+
+			// Should exit with code 0
+			expect(exitCode).toBe(0)
+
+			// Should show usage information
+			expect(stdout).toContain("Usage:")
+			expect(stdout).toContain("preuninstall.mjs")
+			expect(stdout).toContain("Options:")
+			expect(stdout).toContain("--dry-run")
+			expect(stdout).toContain("--verbose")
+			expect(stdout).toContain("--help")
+
+			// Should not have errors
+			expect(stderr).toBe("")
+		})
 	})
 
 	describe("full install/uninstall cycle", () => {