@specsage/cli 0.1.7 → 0.1.8

package/lib/browser.js

@@ -248,6 +248,7 @@ async function enumerateElements() {
             w: Math.round(box.width),
             h: Math.round(box.height)
           },
+          locator: el,
           mechanism: 'native',
           source: 'native'
         });
@@ -336,6 +337,7 @@ async function enumerateElements() {
             w: Math.round(box.width),
             h: Math.round(box.height)
           },
+          locator: el,
           mechanism: 'scripted',
           source: source
         });
@@ -427,7 +429,8 @@ async function enumerateElements() {
   }
 
   lastElements = uniqueElements;
-  return lastElements;
+  // Return elements without the locator property (not JSON-serializable)
+  return lastElements.map(({ locator, ...rest }) => rest);
 }
 
 async function debugOverlay(x, y) {
@@ -499,9 +502,26 @@ async function handleCommand(msg) {
         if (!element) throw new Error(`Element not found: ${element_id}`);
         lastClickedElement = element; // Store for keypress context
 
-        const { x, y, w, h } = element.bounding_box;
+        let { x, y, w, h } = element.bounding_box;
+
+        // Auto-scroll element into view if its center is outside the viewport
+        const viewportSize = page.viewportSize();
+        let centerY = y + h / 2;
+        if (element.locator && (centerY < 0 || centerY >= viewportSize.height)) {
+          await element.locator.scrollIntoViewIfNeeded({ timeout: 3000 });
+          await new Promise(r => setTimeout(r, 200));
+          const newBox = await element.locator.boundingBox();
+          if (newBox) {
+            x = Math.round(newBox.x);
+            y = Math.round(newBox.y);
+            w = Math.round(newBox.width);
+            h = Math.round(newBox.height);
+            element.bounding_box = { x, y, w, h };
+          }
+        }
+
         const centerX = x + w / 2;
-        const centerY = y + h / 2;
+        centerY = y + h / 2;
 
         await debugOverlay(centerX, centerY);
 
@@ -559,9 +579,26 @@ async function handleCommand(msg) {
         if (!element) throw new Error(`Element not found: ${element_id}`);
 
         // Find the select element by its bounding box center
-        const { x, y, w, h } = element.bounding_box;
+        let { x, y, w, h } = element.bounding_box;
+
+        // Auto-scroll element into view if its center is outside the viewport
+        const viewportSize = page.viewportSize();
+        let centerY = y + h / 2;
+        if (element.locator && (centerY < 0 || centerY >= viewportSize.height)) {
+          await element.locator.scrollIntoViewIfNeeded({ timeout: 3000 });
+          await new Promise(r => setTimeout(r, 200));
+          const newBox = await element.locator.boundingBox();
+          if (newBox) {
+            x = Math.round(newBox.x);
+            y = Math.round(newBox.y);
+            w = Math.round(newBox.width);
+            h = Math.round(newBox.height);
+            element.bounding_box = { x, y, w, h };
+          }
+        }
+
         const centerX = x + w / 2;
-        const centerY = y + h / 2;
+        centerY = y + h / 2;
 
         // Use Playwright's selectOption by locating the element at the position
         const selectEl = page.locator('select').filter({

package/lib/cli.rb

@@ -71,6 +71,19 @@ class SpecSageCLI
     parser.parse!(@args)
   end
 
+  def build_ci_metadata
+    metadata = {}
+    metadata[:commit_sha] = ENV['GITHUB_SHA'] if ENV['GITHUB_SHA']
+    metadata[:branch] = ENV['GITHUB_REF_NAME'] if ENV['GITHUB_REF_NAME']
+    metadata[:pr_number] = ENV['GITHUB_PR_NUMBER'].to_i if ENV['GITHUB_PR_NUMBER']
+    metadata[:commit_message] = ENV['GITHUB_COMMIT_MESSAGE'] if ENV['GITHUB_COMMIT_MESSAGE']
+    metadata
+  end
+
+  def self_test_mode?
+    ENV['SPECSAGE_APP_TEST'] == 'true'
+  end
+
   def run_ci_mode
     website = ENV['TARGET_WEBSITE_SLUG']
     api_key = ENV['SPEC_SAGE_API_KEY']
@@ -91,6 +104,9 @@ class SpecSageCLI
     puts "Version: #{VERSION}"
     puts "Website: #{website}"
     puts "Base URL: #{base_url}"
+    if self_test_mode?
+      puts "Self-test mode: orchestration via #{base_url}"
+    end
     puts ""
 
     publisher = ResultsUploader.new(api_key: api_key)
@@ -108,7 +124,7 @@ class SpecSageCLI
     puts ""
 
     begin
-      run_response = publisher.create_ci_run(website)
+      run_response = publisher.create_ci_run(website, ci_metadata: build_ci_metadata)
       server_run_id = run_response['server_run_id']
     rescue ResultsUploader::UploadError => e
       puts "Error creating run: #{e.message}"
@@ -129,7 +145,8 @@ class SpecSageCLI
         visible: @options[:visible],
         record: @options[:record],
         publisher: publisher,
-        server_run_id: server_run_id
+        server_run_id: server_run_id,
+        orchestration_base_url: self_test_mode? ? base_url : nil
       )
 
       verdict = runner.run

package/lib/results_uploader.rb

@@ -58,8 +58,11 @@ class ResultsUploader
 
   # Create a CI run on the server
   # Returns hash with server_run_id and base_url
-  def create_ci_run(website_identifier)
-    post("/api/v1/ci/runs", { website: website_identifier })
+  # ci_metadata can include: commit_sha, branch, pr_number, commit_message
+  def create_ci_run(website_identifier, ci_metadata: {})
+    body = { website: website_identifier }
+    body.merge!(ci_metadata) if ci_metadata.any?
+    post("/api/v1/ci/runs", body)
   end
 
   private

package/lib/runner.rb

@@ -21,7 +21,9 @@ class Runner
   SAFE_PATH_SEGMENT = /\A[a-zA-Z0-9_-]+\z/
 
   # Initialize runner with scenario data from server
-  def initialize(scenario_data, visible: false, record: false, publisher: nil, server_run_id: nil)
+  # orchestration_base_url: URL for step submissions (LLM orchestration). Defaults to publisher.base_url.
+  #   Set to CI_APP_URL when SPECSAGE_APP_TEST=true to test server code locally.
+  def initialize(scenario_data, visible: false, record: false, publisher: nil, server_run_id: nil, orchestration_base_url: nil)
     @scenario = normalize_scenario_data(scenario_data)
     @scenario_id = @scenario['id']
     @scenario_name = @scenario['name'] || @scenario['id'] || 'unnamed'
@@ -34,6 +36,7 @@ class Runner
     @next_request_id = 1
     @node_channel_poisoned = false
     @publisher = publisher
+    @orchestration_base_url = orchestration_base_url || publisher&.base_url
     @step_client = nil
     @server_run_id = server_run_id
     @credentials = {} # Credentials received from server { "NAME" => "value" }
@@ -47,7 +50,7 @@ class Runner
     raise ArgumentError, 'server_run_id is required' unless @server_run_id
 
     @step_client = StepClient.new(
-      base_url: @publisher.base_url,
+      base_url: @orchestration_base_url,
       server_run_id: @server_run_id,
       api_key: @publisher.api_key
     )

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@specsage/cli",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "SpecSage CLI - AI-powered end-to-end testing automation (Node wrapper for Ruby CLI)",
   "type": "module",
   "bin": {
@@ -31,6 +31,6 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "playwright": "^1.57.0"
+    "playwright": "1.57.0"
   }
 }