style: trim redundant comments on subagent param floor

Collapse the repeated subagent/fast rationale to a single home (the
plugin config hook where the map originates); reduce the type-field
docstrings to one-liners and drop the self-evident debug-log comment.

Author: justin-carper

src/plugin/index.ts

@@ -118,13 +118,11 @@ export const CursorPlugin: Plugin = async (input) => {
 				? { ...userMcp, ...translateMcpServers(config.mcp) }
 				: userMcp;
 
-			// Per-model floor params (e.g. { "composer-2.5": { fast: "false" } }).
-			// opencode merges each model's own `options.params` on the normal chat
-			// path, but a subagent that inherits its parent agent's model can reach
-			// the provider with the bare model id and no params. Threading these
-			// defaults through the (per-provider, not per-request) provider options
-			// lets the provider re-apply them as a floor so `fast` never silently
-			// falls back to Cursor's server-side `true`.
+			// opencode forwards a model's own options.params on the normal chat
+			// path, but a subagent inheriting its parent's model reaches the provider
+			// with them dropped — letting Cursor's server-side `fast: true` apply.
+			// Thread the defaults through provider options (per-provider, survives
+			// the drop) so the provider can re-apply them as a floor.
 			const modelParamDefaults: Record<string, Record<string, string>> = {};
 			for (const item of models) {
 				const params = defaultModelParams(item);

src/provider/controls.ts

@@ -6,13 +6,9 @@ export interface StaticControls {
   /** Default Cursor model params (id -> value), e.g. { thinking: "high" }. */
   params?: Record<string, string>;
   /**
-   * Per-model floor params applied UNDER {@link params} and per-request options.
-   * Carries this model's non-reasoning boolean defaults (e.g. `{ fast: "false" }`)
-   * so a call that reaches the provider with the bare model id and no params —
-   * notably an opencode subagent inheriting its parent's model — still pins
-   * `fast` off instead of inheriting Cursor's server-side `fast: true` default.
-   * The normal chat path already carries these via the model's opencode
-   * `options.params`, so re-applying them here is a no-op there.
+   * Per-model floor params, applied UNDER {@link params} and per-request options
+   * (an explicit param always wins). Pins Cursor's boolean toggles, e.g.
+   * `{ fast: "false" }`, when a turn arrives with no params of its own.
    */
   defaults?: Record<string, string>;
 }

src/provider/index.ts

@@ -34,11 +34,9 @@ export interface CursorProviderOptions {
 	/** Default Cursor model params (id -> value), e.g. { thinking: "high" }. */
 	params?: Record<string, string>;
 	/**
-	 * Per-model floor params keyed by model id (e.g. `{ "composer-2.5": { fast:
-	 * "false" } }`). Seeded by the plugin's `config` hook from the discovered
-	 * catalog so subagents that inherit a model without its options.params still
-	 * pin Cursor's boolean toggles to their opencode defaults. Applied under
-	 * `params` and per-request options.
+	 * Per-model floor params keyed by model id, e.g. `{ "composer-2.5": { fast:
+	 * "false" } }`. Seeded by the plugin's `config` hook; applied under `params`
+	 * and per-request options.
 	 */
 	modelParamDefaults?: Record<string, Record<string, string>>;
 	/**

src/provider/language-model.ts

@@ -56,13 +56,8 @@ export interface CursorModelConfig {
 	/** Default Cursor model params (id -> value); overridable per-request. */
 	params?: Record<string, string>;
 	/**
-	 * Per-model floor params keyed by model id (e.g. `{ "composer-2.5": { fast:
-	 * "false" } }`). Applied under {@link params} and per-request options so a
-	 * call arriving with the bare model id and no params — notably an opencode
-	 * subagent that inherited its parent agent's model — still pins Cursor's
-	 * boolean toggles (like `fast`) to their opencode defaults instead of
-	 * silently inheriting Cursor's server-side `fast: true`. Seeded by the
-	 * plugin's `config` hook from the discovered catalog.
+	 * Per-model floor params keyed by model id, seeded by the plugin's `config`
+	 * hook. Passed as {@link resolveControls}'s `defaults` for the active model.
 	 */
 	modelParamDefaults?: Record<string, Record<string, string>>;
 	/** MCP servers forwarded to the Cursor agent from opencode's config. */
@@ -158,10 +153,6 @@ export class CursorLanguageModel implements LanguageModelV3 {
 			providerOptions,
 		);
 		if (process.env["OPENCODE_CURSOR_DEBUG"] === "1") {
-			// Root-cause instrument: shows the exact ModelSelection sent to Cursor.
-			// A subagent that inherited its parent's model with dropped params shows
-			// up here as a selection missing the `fast: "false"` floor before the
-			// modelParamDefaults guard re-applies it.
 			console.error(
 				`[cursor:debug] model=${this.modelId} selection=${JSON.stringify(modelSelection)}`,
 			);