fix: resolve CI build errors - SDK execute method, StateMachineConfig imports, and TypeScript strict type issues

Copilot · hotlong · Copilot · commit eb7a39917f18 · 2026-02-10T00:35:16.000Z
Co-authored-by: hotlong &lt;50353452+hotlong@users.noreply.github.com&gt;
diff --git a/packages/drivers/sdk/src/index.ts b/packages/drivers/sdk/src/index.ts
@@ -545,19 +545,19 @@ export class RemoteDriver implements Driver {
      * @example
      * ```typescript
      * // Execute a custom workflow
-     * const result = await driver.execute('/api/workflows/approve', {
+     * const result = await driver.executeCustomEndpoint('/api/workflows/approve', {
      *   workflowId: 'wf_123',
      *   comment: 'Approved'
      * });
      * 
      * // Use default execute endpoint
-     * const result = await driver.execute(undefined, {
+     * const result = await driver.executeCustomEndpoint(undefined, {
      *   action: 'calculateMetrics',
      *   params: { year: 2024 }
      * });
      * ```
      */
-    async execute(endpoint?: string, payload?: any, _options?: any): Promise<any> {
+    async executeCustomEndpoint(endpoint?: string, payload?: any, _options?: any): Promise<any> {
         return this.retryWithBackoff(async () => {
             const targetEndpoint = endpoint 
                 ? this.buildEndpoint(endpoint)
diff --git a/packages/drivers/sdk/test/remote-driver.test.ts b/packages/drivers/sdk/test/remote-driver.test.ts
@@ -688,7 +688,7 @@ describe('RemoteDriver', () => {
                 json: async () => mockResponse
             });
 
-            const result = await driver.execute('/api/custom', payload);
+            const result = await driver.executeCustomEndpoint('/api/custom', payload);
 
             expect(global.fetch).toHaveBeenCalledWith(
                 'http://localhost:3000/api/custom',
@@ -708,7 +708,7 @@ describe('RemoteDriver', () => {
                 json: async () => ({ success: true })
             });
 
-            await driver.execute(undefined, payload);
+            await driver.executeCustomEndpoint(undefined, payload);
 
             expect(global.fetch).toHaveBeenCalledWith(
                 'http://localhost:3000/api/execute',
@@ -729,7 +729,7 @@ describe('RemoteDriver', () => {
                 })
             });
 
-            await expect(driver.execute('/api/test', {}))
+            await expect(driver.executeCustomEndpoint('/api/test', {}))
                 .rejects
                 .toThrow('Server error');
         });
diff --git a/packages/foundation/core/src/query/query-analyzer.ts b/packages/foundation/core/src/query/query-analyzer.ts
@@ -271,7 +271,7 @@ export class QueryAnalyzer {
      * @private
      */
     private getSchema(objectName: string): ObjectConfig {
-        const obj = this.metadata.get('object', objectName);
+        const obj = this.metadata.get<ObjectConfig>('object', objectName);
         if (!obj) {
             throw new ObjectQLError({ code: 'NOT_FOUND', message: `Object '${objectName}' not found` });
         }
diff --git a/packages/foundation/core/src/query/query-service.ts b/packages/foundation/core/src/query/query-service.ts
@@ -132,7 +132,7 @@ export class QueryService {
      * @private
      */
     private getSchema(objectName: string): ObjectConfig {
-        const obj = this.metadata.get('object', objectName);
+        const obj = this.metadata.get<ObjectConfig>('object', objectName);
         if (!obj) {
             throw new ObjectQLError({ code: 'NOT_FOUND', message: `Object '${objectName}' not found in metadata` });
         }
diff --git a/packages/foundation/core/src/repository.ts b/packages/foundation/core/src/repository.ts
@@ -28,7 +28,7 @@ export class ObjectRepository {
     }
     
     private getKernel(): ObjectKernel {
-        return this.app.getKernel();
+        return this.app.getKernel() as ObjectKernel;
     }
 
     private getOptions(extra: Record<string, unknown> = {}) {
diff --git a/packages/foundation/platform-node/src/loader.ts b/packages/foundation/platform-node/src/loader.ts
@@ -44,7 +44,7 @@ export class ObjectLoader {
                             // Object name does not match filename — using doc.name
                         }
 
-                        const packageEntry = ctx.registry.getEntry('package-map', ctx.file);
+                        const packageEntry = ctx.registry.getEntry<{ package?: string }>('package-map', ctx.file);
                         registerObject(ctx.registry, doc, ctx.file, ctx.packageName || (packageEntry && packageEntry.package));
                         return;
                     }
@@ -295,7 +295,7 @@ function registerObject(registry: MetadataRegistry, obj: any, file: string, pack
     // --- End Smart Defaults ---
 
     // Check for existing object to Merge
-    const existing = registry.getEntry('object', obj.name);
+    const existing = registry.getEntry<{ content?: ObjectConfig }>('object', obj.name);
     if (existing) {
         const base = existing.content as ObjectConfig;
         
diff --git a/packages/foundation/plugin-security/src/storage-database.ts b/packages/foundation/plugin-security/src/storage-database.ts
@@ -120,7 +120,7 @@ export class DatabasePermissionStorage implements IPermissionStorage {
       });
       const row = Array.isArray(rows) ? rows[0] : undefined;
       if (!row) return undefined;
-      return JSON.parse(row.config) as PermissionConfig;
+      return JSON.parse((row as Record<string, unknown>).config as string) as PermissionConfig;
     } catch {
       return undefined;
     }
@@ -134,8 +134,9 @@ export class DatabasePermissionStorage implements IPermissionStorage {
       const items = Array.isArray(rows) ? rows : [];
       for (const row of items) {
         try {
-          const config = JSON.parse(row.config) as PermissionConfig;
-          result.set(row.object_name, config);
+          const rowData = row as Record<string, unknown>;
+          const config = JSON.parse(rowData.config as string) as PermissionConfig;
+          result.set(rowData.object_name as string, config);
         } catch {
           // Skip rows with corrupt JSON
         }
@@ -154,8 +155,9 @@ export class DatabasePermissionStorage implements IPermissionStorage {
       // Create a shallow copy to avoid issues when deleting during iteration
       const items = Array.isArray(rows) ? [...rows] : [];
       for (const row of items) {
-        if (row._id || row.id || row.object_name) {
-          await driver.delete(this.tableName, row._id ?? row.id ?? row.object_name, {});
+        const rowData = row as Record<string, unknown>;
+        if (rowData._id || rowData.id || rowData.object_name) {
+          await driver.delete(this.tableName, (rowData._id ?? rowData.id ?? rowData.object_name) as string | number, {});
         }
       }
     } catch {
@@ -187,8 +189,9 @@ export class DatabasePermissionStorage implements IPermissionStorage {
       });
       const row = Array.isArray(rows) ? rows[0] : undefined;
       if (row) {
-        const id = row._id ?? row.id ?? row.object_name;
-        await driver.update(this.tableName, id, {
+        const rowData = row as Record<string, unknown>;
+        const id = rowData._id ?? rowData.id ?? rowData.object_name;
+        await driver.update(this.tableName, id as string | number, {
           config: JSON.stringify(config),
           updated_at: new Date().toISOString(),
         }, {});
@@ -214,8 +217,9 @@ export class DatabasePermissionStorage implements IPermissionStorage {
       });
       const row = Array.isArray(rows) ? rows[0] : undefined;
       if (row) {
-        const id = row._id ?? row.id ?? row.object_name;
-        await driver.delete(this.tableName, id, {});
+        const rowData = row as Record<string, unknown>;
+        const id = rowData._id ?? rowData.id ?? rowData.object_name;
+        await driver.delete(this.tableName, id as string | number, {});
       }
     } catch {
       // Row may not exist
diff --git a/packages/foundation/plugin-validator/src/validator.ts b/packages/foundation/plugin-validator/src/validator.ts
@@ -364,7 +364,7 @@ export class Validator {
         }
 
         // Check if transition is allowed
-        const transitions = rule.transitions?.[oldState];
+        const transitions = rule.transitions?.[oldState as string];
         if (!transitions) {
             return {
                 rule: rule.name,
@@ -384,7 +384,7 @@ export class Validator {
             allowedNext = transitions.allowed_next || [];
         }
 
-        const isAllowed = allowedNext.includes(newState);
+        const isAllowed = allowedNext.includes(newState as string);
 
         return {
             rule: rule.name,
@@ -480,7 +480,7 @@ export class Validator {
 
         try {
             // Query database to count existing records with same field values
-            const count = await context.api.count(objectName, filters);
+            const count = await (context.api as { count: (obj: string, filters: Record<string, unknown>) => Promise<number> }).count(objectName, filters);
 
             const valid = count === 0;
 
diff --git a/packages/foundation/plugin-workflow/__tests__/workflow-plugin.spec.ts b/packages/foundation/plugin-workflow/__tests__/workflow-plugin.spec.ts
@@ -8,7 +8,7 @@
 
 import { describe, it, expect, beforeEach } from 'vitest';
 import { WorkflowPlugin } from '../src/workflow-plugin';
-import type { _StateMachineConfig } from '@objectql/types';
+import type { StateMachineConfig } from '@objectql/types';
 
 describe('WorkflowPlugin', () => {
   let plugin: WorkflowPlugin;
diff --git a/packages/foundation/plugin-workflow/src/types.ts b/packages/foundation/plugin-workflow/src/types.ts
@@ -11,7 +11,7 @@
  * These complement the protocol types from @objectql/types
  */
 
-import type { _StateMachineConfig } from '@objectql/types';
+import type { StateMachineConfig } from '@objectql/types';
 
 /**
  * Execution context for state machine operations
diff --git a/packages/tools/cli/src/commands/repl.ts b/packages/tools/cli/src/commands/repl.ts
@@ -73,7 +73,7 @@ export async function startRepl(configPath?: string) {
         r.context.object = (name: string) => app.getObject(name);
         
         // Helper to get a repo quickly: tasks.find() instead of app.object('tasks').find()
-        const objects = app.metadata.list('object');
+        const objects = app.metadata.list<{ name: string }>('object');
         for (const obj of objects) {
             // Inject repositories as top-level globals if valid identifiers
             if (/^[a-zA-Z_$][a-zA-Z0-9_$]*$/.test(obj.name)) {

Original file line number	Diff line number	Diff line change
`@@ -271,7 +271,7 @@ export class QueryAnalyzer {`
`271`	`271`	`* @private`
`272`	`272`	`*/`
`273`	`273`	`private getSchema(objectName: string): ObjectConfig {`
`274`		`- const obj = this.metadata.get('object', objectName);`
	`274`	`+ const obj = this.metadata.get<ObjectConfig>('object', objectName);`
`275`	`275`	`if (!obj) {`
`276`	`276`	throw new ObjectQLError({ code: 'NOT_FOUND', message: `Object '${objectName}' not found` });
`277`	`277`	`}`
Original file line number	Diff line number	Diff line change
`@@ -132,7 +132,7 @@ export class QueryService {`
`132`	`132`	`* @private`
`133`	`133`	`*/`
`134`	`134`	`private getSchema(objectName: string): ObjectConfig {`
`135`		`- const obj = this.metadata.get('object', objectName);`
	`135`	`+ const obj = this.metadata.get<ObjectConfig>('object', objectName);`
`136`	`136`	`if (!obj) {`
`137`	`137`	throw new ObjectQLError({ code: 'NOT_FOUND', message: `Object '${objectName}' not found in metadata` });
`138`	`138`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ export class ObjectRepository {`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`private getKernel(): ObjectKernel {`
`31`		`- return this.app.getKernel();`
	`31`	`+ return this.app.getKernel() as ObjectKernel;`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`private getOptions(extra: Record<string, unknown> = {}) {`
Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ export class ObjectLoader {`
`44`	`44`	`// Object name does not match filename — using doc.name`
`45`	`45`	`}`
`46`	`46`
`47`		`- const packageEntry = ctx.registry.getEntry('package-map', ctx.file);`
	`47`	`+ const packageEntry = ctx.registry.getEntry<{ package?: string }>('package-map', ctx.file);`
`48`	`48`	`registerObject(ctx.registry, doc, ctx.file, ctx.packageName \|\| (packageEntry && packageEntry.package));`
`49`	`49`	`return;`
`50`	`50`	`}`
`@@ -295,7 +295,7 @@ function registerObject(registry: MetadataRegistry, obj: any, file: string, pack`
`295`	`295`	`// --- End Smart Defaults ---`
`296`	`296`
`297`	`297`	`// Check for existing object to Merge`
`298`		`- const existing = registry.getEntry('object', obj.name);`
	`298`	`+ const existing = registry.getEntry<{ content?: ObjectConfig }>('object', obj.name);`
`299`	`299`	`if (existing) {`
`300`	`300`	`const base = existing.content as ObjectConfig;`
`301`	`301`