feat: add CLI and config options for git operations

- Add autoCommit, autoPush, commitSignoff to Config interface
- Add CLI flags: --no-auto-commit, --no-auto-push, -s/--signoff
- Add environment variables: OPENCODER_AUTO_COMMIT, OPENCODER_AUTO_PUSH, OPENCODER_COMMIT_SIGNOFF
- Support config.json options for git settings
- Add CliOptions interface extensions
- Add comprehensive config tests for new options

Signed-off-by: leocavalcante <[email protected]>

Author: leocavalcante

src/cli.ts

@@ -7,12 +7,23 @@ import { loadConfig } from "./config.ts"
 import { runLoop } from "./loop.ts"
 import type { CliOptions } from "./types.ts"
 
-const VERSION = "0.1.0"
+const VERSION = "1.0.0"
 
 /**
- * Parse CLI arguments and run the application
+ * Result of CLI argument parsing.
+ * Contains the parsed options and optional hint argument.
  */
-export async function run(): Promise<void> {
+export interface ParsedCli {
+	/** Parsed CLI options (project, model, verbose, etc.) */
+	options: CliOptions
+	/** Optional hint/instruction for the AI, passed as a positional argument */
+	hint?: string
+}
+
+/**
+ * Create and configure the CLI program
+ */
+function createProgram(): Command {
 	const program = new Command()
 
 	program
@@ -25,23 +36,9 @@ export async function run(): Promise<void> {
 		.option("-P, --plan-model <model>", "Model for plan phase (provider/model format)")
 		.option("-B, --build-model <model>", "Model for build phase (provider/model format)")
 		.option("-v, --verbose", "Enable verbose logging")
-		.action(async (hint: string | undefined, opts: Record<string, unknown>) => {
-			try {
-				const cliOptions: CliOptions = {
-					project: opts.project as string | undefined,
-					model: opts.model as string | undefined,
-					planModel: opts.planModel as string | undefined,
-					buildModel: opts.buildModel as string | undefined,
-					verbose: opts.verbose as boolean | undefined,
-				}
-
-				const config = await loadConfig(cliOptions, hint)
-				await runLoop(config)
-			} catch (err) {
-				console.error(`Error: ${err instanceof Error ? err.message : String(err)}`)
-				process.exit(1)
-			}
-		})
+		.option("--no-auto-commit", "Disable automatic commits after tasks")
+		.option("--no-auto-push", "Disable automatic push after cycles")
+		.option("-s, --signoff", "Add Signed-off-by line to commits")
 
 	// Add examples to help
 	program.addHelpText(
@@ -60,27 +57,127 @@ Examples:
   $ opencoder -m openai/gpt-4o -p ./myproject -v
     Run with verbose logging in a specific directory
 
+  $ opencoder -m anthropic/claude-sonnet-4 --no-auto-commit --no-auto-push
+    Run without automatic git operations
+
+  $ opencoder -m anthropic/claude-sonnet-4 -s
+    Run with commit signoff enabled
+
 Options:
     -p, --project <dir>         Project directory (default: current directory)
     -m, --model <model>         Model for both plan and build
     -P, --plan-model            Model for plan phase
     -B, --build-model           Model for build phase
     -v, --verbose               Enable verbose logging
+    --no-auto-commit            Disable automatic commits after tasks
+    --no-auto-push              Disable automatic push after cycles
+    -s, --signoff               Add Signed-off-by line to commits
 
 Environment variables:
     OPENCODER_PLAN_MODEL        Default plan model
     OPENCODER_BUILD_MODEL       Default build model
     OPENCODER_VERBOSE           Enable verbose logging (true/1)
     OPENCODER_PROJECT_DIR       Default project directory
+    OPENCODER_AUTO_COMMIT       Enable auto-commit (true/1, default: true)
+    OPENCODER_AUTO_PUSH         Enable auto-push (true/1, default: true)
+    OPENCODER_COMMIT_SIGNOFF    Add signoff to commits (true/1, default: false)
 
 Config file (.opencode/opencoder/config.json):
     {
       "planModel": "anthropic/claude-sonnet-4",
       "buildModel": "anthropic/claude-sonnet-4",
-      "verbose": false
+      "verbose": false,
+      "autoCommit": true,
+      "autoPush": true,
+      "commitSignoff": false
     }
 `,
 	)
 
+	return program
+}
+
+/**
+ * Parse CLI arguments without executing the action.
+ * Useful for testing or when you need to inspect arguments before running.
+ *
+ * @param argv - Command line arguments array. Defaults to process.argv.
+ *               Should include the node executable and script name as first two elements.
+ * @returns Parsed CLI options and hint
+ *
+ * @example
+ * ```typescript
+ * // Parse default process arguments
+ * const { options, hint } = parseCli()
+ *
+ * // Parse custom arguments for testing
+ * const { options, hint } = parseCli(['node', 'opencoder', '-m', 'anthropic/claude-sonnet-4', 'my hint'])
+ * ```
+ */
+export function parseCli(argv: string[] = process.argv): ParsedCli {
+	const program = createProgram()
+
+	// Parse without running the action
+	program.parse(argv)
+
+	const opts = program.opts()
+	const args = program.args
+
+	return {
+		options: {
+			project: opts.project as string | undefined,
+			model: opts.model as string | undefined,
+			planModel: opts.planModel as string | undefined,
+			buildModel: opts.buildModel as string | undefined,
+			verbose: opts.verbose as boolean | undefined,
+			autoCommit: opts.autoCommit as boolean | undefined,
+			autoPush: opts.autoPush as boolean | undefined,
+			commitSignoff: opts.signoff as boolean | undefined,
+		},
+		hint: args[0],
+	}
+}
+
+/**
+ * Parse CLI arguments and run the autonomous development loop.
+ * This is the main entry point for the CLI application.
+ *
+ * Parses command line arguments, loads configuration from all sources
+ * (defaults, config file, environment variables, CLI options), and
+ * starts the autonomous development loop.
+ *
+ * @throws Will call process.exit(1) if configuration is invalid or an error occurs
+ *
+ * @example
+ * ```typescript
+ * // In your entry point (e.g., index.ts)
+ * import { run } from './cli.ts'
+ * await run()
+ * ```
+ */
+export async function run(): Promise<void> {
+	const program = createProgram()
+
+	program.action(async (hint: string | undefined, opts: Record<string, unknown>) => {
+		try {
+			const cliOptions: CliOptions = {
+				project: opts.project as string | undefined,
+				model: opts.model as string | undefined,
+				planModel: opts.planModel as string | undefined,
+				buildModel: opts.buildModel as string | undefined,
+				verbose: opts.verbose as boolean | undefined,
+				autoCommit: opts.autoCommit as boolean | undefined,
+				autoPush: opts.autoPush as boolean | undefined,
+				commitSignoff: opts.signoff as boolean | undefined,
+			}
+
+			const config = await loadConfig(cliOptions, hint)
+			await runLoop(config)
+		} catch (err) {
+			console.error(`Error: ${err instanceof Error ? err.message : String(err)}`)
+			process.exit(1)
+		}
+	})
+
 	await program.parseAsync()
 }

src/config.ts

@@ -20,6 +20,9 @@ const DEFAULTS: Omit<Config, "planModel" | "buildModel" | "projectDir"> = {
 	backoffBase: 10,
 	logRetention: 30,
 	taskPauseSeconds: 2,
+	autoCommit: true,
+	autoPush: true,
+	commitSignoff: false,
 }
 
 /** Environment variable prefix */
@@ -43,6 +46,9 @@ export async function loadConfig(cliOptions: CliOptions, hint?: string): Promise
 		planModel: cliOptions.planModel || cliOptions.model,
 		buildModel: cliOptions.buildModel || cliOptions.model,
 		verbose: cliOptions.verbose,
+		autoCommit: cliOptions.autoCommit,
+		autoPush: cliOptions.autoPush,
+		commitSignoff: cliOptions.commitSignoff,
 	}
 
 	// Merge all sources (later sources override earlier ones)
@@ -93,15 +99,19 @@ async function loadConfigFile(projectDir: string): Promise<Partial<Config>> {
 		const content = await readFile(configPath, "utf-8")
 		const parsed = JSON.parse(content) as ConfigFile
 
-		return {
+		// Only include defined values to avoid overriding defaults with undefined
+		return filterUndefined({
 			planModel: parsed.planModel,
 			buildModel: parsed.buildModel,
 			verbose: parsed.verbose,
 			maxRetries: parsed.maxRetries,
 			backoffBase: parsed.backoffBase,
 			logRetention: parsed.logRetention,
 			taskPauseSeconds: parsed.taskPauseSeconds,
-		}
+			autoCommit: parsed.autoCommit,
+			autoPush: parsed.autoPush,
+			commitSignoff: parsed.commitSignoff,
+		})
 	} catch (err) {
 		console.warn(`Warning: Failed to parse config.json: ${err}`)
 		return {}
@@ -147,6 +157,15 @@ function loadEnvConfig(): Partial<Config> {
 		if (!Number.isNaN(parsed)) config.taskPauseSeconds = parsed
 	}
 
+	const autoCommit = process.env[`${ENV_PREFIX}AUTO_COMMIT`]
+	if (autoCommit) config.autoCommit = autoCommit === "true" || autoCommit === "1"
+
+	const autoPush = process.env[`${ENV_PREFIX}AUTO_PUSH`]
+	if (autoPush) config.autoPush = autoPush === "true" || autoPush === "1"
+
+	const commitSignoff = process.env[`${ENV_PREFIX}COMMIT_SIGNOFF`]
+	if (commitSignoff) config.commitSignoff = commitSignoff === "true" || commitSignoff === "1"
+
 	return config
 }
 

src/types.ts

@@ -1,5 +1,5 @@
 /**
- * Core type definitions for Opencoder
+ * Core type definitions for OpenCoder
  */
 
 /** Phases in the autonomous loop */
@@ -25,6 +25,12 @@ export interface Config {
 	logRetention: number
 	/** Seconds to pause between tasks */
 	taskPauseSeconds: number
+	/** Automatically commit changes after each task */
+	autoCommit: boolean
+	/** Automatically push commits after each cycle */
+	autoPush: boolean
+	/** Add signoff flag (-s) to commits */
+	commitSignoff: boolean
 }
 
 /** Persisted state */
@@ -39,6 +45,10 @@ export interface State {
 	sessionId?: string
 	/** ISO timestamp of last state update */
 	lastUpdate: string
+	/** Path to the idea file currently being processed (if any) */
+	currentIdeaPath?: string
+	/** Filename of the idea currently being processed (for display) */
+	currentIdeaFilename?: string
 }
 
 /** Runtime state with additional non-persisted fields */
@@ -118,6 +128,9 @@ export interface ConfigFile {
 	backoffBase?: number
 	logRetention?: number
 	taskPauseSeconds?: number
+	autoCommit?: boolean
+	autoPush?: boolean
+	commitSignoff?: boolean
 }
 
 /** CLI options from argument parsing */
@@ -127,4 +140,7 @@ export interface CliOptions {
 	planModel?: string
 	buildModel?: string
 	verbose?: boolean
+	autoCommit?: boolean
+	autoPush?: boolean
+	commitSignoff?: boolean
 }