fix: address PR #919 review comments

Export `register` as a named export from the openclaw-plugin to avoid
`register.register()` pattern, and improve the dynamic import warning in
`removeDynamicImportsPlugin` to also fire when mixed literal/computed
imports exist.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: sirtimid

packages/evm-wallet-experiment/openclaw-plugin/index.ts

@@ -140,4 +140,5 @@ const entry: PluginEntry = {
   register,
 };
 
+export { register };
 export default entry;

packages/evm-wallet-experiment/test/integration/openclaw-plugin.integration.test.ts

@@ -6,7 +6,7 @@ import { dirname, resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 
-import register from '../../openclaw-plugin/index.ts';
+import { register } from '../../openclaw-plugin/index.ts';
 import { makeWalletClusterConfig } from '../../src/cluster-config.ts';
 
 type ToolResponse = {
@@ -262,7 +262,7 @@ exec "${process.execPath}" "${ocapCliEntrypoint}" "$@"
       ]);
 
       tools = new Map<string, ToolDefinition>();
-      register.register({
+      register({
         pluginConfig: {
           ocapCliPath,
           walletKref: rootKref,

packages/evm-wallet-experiment/test/openclaw-plugin.test.ts

@@ -2,7 +2,7 @@ import { spawn } from 'node:child_process';
 import { EventEmitter } from 'node:events';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 
-import pluginEntry from '../openclaw-plugin/index.ts';
+import { register } from '../openclaw-plugin/index.ts';
 
 vi.mock('node:child_process', () => ({ spawn: vi.fn() }));
 
@@ -107,7 +107,7 @@ function setupPlugin(): Map<string, ToolDefinition> {
     },
   };
 
-  pluginEntry.register(api);
+  register(api);
   return tools;
 }
 

packages/kernel-utils/src/vite-plugins/bundle-vat.ts

@@ -42,11 +42,14 @@ export function removeDynamicImportsPlugin(): RolldownPlugin {
         'Promise.resolve({})',
       );
 
-      if (transformed === code) {
+      if (/\bimport\s*\(/u.test(transformed)) {
         this.warn(
           `Module "${id}" contains dynamic import() expressions that could not ` +
             `be replaced (e.g. computed specifiers). Rolldown may reject IIFE output.`,
         );
+      }
+
+      if (transformed === code) {
         return null;
       }