Follow-up to #2336 (umbrella) / PR #2463 (partial sweep). Covers Area 5 deferred from that PR.
Scope
handleDirectiveOnly, applyInlineDirectivesFastLane, and their params carry aliasIndex, allowedModelKeys, allowedModelCatalog, resetModelOverride, defaultProvider, defaultModel, initialModelLabel, formatModelSwitchEvent. After #2335 these are always empty/trivial because createModelSelectionState is gone, but the whole directive-handling model-switching path is still wired.
AC
Context
Follow-up to #2336 (umbrella) / PR #2463 (partial sweep). Covers Area 5 deferred from that PR.
Scope
handleDirectiveOnly,applyInlineDirectivesFastLane, and their params carryaliasIndex,allowedModelKeys,allowedModelCatalog,resetModelOverride,defaultProvider,defaultModel,initialModelLabel,formatModelSwitchEvent. After #2335 these are always empty/trivial becausecreateModelSelectionStateis gone, but the whole directive-handling model-switching path is still wired.src/auto-reply/reply/directive-handling.params.ts— delete the model-state fields fromHandleDirectiveOnlyCoreParams.src/auto-reply/reply/directive-handling.impl.ts— delete model-switching logic and unused params.src/auto-reply/reply/directive-handling.fast-lane.ts— delete destructures andhandleDirectiveOnlycall wiring.src/auto-reply/reply/get-reply-directives-apply.ts— delete the inline empty-defaultdirectiveModelStatecompat shim (introduced in gut(auto-reply): remove dead thinking-level resolution plumbing (#2334) #2335) and remainingaliasIndex/initialModelLabel/formatModelSwitchEventthreading.AC
pnpm check+pnpm testpass.Context