refactor(kimi-web-search): rename tools to snake_case for consistency

[ckanner]

Summary

Renames SearchWeb/FetchURL to kimi_search_web/kimi_fetch_url to align with senpi/pi-mono tool naming conventions.

Problem

The current tool names use PascalCase (SearchWeb, FetchURL), inherited from Kimi CLI's class names. This is inconsistent with every other tool in the senpi ecosystem:

Tool	Naming
read, write, edit, bash	lowercase
apply_patch, multiedit	snake_case
web_search (anthropic-web-search)	snake_case
todowrite, todoread	lowercase
SearchWeb, FetchURL (kimi-web-search)	PascalCase ❌

Solution

• SearchWeb → kimi_search_web
• FetchURL → kimi_fetch_url

This makes the tool names consistent with:

1. The rest of senpi's builtin extensions
2. Third-party extensions (e.g., pi-websearch uses web_search)
3. The general Unix/CLI convention of snake_case for identifiers

Changes

• packages/coding-agent/src/core/extensions/builtin/kimi-web-search/index.ts
  • name, label, description, promptSnippet updated

Checklist

• Pre-commit checks pass (Biome + tsgo + browser-smoke + web-ui)
• Atomic commit with clear rationale
• No breaking changes to tool schema (only the tool name changed)

Migration

Users with permission rules for SearchWeb/FetchURL need to update:

"permission": {
  "kimi_search_web": "allow",
  "kimi_fetch_url": "allow"
}

Refs: #10

---

Summary by cubic

Renamed SearchWeb and FetchURL to kimi_search_web and kimi_fetch_url for consistent snake_case tool names across senpi/pi-mono, and updated regression tests. Aligns with #10; no schema or behavior changes.

• Migration
  • Update any permission rules or references: SearchWeb → kimi_search_web, FetchURL → kimi_fetch_url.

^{Written for commit ba84934. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 1 file

_{Re-trigger cubic}

[code-yeongyu]

[sisyphus-bot] Rebased this branch cleanly onto the latest main (no conflicts) and verified the diff end to end before merging.

Verification I ran on the rebased tree:

• Static gate: git grep -n "SearchWeb|FetchURL" -- packages/coding-agent/src packages/coding-agent/test shows no remaining tool-registration usages. The only hits are the internal TypeScript type identifiers (SearchWebInput / FetchUrlInput), which are out of scope for a tool-name rename, plus the historical note in changes.md. Both kimi_search_web and kimi_fetch_url are present in index.ts (name, label, description, promptSnippet) and in the 3592 regression test.
• Functional gate: npm ci --ignore-scripts at the repo root, then npx vitest --run test/suite/regressions/3592-no-builtin-tools-keeps-extension-tools.test.ts (3/3 passed), and npx tsgo --noEmit from the repo root (clean, exit 0).

Merged via a merge commit. Thanks for the tidy, focused change, ckanner.