@opice/cli 0.4.1 → 0.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@opice/cli",
-	"version": "0.4.1",
+	"version": "0.5.0",
 	"description": "CLI for opice — scaffolds projects and wraps `bun test` to stream E2E results to the reporting platform",
 	"type": "module",
 	"bin": {

package/src/cli.ts

@@ -14,10 +14,14 @@ Commands:
       Scaffold opice.config.json in the current project. Pass
       --with-workflow to also drop a .github/workflows/opice.yml.
 
-  test [bun test args...]
+  test [--retries=N] [bun test args...]
       Wrapper around 'bun test' that exports OPICE_* env vars from
       opice.config.json + git so the harness reporter streams results
       to the platform. All trailing args pass through to bun test.
+      --retries=N sets a default retry budget for every scenario (a
+      flaky scenario that fails then passes is reported as flaky, not
+      failed). Falls back to "retries" in opice.config.json; a
+      per-scenario walkthrough/meta retries overrides both.
 
   failures <run-url|run-id> [--json]
       Pull a failed run's details (failed scenarios, the failing step,

package/src/commands/test.ts

@@ -43,8 +43,18 @@ export async function testCommand(args: string[]): Promise<number> {
 		...(git.commit ? { OPICE_COMMIT: git.commit } : {}),
 	}
 
-	// Always invoke `bun test`; any user-passed args go after.
-	const child = spawn('bun', ['test', ...args], { stdio: 'inherit', env })
+	// `--retries=N` (opice's spelling) → bun's `--retry=N`, the global default
+	// retry budget for every scenario. CLI flag wins over opice.config.json's
+	// `retries`. A per-scenario `walkthrough`/meta `retries` overrides both.
+	const { retries, rest } = extractRetries(args)
+	const resolvedRetries = retries ?? config?.retries
+	const bunArgs = ['test', ...rest]
+	// Don't clobber an explicit `--retry` the caller passed through to bun.
+	if (resolvedRetries !== undefined && !rest.some((a) => a === '--retry' || a.startsWith('--retry='))) {
+		bunArgs.push(`--retry=${resolvedRetries}`)
+	}
+
+	const child = spawn('bun', bunArgs, { stdio: 'inherit', env })
 
 	const exitCode = await new Promise<number>((resolve) => {
 		child.on('exit', (code) => resolve(code ?? 1))
@@ -57,6 +67,33 @@ export async function testCommand(args: string[]): Promise<number> {
 	return exitCode
 }
 
+/**
+ * Pull opice's `--retries=N` / `--retries N` out of the arg list (so it isn't
+ * forwarded to bun, which only knows `--retry`). Returns the parsed budget and
+ * the remaining args. An invalid value is ignored (falls through to config).
+ */
+function extractRetries(args: string[]): { retries: number | undefined; rest: string[] } {
+	const rest: string[] = []
+	let retries: number | undefined
+	for (let i = 0; i < args.length; i++) {
+		const arg = args[i]
+		if (arg === undefined) continue
+		if (arg.startsWith('--retries=')) {
+			const n = Number(arg.slice('--retries='.length))
+			if (Number.isInteger(n) && n >= 0) retries = n
+		} else if (arg === '--retries') {
+			const n = Number(args[i + 1])
+			if (Number.isInteger(n) && n >= 0) {
+				retries = n
+				i++ // consume the value
+			}
+		} else {
+			rest.push(arg)
+		}
+	}
+	return { retries, rest }
+}
+
 async function finalizeHandoffs(childPid: number | undefined, slug: string | undefined): Promise<void> {
 	let files: string[]
 	try {

package/src/config.ts

@@ -4,6 +4,13 @@ import path from 'node:path'
 export interface OpiceConfig {
 	project: string
 	endpoint: string
+	/**
+	 * Default retry budget applied to every scenario (passed to `bun test
+	 * --retry`). A flaky scenario that fails then passes within the budget is
+	 * reported as passed-but-flaky. Overridden by `opice test --retries=N` on
+	 * the command line, and by a per-scenario `walkthrough`/meta `retries`.
+	 */
+	retries?: number
 }
 
 const CONFIG_NAME = 'opice.config.json'