feat(tools): aggregate sema_search/sema_reindex/sema_stats into one action tool

[dearbld]

Aggregate sema_search / sema_reindex / sema_stats into one action tool

Base: upstream 601c3f5 (the commit this fork tracks) · stacked series: this PR → pr-2 → pr-4 → pr-5 → pr-3

Motivation

Three flat tool registrations mean three JSON schemas injected into every model round, for a plugin whose actions share one subject (the index). The harness itself aggregates same-family tools behind an action parameter (its built-in code_intel), and this plugin fits the same shape naturally: sema(action=search|stats|reindex).

What changes

• src/tools.ts: the three tools' execute and render logic is kept verbatim (moved into execSearch / execReindex / execStats and the three render helpers); only the tool surface collapses into one sema tool.
  • Parameters are the union of the three schemas with per-action descriptions.
  • The output schema root stays open (additionalProperties: true) because the canonical value shape differs per action; an internal __act field records which sub-tool produced the value so render dispatches. Note the schema compiler rejects an omitted additionalProperties, so the explicit value is required, not stylistic.
• src/index.ts: header comment updated (registers one aggregated tool).
• tests/plugin.test.ts: registration and end-to-end tests updated to call the aggregated tool with action.

Before / after

// before
return [searchTool, reindexTool, statsTool]   // 3 schemas every round

// after
return [semaTool]                             // 1 schema; action picks the sub-tool

Tool output rendering is unchanged from the model's perspective — each action renders exactly as its former flat tool did (including the sema_search: / sema_stats: render prefixes).

Testing

• tsc --noEmit clean under the repo tsconfig (strict + noUncheckedIndexedAccess).
• Full suite 80/80 pass with the updated tests.
• Suggested: assert the rendered blocks of all three actions byte-identical to the pre-aggregation renders (the render helpers are lifted verbatim, so this should hold trivially).

[JohnXu22786]

DSH should remain the primary host, and I accept the single sema(action=...) direction. Please keep that direction, but tighten the contract: make action a required enum, reject unknown actions instead of silently treating them as search, require a non-empty query for search, and update both README.md and README.zh.md with the new API. Compatibility with the old tool names is secondary; it is not the main objection.

[dearbld]

Addressed in 9cef5e0: action is now a required enum ([search, stats, reindex]) in the parameter schema. The dsh tool wrapper enforces it before execute runs — a bogus action now throws a validation error instead of silently running a search. A runtime guard inside execute keeps the rejection explicit as defense in depth, and render surfaces rejected values as plain text (the no-__act branch) instead of falling through to the search renderer. Test added covering both layers.