Files
claude-code/packages/builtin-tools/src/tools/BashTool/__tests__/commandSemantics.test.ts
claude-code-best 2fb1c9dcd8 feat: 工具层及 mcp 大重构 (#252)
* feat: 第一版大重构

* fix: 修复类型问题

* chore: 更新版本到 1.3.2

* Add brave as alternative WebSearchTool

* fix: 修正顺序

* fix: 修复对穷鬼模式的 auto dream 和 session memory 越过

* feat: 穷鬼模式去除 session-summary

* feat: 创建 builtin-tools 包,搬运所有工具实现

将 src/tools/ 下的全部 60 个工具目录迁移至 packages/builtin-tools/src/tools/,
内部导入路径已更新为 src/ alias 模式。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* refactor: 更新 src/ 中所有工具引用至 builtin-tools 包,删除 src/tools/

- src/tools.ts 及 178 个 src/ 文件的 import 路径从 ./tools/ 改为 builtin-tools/tools/
- 删除 src/tools/ 整个目录(已迁移至 packages/builtin-tools/)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* chore: 添加 builtin-tools 路径别名至 tsconfig,更新 bun.lock

- tsconfig.json 新增 builtin-tools/* 和 builtin-tools 路径映射
- 新增 packages/builtin-tools/src 至 include

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* refactor: 为 builtin-tools、mcp-client、agent-tools 添加 @claude-code-best 作用域前缀

所有包名及 import 路径统一添加 @claude-code-best/ 前缀:
- builtin-tools → @claude-code-best/builtin-tools
- mcp-client → @claude-code-best/mcp-client
- agent-tools → @claude-code-best/agent-tools

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: 修复 node 环境没有 bun 的问题

---------

Co-authored-by: Eric-Guo <eric.guocz@gmail.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-13 09:52:05 +08:00

88 lines
4.0 KiB
TypeScript

import { mock, describe, expect, test } from "bun:test";
// Mock commands.ts to cut the heavy shell/prefix.ts → analytics → api chain
mock.module("src/utils/bash/commands.ts", () => ({
splitCommand_DEPRECATED: (cmd: string) =>
cmd.split(/\s*(?:[|;&]+)\s*/).filter(Boolean),
quote: (args: string[]) => args.join(" "),
}));
const { interpretCommandResult } = await import("../commandSemantics");
describe("interpretCommandResult", () => {
// ─── Default semantics ────────────────────────────────────────────
test("exit 0 is not an error for unknown commands", () => {
const result = interpretCommandResult("echo hello", 0, "hello", "");
expect(result.isError).toBe(false);
});
test("non-zero exit is an error for unknown commands", () => {
const result = interpretCommandResult("echo hello", 1, "", "fail");
expect(result.isError).toBe(true);
expect(result.message).toContain("exit code 1");
});
// ─── grep semantics ──────────────────────────────────────────────
test("grep exit 0 is not an error", () => {
const result = interpretCommandResult("grep pattern file", 0, "match", "");
expect(result.isError).toBe(false);
});
test("grep exit 1 means no matches (not error)", () => {
const result = interpretCommandResult("grep pattern file", 1, "", "");
expect(result.isError).toBe(false);
expect(result.message).toBe("No matches found");
});
test("grep exit 2 is an error", () => {
const result = interpretCommandResult("grep pattern file", 2, "", "err");
expect(result.isError).toBe(true);
});
// ─── diff semantics ──────────────────────────────────────────────
test("diff exit 1 means files differ (not error)", () => {
const result = interpretCommandResult("diff a.txt b.txt", 1, "diff", "");
expect(result.isError).toBe(false);
expect(result.message).toBe("Files differ");
});
test("diff exit 2 is an error", () => {
const result = interpretCommandResult("diff a.txt b.txt", 2, "", "err");
expect(result.isError).toBe(true);
});
// ─── test/[ semantics ────────────────────────────────────────────
test("test exit 1 means condition false (not error)", () => {
const result = interpretCommandResult("test -f nofile", 1, "", "");
expect(result.isError).toBe(false);
expect(result.message).toBe("Condition is false");
});
// ─── piped commands ──────────────────────────────────────────────
test("uses last command in pipe for semantics", () => {
// "cat file | grep pattern" → last command is "grep pattern"
const result = interpretCommandResult(
"cat file | grep pattern",
1,
"",
""
);
expect(result.isError).toBe(false);
expect(result.message).toBe("No matches found");
});
// ─── rg (ripgrep) semantics ──────────────────────────────────────
test("rg exit 1 means no matches (not error)", () => {
const result = interpretCommandResult("rg pattern", 1, "", "");
expect(result.isError).toBe(false);
expect(result.message).toBe("No matches found");
});
// ─── find semantics ──────────────────────────────────────────────
test("find exit 1 is partial success", () => {
const result = interpretCommandResult("find . -name '*.ts'", 1, "", "");
expect(result.isError).toBe(false);
expect(result.message).toBe("Some directories were inaccessible");
});
});