默认错误规则新增 Tool names must be unique

[ding113]

为了解决 claude code 的 [Bug] /compact fails with "Tool names must be unique" even after disabling all MCP servers (v2.0.70)
需要给现有默认错误规则增加正则，匹配下面的报错样式：

    {"type":"error","error":{"type":"invalid_request_error",
       "message":"tools: Tool names must be unique."},
       "request_id":"req_011CW9cz13pCHJv2SC8NPCaq"}

[github-actions]

Thank you for this feature request.

Analysis:

Based on my analysis of the codebase, the default error rules are defined in src/repository/error-rules.ts (lines 292-479) within the DEFAULT_ERROR_RULES array. These rules are used by the error rule detection engine to identify non-retryable errors.

Proposed Rule:

To match the error pattern you provided:

{"type":"error","error":{"type":"invalid_request_error",
   "message":"tools: Tool names must be unique."},
   "request_id":"..."}

A new rule should be added to DEFAULT_ERROR_RULES array with a pattern like:

{
  pattern: "Tool names must be unique",
  category: "validation_error",
  description: "Duplicate tool names in request (client error, related to MCP server configuration)",
  matchType: "contains" as const,
  isDefault: true,
  isEnabled: true,
  priority: 89,
}

Relevant code:

• src/repository/error-rules.ts (lines 367-386) - Similar tool use validation rules exist as precedent:
  • \tool_use` ids must be unique` (line 369)
  • unexpected.*tool_use_id.*tool_result (line 379)

Synchronization:
After adding the rule, running bun run scripts/init-error-rules.ts or restarting the application with AUTO_MIGRATE=true will sync the new default rule to the database via syncDefaultErrorRules().

---

Automated response from Claude AI

[ding113]

@claude fix it

[ding113]

I've created a fix for this issue.

Pull Request: #367

Changes made:

• Added new default error rule to src/repository/error-rules.ts
• Pattern: Tool names must be unique (uses contains match type)
• Category: validation_error with priority 89 (same as other tool validation errors)
• Includes Chinese localized override response: "工具名称重复，请检查 MCP 服务器配置确保工具名称唯一"

Synchronization:
After merging, the new rule will be synced to the database on:

• Application restart with AUTO_MIGRATE=true
• Or by running bun run scripts/init-error-rules.ts

Please review and let me know if you need any adjustments.

---

Response from Claude AI