Merge pull request #97 from probitas-test/fix/deno-option-parse

Fix --deno-* option parsing and add cache command

Author: lambdalisue

README.md

@@ -487,6 +487,9 @@ probitas list [options]
 # Initialize configuration file
 probitas init
 
+# Cache scenario dependencies
+probitas cache [options]
+
 # Format scenario files
 probitas fmt
 

assets/usage-cache.txt

@@ -0,0 +1,30 @@
+probitas cache - Cache scenario dependencies
+
+Usage: probitas cache [options] [paths...]
+
+Arguments:
+  [paths...]  Scenario files or directories
+              Defaults to current directory
+
+Options:
+  -h, --help             Show help message
+  -r, --reload           Force re-download of cached dependencies
+  --include <pattern>    Include pattern for file discovery
+  --exclude <pattern>    Exclude pattern for file discovery
+  --config <path>        Path to probitas config file
+  -v, --verbose          Verbose output
+  -q, --quiet            Suppress output
+  -d, --debug            Debug output
+
+Note:
+  Runs `deno cache` on discovered scenario files to pre-download dependencies.
+  Uses includes/excludes from probitas config (same as run/list).
+
+Examples:
+  probitas cache                    # Cache dependencies for all scenarios
+  probitas cache api/               # Cache dependencies in api directory
+  probitas cache -r                 # Force re-download all dependencies
+  probitas cache -r api/            # Force re-download for api directory
+  probitas cache --include "e2e/**/*.probitas.ts"
+
+Documentation: https://probitas-test.github.io/documents/

assets/usage-list.txt

@@ -18,8 +18,8 @@ Options:
   --no-env                   Skip loading .env file
   -r, --reload               Reload dependencies before running
   --deno-<option>[=<value>] Pass options to deno subprocess
-                            Value options: --deno-config=custom.json, --deno-lock=custom.lock
-                            Boolean flags: --deno-no-lock, --deno-no-prompt
+                            With value: --deno-config=custom.json, --deno-reload=jsr:@std/http
+                            Without value: --deno-reload, --deno-no-lock, --deno-no-prompt
 
 Selector Format:
   [!][type:]value

assets/usage-run.txt

@@ -30,8 +30,8 @@ Options:
   -d, --debug               Debug output (maximum detail)
   --no-color                Disable colored output
   --deno-<option>[=<value>] Pass options to deno subprocess
-                            Value options: --deno-config=custom.json, --deno-lock=custom.lock
-                            Boolean flags: --deno-no-lock, --deno-no-prompt
+                            With value: --deno-config=custom.json, --deno-reload=jsr:@std/http
+                            Without value: --deno-reload, --deno-no-lock, --deno-no-prompt
 
 Selector Format:
   [!][type:]value

assets/usage.txt

@@ -6,6 +6,7 @@ Commands:
   init     Initialize a new probitas project
   run      Run scenarios
   list     List available scenarios
+  cache    Cache scenario dependencies (runs deno cache)
   fmt      Format scenario files (runs deno fmt)
   lint     Lint scenario files (runs deno lint)
   check    Type-check scenario files (runs deno check)

src/cli.ts

@@ -7,6 +7,7 @@
 import { parseArgs } from "@std/cli";
 import { getLogger } from "@logtape/logtape";
 import { EXIT_CODE } from "./cli/constants.ts";
+import { cacheCommand } from "./cli/commands/cache.ts";
 import { checkCommand } from "./cli/commands/check.ts";
 import { fmtCommand } from "./cli/commands/fmt.ts";
 import { initCommand } from "./cli/commands/init.ts";
@@ -81,6 +82,9 @@ export async function main(args: string[]): Promise<number> {
     case "list":
       return await listCommand(commandArgs, cwd);
 
+    case "cache":
+      return await cacheCommand(commandArgs, cwd);
+
     case "fmt":
       return await fmtCommand(commandArgs, cwd);
 

src/cli/commands/_deno.ts

@@ -26,6 +26,10 @@ export interface DenoSubcommandOptions {
   usageAsset: string;
   /** Extra arguments to pass to the deno command */
   extraArgs?: readonly string[];
+  /** Enable -r/--reload option support */
+  supportReload?: boolean;
+  /** Use deno.json/deno.lock config (default: false, adds --no-config) */
+  useConfig?: boolean;
 }
 
 /**
@@ -46,13 +50,14 @@ export async function runDenoSubcommand(
   try {
     const parsed = parseArgs(args, {
       string: ["config", "include", "exclude"],
-      boolean: ["help", "quiet", "verbose", "debug"],
+      boolean: ["help", "quiet", "verbose", "debug", "reload"],
       collect: ["include", "exclude"],
       alias: {
         h: "help",
         v: "verbose",
         q: "quiet",
         d: "debug",
+        r: "reload",
       },
       default: {
         include: undefined,
@@ -108,10 +113,11 @@ export async function runDenoSubcommand(
       fileCount: scenarioFiles.length,
     });
 
-    // Run deno command with --no-config
+    // Run deno command (with --no-config unless useConfig is true)
     const denoArgs = [
       subcommand,
-      "--no-config",
+      ...(options.useConfig ? [] : ["--no-config"]),
+      ...(options.supportReload && parsed.reload ? ["--reload"] : []),
       ...(options.extraArgs ?? []),
       ...scenarioFiles,
     ];

src/cli/commands/cache.ts

@@ -0,0 +1,27 @@
+/**
+ * Cache command for Probitas CLI
+ *
+ * Runs `deno cache` on discovered scenario files to pre-download dependencies.
+ *
+ * @module
+ */
+
+import { runDenoSubcommand } from "./_deno.ts";
+
+/**
+ * Run the cache command
+ *
+ * @param args - Command-line arguments
+ * @param cwd - Current working directory
+ * @returns Exit code
+ */
+export async function cacheCommand(
+  args: string[],
+  cwd: string,
+): Promise<number> {
+  return await runDenoSubcommand("cache", args, cwd, {
+    usageAsset: "usage-cache.txt",
+    supportReload: true,
+    useConfig: true,
+  });
+}

src/cli/commands/list.ts

@@ -102,11 +102,20 @@ export async function listCommand(
     // Configure logging with determined log level
     try {
       await configureLogging(logLevel);
-      logger.debug("List command started", { args, cwd, logLevel, denoArgs });
     } catch {
       // Silently ignore logging configuration errors (e.g., in test environments)
     }
 
+    // Add --reload to denoArgs if -r/--reload is specified
+    const finalDenoArgs = parsed.reload ? [...denoArgs, "--reload"] : denoArgs;
+
+    logger.debug("List command started", {
+      args,
+      cwd,
+      logLevel,
+      denoArgs: finalDenoArgs,
+    });
+
     // Load environment variables before loading configuration
     // This allows config files to reference environment variables
     await loadEnvironment(cwd, {
@@ -172,7 +181,7 @@ export async function listCommand(
     const { allScenarios, filteredScenarios } = await runListSubprocess(
       scenarioFiles,
       selectors,
-      denoArgs,
+      finalDenoArgs,
       cwd,
     );
 

src/cli/commands/mod.ts

@@ -4,6 +4,7 @@
  * @module
  */
 
+export { cacheCommand } from "./cache.ts";
 export { checkCommand } from "./check.ts";
 export { fmtCommand } from "./fmt.ts";
 export { initCommand } from "./init.ts";