diff --git a/BUILD_PLAN.md b/BUILD_PLAN.md index daa07a385..c6f7e59a0 100644 --- a/BUILD_PLAN.md +++ b/BUILD_PLAN.md @@ -12,7 +12,6 @@ These came up during recent ports and refactor passes — tracked here so they d | Follow-up | Why | Tier | Effort | |---|---|---|---| -| **Remove OpenRouter** | User confirmed we don't use OpenRouter. ~248 model entries in `models.generated.ts` + ~84 file mentions. Also removes `provider: "openrouter"` xiaomi entries (we now have direct via `xiaomi-token-plan-*`). | 1 | 1-2 days | | **Search provider registry refactor** | Adding minimax took 9 files because the provider list is duplicated across `provider.ts` (type + VALID_PREFERENCES), `native-search.ts`, `command-search-provider.ts` (CLI), `tool-search.ts` + `tool-llm-context.ts` (two separate execute paths!), `preferences-types.ts`, `preferences-validation.ts`, manifest, docs. A single `SearchProviderRegistry` array would let everything iterate. | 2 | 3-5 days | | **Minimax search tests** | Search agent ported the feature but explicitly skipped tests because bunker's tests don't match our preferences/provider export shape. Need: `getMiniMaxSearchApiKey()` priority order, `resolveSearchProvider()` returning "minimax", `/search-provider minimax` CLI behavior, no-key error messages, `executeMiniMaxSearch` request shape. | 1 | 0.5 day | | **Product-audit phase machine wire-up** | Slim port (commit `a8cf2cd94`) shipped the prompt + `sf_product_audit` tool + workflow template, but doesn't yet dispatch into PhaseMerge or PhaseComplete. The tool is callable; the phase doesn't auto-fire. | 2 | 0.5 day |