@specsage/cli 0.1.11 → 0.1.13

package/lib/browser.js

@@ -1,5 +1,8 @@
 #!/usr/bin/env node
 
+// DO NOT EDIT packages/cli/lib/browser.js - it is copied from this file during npm publish.
+// See bin/publish_npm_package for details.
+
 import { chromium } from "playwright";
 import fs from "fs";
 import path from "path";

package/lib/client_errors.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+# Shared base error type for step clients (StepClient, DirectStepClient).
+# Runner can rescue this base type to handle errors from either client.
+# Each client defines its own StepError subclass for API stability.
+class StepClientError < StandardError; end

package/lib/dialogs.js

@@ -2,6 +2,9 @@
  * Dialog handling module for SpecSage browser automation.
  * Captures JavaScript dialogs (alert, confirm, prompt) and exposes them
  * to the AI agent via visual overlays and pseudo-elements.
+ *
+ * DO NOT EDIT packages/cli/lib/dialogs.js - it is copied from this file during npm publish.
+ * See bin/publish_npm_package for details.
  */
 
 // Pending dialog state

package/lib/runner.rb

@@ -1,6 +1,9 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
+# DO NOT EDIT packages/cli/lib/runner.rb - it is copied from this file during npm publish.
+# See bin/publish_npm_package for details.
+
 require 'json'
 require 'timeout'
 require 'fileutils'
@@ -8,6 +11,7 @@ require 'fileutils'
 # Determine SpecSage home directory for locating resources
 SPECSAGE_HOME ||= File.expand_path('..', __dir__)
 
+require_relative 'client_errors'
 require_relative 'step_client'
 
 class Runner
@@ -22,7 +26,8 @@ class Runner
 
   # Initialize runner with scenario data from server
   # @param all_scenarios [Hash] optional map of scenario_id => scenario_data for pre-scenario lookup
-  def initialize(scenario_data, visible: false, record: false, publisher: nil, server_run_id: nil, all_scenarios: nil)
+  # @param step_client [StepClient, DirectStepClient] optional pre-configured client for step processing
+  def initialize(scenario_data, visible: false, record: false, publisher: nil, server_run_id: nil, all_scenarios: nil, step_client: nil)
     @scenario = normalize_scenario_data(scenario_data)
     @scenario_id = @scenario['id']
     @scenario_name = @scenario['name'] || @scenario['id'] || 'unnamed'
@@ -35,7 +40,7 @@ class Runner
     @next_request_id = 1
     @node_channel_poisoned = false
     @publisher = publisher
-    @step_client = nil
+    @step_client = step_client # Accept pre-configured client (nil = create StepClient in run())
     @server_run_id = server_run_id
     @credentials = {} # Credentials received from server { "NAME" => "value" }
     @max_steps = nil # Max browser actions allowed, received from server on first step
@@ -48,7 +53,10 @@ class Runner
 
     raise ArgumentError, 'server_run_id is required' unless @server_run_id
 
-    @step_client = StepClient.new(
+    # Only create StepClient if not injected (CLI path uses HTTP, Sidekiq injects DirectStepClient)
+    # TODO: Validate publisher presence here when step_client is nil. Currently crashes with
+    # NoMethodError on @publisher.base_url if caller omits both step_client and publisher.
+    @step_client ||= StepClient.new(
       base_url: @publisher.base_url,
       server_run_id: @server_run_id,
       api_key: @publisher.api_key
@@ -69,7 +77,7 @@ class Runner
           stop_node_process
           upload_video
           cleanup_temp_dir
-          return
+          return 'ERROR'
         end
 
         pre_scenario_name = pre_scenario_data['name'] || pre_scenario_id
@@ -92,14 +100,14 @@ class Runner
           stop_node_process
           upload_video
           cleanup_temp_dir
-          return
+          return 'FAIL'
         elsif result[:verdict] == 'ERROR'
           log "Pre-scenario '#{pre_scenario_name}' ERROR - skipping main scenario"
           send_main_scenario_verdict('PRE_SCENARIO_ERROR', "Pre-scenario '#{pre_scenario_name}' errored: #{result[:reason]}")
           stop_node_process
           upload_video
           cleanup_temp_dir
-          return
+          return 'ERROR'
         end
 
         log "Pre-scenario '#{pre_scenario_name}' PASSED"
@@ -109,7 +117,7 @@ class Runner
     end
 
     # Run the main scenario
-    run_single_scenario(
+    result = run_single_scenario(
       scenario_id: @scenario_id,
       scenario_data: @scenario,
       pre_scenario_for_id: nil,
@@ -119,16 +127,20 @@ class Runner
     stop_node_process
     upload_video
     cleanup_temp_dir
-  rescue StepClient::StepError => e
+
+    result[:verdict]
+  rescue StepClientError => e
     send_client_verdict_if_needed('ERROR', "Server error: #{e.message}")
     stop_node_process
     upload_video
     cleanup_temp_dir
+    'ERROR'
   rescue StandardError => e
     send_client_verdict_if_needed('ERROR', e.message)
     stop_node_process
     upload_video
     cleanup_temp_dir
+    'ERROR'
   end
 
   # Run a single scenario (either pre-scenario or main scenario)
@@ -238,7 +250,7 @@ class Runner
       status: status,
       reason: reason
     )
-  rescue StepClient::StepError => e
+  rescue StepClientError => e
     log "Warning: Failed to send main scenario verdict: #{e.message}"
   end
 
@@ -580,7 +592,7 @@ class Runner
     log "Uploading video (#{@video_data.bytesize} bytes)..."
     @step_client.upload_video(scenario_id: @scenario_id, video_data: @video_data)
     log "Video uploaded successfully."
-  rescue StepClient::StepError => e
+  rescue StepClientError => e
     log "Warning: Failed to upload video: #{e.message}"
   end
 
@@ -634,7 +646,7 @@ class Runner
       status: status,
       reason: reason
     )
-  rescue StepClient::StepError => e
+  rescue StepClientError => e
     log "Warning: Failed to send verdict to server: #{e.message}"
   end
 

package/lib/step_client.rb

@@ -1,12 +1,18 @@
 # frozen_string_literal: true
 
+# DO NOT EDIT packages/cli/lib/step_client.rb - it is copied from this file during npm publish.
+# See bin/publish_npm_package for details.
+
 require "net/http"
 require "uri"
 require "json"
 require "base64"
+require_relative "client_errors"
 
 class StepClient
-  class StepError < StandardError; end
+  # Subclass shared error for API stability (supports subclassing, .class checks, .name)
+  # Runner can rescue either StepClient::StepError or the base StepClientError
+  class StepError < StepClientError; end
 
   attr_reader :server_run_id
 
@@ -56,6 +62,10 @@ class StepClient
 
     response = post("/api/runs/#{@server_run_id}/step", body)
 
+    # TODO: Credentials defaulted to {} may change semantics. Server returns nil when no
+    # credentials are present (after first step). Returning {} instead of nil may cause
+    # downstream code to branch incorrectly (`if credentials` is always truthy). Consider
+    # returning nil when server returns nil, or document this as intentional contract.
     {
       action: response["action"],
       step_number: response["step_number"],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@specsage/cli",
-  "version": "0.1.11",
+  "version": "0.1.13",
   "description": "SpecSage CLI - AI-powered end-to-end testing automation (Node wrapper for Ruby CLI)",
   "type": "module",
   "bin": {