Address code review feedback - improve type safety and documentation

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/runtime/src/interfaces/data-engine.ts

@@ -26,11 +26,11 @@ export interface QueryOptions {
     select?: string[];
     /** Sort order */
     sort?: Record<string, 1 | -1 | 'asc' | 'desc'>;
-    /** Limit number of results */
+    /** Limit number of results (alternative name for top, used by some drivers) */
     limit?: number;
     /** Skip number of results (for pagination) */
     skip?: number;
-    /** Maximum number of results (alternative to limit) */
+    /** Maximum number of results (OData-style, takes precedence over limit if both specified) */
     top?: number;
 }
 

packages/runtime/src/interfaces/http-server.ts

@@ -54,9 +54,9 @@ export interface IHttpResponse {
     /**
      * Set response header
      * @param name - Header name
-     * @param value - Header value
+     * @param value - Header value (string or array of strings for multi-value headers)
      */
-    header(name: string, value: string): IHttpResponse;
+    header(name: string, value: string | string[]): IHttpResponse;
 }
 
 /**

packages/runtime/src/test-interfaces.ts

@@ -5,7 +5,7 @@
  * and IDataEngine interfaces without depending on concrete implementations.
  */
 
-import { IHttpServer, IDataEngine, RouteHandler, IHttpRequest, IHttpResponse } from './index.js';
+import { IHttpServer, IDataEngine, RouteHandler, IHttpRequest, IHttpResponse, Middleware, QueryOptions } from './index.js';
 
 /**
  * Example: Mock HTTP Server Plugin
@@ -41,7 +41,7 @@ class MockHttpServer implements IHttpServer {
         console.log(`✅ Registered PATCH ${path}`);
     }
 
-    use(path: any, handler?: any): void {
+    use(path: string | Middleware, handler?: Middleware): void {
         console.log(`✅ Registered middleware`);
     }
 
@@ -77,7 +77,7 @@ class MockDataEngine implements IDataEngine {
         return record;
     }
 
-    async find(objectName: string, query?: any): Promise<any[]> {
+    async find(objectName: string, query?: QueryOptions): Promise<any[]> {
         const objectStore = this.store.get(objectName);
         if (!objectStore) {
             return [];
@@ -149,12 +149,15 @@ async function testInterfaces() {
     });
 
     const users = await dataEngine.find('user');
+    console.log(`Found ${users.length} users after inserts`);
 
     const updatedUser = await dataEngine.update('user', user1.id, {
         name: 'John Updated'
     });
+    console.log(`Updated user:`, updatedUser);
 
     const deleted = await dataEngine.delete('user', user2.id);
+    console.log(`Delete result: ${deleted}`);
 
     console.log('\n✅ All interface tests passed!\n');