Merge pull request #156 from objectstack-ai/copilot/update-ci-pipeline-configuration

Author: xuyushun441-sys

packages/plugins/server/package.json

@@ -37,9 +37,11 @@
     "@nestjs/cli": "^10.0.0",
     "@nestjs/schematics": "^10.0.0",
     "@nestjs/testing": "^10.0.0",
+    "@types/better-sqlite3": "^7.6.11",
     "@types/express": "^5.0.6",
     "@types/jest": "^30.0.0",
     "@types/node": "^20.10.0",
+    "@types/pg": "^8.11.10",
     "jest": "^30.2.0",
     "source-map-support": "^0.5.21",
     "ts-jest": "^29.4.6",

packages/plugins/server/src/auth/auth.controller.ts

@@ -1,17 +1,19 @@
 import { Controller, All, Req, Res } from '@nestjs/common';
+import { Request, Response } from 'express';
 import { getAuth } from './auth.client.js';
 
 @Controller('api/auth')
 export class AuthController {
     @All('*')
-    async handleAuth(@Req() req, @Res() res) {
+    async handleAuth(@Req() req: Request, @Res() res: Response) {
         try {
             const auth = await getAuth();
             const { toNodeHandler } = await import('better-auth/node');
             return toNodeHandler(auth)(req, res);
         } catch (error) {
             console.error('Auth Error:', error);
-            res.status(500).json({ error: 'Internal Server Error', details: error.message });
+            const errorMessage = error instanceof Error ? error.message : 'Unknown error';
+            res.status(500).json({ error: 'Internal Server Error', details: errorMessage });
         }
     }
 }

packages/plugins/server/src/auth/auth.middleware.ts

@@ -1,6 +1,7 @@
 import { Injectable, NestMiddleware } from '@nestjs/common';
 import { Request, Response, NextFunction } from 'express';
 import { getAuth } from './auth.client.js';
+import './express.d.ts';
 
 @Injectable()
 export class AuthMiddleware implements NestMiddleware {
@@ -26,7 +27,7 @@ export class AuthMiddleware implements NestMiddleware {
           // Ensure we have an array of roles. better-auth usually provides a single role string on user object.
           const roles = [role];
 
-          req['user'] = {
+          req.user = {
               userId: session.user.id,
               id: session.user.id,
               ...session.user,
@@ -40,7 +41,7 @@ export class AuthMiddleware implements NestMiddleware {
           // Fallback for dev/test: trust x-user-id header
           const userId = req.headers['x-user-id'] as string;
           const isAdmin = userId === 'admin';
-          req['user'] = {
+          req.user = {
               userId: userId,
               id: userId,
               roles: isAdmin ? ['admin'] : ['user'],
@@ -49,15 +50,15 @@ export class AuthMiddleware implements NestMiddleware {
           };
       } else {
           // Anonymous user
-          req['user'] = {
+          req.user = {
               roles: ['guest'],
               isSystem: false
           };
       }
     } catch (e) {
       // ignore auth error
       console.error("Auth Middleware Error:", e);
-      req['user'] = {
+      req.user = {
         roles: ['guest'],
         isSystem: false
       }

packages/plugins/server/src/auth/express.d.ts

@@ -0,0 +1,14 @@
+// Type augmentation for Express Request
+declare namespace Express {
+  export interface Request {
+    user?: {
+      userId?: string;
+      id?: string;
+      roles?: string[];
+      spaceId?: string;
+      sessionId?: string;
+      isSystem?: boolean;
+      [key: string]: any;
+    };
+  }
+}

packages/runtime/src/kernel.ts

@@ -32,6 +32,8 @@ export class ObjectKernel {
   constructor() {
     this.logger = createLogger('ObjectKernel');
     this.context = new PluginContextImpl(this.logger);
+    // Set the kernel reference in the context
+    this.context.setKernel(this);
   }
 
   /**

packages/runtime/src/plugin-context.ts

@@ -8,6 +8,7 @@ export class PluginContextImpl implements PluginContext {
   private services: Map<string, any> = new Map();
   private hooks: Map<string, Function[]> = new Map();
   public logger: Logger;
+  private kernel?: any;
 
   constructor(
     logger: Logger,
@@ -91,4 +92,18 @@ export class PluginContextImpl implements PluginContext {
   getHooks(): Map<string, Function[]> {
     return this.hooks;
   }
+
+  /**
+   * Get the kernel instance (for advanced use cases)
+   */
+  getKernel(): any {
+    return this.kernel;
+  }
+
+  /**
+   * Set the kernel instance (for internal use)
+   */
+  setKernel(kernel: any): void {
+    this.kernel = kernel;
+  }
 }

packages/runtime/src/types.ts

@@ -25,6 +25,11 @@ export interface PluginContext {
   getService<T = any>(name: string): T;
   hasService(name: string): boolean;
 
+  /**
+   * Get all registered services
+   */
+  getServices(): Map<string, any>;
+
   /**
    * Event System (Hook System)
    */
@@ -35,6 +40,13 @@ export interface PluginContext {
    * Logger
    */
   logger: Logger;
+
+  /**
+   * Get the kernel instance (for advanced use cases)
+   * @returns Kernel instance
+   * @note Returns 'any' to avoid circular dependency with ObjectKernel
+   */
+  getKernel(): any;
 }
 
 /**