refactor: reorganize task commands and add auto-fix to pre-commit

- Delete founder/ commands (moved to ops/)
- Add backend:db:* commands with clear dev/test cluster separation
- Add qa:fix command for auto-fixing linting issues
- Add qa:rules:check command for founder rules validation
- Update pre-commit hook to auto-fix code quality on commit
- Fix 170+ linting issues via biome check --write
- Fix non-null assertions and empty type definitions
- Update help text to reference ops:* instead of founder:*

Commands now organized by clear boundaries:
- backend/ → Database management (dev/test clusters)
- frontend/ → Frontend development
- ops/ → Service lifecycle management
- qa/ → All testing and quality checks
- shared/ → Cross-cutting utilities

Author: nirukk52

.cursor/Taskfile.yml

@@ -11,10 +11,6 @@ includes:
     taskfile: ./commands/shared/Taskfile.yml
     dir: ./commands/shared
 
-  founder:
-    taskfile: ./commands/founder/Taskfile.yml
-    dir: ./commands/founder
-  
   backend:
     taskfile: ./commands/backend/Taskfile.yml
     dir: ./commands/backend
@@ -52,14 +48,14 @@ tasks:
       - echo "ScreenGraph Unified Automation System"
       - echo ""
       - echo "Common commands:"
-      - echo "  task founder:servers:start    # Start backend + frontend"
-      - echo "  task founder:servers:stop     # Stop all services"
-      - echo "  task founder:servers:status   # Check service status"
-      - echo "  task qa:smoke:backend         # Run backend smoke tests"
-      - echo "  task qa:smoke:frontend        # Run frontend smoke tests"
-      - echo "  task founder:rules:check      # Validate founder rules"
-      - echo "  node automation/scripts/env.mjs status  # Show standard ports"
+      - echo "  task ops:servers:start        - Start backend + frontend"
+      - echo "  task ops:servers:stop         - Stop all services"
+      - echo "  task ops:servers:status       - Check service status"
+      - echo "  task qa:smoke:backend         - Run backend smoke tests"
+      - echo "  task qa:smoke:frontend        - Run frontend smoke tests"
+      - echo "  task qa:rules:check           - Validate founder rules"
+      - echo "  task backend:db:status        - Show database cluster info"
       - echo ""
-      - echo "Run 'task --list-all' for all available tasks"
+      - echo "Run task --list-all for all available tasks"
     silent: true
 

.cursor/commands/founder/Taskfile.yml

@@ -138,3 +138,21 @@ tasks:
       - lsof -ti:${APPIUM_PORT:-4723} 2>/dev/null | xargs kill -9 2>/dev/null || true
       - echo "✅ Appium server stopped"
     silent: false
+
+  # Rules validation
+  rules:check:
+    desc: "Validate all founder rules"
+    cmds:
+      - node ../../../automation/scripts/check-founder-rules.mjs
+    silent: false
+
+  # Auto-fix code quality issues
+  fix:
+    desc: "Auto-fix linting and formatting issues"
+    cmds:
+      - echo "🔧 Auto-fixing backend..."
+      - cd ../../../backend && bunx biome check --write .
+      - echo "🔧 Auto-fixing frontend..."
+      - cd ../../../frontend && bunx biome check --write .
+      - echo "✅ Auto-fix complete"
+    silent: false

.cursor/commands/qa/Taskfile.yml

@@ -1,8 +1,14 @@
 #!/bin/sh
 # Husky pre-commit hook
-# Validates founder rules before allowing commit
+# Auto-fixes and validates code before allowing commit
 
-cd .cursor && task founder:rules:check
+# Auto-fix linting and formatting
+echo "🔧 Auto-fixing code quality issues..."
+cd .cursor && task qa:fix
+
+# Validate founder rules
+echo "📋 Validating founder rules..."
+cd .cursor && task qa:rules:check
 
 # Cursor docs validation (informational, won't block commit)
 cd .. && node automation/scripts/validate-cursor-docs.mjs || echo "⚠️  Cursor docs validation failed (non-blocking)"

.husky/pre-commit

@@ -1,9 +1,4 @@
-import {
-  AppCrashedError,
-  AppNotInstalledError,
-  DeviceOfflineError,
-  TimeoutError,
-} from "./errors";
+import { AppCrashedError, AppNotInstalledError, DeviceOfflineError, TimeoutError } from "./errors";
 
 /**
  * DriverErrorKind enumerates normalized error categories surfaced by Appium adapters.
@@ -41,4 +36,3 @@ export function mapAdapterErrorToDriverErrorKind(error: unknown): DriverErrorKin
 
   return DriverErrorKind.INTERNAL_DRIVER_ERROR;
 }
-

backend/agent/adapters/appium/error-kinds.ts

@@ -1,7 +1,10 @@
 import type { ApplicationForegroundContext } from "../../../domain/entities";
-import type { AppLifecyclePort, AppLifecycleBudget } from "../../../ports/appium/app-lifecycle.port";
-import { AppNotInstalledError, AppCrashedError, TimeoutError } from "../errors";
-import { DriverErrorKind, mapAdapterErrorToDriverErrorKind } from "../error-kinds";
+import type {
+  AppLifecycleBudget,
+  AppLifecyclePort,
+} from "../../../ports/appium/app-lifecycle.port";
+import { type DriverErrorKind, mapAdapterErrorToDriverErrorKind } from "../error-kinds";
+import { AppCrashedError, AppNotInstalledError, TimeoutError } from "../errors";
 import type { SessionContext } from "./session-context";
 
 /**

backend/agent/adapters/appium/webdriverio/app-lifecycle.adapter.ts

@@ -1,5 +1,5 @@
-import type { PackageManagerPort, PackageInfo } from "../../../ports/appium/package-manager.port";
-import { AppNotInstalledError, TimeoutError, InvalidArgumentError } from "../errors";
+import type { PackageInfo, PackageManagerPort } from "../../../ports/appium/package-manager.port";
+import { AppNotInstalledError, InvalidArgumentError, TimeoutError } from "../errors";
 import type { SessionContext } from "./session-context";
 
 /**
@@ -73,7 +73,10 @@ export class WebDriverIOPackageManagerAdapter implements PackageManagerPort {
    *   TimeoutError: If installation timed out
    *   InvalidArgumentError: If APK is invalid or corrupted
    */
-  async installFromObjectStorage(ref: string, _expectedSha256?: string): Promise<{ packageId: string }> {
+  async installFromObjectStorage(
+    ref: string,
+    _expectedSha256?: string,
+  ): Promise<{ packageId: string }> {
     try {
       // WebDriverIO will upload and install the APK located at ref (path accessible to Appium server)
       await this.context.driver.installApp(ref);
@@ -113,13 +116,13 @@ export class WebDriverIOPackageManagerAdapter implements PackageManagerPort {
       const output = typeof result === "string" ? result : JSON.stringify(result);
       // Look for a line like: "SHA-256 digest: ABCDEF..."
       const shaMatch = output.match(/SHA-256 digest:\s*([A-Fa-f0-9:]+)/);
-      if (shaMatch && shaMatch[1]) {
+      if (shaMatch?.[1]) {
         // Normalize to lowercase without colons
         return shaMatch[1].replace(/:/g, "").toLowerCase();
       }
       // Fallback: attempt to parse legacy signature lines
       const legacyMatch = output.match(/signatures=\[([A-Fa-f0-9:]+)\]/);
-      if (legacyMatch && legacyMatch[1]) {
+      if (legacyMatch?.[1]) {
         return legacyMatch[1].replace(/:/g, "").toLowerCase();
       }
       throw new InvalidArgumentError("Unable to parse SHA-256 signature from dumpsys output");
@@ -147,4 +150,3 @@ export class WebDriverIOPackageManagerAdapter implements PackageManagerPort {
     }
   }
 }
-

backend/agent/adapters/appium/webdriverio/package-manager.adapter.ts

@@ -1,10 +1,10 @@
+import log from "encore.dev/log";
 import { remote } from "webdriverio";
+import { AGENT_ACTORS, MODULES } from "../../../../logging/logger";
 import type { DeviceRuntimeContext } from "../../../domain/entities";
-import type { SessionPort, DeviceConfiguration } from "../../../ports/appium/session.port";
+import type { DeviceConfiguration, SessionPort } from "../../../ports/appium/session.port";
 import { DeviceOfflineError, TimeoutError } from "../errors";
 import type { SessionContext } from "./session-context";
-import log from "encore.dev/log";
-import { MODULES, AGENT_ACTORS } from "../../../../logging/logger";
 
 interface RemoteOptions {
   hostname: string;
@@ -63,18 +63,18 @@ export class WebDriverIOSessionAdapter implements SessionPort {
 
   /**
    * Lazily initialize the Appium session if not already created.
-   * 
+   *
    * PURPOSE:
    * --------
    * Create the actual WebDriverIO session on first use (not during EnsureDevice).
    * Defers session creation until app context is available in ProvisionApp.
-   * 
+   *
    * Args:
    *   config: Device configuration (needed for session creation)
-   * 
+   *
    * Returns:
    *   SessionContext with active WebDriverIO driver
-   * 
+   *
    * Raises:
    *   DeviceOfflineError: If session creation fails
    *   TimeoutError: If session creation times out
@@ -86,9 +86,9 @@ export class WebDriverIOSessionAdapter implements SessionPort {
     });
 
     // If we already have a real driver, return existing context
-    if (this.context?.driver && this.context.driver.sessionId) {
-      logger.info("Session already initialized, reusing", { 
-        sessionId: this.context.driver.sessionId 
+    if (this.context?.driver?.sessionId) {
+      logger.info("Session already initialized, reusing", {
+        sessionId: this.context.driver.sessionId,
       });
       return this.context;
     }
@@ -154,14 +154,16 @@ export class WebDriverIOSessionAdapter implements SessionPort {
       logger.error("Failed to create Appium session", {
         error: error instanceof Error ? error.message : String(error),
       });
-      
+
       if (error instanceof Error) {
         const errorMsg = error.message.toLowerCase();
         if (errorMsg.includes("timeout")) {
           throw new TimeoutError(`Session creation timeout: ${error.message}`);
         }
       }
-      throw new DeviceOfflineError(`Failed to create session: ${error instanceof Error ? error.message : String(error)}`);
+      throw new DeviceOfflineError(
+        `Failed to create session: ${error instanceof Error ? error.message : String(error)}`,
+      );
     }
   }
 
@@ -204,7 +206,7 @@ export class WebDriverIOSessionAdapter implements SessionPort {
       if (this.context?.driver === null || (this.context && !this.context.driver.sessionId)) {
         logger.info("Lazy initialization: creating Appium session");
         await this.ensureSessionInitialized(config);
-        
+
         if (!this.context) {
           throw new DeviceOfflineError("Session initialization failed");
         }
@@ -222,7 +224,7 @@ export class WebDriverIOSessionAdapter implements SessionPort {
       logger.info("Deferring session creation until app context is available");
 
       const deviceRuntimeContextId = `pending-${Date.now()}`;
-      
+
       // Store minimal context for later
       const capabilities = {
         platformName: config.platformName,
@@ -242,16 +244,17 @@ export class WebDriverIOSessionAdapter implements SessionPort {
         healthProbeStatus: "HEALTHY" as const,
       };
     } catch (error) {
-      const errorDetails = error instanceof Error 
-        ? {
-            name: error.name,
-            message: error.message,
-            stack: error.stack,
-          }
-        : String(error);
-      
+      const errorDetails =
+        error instanceof Error
+          ? {
+              name: error.name,
+              message: error.message,
+              stack: error.stack,
+            }
+          : String(error);
+
       logger.error("ensureDevice failed", { error: errorDetails, config });
-      
+
       if (error instanceof Error) {
         const errorMsg = error.message.toLowerCase();
         if (errorMsg.includes("econnrefused") || errorMsg.includes("econnreset")) {
@@ -261,7 +264,9 @@ export class WebDriverIOSessionAdapter implements SessionPort {
           throw new TimeoutError(`Connection timeout: ${error.message}`);
         }
       }
-      throw new DeviceOfflineError(`Failed to ensure device: ${error instanceof Error ? error.message : String(error)}`);
+      throw new DeviceOfflineError(
+        `Failed to ensure device: ${error instanceof Error ? error.message : String(error)}`,
+      );
     }
   }
 

backend/agent/adapters/appium/webdriverio/session.adapter.ts

@@ -1,5 +1,5 @@
-import type { LLMPort } from "../../ports/llm";
 import type { ActionCandidate, ActionDecision } from "../../domain/actions";
+import type { LLMPort } from "../../ports/llm";
 
 /**
  * FakeLLM offers deterministic LLM behavior for tests and demos.

backend/agent/adapters/fakes/fake-llm.ts

@@ -1,4 +1,4 @@
-import type { PackageManagerPort, PackageInfo } from "../../ports/appium/package-manager.port";
+import type { PackageInfo, PackageManagerPort } from "../../ports/appium/package-manager.port";
 
 /**
  * Fake implementation of PackageManagerPort for testing.
@@ -20,7 +20,10 @@ export class FakePackageManagerAdapter implements PackageManagerPort {
     return this.installedPackages[packageId] || { installed: false };
   }
 
-  async installFromObjectStorage(ref: string, expectedSha256?: string): Promise<{ packageId: string }> {
+  async installFromObjectStorage(
+    ref: string,
+    expectedSha256?: string,
+  ): Promise<{ packageId: string }> {
     // Extract package ID from ref (mock implementation)
     const packageId = ref.split("/").pop()?.replace(".apk", "") || "com.example.app";
     this.installedPackages[packageId] = {
@@ -38,4 +41,3 @@ export class FakePackageManagerAdapter implements PackageManagerPort {
     return "fake-signature-hash";
   }
 }
-