@kodrunhq/opencode-autopilot 1.15.0 → 1.15.2

package/README.md

@@ -62,6 +62,20 @@ npm install -g @kodrunhq/opencode-autopilot
 
 Launch OpenCode. The plugin auto-installs agents, skills, and commands on first load and shows a welcome toast.
 
+### Agent visibility defaults
+
+Primary Tab-cycle agents provided by this plugin are:
+
+- `autopilot`
+- `coder`
+- `debugger`
+- `planner`
+- `researcher`
+- `reviewer`
+
+OpenCode native `plan` and `build` are suppressed by the plugin config hook to avoid
+duplicate planning/building entries in the primary Tab menu.
+
 ### Verify your setup
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@kodrunhq/opencode-autopilot",
-	"version": "1.15.0",
+	"version": "1.15.2",
 	"description": "Curated agents, skills, and commands for the OpenCode AI coding CLI — autonomous orchestrator, multi-agent code review, model fallback, and in-session asset creation tools.",
 	"main": "src/index.ts",
 	"keywords": [

package/src/agents/index.ts

@@ -71,6 +71,60 @@ function registerAgents(
 	}
 }
 
+const nativeSuppressionPatch = Object.freeze({
+	disable: true,
+	mode: "subagent" as const,
+	hidden: true,
+});
+
+const optionalNativePlanBuildKeys = Object.freeze(["Plan", "Build", "Planner", "Builder"] as const);
+
+function isObjectRecord(value: unknown): value is Record<string, unknown> {
+	return typeof value === "object" && value !== null;
+}
+
+function mergeSuppressionPatch(entry: unknown): Record<string, unknown> {
+	if (isObjectRecord(entry)) {
+		return {
+			...entry,
+			...nativeSuppressionPatch,
+		};
+	}
+
+	return { ...nativeSuppressionPatch };
+}
+
+function suppressNativePlanBuildAgents(config: Config): void {
+	if (!config.agent) return;
+
+	const agentRef = config.agent as Record<string, unknown>;
+
+	// Deterministically suppress native lowercase keys even if OpenCode did not
+	// pre-populate them before configHook execution.
+	for (const key of ["plan", "build"] as const) {
+		agentRef[key] = mergeSuppressionPatch(agentRef[key]);
+	}
+
+	// Also suppress optional native variants when present.
+	for (const key of optionalNativePlanBuildKeys) {
+		if (agentRef[key] !== undefined) {
+			agentRef[key] = mergeSuppressionPatch(agentRef[key]);
+		}
+	}
+}
+
+function suppressLegacyModePlanBuild(config: Config): void {
+	if (!config.mode) return;
+
+	const modeRef = config.mode as Record<string, unknown>;
+	for (const key of ["plan", "build", ...optionalNativePlanBuildKeys] as const) {
+		const existing = modeRef[key];
+		if (existing === undefined) continue;
+		if (!isObjectRecord(existing)) continue;
+		modeRef[key] = mergeSuppressionPatch(existing);
+	}
+}
+
 export async function configHook(config: Config, configPath?: string): Promise<void> {
 	if (!config.agent) {
 		config.agent = {};
@@ -89,25 +143,16 @@ export async function configHook(config: Config, configPath?: string): Promise<v
 	const groups: Readonly<Record<string, GroupModelAssignment>> = pluginConfig?.groups ?? {};
 	const overrides: Readonly<Record<string, AgentOverride>> = pluginConfig?.overrides ?? {};
 
-	// Snapshot built-in agent keys BEFORE we register ours — we only suppress
-	// built-in Plan variants, not our own custom "planner" agent.
-	const builtInKeys = new Set(Object.keys(config.agent));
-
 	// Register standard agents and pipeline agents (v2 orchestrator subagents)
 	registerAgents(agents, config, groups, overrides);
 	registerAgents(pipelineAgents, config, groups, overrides);
 
-	// Suppress built-in Plan agent — our planner agent replaces it (D-17).
-	// Only disable keys that existed before our registration (built-ins).
-	const planVariants = ["Plan", "plan", "Planner", "planner"] as const;
-	for (const variant of planVariants) {
-		if (builtInKeys.has(variant) && config.agent[variant] !== undefined) {
-			config.agent[variant] = {
-				...config.agent[variant],
-				disable: true,
-			};
-		}
-	}
+	// Suppress native built-in Plan/Build agents. This is deterministic and does
+	// not rely on whether OpenCode pre-populated keys before configHook runs.
+	suppressNativePlanBuildAgents(config);
+
+	// Backward compatibility for legacy mode.plan/mode.build config shape.
+	suppressLegacyModePlanBuild(config);
 }
 
 export { autopilotAgent } from "./autopilot";

package/src/agents/researcher.ts

@@ -2,7 +2,7 @@ import type { AgentConfig } from "@opencode-ai/sdk";
 
 export const researcherAgent: Readonly<AgentConfig> = Object.freeze({
 	description: "Searches the web about a topic and produces a comprehensive report with sources",
-	mode: "subagent",
+	mode: "all",
 	prompt: `You are a research specialist. Your job is to thoroughly investigate a given topic and produce a clear, well-organized report.
 
 ## Instructions

package/src/health/checks.ts

@@ -90,6 +90,85 @@ export async function agentHealthCheck(config: Config | null): Promise<HealthRes
 	});
 }
 
+/**
+ * Check that OpenCode native plan/build agents are suppressed by the plugin.
+ * Contract: both entries must have disable=true, mode=subagent, hidden=true.
+ */
+export async function nativeAgentSuppressionHealthCheck(
+	config: Config | null,
+): Promise<HealthResult> {
+	if (!config?.agent) {
+		return Object.freeze({
+			name: "native-agent-suppression",
+			status: "fail" as const,
+			message: "No OpenCode config or agent map available",
+		});
+	}
+
+	const agentMap = config.agent as Record<string, unknown>;
+	const issues: string[] = [];
+	const requiredKeys = ["plan", "build"] as const;
+	const optionalKeys = ["Plan", "Build", "Planner", "Builder"] as const;
+
+	for (const key of requiredKeys) {
+		const raw = agentMap[key];
+		if (raw === undefined) {
+			issues.push(`${key}: missing config entry`);
+			continue;
+		}
+		if (typeof raw !== "object" || raw === null) {
+			issues.push(`${key}: invalid config entry type`);
+			continue;
+		}
+
+		const entry = raw as Record<string, unknown>;
+		if (entry.disable !== true) {
+			issues.push(`${key}: disable must be true`);
+		}
+		if (entry.mode !== "subagent") {
+			issues.push(`${key}: mode must be subagent`);
+		}
+		if (entry.hidden !== true) {
+			issues.push(`${key}: hidden must be true`);
+		}
+	}
+
+	for (const key of optionalKeys) {
+		const raw = agentMap[key];
+		if (raw === undefined) continue;
+		if (typeof raw !== "object" || raw === null) {
+			issues.push(`${key}: invalid config entry type`);
+			continue;
+		}
+
+		const entry = raw as Record<string, unknown>;
+		if (entry.disable !== true) {
+			issues.push(`${key}: disable must be true`);
+		}
+		if (entry.mode !== "subagent") {
+			issues.push(`${key}: mode must be subagent`);
+		}
+		if (entry.hidden !== true) {
+			issues.push(`${key}: hidden must be true`);
+		}
+	}
+
+	if (issues.length > 0) {
+		return Object.freeze({
+			name: "native-agent-suppression",
+			status: "fail" as const,
+			message: `${issues.length} native suppression issue(s) found`,
+			details: Object.freeze([...issues]),
+		});
+	}
+
+	return Object.freeze({
+		name: "native-agent-suppression",
+		status: "pass" as const,
+		message: "Native plan/build agents are suppressed",
+	});
+}
+
 /**
  * Check that the source and target asset directories exist and are accessible.
  */

package/src/health/runner.ts

@@ -5,6 +5,7 @@ import {
 	commandHealthCheck,
 	configHealthCheck,
 	memoryHealthCheck,
+	nativeAgentSuppressionHealthCheck,
 	skillHealthCheck,
 } from "./checks";
 import type { HealthReport, HealthResult } from "./types";
@@ -45,6 +46,7 @@ export async function runHealthChecks(options?: {
 	const settled = await Promise.allSettled([
 		configHealthCheck(options?.configPath),
 		agentHealthCheck(options?.openCodeConfig ?? null),
+		nativeAgentSuppressionHealthCheck(options?.openCodeConfig ?? null),
 		assetHealthCheck(options?.assetsDir, options?.targetDir),
 		skillHealthCheck(options?.projectRoot ?? process.cwd()),
 		memoryHealthCheck(options?.targetDir),
@@ -54,6 +56,7 @@ export async function runHealthChecks(options?: {
 	const fallbackNames = [
 		"config-validity",
 		"agent-injection",
+		"native-agent-suppression",
 		"asset-directories",
 		"skill-loading",
 		"memory-db",

package/src/index.ts

@@ -34,7 +34,7 @@ import {
 import { ocCreateAgent } from "./tools/create-agent";
 import { ocCreateCommand } from "./tools/create-command";
 import { ocCreateSkill } from "./tools/create-skill";
-import { ocDoctor } from "./tools/doctor";
+import { ocDoctor, setOpenCodeConfig as setDoctorOpenCodeConfig } from "./tools/doctor";
 import { ocForensics } from "./tools/forensics";
 import { ocHashlineEdit } from "./tools/hashline-edit";
 import { ocLogs } from "./tools/logs";
@@ -255,6 +255,7 @@ const plugin: Plugin = async (input) => {
 		config: async (cfg: Config) => {
 			openCodeConfig = cfg;
 			setOpenCodeConfig(cfg);
+			setDoctorOpenCodeConfig(cfg);
 			await configHook(cfg);
 		},
 		"chat.message": async (

package/src/tools/doctor.ts

@@ -6,6 +6,12 @@ import { runHealthChecks } from "../health/runner";
 import type { HealthResult } from "../health/types";
 import { getProjectArtifactDir } from "../utils/paths";
 
+let openCodeConfig: Config | null = null;
+
+export function setOpenCodeConfig(config: Config | null): void {
+	openCodeConfig = config;
+}
+
 /**
  * A single check in the doctor report, with an optional fix suggestion.
  */
@@ -63,6 +69,8 @@ const FIX_SUGGESTIONS: Readonly<Record<string, string>> = Object.freeze({
 	"config-validity":
 		"Run `bunx @kodrunhq/opencode-autopilot configure` to reconfigure, or delete ~/.config/opencode/opencode-autopilot.json to reset",
 	"agent-injection": "Restart OpenCode to trigger agent re-injection via config hook",
+	"native-agent-suppression":
+		"Restart OpenCode and verify plugin config hook runs. If issue persists, check for conflicting OpenCode config or another plugin overriding agent entries",
 	"asset-directories": "Restart OpenCode to trigger asset reinstallation",
 	"skill-loading": "Ensure skills directory exists in ~/.config/opencode/skills/",
 	"memory-db":
@@ -143,9 +151,12 @@ export async function doctorCore(options?: DoctorOptions): Promise<string> {
 export const ocDoctor = tool({
 	description:
 		"Run plugin health diagnostics. Reports pass/fail status for config, agents, " +
-		"assets, and hooks. Like `brew doctor` for opencode-autopilot.",
+		"native suppression, assets, and hooks. Like `brew doctor` for opencode-autopilot.",
 	args: {},
-	async execute() {
-		return doctorCore();
+	async execute(_args, context) {
+		return doctorCore({
+			openCodeConfig,
+			projectRoot: context.directory,
+		});
 	},
 });