cdp-skill 1.0.8 → 1.0.14

package/src/runner/step-validator.js

@@ -7,10 +7,10 @@
  * - validateStepInternal(step) → string[] - Internal per-step validation
  *
  * DEPENDENCIES:
- * - ./context-helpers.js: STEP_TYPES
+ * - ./step-registry.js: getAllStepTypes, getStepConfig, validateHooks
  */
 
-import { STEP_TYPES } from './context-helpers.js';
+import { getAllStepTypes, getStepConfig, validateHooks } from './step-registry.js';
 
 /**
  * Validate a single step definition
@@ -25,10 +25,11 @@ export function validateStepInternal(step) {
     return errors;
   }
 
-  const definedActions = STEP_TYPES.filter(type => step[type] !== undefined);
+  const allStepTypes = getAllStepTypes();
+  const definedActions = allStepTypes.filter(type => step[type] !== undefined);
 
   if (definedActions.length === 0) {
-    errors.push(`unknown step type, expected one of: ${STEP_TYPES.join(', ')}`);
+    errors.push(`unknown step type, expected one of: ${allStepTypes.join(', ')}`);
     return errors;
   }
 
@@ -40,759 +41,20 @@ export function validateStepInternal(step) {
   const action = definedActions[0];
   const params = step[action];
 
-  switch (action) {
-    case 'goto':
-      // Support both string URL and object format with options
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('goto requires a non-empty URL string');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.url || typeof params.url !== 'string') {
-          errors.push('goto requires a non-empty url property');
-        }
-        // Validate waitUntil if provided
-        if (params.waitUntil !== undefined) {
-          const validWaitUntil = ['commit', 'domcontentloaded', 'load', 'networkidle'];
-          if (!validWaitUntil.includes(params.waitUntil)) {
-            errors.push(`goto waitUntil must be one of: ${validWaitUntil.join(', ')}`);
-          }
-        }
-      } else {
-        errors.push('goto requires a URL string or object with url property');
-      }
-      break;
-
-    case 'reload':
-      // reload can be boolean true or object with options
-      if (params !== true && typeof params !== 'object') {
-        errors.push('reload requires true or params object');
-      }
-      if (typeof params === 'object' && params.waitUntil !== undefined) {
-        const validWaitUntil = ['commit', 'domcontentloaded', 'load', 'networkidle'];
-        if (!validWaitUntil.includes(params.waitUntil)) {
-          errors.push(`reload waitUntil must be one of: ${validWaitUntil.join(', ')}`);
-        }
-      }
-      break;
-
-    case 'wait':
-      // Support numeric value for simple delay: { "wait": 2000 }
-      if (typeof params === 'number') {
-        if (params < 0) {
-          errors.push('wait time must be a non-negative number');
-        }
-      } else if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('wait selector cannot be empty');
-        }
-      } else if (params && typeof params === 'object') {
-        const hasSelector = params.selector !== undefined;
-        const hasText = params.text !== undefined;
-        const hasTextRegex = params.textRegex !== undefined;
-        const hasTime = params.time !== undefined;
-        const hasUrlContains = params.urlContains !== undefined;
-        if (!hasSelector && !hasText && !hasTextRegex && !hasTime && !hasUrlContains) {
-          errors.push('wait requires selector, text, textRegex, time, or urlContains');
-        }
-        if (hasSelector && typeof params.selector !== 'string') {
-          errors.push('wait selector must be a string');
-        }
-        if (hasText && typeof params.text !== 'string') {
-          errors.push('wait text must be a string');
-        }
-        if (hasTextRegex && typeof params.textRegex !== 'string') {
-          errors.push('wait textRegex must be a string');
-        }
-        if (hasTime && (typeof params.time !== 'number' || params.time < 0)) {
-          errors.push('wait time must be a non-negative number');
-        }
-        if (hasUrlContains && typeof params.urlContains !== 'string') {
-          errors.push('wait urlContains must be a string');
-        }
-        if (params.minCount !== undefined && (typeof params.minCount !== 'number' || params.minCount < 0)) {
-          errors.push('wait minCount must be a non-negative number');
-        }
-        if (params.caseSensitive !== undefined && typeof params.caseSensitive !== 'boolean') {
-          errors.push('wait caseSensitive must be a boolean');
-        }
-        if (params.hidden !== undefined && typeof params.hidden !== 'boolean') {
-          errors.push('wait hidden must be a boolean');
-        }
-      } else {
-        errors.push('wait requires a number (ms), selector string, or params object');
-      }
-      break;
-
-    case 'click':
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('click selector cannot be empty');
-        }
-      } else if (params && typeof params === 'object') {
-        // Check for coordinate-based click
-        const hasCoordinates = typeof params.x === 'number' && typeof params.y === 'number';
-        // Check for text-based click
-        const hasText = typeof params.text === 'string';
-        // Check for multi-selector fallback
-        const hasSelectors = Array.isArray(params.selectors);
-        if (!params.selector && !params.ref && !hasCoordinates && !hasText && !hasSelectors) {
-          errors.push('click requires selector, ref, text, selectors array, or x/y coordinates');
-        } else if (params.selector && typeof params.selector !== 'string') {
-          errors.push('click selector must be a string');
-        } else if (params.ref && typeof params.ref !== 'string') {
-          errors.push('click ref must be a string');
-        } else if (hasText && params.text.length === 0) {
-          errors.push('click text cannot be empty');
-        } else if (hasSelectors && params.selectors.length === 0) {
-          errors.push('click selectors array cannot be empty');
-        } else if (hasCoordinates) {
-          if (params.x < 0 || params.y < 0) {
-            errors.push('click coordinates must be non-negative');
-          }
-        }
-      } else {
-        errors.push('click requires a selector string or params object');
-      }
-      break;
-
-    case 'fill':
-      if (!params || typeof params !== 'object') {
-        errors.push('fill requires an object with selector/ref/label and value');
-      } else {
-        if (!params.selector && !params.ref && !params.label) {
-          errors.push('fill requires selector, ref, or label');
-        } else if (params.selector && typeof params.selector !== 'string') {
-          errors.push('fill selector must be a string');
-        } else if (params.ref && typeof params.ref !== 'string') {
-          errors.push('fill ref must be a string');
-        } else if (params.label && typeof params.label !== 'string') {
-          errors.push('fill label must be a string');
-        }
-        if (params.value === undefined) {
-          errors.push('fill requires value');
-        }
-      }
-      break;
-
-    case 'fillForm':
-      if (!params || typeof params !== 'object') {
-        errors.push('fillForm requires an object mapping selectors/refs to values');
-      } else {
-        // Support both formats:
-        // Simple: {"#firstName": "John", "#lastName": "Doe"}
-        // Extended: {"fields": {"#firstName": "John"}, "react": true}
-        let fields;
-        if (params.fields && typeof params.fields === 'object') {
-          fields = params.fields;
-          // Validate react option if present
-          if (params.react !== undefined && typeof params.react !== 'boolean') {
-            errors.push('fillForm react option must be a boolean');
-          }
-        } else {
-          fields = params;
-        }
-        const entries = Object.entries(fields);
-        if (entries.length === 0) {
-          errors.push('fillForm requires at least one field');
-        }
-      }
-      break;
-
-    case 'press':
-      if (typeof params !== 'string' || params.length === 0) {
-        errors.push('press requires a non-empty key string');
-      }
-      break;
-
-    case 'query':
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('query selector cannot be empty');
-        }
-      } else if (params && typeof params === 'object') {
-        // Support both CSS selector and role-based queries
-        if (!params.selector && !params.role) {
-          errors.push('query requires selector or role');
-        }
-        // Role can be string or array of strings (compound roles)
-        if (params.role) {
-          if (typeof params.role !== 'string' && !Array.isArray(params.role)) {
-            errors.push('query role must be a string or array of strings');
-          }
-          if (Array.isArray(params.role) && !params.role.every(r => typeof r === 'string')) {
-            errors.push('query role array must contain only strings');
-          }
-        }
-        // Validate nameExact and nameRegex are not both set
-        if (params.nameExact && params.nameRegex) {
-          errors.push('query cannot have both nameExact and nameRegex');
-        }
-      } else {
-        errors.push('query requires a selector string or params object');
-      }
-      break;
-
-    case 'inspect':
-      // inspect can be boolean or object with options
-      break;
-
-    case 'scroll':
-      if (typeof params === 'string') {
-        if (!['top', 'bottom', 'up', 'down'].includes(params) && params.length === 0) {
-          errors.push('scroll requires direction (top/bottom/up/down) or selector');
-        }
-      } else if (params && typeof params === 'object') {
-        // selector, x, y, deltaX, deltaY are all valid
-      } else if (typeof params !== 'string') {
-        errors.push('scroll requires direction string or params object');
-      }
-      break;
-
-    case 'console':
-      // console can be boolean or object with filter options
-      break;
-
-    case 'pdf':
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('pdf path cannot be empty');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.path) {
-          errors.push('pdf requires path');
-        } else if (typeof params.path !== 'string') {
-          errors.push('pdf path must be a string');
-        }
-      } else {
-        errors.push('pdf requires a path string or params object');
-      }
-      break;
-
-    case 'eval':
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('eval expression cannot be empty');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.expression) {
-          errors.push('eval requires expression');
-        } else if (typeof params.expression !== 'string') {
-          errors.push('eval expression must be a string');
-        }
-      } else {
-        errors.push('eval requires an expression string or params object');
-      }
-      break;
-
-    case 'snapshot':
-      // snapshot can be boolean or object with options
-      if (params !== true && params !== false && typeof params !== 'object') {
-        errors.push('snapshot requires true or params object');
-      }
-      if (typeof params === 'object') {
-        if (params.mode && !['ai', 'full'].includes(params.mode)) {
-          errors.push('snapshot mode must be "ai" or "full"');
-        }
-        if (params.detail && !['summary', 'interactive', 'full'].includes(params.detail)) {
-          errors.push('snapshot detail must be "summary", "interactive", or "full"');
-        }
-        if (params.inlineLimit !== undefined && (typeof params.inlineLimit !== 'number' || params.inlineLimit < 0)) {
-          errors.push('snapshot inlineLimit must be a non-negative number');
-        }
-      }
-      break;
-
-    case 'snapshotSearch':
-      // snapshotSearch requires at least one search criterion
-      if (!params || typeof params !== 'object') {
-        errors.push('snapshotSearch requires an object with search parameters');
-      } else {
-        const hasText = params.text !== undefined;
-        const hasPattern = params.pattern !== undefined;
-        const hasRole = params.role !== undefined;
-        if (!hasText && !hasPattern && !hasRole) {
-          errors.push('snapshotSearch requires at least one of: text, pattern, or role');
-        }
-        if (hasText && typeof params.text !== 'string') {
-          errors.push('snapshotSearch text must be a string');
-        }
-        if (hasPattern && typeof params.pattern !== 'string') {
-          errors.push('snapshotSearch pattern must be a string (regex)');
-        }
-        if (hasRole && typeof params.role !== 'string') {
-          errors.push('snapshotSearch role must be a string');
-        }
-        if (params.limit !== undefined && (typeof params.limit !== 'number' || params.limit < 1)) {
-          errors.push('snapshotSearch limit must be a positive number');
-        }
-        if (params.context !== undefined && (typeof params.context !== 'number' || params.context < 0)) {
-          errors.push('snapshotSearch context must be a non-negative number');
-        }
-      }
-      break;
-
-    case 'hover':
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('hover selector cannot be empty');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.selector && !params.ref) {
-          errors.push('hover requires selector or ref');
-        }
-      } else {
-        errors.push('hover requires a selector string or params object');
-      }
-      break;
-
-    case 'viewport':
-      // Support both device preset strings and explicit config objects
-      if (typeof params === 'string') {
-        // Device preset name - validation happens at execution time
-        if (params.length === 0) {
-          errors.push('viewport preset name cannot be empty');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.width || typeof params.width !== 'number') {
-          errors.push('viewport requires numeric width');
-        }
-        if (!params.height || typeof params.height !== 'number') {
-          errors.push('viewport requires numeric height');
-        }
-      } else {
-        errors.push('viewport requires a device preset string or object with width and height');
-      }
-      break;
-
-    case 'cookies':
-      if (!params || typeof params !== 'object') {
-        errors.push('cookies requires an object with action (get, set, or clear)');
-      } else {
-        if (params.set && !Array.isArray(params.set)) {
-          errors.push('cookies set requires an array of cookie objects');
-        }
-      }
-      break;
-
-    case 'back':
-      if (params !== true && typeof params !== 'object') {
-        errors.push('back requires true or params object');
-      }
-      break;
-
-    case 'forward':
-      if (params !== true && typeof params !== 'object') {
-        errors.push('forward requires true or params object');
-      }
-      break;
-
-    case 'waitForNavigation':
-      if (params !== true && typeof params !== 'object') {
-        errors.push('waitForNavigation requires true or params object');
-      }
-      if (typeof params === 'object' && params.timeout !== undefined) {
-        if (typeof params.timeout !== 'number' || params.timeout < 0) {
-          errors.push('waitForNavigation timeout must be a non-negative number');
-        }
-      }
-      break;
-
-    case 'listTabs':
-      // listTabs can be boolean true
-      if (params !== true) {
-        errors.push('listTabs requires true');
-      }
-      break;
-
-    case 'closeTab':
-      if (typeof params !== 'string' || params.length === 0) {
-        errors.push('closeTab requires a non-empty targetId string');
-      }
-      break;
-
-    case 'openTab':
-      // openTab can be:
-      // - true: just open a blank tab
-      // - string: open tab and navigate to URL
-      // - object with options: {url: "...", viewport: {...}}
-      if (params !== true && typeof params !== 'string' && (typeof params !== 'object' || params === null)) {
-        errors.push('openTab must be true, a URL string, or an options object');
-      }
-      if (typeof params === 'object' && params !== null && params.url !== undefined && typeof params.url !== 'string') {
-        errors.push('openTab url must be a string');
-      }
-      break;
-
-    case 'type':
-      if (!params || typeof params !== 'object') {
-        errors.push('type requires an object with selector and text');
-      } else {
-        if (!params.selector) {
-          errors.push('type requires selector');
-        } else if (typeof params.selector !== 'string') {
-          errors.push('type selector must be a string');
-        }
-        if (params.text === undefined) {
-          errors.push('type requires text');
-        }
-      }
-      break;
-
-    case 'select':
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('select selector cannot be empty');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.selector) {
-          errors.push('select requires selector');
-        } else if (typeof params.selector !== 'string') {
-          errors.push('select selector must be a string');
-        }
-        if (params.start !== undefined && typeof params.start !== 'number') {
-          errors.push('select start must be a number');
-        }
-        if (params.end !== undefined && typeof params.end !== 'number') {
-          errors.push('select end must be a number');
-        }
-      } else {
-        errors.push('select requires a selector string or params object');
-      }
-      break;
-
-    case 'validate':
-      if (typeof params !== 'string' || params.length === 0) {
-        errors.push('validate requires a non-empty selector string');
-      }
-      break;
-
-    case 'submit':
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('submit requires a non-empty form selector');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.selector) {
-          errors.push('submit requires selector');
-        } else if (typeof params.selector !== 'string') {
-          errors.push('submit selector must be a string');
-        }
-      } else {
-        errors.push('submit requires a selector string or params object');
-      }
-      break;
-
-    case 'assert':
-      if (!params || typeof params !== 'object') {
-        errors.push('assert requires an object with url, text, or selector');
-      } else {
-        const hasUrl = params.url !== undefined;
-        const hasText = params.text !== undefined;
-        if (!hasUrl && !hasText) {
-          errors.push('assert requires url or text');
-        }
-        if (hasUrl && typeof params.url !== 'object') {
-          errors.push('assert url must be an object (e.g., { contains: "..." })');
-        }
-        if (hasUrl && params.url && !params.url.contains && !params.url.equals && !params.url.startsWith && !params.url.endsWith && !params.url.matches) {
-          errors.push('assert url requires contains, equals, startsWith, endsWith, or matches');
-        }
-        if (hasText && typeof params.text !== 'string') {
-          errors.push('assert text must be a string');
-        }
-        if (params.selector && typeof params.selector !== 'string') {
-          errors.push('assert selector must be a string');
-        }
-      }
-      break;
-
-    case 'queryAll':
-      if (!params || typeof params !== 'object') {
-        errors.push('queryAll requires an object mapping names to selectors');
-      } else {
-        const entries = Object.entries(params);
-        if (entries.length === 0) {
-          errors.push('queryAll requires at least one query');
-        }
-        for (const [name, selector] of entries) {
-          if (typeof selector !== 'string' && typeof selector !== 'object') {
-            errors.push(`queryAll "${name}" must be a selector string or query object`);
-          }
-        }
-      }
-      break;
-
-    case 'switchToFrame':
-      // Can be string (selector/name), number (index), or object
-      if (params === null || params === undefined) {
-        errors.push('switchToFrame requires a selector, index, or options object');
-      }
-      break;
-
-    case 'switchToMainFrame':
-      // No validation needed, params can be true or anything
-      break;
-
-    case 'listFrames':
-      // No validation needed
-      break;
-
-    case 'drag':
-      if (!params || typeof params !== 'object') {
-        errors.push('drag requires an object with source and target');
-      } else {
-        if (!params.source) {
-          errors.push('drag requires a source selector or coordinates');
-        }
-        if (!params.target) {
-          errors.push('drag requires a target selector or coordinates');
-        }
-      }
-      break;
-
-    case 'formState':
-      if (typeof params !== 'string' && (!params || !params.selector)) {
-        errors.push('formState requires a selector string or object with selector');
-      }
-      break;
-
-    case 'extract':
-      if (typeof params !== 'string' && (!params || !params.selector)) {
-        errors.push('extract requires a selector string or object with selector');
-      }
-      break;
-
-    case 'selectOption':
-      // selectOption: {"selector": "#dropdown", "value": "optionValue"}
-      // or: {"selector": "#dropdown", "label": "Option Text"}
-      // or: {"selector": "#dropdown", "index": 2}
-      if (!params || typeof params !== 'object') {
-        errors.push('selectOption requires an object with selector and value/label/index');
-      } else {
-        if (!params.selector) {
-          errors.push('selectOption requires selector');
-        }
-        if (params.value === undefined && params.label === undefined && params.index === undefined && !params.values) {
-          errors.push('selectOption requires value, label, index, or values');
-        }
-      }
-      break;
-
-    case 'getDom':
-      // getDom: true (full page) or selector string or object with selector
-      if (params !== true && typeof params !== 'string' && (typeof params !== 'object' || params === null)) {
-        errors.push('getDom requires true, a selector string, or an options object');
-      }
-      if (typeof params === 'object' && params !== null && params.selector && typeof params.selector !== 'string') {
-        errors.push('getDom selector must be a string');
-      }
-      break;
-
-    case 'getBox':
-      // getBox: "s1e1" or ["s1e1", "s1e2"] or {"refs": ["s1e1", "s1e2"]}
-      // Versioned ref format: s{snapshotId}e{elementId}
-      if (typeof params === 'string') {
-        if (!/^s\d+e\d+$/.test(params)) {
-          errors.push('getBox ref must be in format "s{N}e{M}" (e.g., "s1e1", "s2e34")');
-        }
-      } else if (Array.isArray(params)) {
-        if (params.length === 0) {
-          errors.push('getBox refs array cannot be empty');
-        }
-        for (const ref of params) {
-          if (typeof ref !== 'string' || !/^s\d+e\d+$/.test(ref)) {
-            errors.push('getBox refs must be strings in format "s{N}e{M}"');
-            break;
-          }
-        }
-      } else if (typeof params === 'object' && params !== null) {
-        if (!params.refs && !params.ref) {
-          errors.push('getBox requires ref or refs');
-        }
-      } else {
-        errors.push('getBox requires a ref string, array of refs, or options object');
-      }
-      break;
-
-    case 'fillActive':
-      // fillActive: "text" or {"value": "text", "clear": true}
-      if (typeof params === 'string') {
-        // Simple string value is fine
-      } else if (typeof params === 'object' && params !== null) {
-        if (params.value === undefined) {
-          errors.push('fillActive requires value');
-        }
-      } else {
-        errors.push('fillActive requires a string value or options object with value');
-      }
-      break;
-
-    case 'refAt':
-      // refAt: {"x": 100, "y": 200}
-      if (!params || typeof params !== 'object') {
-        errors.push('refAt requires an object with x and y coordinates');
-      } else {
-        if (typeof params.x !== 'number') {
-          errors.push('refAt requires x coordinate as a number');
-        }
-        if (typeof params.y !== 'number') {
-          errors.push('refAt requires y coordinate as a number');
-        }
-      }
-      break;
-
-    case 'elementsAt':
-      // elementsAt: [{"x": 100, "y": 200}, {"x": 300, "y": 400}]
-      if (!Array.isArray(params)) {
-        errors.push('elementsAt requires an array of {x, y} coordinates');
-      } else if (params.length === 0) {
-        errors.push('elementsAt array cannot be empty');
-      } else {
-        for (let i = 0; i < params.length; i++) {
-          const coord = params[i];
-          if (!coord || typeof coord !== 'object') {
-            errors.push(`elementsAt[${i}] must be an object with x and y`);
-          } else if (typeof coord.x !== 'number' || typeof coord.y !== 'number') {
-            errors.push(`elementsAt[${i}] requires x and y as numbers`);
-          }
-        }
-      }
-      break;
-
-    case 'elementsNear':
-      // elementsNear: {"x": 100, "y": 200, "radius": 50}
-      if (!params || typeof params !== 'object') {
-        errors.push('elementsNear requires an object with x, y, and optional radius');
-      } else {
-        if (typeof params.x !== 'number') {
-          errors.push('elementsNear requires x coordinate as a number');
-        }
-        if (typeof params.y !== 'number') {
-          errors.push('elementsNear requires y coordinate as a number');
-        }
-        if (params.radius !== undefined && typeof params.radius !== 'number') {
-          errors.push('elementsNear radius must be a number');
-        }
-      }
-      break;
-
-    case 'pageFunction':
-      // pageFunction: "() => document.title" or {fn, refs, timeout}
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('pageFunction requires a non-empty function string');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.fn || typeof params.fn !== 'string') {
-          errors.push('pageFunction requires a non-empty fn string');
-        }
-        if (params.refs !== undefined && typeof params.refs !== 'boolean') {
-          errors.push('pageFunction refs must be a boolean');
-        }
-        if (params.timeout !== undefined && (typeof params.timeout !== 'number' || params.timeout < 0)) {
-          errors.push('pageFunction timeout must be a non-negative number');
-        }
-      } else {
-        errors.push('pageFunction requires a function string or params object');
-      }
-      break;
-
-    case 'poll':
-      // poll: "() => condition" or {fn, interval, timeout}
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('poll requires a non-empty function string');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.fn || typeof params.fn !== 'string') {
-          errors.push('poll requires a non-empty fn string');
-        }
-        if (params.interval !== undefined && (typeof params.interval !== 'number' || params.interval < 0)) {
-          errors.push('poll interval must be a non-negative number');
-        }
-        if (params.timeout !== undefined && (typeof params.timeout !== 'number' || params.timeout < 0)) {
-          errors.push('poll timeout must be a non-negative number');
-        }
-      } else {
-        errors.push('poll requires a function string or params object');
-      }
-      break;
-
-    case 'pipeline':
-      // pipeline: [{find, fill}, ...] or {steps, timeout}
-      {
-        const pipelineSteps = Array.isArray(params) ? params : (params && params.steps);
-        if (!Array.isArray(pipelineSteps) || pipelineSteps.length === 0) {
-          errors.push('pipeline requires a non-empty array of micro-operations');
-        } else {
-          for (let pi = 0; pi < pipelineSteps.length; pi++) {
-            const op = pipelineSteps[pi];
-            if (!op || typeof op !== 'object') {
-              errors.push(`pipeline step ${pi}: must be an object`);
-              continue;
-            }
-            const hasMicroOp = op.find || op.waitFor || op.sleep !== undefined || op.return;
-            if (!hasMicroOp) {
-              errors.push(`pipeline step ${pi}: unrecognized micro-op (need find, waitFor, sleep, or return)`);
-            }
-            if (op.find) {
-              const hasAction = op.fill !== undefined || op.click !== undefined ||
-                op.type !== undefined || op.check !== undefined || op.select !== undefined;
-              if (!hasAction) {
-                errors.push(`pipeline step ${pi}: find requires an action (fill, click, type, check, or select)`);
-              }
-            }
-          }
-        }
-        if (!Array.isArray(params) && params && params.timeout !== undefined) {
-          if (typeof params.timeout !== 'number' || params.timeout < 0) {
-            errors.push('pipeline timeout must be a non-negative number');
-          }
-        }
-      }
-      break;
-
-    case 'writeSiteProfile':
-      if (!params || typeof params !== 'object') {
-        errors.push('writeSiteProfile requires an object with domain and content');
-      } else {
-        if (!params.domain || typeof params.domain !== 'string') {
-          errors.push('writeSiteProfile requires a non-empty domain string');
-        }
-        if (!params.content || typeof params.content !== 'string') {
-          errors.push('writeSiteProfile requires a non-empty content string');
-        }
-      }
-      break;
-
-    case 'readSiteProfile':
-      if (typeof params === 'string') {
-        if (params.length === 0) {
-          errors.push('readSiteProfile requires a non-empty domain string');
-        }
-      } else if (params && typeof params === 'object') {
-        if (!params.domain || typeof params.domain !== 'string') {
-          errors.push('readSiteProfile requires a non-empty domain string');
-        }
-      } else {
-        errors.push('readSiteProfile requires a domain string or object with domain');
-      }
-      break;
+  // Get step configuration from registry
+  const stepConfig = getStepConfig(action);
+  if (!stepConfig) {
+    errors.push(`No configuration found for step type: ${action}`);
+    return errors;
   }
 
+  // Run step-specific validation from registry
+  const stepErrors = stepConfig.validate(params);
+  errors.push(...stepErrors);
+
   // Validate hooks on action steps (readyWhen, settledWhen, observe)
-  if (typeof params === 'object' && params !== null) {
-    if (params.readyWhen !== undefined && typeof params.readyWhen !== 'string') {
-      errors.push('readyWhen must be a function string');
-    }
-    if (params.settledWhen !== undefined && typeof params.settledWhen !== 'string') {
-      errors.push('settledWhen must be a function string');
-    }
-    if (params.observe !== undefined && typeof params.observe !== 'string') {
-      errors.push('observe must be a function string');
-    }
-  }
+  const hookErrors = validateHooks(params);
+  errors.push(...hookErrors);
 
   return errors;
 }