@zhin.js/core 1.0.15 → 1.0.17

package/src/plugin.ts

@@ -9,7 +9,7 @@ import { PermissionItem,PermissionChecker } from './permissions.js';
 import {Message} from './message.js'
 import { Schema } from '@zhin.js/hmr';
 import {Dependency, Logger,} from "@zhin.js/hmr";
-import {App} from "./app";
+import {App} from "./app.js";
 import {MessageCommand} from "./command.js";
 import {Component, renderComponents} from "./component.js";
 import { PluginError, MessageError, errorManager } from './errors.js';
@@ -194,7 +194,25 @@ export class Plugin extends Dependency<Plugin> {
             for (const middleware of this.middlewares) {
                 this.dispatch('middleware.remove', middleware)
             }
-            this.middlewares = []
+            this.middlewares.length = 0;  // 清空数组但保持引用
+            
+            // 清理 commands
+            this.commands.length = 0;
+            
+            // 清理权限
+            this.permissions.length = 0;
+            
+            // 清理组件 Map
+            this.components.clear();
+            
+            // 清理定义 Map
+            this.definitions.clear();
+            
+            // 清理 crons（在 dispose 事件中已处理，这里确保清空）
+            this.crons.length = 0;
+            
+            // 清空 logger 引用
+            this.#logger = undefined;
             
             // 调用父类的dispose方法
             super.dispose()

package/tests/command.test.ts

@@ -611,6 +611,83 @@ describe('Command系统测试', () => {
     })
   })
 
+  describe('帮助系统测试', () => {
+    it('应该正确设置和获取描述信息', () => {
+      const command = new MessageCommand('help')
+        .desc('这是命令描述', '可以有多行描述')
+
+      expect(command.helpInfo.desc).toEqual(['这是命令描述', '可以有多行描述'])
+    })
+
+    it('应该正确设置和获取用法信息', () => {
+      const command = new MessageCommand('help')
+        .usage('help', 'help <command>')
+
+      expect(command.helpInfo.usage).toEqual(['help', 'help <command>'])
+    })
+
+    it('应该正确设置和获取示例信息', () => {
+      const command = new MessageCommand('help')
+        .examples('help', 'help echo', 'help admin')
+
+      expect(command.helpInfo.examples).toEqual(['help', 'help echo', 'help admin'])
+    })
+
+    it('应该支持链式调用设置帮助信息', () => {
+      const command = new MessageCommand('test')
+        .desc('测试命令', '用于测试功能')
+        .usage('test', 'test <arg>')
+        .examples('test', 'test hello')
+        .action(() => 'Test response')
+
+      expect(command.helpInfo.pattern).toBe('test')
+      expect(command.helpInfo.desc).toEqual(['测试命令', '用于测试功能'])
+      expect(command.helpInfo.usage).toEqual(['test', 'test <arg>'])
+      expect(command.helpInfo.examples).toEqual(['test', 'test hello'])
+    })
+
+    it('应该正确生成帮助文本', () => {
+      const command = new MessageCommand('greet')
+        .desc('打招呼命令')
+        .usage('greet <name>')
+        .examples('greet Alice')
+
+      const help = command.help
+      
+      expect(help).toContain('greet')
+      expect(help).toContain('打招呼命令')
+      expect(help).toContain('greet <name>')
+      expect(help).toContain('greet Alice')
+    })
+
+    it('应该处理没有帮助信息的情况', () => {
+      const command = new MessageCommand('simple')
+
+      expect(command.helpInfo.desc).toEqual([])
+      expect(command.helpInfo.usage).toEqual([])
+      expect(command.helpInfo.examples).toEqual([])
+      expect(command.help).toBe('simple')
+    })
+
+    it('应该正确返回 helpInfo 对象结构', () => {
+      const command = new MessageCommand('info')
+        .desc('信息命令')
+        .usage('info')
+        .examples('info')
+
+      const helpInfo = command.helpInfo
+
+      expect(helpInfo).toHaveProperty('pattern')
+      expect(helpInfo).toHaveProperty('desc')
+      expect(helpInfo).toHaveProperty('usage')
+      expect(helpInfo).toHaveProperty('examples')
+      expect(typeof helpInfo.pattern).toBe('string')
+      expect(Array.isArray(helpInfo.desc)).toBe(true)
+      expect(Array.isArray(helpInfo.usage)).toBe(true)
+      expect(Array.isArray(helpInfo.examples)).toBe(true)
+    })
+  })
+
   describe('权限系统测试', () => {
     it('应该正确处理权限检查失败', async () => {
       const command = new MessageCommand('admin')

package/tests/component-new.test.ts

@@ -342,7 +342,9 @@ describe('函数式组件系统测试', () => {
         bot: 'test'
       }
 
-      await expect(renderComponents(componentMap, options)).rejects.toThrow('Test error')
+      const result = await renderComponents(componentMap, options)
+      // 错误会被捕获并转换为错误消息
+      expect(result.content).toMatch(/\[error Error: Test error\]/)
     })
   })
 })

package/tests/jsx.test.ts

@@ -218,15 +218,15 @@ describe('JSX消息组件系统测试', () => {
   })
 
   describe('错误处理', () => {
-    it('应该处理组件执行错误', async () => {
+    it('应该捕获组件执行错误并返回错误消息', async () => {
       const ErrorComponent: Component<{}> = async () => {
         throw new Error('Component error')
       }
 
       const element = jsx(ErrorComponent, {})
       
-      await expect(renderJSX(element as MessageComponent<any>, mockContext))
-        .rejects.toThrow('Component error')
+      const result = await renderJSX(element as MessageComponent<any>, mockContext)
+      expect(result).toBe('❌ 组件渲染失败: Component error')
     })
 
     it('应该处理无效的元素类型', async () => {
@@ -235,9 +235,28 @@ describe('JSX消息组件系统测试', () => {
         data: { children: 'test' }
       }
       
-      // 由于实现中会尝试调用type作为函数，这里应该处理错误
-      await expect(renderJSX(invalidElement as MessageComponent<any>, mockContext))
-        .rejects.toThrow()
+      const result = await renderJSX(invalidElement as MessageComponent<any>, mockContext)
+      expect(result).toMatch(/❌ 组件渲染失败/)
+    })
+    
+    it('应该处理异步组件返回 Promise 的情况', async () => {
+      const AsyncPromiseComponent: Component<{ text: string }> = async (props) => {
+        // 返回一个 Promise（会被自动 await）
+        return Promise.resolve(`Async: ${props.text}`)
+      }
+
+      const element = jsx(AsyncPromiseComponent, { text: 'test' })
+      const result = await renderJSX(element as MessageComponent<any>, mockContext)
+      
+      expect(result).toBe('Async: test')
+    })
+    
+    it('应该处理子组件为 Promise 的情况', async () => {
+      const AsyncChild = Promise.resolve('Async Child Content')
+      const element = jsx('div', { children: [AsyncChild] })
+      
+      const result = await renderJSX(element as MessageComponent<any>, mockContext)
+      expect(result).toBe('Async Child Content')
     })
   })