@elizaos/cli 1.4.2 → 1.4.4

package/templates/project-tee-starter/src/__tests__/config.test.ts

@@ -18,6 +18,46 @@ describe('Plugin Configuration', () => {
     );
   });
 
+  it('should parse and validate config during initialization', async () => {
+    // Mock runtime for testing
+    const mockRuntime = {
+      logger: {
+        info: () => {},
+        warn: () => {},
+        debug: () => {},
+        error: () => {},
+      },
+    } as any;
+
+    // Test that config parsing happens during init, not at import time
+    const originalEnv = {
+      NODE_ENV: process.env.NODE_ENV,
+      TEE_MODE: process.env.TEE_MODE,
+      WALLET_SECRET_SALT: process.env.WALLET_SECRET_SALT,
+    };
+
+    try {
+      // Set test environment with valid defaults
+      process.env.NODE_ENV = 'test';
+      process.env.TEE_MODE = 'OFF';
+      process.env.WALLET_SECRET_SALT = 'test_salt_12345';
+
+      // Config should be parsed and validated during init without throwing
+      await expect(teeStarterPlugin.init({}, mockRuntime)).resolves.toBeUndefined();
+
+      // Test with invalid config should fail validation during init
+      const invalidConfig = { TEE_MODE: 'INVALID_MODE' };
+      await expect(teeStarterPlugin.init(invalidConfig, mockRuntime)).rejects.toThrow(
+        'Invalid plugin configuration'
+      );
+    } finally {
+      // Restore original environment
+      process.env.NODE_ENV = originalEnv.NODE_ENV;
+      process.env.TEE_MODE = originalEnv.TEE_MODE;
+      process.env.WALLET_SECRET_SALT = originalEnv.WALLET_SECRET_SALT;
+    }
+  });
+
   it('should be a TEE-focused plugin with appropriate components', () => {
     // Verify plugin has TEE-specific components
     expect(teeStarterPlugin.actions).toEqual([]);

package/templates/project-tee-starter/src/index.ts

@@ -1,7 +1,6 @@
 import { logger, type IAgentRuntime, type Project, type ProjectAgent } from '@elizaos/core';
 import teeStarterPlugin, { StarterService } from './plugin.ts';
 import { mrTeeCharacter as character } from './character.ts';
-import ProjectTeeStarterTestSuite from './__tests__/e2e/project-tee-starter.e2e';
 
 const initCharacter = ({ runtime }: { runtime: IAgentRuntime }) => {
   logger.info(`Initializing character: ${character.name}`);
@@ -11,7 +10,7 @@ const initCharacter = ({ runtime }: { runtime: IAgentRuntime }) => {
 export const projectAgent: ProjectAgent = {
   character,
   init: async (runtime: IAgentRuntime) => await initCharacter({ runtime }),
-  tests: [ProjectTeeStarterTestSuite], // Export tests from ProjectAgent
+  plugins: [teeStarterPlugin], // Add any additional plugins here
 };
 
 const project: Project = {

package/templates/project-tee-starter/src/plugin.ts

@@ -213,34 +213,36 @@ export class StarterService extends Service {
 const teeStarterPlugin: Plugin = {
   name: 'mr-tee-starter-plugin',
   description: "Mr. TEE's starter plugin - using plugin-tee for attestation",
-  config: {
-    TEE_MODE: process.env.TEE_MODE,
-    TEE_VENDOR: process.env.TEE_VENDOR,
-    WALLET_SECRET_SALT: process.env.WALLET_SECRET_SALT,
-  },
+  // Use dynamic getters so tests/CI always see current env values
+  config: Object.defineProperties(
+    {},
+    {
+      TEE_MODE: {
+        get: () => process.env.TEE_MODE,
+        enumerable: true,
+      },
+      TEE_VENDOR: {
+        get: () => process.env.TEE_VENDOR,
+        enumerable: true,
+      },
+      WALLET_SECRET_SALT: {
+        get: () => process.env.WALLET_SECRET_SALT,
+        enumerable: true,
+      },
+    }
+  ) as Record<string, string>,
   async init(config: Record<string, string>, runtime: IAgentRuntime) {
     logger.info('*** Initializing Mr. TEE plugin ***');
     try {
       // Merge process.env values with config, config takes precedence
-      const mergedConfig = {
+      const rawConfig = {
         TEE_MODE: config.TEE_MODE ?? process.env.TEE_MODE,
         TEE_VENDOR: config.TEE_VENDOR ?? process.env.TEE_VENDOR,
         WALLET_SECRET_SALT: config.WALLET_SECRET_SALT ?? process.env.WALLET_SECRET_SALT,
       };
 
-      // Apply test defaults if in test environment
-      const isTestEnvironment = process.env.NODE_ENV === 'test' || process.argv.includes('test');
-
-      if (isTestEnvironment) {
-        // Apply test-only defaults - NEVER use these in production
-        mergedConfig.TEE_MODE = mergedConfig.TEE_MODE || 'OFF';
-        mergedConfig.TEE_VENDOR = mergedConfig.TEE_VENDOR || 'phala';
-        // Test salt - this is ONLY for test environments and should NEVER be used in production
-        mergedConfig.WALLET_SECRET_SALT =
-          mergedConfig.WALLET_SECRET_SALT || 'test_default_salt_12345';
-      }
-
-      const validatedConfig = await configSchema.parseAsync(mergedConfig);
+      // Parse and validate configuration with schema (includes test defaults)
+      const validatedConfig = configSchema.parse(rawConfig);
 
       // Production safety check - ensure test defaults aren't used in production
       if (