fix: resolve @hotcrm/core module for Vercel build

- Re-include @hotcrm/core in pnpm workspace (no circular deps)
- Add @hotcrm/core as root workspace dependency for hoisting
- Build core first in Vercel build command
- Enable declaration output for core tsconfig
- Add flexible Logger interface to avoid strict pino overload errors
- Fix pino logger calls in AI package to use structured logging

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

Author: Copilot

package.json

@@ -51,6 +51,7 @@
   "author": "HotCRM Team",
   "license": "MIT",
   "dependencies": {
+    "@hotcrm/core": "workspace:*",
     "pino": "^10.3.1",
     "pino-pretty": "^13.1.3"
   },

packages/ai/src/cache-manager.ts

@@ -92,7 +92,7 @@ export class CacheManager {
 
       logger.info('[Cache] Redis initialization placeholder - using in-memory cache');
     } catch (error) {
-      logger.error('[Cache] Redis initialization failed:', error);
+      logger.error({ err: error }, '[Cache] Redis initialization failed');
 
       if (this.config.useMemoryFallback) {
         logger.info('[Cache] Falling back to in-memory cache');
@@ -136,7 +136,7 @@ export class CacheManager {
 
       return entry.data as T;
     } catch (error) {
-      logger.error('[Cache] Get error:', error);
+      logger.error({ err: error }, '[Cache] Get error');
       return null;
     }
   }
@@ -169,7 +169,7 @@ export class CacheManager {
       // Clean up expired entries periodically
       this.cleanupExpired();
     } catch (error) {
-      logger.error('[Cache] Set error:', error);
+      logger.error({ err: error }, '[Cache] Set error');
     }
   }
 
@@ -185,7 +185,7 @@ export class CacheManager {
 
       this.memoryCache.delete(key);
     } catch (error) {
-      logger.error('[Cache] Delete error:', error);
+      logger.error({ err: error }, '[Cache] Delete error');
     }
   }
 
@@ -201,7 +201,7 @@ export class CacheManager {
 
       this.memoryCache.clear();
     } catch (error) {
-      logger.error('[Cache] Clear error:', error);
+      logger.error({ err: error }, '[Cache] Clear error');
     }
   }
 

packages/ai/src/prediction-service.ts

@@ -194,7 +194,7 @@ export class PredictionService {
           metadata: result.metadata
         }));
       } catch (error) {
-        logger.warn('[PredictionService] Batch prediction failed, falling back to sequential:', error);
+        logger.warn({ err: error }, '[PredictionService] Batch prediction failed, falling back to sequential');
       }
     }
 
@@ -241,7 +241,7 @@ export class PredictionService {
         const result = await provider.predict<T>(model.id, { features, context });
         return result;
       } catch (error) {
-        logger.warn(`[PredictionService] Provider prediction failed for ${model.id}, falling back to mock:`, error);
+        logger.warn({ err: error }, `[PredictionService] Provider prediction failed for ${model.id}, falling back to mock`);
       }
     }
 

packages/ai/src/providers/aws-sagemaker-provider.ts

@@ -69,7 +69,7 @@ export class AWSSageMakerProvider extends BaseMLProvider {
 
       return true;
     } catch (error) {
-      logger.error('[SageMaker] Validation failed:', error);
+      logger.error({ err: error }, '[SageMaker] Validation failed');
       return false;
     }
   }
@@ -115,7 +115,7 @@ export class AWSSageMakerProvider extends BaseMLProvider {
         }
       };
     } catch (error) {
-      logger.error('[SageMaker] Prediction error:', error);
+      logger.error({ err: error }, '[SageMaker] Prediction error');
       throw new Error(`SageMaker prediction failed: ${error instanceof Error ? error.message : 'Unknown error'}`);
     }
   }

packages/ai/src/providers/azure-ml-provider.ts

@@ -86,7 +86,7 @@ export class AzureMLProvider extends BaseMLProvider {
 
       return true;
     } catch (error) {
-      logger.error('[Azure ML] Validation failed:', error);
+      logger.error({ err: error }, '[Azure ML] Validation failed');
       return false;
     }
   }
@@ -131,7 +131,7 @@ export class AzureMLProvider extends BaseMLProvider {
         }
       };
     } catch (error) {
-      logger.error('[Azure ML] Prediction error:', error);
+      logger.error({ err: error }, '[Azure ML] Prediction error');
 
       const err = error as any;
       if (err.isAxiosError) {
@@ -178,7 +178,7 @@ export class AzureMLProvider extends BaseMLProvider {
         }
       }));
     } catch (error) {
-      logger.error('[Azure ML] Batch prediction error:', error);
+      logger.error({ err: error }, '[Azure ML] Batch prediction error');
       throw new Error(`Azure ML batch prediction failed: ${error instanceof Error ? error.message : 'Unknown error'}`);
     }
   }

packages/ai/src/providers/openai-provider.ts

@@ -89,7 +89,7 @@ export class OpenAIProvider extends BaseMLProvider {
 
       return true;
     } catch (error) {
-      logger.error('[OpenAI] Validation failed:', error);
+      logger.error({ err: error }, '[OpenAI] Validation failed');
       return false;
     }
   }
@@ -131,7 +131,7 @@ export class OpenAIProvider extends BaseMLProvider {
         }
       };
     } catch (error) {
-      logger.error('[OpenAI] Prediction error:', error);
+      logger.error({ err: error }, '[OpenAI] Prediction error');
 
       const err = error as any;
       if (err.isAxiosError) {

packages/core/src/logger.ts

@@ -20,16 +20,33 @@ export interface LoggerOptions {
   level?: LogLevel;
 }
 
+/** Flexible logger interface compatible with pino but lenient on call signatures */
+export interface Logger {
+  fatal(msg: string, ...args: any[]): void;
+  fatal(obj: unknown, msg?: string, ...args: any[]): void;
+  error(msg: string, ...args: any[]): void;
+  error(obj: unknown, msg?: string, ...args: any[]): void;
+  warn(msg: string, ...args: any[]): void;
+  warn(obj: unknown, msg?: string, ...args: any[]): void;
+  info(msg: string, ...args: any[]): void;
+  info(obj: unknown, msg?: string, ...args: any[]): void;
+  debug(msg: string, ...args: any[]): void;
+  debug(obj: unknown, msg?: string, ...args: any[]): void;
+  trace(msg: string, ...args: any[]): void;
+  trace(obj: unknown, msg?: string, ...args: any[]): void;
+  child(bindings: Record<string, any>): Logger;
+}
+
 /**
  * Create a child logger scoped to a specific module.
  *
  * @param module  Hierarchical module name, e.g. 'crm:account' or 'ai:cache'
  * @param options Optional overrides
  */
-export function createLogger(module: string, options?: LoggerOptions): pino.Logger {
+export function createLogger(module: string, options?: LoggerOptions): Logger {
   const level = options?.level ?? (process.env.LOG_LEVEL as LogLevel) ?? 'info';
-  return pino({ name: module, level });
+  return pino({ name: module, level }) as unknown as Logger;
 }
 
 /** Shared root logger instance */
-export const logger: pino.Logger = createLogger('hotcrm');
+export const logger: Logger = createLogger('hotcrm');

packages/core/tsconfig.json

@@ -2,7 +2,8 @@
   "extends": "../../base.tsconfig.json",
   "compilerOptions": {
     "outDir": "./dist",
-    "rootDir": "./src"
+    "rootDir": "./src",
+    "declaration": true
   },
   "include": ["src/**/*"],
   "exclude": ["node_modules", "dist"]

pnpm-lock.yaml

@@ -1,8 +1,7 @@
 packages:
   - 'apps/*'
   - 'packages/*'
-  # core and server are internal-only packages (private: true) excluded from
+  # server is an internal-only package (private: true) excluded from
   # workspace linking to avoid circular dependency issues during builds.
-  # They are consumed via tsconfig path aliases instead.
-  - '!packages/core'
+  # It is consumed via tsconfig path aliases instead.
   - '!packages/server'