chore: 优化 FIT 启动流程和自动化测试配置

CodeCasterX · claude · CodeCasterX · commit 726257cd5de6 · 2025-12-18T11:02:00.000+08:00
- 添加 --plugin-dir 参数支持，允许灵活指定插件目录 - 优化测试命令的清理步骤，分开执行删除操作 - 完善 Claude Code 日常开发权限配置 - 新增 Git 提交规范，明确禁止自动提交代码 - 改进 fit.js 参数解析逻辑，支持相对路径和绝对路径 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
diff --git a/.agent/workflows/test.md b/.agent/workflows/test.md
@@ -23,23 +23,19 @@ description: 执行完整的测试流程，包括单元测试、构建验证和
 
 ### 3. 启动 FIT 服务进行集成测试
 
-**关键要求：**
-- `fit` 命令无法在包含 FIT 框架的目录中执行
-- 必须在一个新建的动态插件目录下执行
-
 **启动步骤：**
 
-创建动态插件目录并启动 FIT 服务：
-```bash
-mkdir -p dynamic-plugins
-cd dynamic-plugins
-../build/bin/fit start
-```
+1. 创建动态插件目录：
+   - `run_command("mkdir -p dynamic-plugins")`
 
-- `run_command("mkdir -p dynamic-plugins && cd dynamic-plugins && ../build/bin/fit start", timeout=120000, run_in_background=true)`
+2. 启动 FIT 服务：
+   - `run_command("build/bin/fit start --plugin-dir=dynamic-plugins", timeout=120000, run_in_background=true)`
 
 使用后台运行模式，超时时间设置为 120 秒（2分钟），给服务足够的启动时间。
 
+**说明：**
+- 使用 `--plugin-dir=dynamic-plugins` 参数指定插件目录
+
 **启动成功的判断标准：**
 - 输出日志中包含启动成功的关键信息
 - 没有出现 ERROR 或 FATAL 级别的日志
@@ -71,7 +67,8 @@ cd dynamic-plugins
 
 测试完成后，停止 FIT 服务并清理动态创建的目录：
 - `run_command("pkill -f fit-discrete-launcher")`
-- `run_command("rm -rf dynamic-plugins build")`
+- `run_command("rm -rf build")`
+- `run_command("rm -rf dynamic-plugins")`
 
 ### 6. 生成测试报告
 
diff --git a/.claude/commands/test.md b/.claude/commands/test.md
@@ -28,21 +28,23 @@ mvn clean install
 
 ## 3. 启动 FIT 服务进行集成测试
 
-**关键要求：**
-- `fit` 命令无法在包含 FIT 框架的目录中执行
-- 必须在一个新建的动态插件目录下执行
-
 **启动步骤：**
 
-**创建动态插件目录并启动**
-```bash run_in_background timeout=120000
+**1. 创建动态插件目录**
+```bash
 mkdir -p dynamic-plugins
-cd dynamic-plugins
-../build/bin/fit start
+```
+
+**2. 启动 FIT 服务**
+```bash run_in_background timeout=120000
+build/bin/fit start --plugin-dir=dynamic-plugins
 ```
 
 使用后台运行模式，超时时间设置为 120 秒（2分钟），给服务足够的启动时间。
 
+**说明：**
+- 使用 `--plugin-dir=dynamic-plugins` 参数指定插件目录
+
 **启动成功的判断标准：**
 - 输出日志中包含启动成功的关键信息
 - 没有出现 ERROR 或 FATAL 级别的日志
@@ -85,7 +87,8 @@ pkill -f fit-discrete-launcher
 
 ```bash
 # 删除动态创建的目录
-rm -rf dynamic-plugins build
+rm -rf build
+rm -rf dynamic-plugins
 ```
 
 **测试报告：**
diff --git a/.claude/project-rules.md b/.claude/project-rules.md
@@ -187,6 +187,62 @@ Bash("OWNER_GROUP=$(ls -l README.md | awk '{print $3\":\"$4}') && sudo chown $OW
 - 新功能：`[模块名] Add <feature-description>`
 - 功能增强：`[模块名] Enhance <feature-description>`
 
+## Git 提交和版本控制规范
+
+### 规则 3: 禁止自动提交代码
+
+**⚠️ 重要规则（CRITICAL）：**
+
+**绝对不要**自动执行 `git commit` 或 `git add` 命令，除非用户明确使用 `/commit` 命令请求提交。
+
+**原因：**
+- 项目配置了 `/commit` 命令，说明用户希望自己控制提交时机
+- 用户可能需要在提交前进行额外的检查或修改
+- 自动提交会剥夺用户对版本控制的控制权
+
+**正确做法：**
+
+1. **完成代码修改后**
+   - ✅ 告知用户有哪些文件被修改
+   - ✅ 展示修改的内容摘要
+   - ✅ 提醒用户可以使用 `/commit` 命令进行提交
+   - ❌ 不要自动执行 `git add` 或 `git commit`
+
+2. **仅在用户明确请求时才提交**
+   - ✅ 用户运行 `/commit` 命令
+   - ✅ 用户明确说"提交这些修改"、"commit these changes"
+   - ❌ 不要在用户只是让你"修复问题"或"完成任务"时自动提交
+
+3. **示例对比**
+
+   **❌ 错误做法：**
+   ```
+   User: 请修复测试命令的清理步骤
+   Assistant:
+   [修复代码]
+   [自动执行 git add 和 git commit]  # ❌ 错误！
+   ```
+
+   **✅ 正确做法：**
+   ```
+   User: 请修复测试命令的清理步骤
+   Assistant:
+   [修复代码]
+   我已经完成了修复，修改了以下文件：
+   - .claude/commands/test.md
+   - .agent/workflows/test.md
+
+   你可以使用 `/commit` 命令来提交这些修改。
+   # ✅ 正确！等待用户决定何时提交
+   ```
+
+**例外情况：**
+
+只有在以下情况下可以自动提交：
+1. 用户明确说"修复后直接提交"或"fix and commit"
+2. 用户使用了 `/commit` 命令
+3. 某个 workflow 或 slash command 的定义中明确要求自动提交
+
 ## 通用最佳实践
 
 ### 文件操作
@@ -195,6 +251,7 @@ Bash("OWNER_GROUP=$(ls -l README.md | awk '{print $3\":\"$4}') && sudo chown $OW
 - 批量操作后统一修改权限
 
 ### Git 操作
+- **绝对不要自动提交代码**（参见规则 3）
 - 提交前检查 `.github/` 目录中的规范
 - 遵循项目的 commit message 格式
 - PR 描述要完整、清晰
diff --git a/.claude/settings.json b/.claude/settings.json
@@ -7,7 +7,7 @@
       "WebFetch",
       "Bash(gh:*)",
       "Bash(git:*)",
-      "Bash(mvn:*)",
+      "Bash(mvn clean install:*)",
       "Bash(npm:*)",
       "Bash(find:*)",
       "Bash(cat:*)",
@@ -20,28 +20,13 @@
       "Bash(echo:*)",
       "Bash(mkdir:*)",
       "Bash(cd:*)",
-      "Bash(rm -rf build)",
-      "Bash(rm -rf dynamic-plugins)",
-      "Bash(rm -rf target)",
-      "Bash(rm -rf node_modules)",
-      "Bash(rm -rf dist)",
-      "Bash(rm -rf .tmp)",
-      "Bash(rm -rf coverage)",
-      "Bash(rm -rf .pytest_cache)",
-      "Bash(rm -rf __pycache__)",
-      "Bash(rm -f *.log)",
-      "Bash(rm -f *.tmp)"
+      "Bash(sleep:*)",
+      "Bash(tail:*)",
+      "Bash(head:*)",
+      "Bash(build/bin/fit start:*)",
+      "Bash(rm:*)"
     ],
     "deny": [
-      "Bash(rm -rf /)",
-      "Bash(rm -rf /*)",
-      "Bash(rm -rf ~)",
-      "Bash(rm -rf ~/*)",
-      "Bash(rm -rf .)",
-      "Bash(rm -rf ./)",
-      "Bash(rm -rf ..)",
-      "Bash(rm -rf ../)",
-      "Bash(rm -rf *)",
       "Bash(sudo rm:*)",
       "Read(.env)",
       "Read(.env.*)",
diff --git a/framework/fit/java/fit-launcher/src/main/resources/bin/fit.js b/framework/fit/java/fit-launcher/src/main/resources/bin/fit.js
@@ -63,6 +63,7 @@ function start(args) {
     // 初始化参数数组
     const javaArgs = [];
     const programArgs = [];
+    let pluginDir = null; // 默认不设置插件目录
 
     // 如果是 debug 模式，添加 debug 参数
     if (isDebugMode) {
@@ -72,9 +73,23 @@ function start(args) {
     // 跳过第一个参数（start 或 debug）
     const remainingArgs = args.slice(1);
 
-    // 分离 Java 参数和程序参数
+    // 分离 Java 参数、程序参数和插件目录参数
     for (const arg of remainingArgs) {
-        if (arg.startsWith('-')) {
+        if (arg.startsWith('--plugin-dir=')) {
+            // 解析插件目录参数
+            pluginDir = arg.substring('--plugin-dir='.length);
+            // 如果是 '.'，使用当前目录
+            if (pluginDir === '.') {
+                pluginDir = currentDir;
+            } else if (!path.isAbsolute(pluginDir)) {
+                // 如果是相对路径，转换为绝对路径
+                pluginDir = path.join(currentDir, pluginDir);
+            }
+        } else if (arg.startsWith('-D') || arg.startsWith('-X') || arg.startsWith('-agentlib:')) {
+            // Java 系统属性和 JVM 参数
+            javaArgs.push(arg);
+        } else if (arg.startsWith('-')) {
+            // 其他以 - 开头的参数作为 Java 参数
             javaArgs.push(arg);
         } else {
             programArgs.push(arg);
@@ -97,13 +112,16 @@ function start(args) {
     const commandArgs = [
         ...javaArgs,
         `-Dsun.io.useCanonCaches=true`,
-        `-Djdk.tls.client.enableSessionTicketExtension=false`,
-        `-Dplugin.fit.dynamic.plugin.directory=${currentDir}`,
-        '-jar',
-        path.join(jarDir, jarFile),
-        ...programArgs
+        `-Djdk.tls.client.enableSessionTicketExtension=false`
     ];
 
+    // 只有当 pluginDir 被指定时才添加插件目录系统属性
+    if (pluginDir) {
+        commandArgs.push(`-Dplugin.fit.dynamic.plugin.directory=${pluginDir}`);
+    }
+
+    commandArgs.push('-jar', path.join(jarDir, jarFile), ...programArgs);
+
     // 打印运行命令
     console.log(`Running command: ${javaCommand} ${commandArgs.join(' ')}`);
 
@@ -472,6 +490,16 @@ function help() {
     console.log('  debug        Start the application in debug mode');
     console.log('  version      Display the version number');
     console.log('  help         Display this help message');
+    console.log('\nStart/Debug Options:');
+    console.log('  --plugin-dir=<path>               Specify plugin directory');
+    console.log('                                    - Use "." for current directory');
+    console.log('                                    - Use relative or absolute path');
+    console.log('                                    - If not specified, no plugin directory is set');
+    console.log('\nStart/Debug Examples:');
+    console.log('  fit start                         # Start without plugin directory');
+    console.log('  fit start --plugin-dir=.          # Use current directory as plugin dir');
+    console.log('  fit start --plugin-dir=plugins    # Use relative path');
+    console.log('  fit start --plugin-dir=/path/to/plugins  # Use absolute path');
     console.log('\nInit Usage:');
     console.log('  fit init                          # Interactive mode (recommended)');
     console.log('  fit init <name>                   # Interactive mode with project name');
