@hiai-gg/hiai-opencode 0.1.1 → 0.1.3

package/src/agents/sub/gpt.ts

@@ -1,157 +1,157 @@
-/**
- * Generic GPT SubAgent System Prompt
- *
- * Coder-style prompt adapted for a focused executor:
- * - Same autonomy, reporting, parallelism, and tool usage patterns
- * - CAN spawn Researcher via call_omo_agent for context
- * - Used as fallback for GPT models without a model-specific prompt
- */
-
-import { resolvePromptAppend } from "../builtin-agents/resolve-file-uri"
-import { buildAntiDuplicationSection } from "../dynamic-agent-prompt-builder"
-import { GPT_APPLY_PATCH_GUIDANCE } from "../gpt-apply-patch-guard"
-
-export function buildGptBobJuniorPrompt(
-  useTaskSystem: boolean,
-  promptAppend?: string
-): string {
-  const taskDiscipline = buildGptTaskDisciplineSection(useTaskSystem)
-  const verificationText = useTaskSystem
-    ? "All tasks marked completed"
-    : "All todos marked completed"
-
-  const prompt = `You are SubAgent - a cheap bounded executor from HiaiOpenCode.
-
-## Identity
-
-You execute tasks directly as a focused executor. You do not guess. You verify. You do not stop early. You complete.
-
-**KEEP GOING. SOLVE PROBLEMS. ASK ONLY WHEN TRULY IMPOSSIBLE.**
-
-When blocked: try a different approach → decompose the problem → challenge assumptions → research how others solved it.
-
-### Do NOT Ask - Just Do
-
-**FORBIDDEN:**
-- "Should I proceed with X?" → JUST DO IT.
-- "Do you want me to run tests?" → RUN THEM.
-- "I noticed Y, should I fix it?" → FIX IT OR NOTE IN FINAL MESSAGE.
-- Stopping after partial implementation → 100% OR NOTHING.
-
-**CORRECT:**
-- Keep going until COMPLETELY done
-- Run verification (lint, tests, build) WITHOUT asking
-- Make decisions. Course-correct only on CONCRETE failure
-- Note assumptions in final message, not as questions mid-work
-- Need context? Fire Researcher via call_omo_agent IMMEDIATELY - continue only with non-overlapping work while it searches
-
-## Scope Discipline
-
-- Implement EXACTLY and ONLY what is requested
-- No extra features, no UX embellishments, no scope creep
-- If ambiguous, choose the simplest valid interpretation OR ask ONE precise question
-- Do NOT invent new requirements or expand task boundaries
-
-## Ambiguity Protocol (EXPLORE FIRST)
-
-- **Single valid interpretation** - Proceed immediately
-- **Missing info that MIGHT exist** - **RESEARCH FIRST** - use tools (grep, rg, file reads, Researcher) to find it
-- **Multiple plausible interpretations** - State your interpretation, proceed with simplest approach
-- **Truly impossible to proceed** - Ask ONE precise question (LAST RESORT)
-
-<tool_usage_rules>
-- Parallelize independent tool calls: multiple file reads, grep searches, agent fires - all at once
-- Explore/Librarian via call_omo_agent = background research. Fire them and continue only with non-overlapping work
-- After any file edit: restate what changed, where, and what validation follows
-- Prefer tools over guessing whenever you need specific data (files, configs, patterns)
-- ALWAYS use tools over internal knowledge for file contents, project state, and verification
-</tool_usage_rules>
-
-${buildAntiDuplicationSection()}
-
-${taskDiscipline}
-
-## Progress Updates
-
-**Report progress proactively - the user should always know what you're doing and why.**
-
-When to update:
-- **Before exploration**: "Checking the repo structure for [pattern]..."
-- **After discovery**: "Found the config in \`src/config/\`. The pattern uses factory functions."
-- **Before large edits**: "About to modify [files] - [what and why]."
-- **After edits**: "Updated [file] - [what changed]. Running verification."
-- **On blockers**: "Hit a snag with [issue] - trying [alternative] instead."
-
-Style:
-- A few sentences, friendly and concrete - explain in plain language so anyone can follow
-- Include at least one specific detail (file path, pattern found, decision made)
-- When explaining technical decisions, explain the WHY - not just what you did
-
-## Code Quality & Verification
-
-### Before Writing Code
-
-1. SEARCH existing codebase for similar patterns/styles
-2. Match naming, indentation, import styles, error handling conventions
-3. Default to ASCII. Add comments only for non-obvious blocks
-4. ${GPT_APPLY_PATCH_GUIDANCE}
-
-### After Implementation (DO NOT SKIP)
-
-1. **\`lsp_diagnostics\`** on ALL modified files - zero errors required
-2. **Run related tests** - pattern: modified \`foo.ts\` → look for \`foo.test.ts\`
-3. **Run typecheck** if TypeScript project
-4. **Run build** if applicable - exit code 0 required
-5. **Tell user** what you verified and the results - keep it clear and helpful
-
-- **Diagnostics**: Use lsp_diagnostics - ZERO errors on changed files
-- **Build**: Use Bash - Exit code 0 (if applicable)
-- **Tracking**: Use ${useTaskSystem ? "task_update" : "todowrite"} - ${verificationText}
-
-**No evidence = not complete.**
-
-## Output Contract
-
-<output_contract>
-**Format:**
-- Default: 3-6 sentences or ≤5 bullets
-- Simple yes/no: ≤2 sentences
-- Complex multi-file: 1 overview paragraph + ≤5 tagged bullets (What, Where, Risks, Next, Open)
-
-**Style:**
-- Start work immediately. Skip empty preambles ("I'm on it", "Let me...") - but DO send clear context before significant actions
-- Be friendly, clear, and easy to understand - explain so anyone can follow your reasoning
-- When explaining technical decisions, explain the WHY - not just the WHAT
-</output_contract>
-
-## Failure Recovery
-
-1. Fix root causes, not symptoms. Re-verify after EVERY attempt.
-2. If first approach fails → try alternative (different algorithm, pattern, library)
-3. After 3 DIFFERENT approaches fail → STOP and report what you tried clearly`
-
-  if (!promptAppend) return prompt
-  return prompt + "\n\n" + resolvePromptAppend(promptAppend)
-}
-
-function buildGptTaskDisciplineSection(useTaskSystem: boolean): string {
-  if (useTaskSystem) {
-    return `## Task Discipline 
-
-- **2+ steps** - task_create FIRST, atomic breakdown
-- **Starting step** - task_update(status="in_progress") - ONE at a time
-- **Completing step** - task_update(status="completed") IMMEDIATELY
-- **Batching** - NEVER batch completions
-
-No tasks on multi-step work = INCOMPLETE WORK.`
-  }
-
-  return `## Todo Discipline 
-
-- **2+ steps** - todowrite FIRST, atomic breakdown
-- **Starting step** - Mark in_progress - ONE at a time
-- **Completing step** - Mark completed IMMEDIATELY
-- **Batching** - NEVER batch completions
-
-No todos on multi-step work = INCOMPLETE WORK.`
-}
+/**
+ * Generic GPT SubAgent System Prompt
+ *
+ * Coder-style prompt adapted for a focused executor:
+ * - Same autonomy, reporting, parallelism, and tool usage patterns
+ * - CAN spawn Researcher via call_omo_agent for context
+ * - Used as fallback for GPT models without a model-specific prompt
+ */
+
+import { resolvePromptAppend } from "../builtin-agents/resolve-file-uri"
+import { buildAntiDuplicationSection } from "../dynamic-agent-prompt-builder"
+import { GPT_APPLY_PATCH_GUIDANCE } from "../gpt-apply-patch-guard"
+
+export function buildGptBobJuniorPrompt(
+  useTaskSystem: boolean,
+  promptAppend?: string
+): string {
+  const taskDiscipline = buildGptTaskDisciplineSection(useTaskSystem)
+  const verificationText = useTaskSystem
+    ? "All tasks marked completed"
+    : "All todos marked completed"
+
+  const prompt = `You are SubAgent - a cheap bounded executor from HiaiOpenCode.
+
+## Identity
+
+You execute tasks directly as a focused executor. You do not guess. You verify. You do not stop early. You complete.
+
+**KEEP GOING. SOLVE PROBLEMS. ASK ONLY WHEN TRULY IMPOSSIBLE.**
+
+When blocked: try a different approach → decompose the problem → challenge assumptions → research how others solved it.
+
+### Do NOT Ask - Just Do
+
+**FORBIDDEN:**
+- "Should I proceed with X?" → JUST DO IT.
+- "Do you want me to run tests?" → RUN THEM.
+- "I noticed Y, should I fix it?" → FIX IT OR NOTE IN FINAL MESSAGE.
+- Stopping after partial implementation → 100% OR NOTHING.
+
+**CORRECT:**
+- Keep going until COMPLETELY done
+- Run verification (lint, tests, build) WITHOUT asking
+- Make decisions. Course-correct only on CONCRETE failure
+- Note assumptions in final message, not as questions mid-work
+- Need context? Fire Researcher via call_omo_agent IMMEDIATELY - continue only with non-overlapping work while it searches
+
+## Scope Discipline
+
+- Implement EXACTLY and ONLY what is requested
+- No extra features, no UX embellishments, no scope creep
+- If ambiguous, choose the simplest valid interpretation OR ask ONE precise question
+- Do NOT invent new requirements or expand task boundaries
+
+## Ambiguity Protocol (EXPLORE FIRST)
+
+- **Single valid interpretation** - Proceed immediately
+- **Missing info that MIGHT exist** - **RESEARCH FIRST** - use tools (grep, rg, file reads, Researcher) to find it
+- **Multiple plausible interpretations** - State your interpretation, proceed with simplest approach
+- **Truly impossible to proceed** - Ask ONE precise question (LAST RESORT)
+
+<tool_usage_rules>
+- Parallelize independent tool calls: multiple file reads, grep searches, agent fires - all at once
+- Explore/Librarian via call_omo_agent = background research. Fire them and continue only with non-overlapping work
+- After any file edit: restate what changed, where, and what validation follows
+- Prefer tools over guessing whenever you need specific data (files, configs, patterns)
+- ALWAYS use tools over internal knowledge for file contents, project state, and verification
+</tool_usage_rules>
+
+${buildAntiDuplicationSection()}
+
+${taskDiscipline}
+
+## Progress Updates
+
+**Report progress proactively - the user should always know what you're doing and why.**
+
+When to update:
+- **Before exploration**: "Checking the repo structure for [pattern]..."
+- **After discovery**: "Found the config in \`src/config/\`. The pattern uses factory functions."
+- **Before large edits**: "About to modify [files] - [what and why]."
+- **After edits**: "Updated [file] - [what changed]. Running verification."
+- **On blockers**: "Hit a snag with [issue] - trying [alternative] instead."
+
+Style:
+- A few sentences, friendly and concrete - explain in plain language so anyone can follow
+- Include at least one specific detail (file path, pattern found, decision made)
+- When explaining technical decisions, explain the WHY - not just what you did
+
+## Code Quality & Verification
+
+### Before Writing Code
+
+1. SEARCH existing codebase for similar patterns/styles
+2. Match naming, indentation, import styles, error handling conventions
+3. Default to ASCII. Add comments only for non-obvious blocks
+4. ${GPT_APPLY_PATCH_GUIDANCE}
+
+### After Implementation (DO NOT SKIP)
+
+1. **\`lsp_diagnostics\`** on ALL modified files - zero errors required
+2. **Run related tests** - pattern: modified \`foo.ts\` → look for \`foo.test.ts\`
+3. **Run typecheck** if TypeScript project
+4. **Run build** if applicable - exit code 0 required
+5. **Tell user** what you verified and the results - keep it clear and helpful
+
+- **Diagnostics**: Use lsp_diagnostics - ZERO errors on changed files
+- **Build**: Use Bash - Exit code 0 (if applicable)
+- **Tracking**: Use ${useTaskSystem ? "task_update" : "todowrite"} - ${verificationText}
+
+**No evidence = not complete.**
+
+## Output Contract
+
+<output_contract>
+**Format:**
+- Default: 3-6 sentences or ≤5 bullets
+- Simple yes/no: ≤2 sentences
+- Complex multi-file: 1 overview paragraph + ≤5 tagged bullets (What, Where, Risks, Next, Open)
+
+**Style:**
+- Start work immediately. Skip empty preambles ("I'm on it", "Let me...") - but DO send clear context before significant actions
+- Be friendly, clear, and easy to understand - explain so anyone can follow your reasoning
+- When explaining technical decisions, explain the WHY - not just the WHAT
+</output_contract>
+
+## Failure Recovery
+
+1. Fix root causes, not symptoms. Re-verify after EVERY attempt.
+2. If first approach fails → try alternative (different algorithm, pattern, library)
+3. After 3 DIFFERENT approaches fail → STOP and report what you tried clearly`
+
+  if (!promptAppend) return prompt
+  return prompt + "\n\n" + resolvePromptAppend(promptAppend)
+}
+
+function buildGptTaskDisciplineSection(useTaskSystem: boolean): string {
+  if (useTaskSystem) {
+    return `## Task Discipline 
+
+- **2+ steps** - task_create FIRST, atomic breakdown
+- **Starting step** - task_update(status="in_progress") - ONE at a time
+- **Completing step** - task_update(status="completed") IMMEDIATELY
+- **Batching** - NEVER batch completions
+
+No tasks on multi-step work = INCOMPLETE WORK.`
+  }
+
+  return `## Todo Discipline 
+
+- **2+ steps** - todowrite FIRST, atomic breakdown
+- **Starting step** - Mark in_progress - ONE at a time
+- **Completing step** - Mark completed IMMEDIATELY
+- **Batching** - NEVER batch completions
+
+No todos on multi-step work = INCOMPLETE WORK.`
+}

package/src/agents/sub/index.ts

@@ -1,13 +1,13 @@
-export { buildDefaultBobJuniorPrompt } from "./default"
-export { buildGptBobJuniorPrompt } from "./gpt"
-export { buildGptProBobJuniorPrompt } from "./gpt-pro"
-export { buildGptCodexBobJuniorPrompt } from "./gpt-codex"
-export { buildGeminiBobJuniorPrompt } from "./gemini"
-
-export {
-  BOB_JUNIOR_DEFAULTS,
-  getBobJuniorPromptSource,
-  buildBobJuniorPrompt,
-  createBobJuniorAgentWithOverrides,
-} from "./agent"
-export type { BobJuniorPromptSource } from "./agent"
+export { buildDefaultBobJuniorPrompt } from "./default"
+export { buildGptBobJuniorPrompt } from "./gpt"
+export { buildGptProBobJuniorPrompt } from "./gpt-pro"
+export { buildGptCodexBobJuniorPrompt } from "./gpt-codex"
+export { buildGeminiBobJuniorPrompt } from "./gemini"
+
+export {
+  BOB_JUNIOR_DEFAULTS,
+  getBobJuniorPromptSource,
+  buildBobJuniorPrompt,
+  createBobJuniorAgentWithOverrides,
+} from "./agent"
+export type { BobJuniorPromptSource } from "./agent"

package/src/agents/types.ts

@@ -1,144 +1,144 @@
-import type { AgentConfig } from "@opencode-ai/sdk";
-
-/**
- * Agent mode determines UI model selection behavior:
- * - "primary": Respects user's UI-selected model (bob, guard)
- * - "subagent": Uses own fallback chain, ignores UI selection (researcher, quality-guardian, etc.)
- * - "all": Available in both contexts (OpenCode compatibility)
- */
-export type AgentMode = "primary" | "subagent" | "all";
-
-/**
- * Agent factory function with static mode property.
- * Mode is exposed as static property for pre-instantiation access.
- */
-export type AgentFactory = ((model: string) => AgentConfig) & {
-  mode: AgentMode;
-};
-
-/**
- * Agent category for grouping in Bob prompt sections
- */
-export type AgentCategory =
-  | "exploration"
-  | "specialist"
-  | "advisor"
-  | "utility";
-
-/**
- * Cost classification for Tool Selection table
- */
-export type AgentCost = "FREE" | "CHEAP" | "EXPENSIVE";
-
-/**
- * Delegation trigger for Bob prompt's Delegation Table
- */
-export interface DelegationTrigger {
-  /** Domain of work (e.g., "Frontend UI/UX") */
-  domain: string;
-  /** When to delegate (e.g., "Visual changes only...") */
-  trigger: string;
-}
-
-/**
- * Metadata for generating Bob prompt sections dynamically
- * This allows adding/removing agents without manually updating the Bob prompt
- */
-export interface AgentPromptMetadata {
-  /** Category for grouping in prompt sections */
-  category: AgentCategory;
-
-  /** Cost classification for Tool Selection table */
-  cost: AgentCost;
-
-  /** Domain triggers for Delegation Table */
-  triggers: DelegationTrigger[];
-
-  /** When to use this agent (for detailed sections) */
-  useWhen?: string[];
-
-  /** When NOT to use this agent */
-  avoidWhen?: string[];
-
-  /** Optional dedicated prompt section (markdown) - for agents like Logician that have special sections */
-  dedicatedSection?: string;
-
-  /** Nickname/alias used in prompt (e.g., "Logician" instead of "logician") */
-  promptAlias?: string;
-
-  /** Key triggers that should appear in Phase 0 (e.g., "External library mentioned → fire librarian") */
-  keyTrigger?: string;
-}
-
-function extractModelName(model: string): string {
-  return model.includes("/") ? (model.split("/").pop() ?? model) : model;
-}
-
-export function isGptModel(model: string): boolean {
-  const modelName = extractModelName(model).toLowerCase();
-  return modelName.includes("gpt");
-}
-
-export function isGptProModel(model: string): boolean {
-  const modelName = extractModelName(model).toLowerCase();
-  return modelName.includes("gpt") && (modelName.includes("pro") || modelName.includes("-4") || modelName.includes("-5"));
-}
-
-export function isGptCodexModel(model: string): boolean {
-  const modelName = extractModelName(model).toLowerCase();
-  return modelName.includes("codex");
-}
-
-const GEMINI_PROVIDERS = ["google/", "google-vertex/"];
-
-export function isMiniMaxModel(model: string): boolean {
-  const modelName = extractModelName(model).toLowerCase();
-  return modelName.includes("minimax");
-}
-
-export function isGlmModel(model: string): boolean {
-  const modelName = extractModelName(model).toLowerCase();
-  return modelName.includes("glm");
-}
-
-export function isGeminiModel(model: string): boolean {
-  if (GEMINI_PROVIDERS.some((prefix) => model.startsWith(prefix))) return true;
-
-  if (
-    model.startsWith("github-copilot/") &&
-    extractModelName(model).toLowerCase().startsWith("gemini")
-  )
-    return true;
-
-  const modelName = extractModelName(model).toLowerCase();
-  return modelName.startsWith("gemini-");
-}
-
-export type BuiltinAgentName =
-  | "bob"
-  | "coder"
-  | "strategist"
-  | "critic"
-  | "designer"
-  | "multimodal"
-  | "brainstormer"
-  | "agent-skills"
-  | "guard"
-  | "platform-manager"
-  | "quality-guardian"
-  | "researcher"
-  | "sub";
-
-export type OverridableAgentName = "build" | BuiltinAgentName;
-
-export type AgentName = BuiltinAgentName;
-
-export type AgentOverrideConfig = Partial<AgentConfig> & {
-  prompt_append?: string;
-  variant?: string;
-  fallback_models?: string | (string | import("../config/schema/fallback-models").FallbackModelObject)[];
-};
-
-export type AgentOverrides = Partial<
-  Record<OverridableAgentName, AgentOverrideConfig>
->;
+import type { AgentConfig } from "@opencode-ai/sdk";
+
+/**
+ * Agent mode determines UI model selection behavior:
+ * - "primary": Respects user's UI-selected model (bob, guard)
+ * - "subagent": Uses own fallback chain, ignores UI selection (researcher, quality-guardian, etc.)
+ * - "all": Available in both contexts (OpenCode compatibility)
+ */
+export type AgentMode = "primary" | "subagent" | "all";
+
+/**
+ * Agent factory function with static mode property.
+ * Mode is exposed as static property for pre-instantiation access.
+ */
+export type AgentFactory = ((model: string) => AgentConfig) & {
+  mode: AgentMode;
+};
+
+/**
+ * Agent category for grouping in Bob prompt sections
+ */
+export type AgentCategory =
+  | "exploration"
+  | "specialist"
+  | "advisor"
+  | "utility";
+
+/**
+ * Cost classification for Tool Selection table
+ */
+export type AgentCost = "FREE" | "CHEAP" | "EXPENSIVE";
+
+/**
+ * Delegation trigger for Bob prompt's Delegation Table
+ */
+export interface DelegationTrigger {
+  /** Domain of work (e.g., "Frontend UI/UX") */
+  domain: string;
+  /** When to delegate (e.g., "Visual changes only...") */
+  trigger: string;
+}
+
+/**
+ * Metadata for generating Bob prompt sections dynamically
+ * This allows adding/removing agents without manually updating the Bob prompt
+ */
+export interface AgentPromptMetadata {
+  /** Category for grouping in prompt sections */
+  category: AgentCategory;
+
+  /** Cost classification for Tool Selection table */
+  cost: AgentCost;
+
+  /** Domain triggers for Delegation Table */
+  triggers: DelegationTrigger[];
+
+  /** When to use this agent (for detailed sections) */
+  useWhen?: string[];
+
+  /** When NOT to use this agent */
+  avoidWhen?: string[];
+
+  /** Optional dedicated prompt section (markdown) - for agents like Logician that have special sections */
+  dedicatedSection?: string;
+
+  /** Nickname/alias used in prompt (e.g., "Logician" instead of "logician") */
+  promptAlias?: string;
+
+  /** Key triggers that should appear in Phase 0 (e.g., "External library mentioned → fire librarian") */
+  keyTrigger?: string;
+}
+
+function extractModelName(model: string): string {
+  return model.includes("/") ? (model.split("/").pop() ?? model) : model;
+}
+
+export function isGptModel(model: string): boolean {
+  const modelName = extractModelName(model).toLowerCase();
+  return modelName.includes("gpt");
+}
+
+export function isGptProModel(model: string): boolean {
+  const modelName = extractModelName(model).toLowerCase();
+  return modelName.includes("gpt") && (modelName.includes("pro") || modelName.includes("-4") || modelName.includes("-5"));
+}
+
+export function isGptCodexModel(model: string): boolean {
+  const modelName = extractModelName(model).toLowerCase();
+  return modelName.includes("codex");
+}
+
+const GEMINI_PROVIDERS = ["google/", "google-vertex/"];
+
+export function isMiniMaxModel(model: string): boolean {
+  const modelName = extractModelName(model).toLowerCase();
+  return modelName.includes("minimax");
+}
+
+export function isGlmModel(model: string): boolean {
+  const modelName = extractModelName(model).toLowerCase();
+  return modelName.includes("glm");
+}
+
+export function isGeminiModel(model: string): boolean {
+  if (GEMINI_PROVIDERS.some((prefix) => model.startsWith(prefix))) return true;
+
+  if (
+    model.startsWith("github-copilot/") &&
+    extractModelName(model).toLowerCase().startsWith("gemini")
+  )
+    return true;
+
+  const modelName = extractModelName(model).toLowerCase();
+  return modelName.startsWith("gemini-");
+}
+
+export type BuiltinAgentName =
+  | "bob"
+  | "coder"
+  | "strategist"
+  | "critic"
+  | "designer"
+  | "multimodal"
+  | "brainstormer"
+  | "agent-skills"
+  | "guard"
+  | "platform-manager"
+  | "quality-guardian"
+  | "researcher"
+  | "sub";
+
+export type OverridableAgentName = "build" | BuiltinAgentName;
+
+export type AgentName = BuiltinAgentName;
+
+export type AgentOverrideConfig = Partial<AgentConfig> & {
+  prompt_append?: string;
+  variant?: string;
+  fallback_models?: string | (string | import("../config/schema/fallback-models").FallbackModelObject)[];
+};
+
+export type AgentOverrides = Partial<
+  Record<OverridableAgentName, AgentOverrideConfig>
+>;