muonroi-cli 1.4.0 → 1.5.0

package/src/pil/layer5-context.ts

@@ -1,171 +0,0 @@
-import { promises as fs } from "node:fs";
-import * as path from "node:path";
-import { searchByText } from "../ee/bridge.js";
-import { truncateToBudget } from "./budget.js";
-import type { PipelineContext } from "./types.js";
-
-// TODO(WhoAmI-L5): when EE v4.0 Who Am I profile is available, replace the
-// hardcoded threshold with profile-derived value:
-//   work_patterns.session_length="medium-long (1-3h)" → 120 min stale threshold
-//   work_patterns.multitasking="sequential-deep"      → extend to 2h
-//   (task-switchers with short sessions → keep 15 min)
-// Also: deduplicate with L3 — if L3 already injected experience-principles,
-// skip that collection here to avoid double-injection.
-const STALE_THRESHOLD_MS = 30 * 60 * 1000;
-
-// Single round-trip for both collections via /api/search; topK=5 yields
-// up to 2 principles + 3 behavioral after server-side scoring.
-const LAYER5_SEARCH_TIMEOUT_MS = 1500;
-
-// Score floor — points below this are dropped as noise. Same default and
-// override knob as Layer 3 to keep behaviour consistent across PIL.
-const LAYER5_SCORE_FLOOR = (() => {
-  const raw = Number(process.env.MUONROI_PIL_SCORE_FLOOR);
-  return Number.isFinite(raw) && raw >= 0 && raw <= 1 ? raw : 0.55;
-})();
-
-async function fetchPrinciples(raw: string, budget: number): Promise<string> {
-  try {
-    const points = await searchByText(
-      raw,
-      ["experience-principles", "experience-behavioral"],
-      5,
-      AbortSignal.timeout(LAYER5_SEARCH_TIMEOUT_MS),
-    );
-    const filtered = points.filter((p) => (p.score ?? 0) >= LAYER5_SCORE_FLOOR);
-    if (filtered.length === 0) return "";
-    const sorted = filtered.slice().sort((a, b) => (b.score ?? 0) - (a.score ?? 0));
-    const lines = sorted.map((p) => {
-      const payload = p.payload as { text?: string; json?: string } | undefined;
-      const text = payload?.text ?? (payload?.json ? (JSON.parse(payload.json) as { solution?: string }).solution : "");
-      return `- ${String(text ?? "").slice(0, 120)}`;
-    });
-    return truncateToBudget(`[principles: Always-loaded experience]\n${lines.join("\n")}`, budget);
-  } catch {
-    return "";
-  }
-}
-
-async function fetchFlowState(cwd: string, budget: number): Promise<string> {
-  try {
-    const activeRunPath = path.join(cwd, ".muonroi-flow", "active-run");
-    const runId = (await fs.readFile(activeRunPath, "utf8")).trim();
-    if (!runId) return "";
-    const statePath = path.join(cwd, ".muonroi-flow", "runs", runId, "state.md");
-    const state = await fs.readFile(statePath, "utf8");
-    const phaseLine = state.match(/phase:\s*(.+)/i)?.[1]?.trim() ?? "unknown";
-    const statusLine = state.match(/status:\s*(.+)/i)?.[1]?.trim() ?? "unknown";
-    return truncateToBudget(`[flow: phase=${phaseLine}, status=${statusLine}, run=${runId}]`, budget);
-  } catch {
-    return "";
-  }
-}
-
-async function fetchRecentFiles(cwd: string, budget: number): Promise<string> {
-  try {
-    const srcDir = path.join(cwd, "src");
-    const entries = await fs.readdir(srcDir, { recursive: true, withFileTypes: true });
-    const tsFiles: Array<{ name: string; mtime: number }> = [];
-    for (const e of entries) {
-      if (!e.isFile()) continue;
-      if (!e.name.endsWith(".ts") && !e.name.endsWith(".tsx")) continue;
-      if (e.name.includes(".test.") || e.name.includes("__test")) continue;
-      const full = path.join(e.parentPath ?? (e as unknown as { path: string }).path, e.name);
-      const stat = await fs.stat(full);
-      tsFiles.push({ name: path.relative(srcDir, full).replace(/\\/g, "/"), mtime: stat.mtimeMs });
-    }
-    tsFiles.sort((a, b) => b.mtime - a.mtime);
-    const top = tsFiles.slice(0, 10).map((f) => f.name);
-    if (top.length === 0) return "";
-    return truncateToBudget(`[recent-files: ${top.join(", ")}]`, budget);
-  } catch {
-    return "";
-  }
-}
-
-export async function layer5Context(ctx: PipelineContext): Promise<PipelineContext> {
-  // Chitchat short-circuit: greetings/small-talk don't need workspace context
-  // (recent-files, flow-state, principles). Inject nothing — keep the model's
-  // attention on a 5-character prompt instead of a 30K-token tool catalog.
-  if (ctx.intentKind === "chitchat") {
-    return {
-      ...ctx,
-      layers: [...ctx.layers, { name: "context-enrichment", applied: false, delta: "skip:chitchat" }],
-    };
-  }
-
-  const cwd = process.cwd();
-  const parts: string[] = [];
-  const deltaSegments: string[] = [];
-
-  // 1. T0/T1 principles — skip when L1's unified call already supplied them
-  //    (L3 already rendered the principles block from ctx._brainData).
-  if (!ctx._brainData) {
-    const principlesBudget = Math.floor(ctx.tokenBudget * 0.12);
-    const principles = await fetchPrinciples(ctx.raw, principlesBudget);
-    if (principles) {
-      parts.push(principles);
-      deltaSegments.push(`principles=${principles.length}ch`);
-    }
-  } else {
-    deltaSegments.push("principles=skipped-l1-unified");
-  }
-
-  // 2. Resume digest (5% budget)
-  const digest = ctx.resumeDigest;
-  if (digest?.trim()) {
-    const isStale = typeof ctx.digestAgeMs === "number" && ctx.digestAgeMs > STALE_THRESHOLD_MS;
-    const stalePrefix = isStale ? "(stale — verify before relying)\n" : "";
-    const hint = `[flow-context: Resume]\n${stalePrefix}${digest.trim()}`;
-    const trimmed = truncateToBudget(hint, Math.floor(ctx.tokenBudget * 0.05));
-    parts.push(trimmed);
-    deltaSegments.push(`digest=${trimmed.length}ch${isStale ? ",stale" : ""}`);
-  }
-
-  // 3. Flow state (5% budget)
-  const flowBudget = Math.floor(ctx.tokenBudget * 0.05);
-  const flowState = await fetchFlowState(cwd, flowBudget);
-  if (flowState) {
-    parts.push(flowState);
-    deltaSegments.push(`flow=${flowState.length}ch`);
-  }
-
-  // 4. Recent file index (3% budget)
-  // PIL-L6 fix — skip recent-files for operational tasks (CI/build/deploy).
-  // Session bca83bcbaad1 logged the inject pulling agent into reading
-  // providers/runtime.ts + mock-model.ts when the real task was just
-  // "gh run view --log-failed". Recent-files biases narrow-scope tasks.
-  const skipRecentFiles = /\b(ci|cd|build|deploy(?:ment)?|action(?:s)?|workflow|pipeline|gh\s+(check|run))\b/i.test(
-    ctx.raw,
-  );
-  if (!skipRecentFiles) {
-    const filesBudget = Math.floor(ctx.tokenBudget * 0.03);
-    const fileIndex = await fetchRecentFiles(cwd, filesBudget);
-    if (fileIndex) {
-      parts.push(fileIndex);
-      deltaSegments.push(`files=${fileIndex.length}ch`);
-    }
-  } else {
-    deltaSegments.push("files=skipped-operational");
-  }
-
-  if (parts.length === 0) {
-    return {
-      ...ctx,
-      layers: [...ctx.layers, { name: "context-enrichment", applied: false, delta: "no-context-sources" }],
-    };
-  }
-
-  return {
-    ...ctx,
-    enriched: `${ctx.enriched}\n${parts.join("\n")}`,
-    layers: [
-      ...ctx.layers,
-      {
-        name: "context-enrichment",
-        applied: true,
-        delta: deltaSegments.join(" "),
-      },
-    ],
-  };
-}

package/src/pil/layer6-output.ts

@@ -1,364 +0,0 @@
-/**
- * src/pil/layer6-output.ts
- *
- * Layer 6: Output optimization.
- * Appends a per-task-type system prompt suffix via applyPilSuffix().
- * Each suffix is tuned to minimize output tokens while preserving quality.
- * Conversational turns (taskType=null) pass through unchanged.
- *
- * PIL-03: When ctx.outputStyle is null and ctx.taskType is not null:
- *   a) 50ms brain rescue — classifyViaBrain for multilingual style detection
- *   b) task-type heuristic — domain-specific default when brain is unavailable
- * Resolved style is propagated back onto ctx.outputStyle so the orchestrator
- * picks it up via applyPilSuffix without falling back to the hard "concise" default.
- */
-
-import type { ToolSet } from "ai";
-import { classifyViaBrain } from "../ee/bridge.js";
-import { getProviderCapabilities } from "../providers/capabilities.js";
-import type { ProviderId } from "../providers/types.js";
-import { buildResponseTools } from "./response-tools.js";
-import type { OutputStyle, PipelineContext, TaskType } from "./types.js";
-
-const VALID_STYLES = ["concise", "balanced", "detailed"] as const;
-
-// Per-task-type fallback style when brain is unavailable.
-//
-// PIL-L6 verbosity fix — debug/analyze flipped from "balanced" → "concise".
-// Users complained about end-of-turn summaries and rambling debug responses.
-// "balanced" pads with rationale prose the user can read from the diff/code
-// already. "detailed" is reserved for explicit user request ("explain").
-
-// Broad detector for self-referential / meta / evaluation prompts about the CLI,
-// prior turns, or the agent's own behavior. Used to relax brevity rules so full
-// answers (with evidence, bullets, file:line) are not suppressed by low budgets
-// or NO_PREAMBLE. No model/provider hardcodes — only prompt content signals.
-// Exported for early detection in pipeline / orchestrator.
-export function isMetaAnalysisPrompt(raw: string): boolean {
-  return /đánh giá|phân tích|cải thiện|fix|debug|nhận xét|đánh giá tổng thể|evaluate.*(cli|system|repo)|improve.*(cli|repo)|your assessment|how would you improve|trả lời tự nhiên|natural response|sau fix|phỏng vấn|discovery|native|agent.*inside|cli.*bên trong|context|previous turn|input.*vừa rồi|mù context/i.test(
-    raw,
-  );
-}
-const TASK_TYPE_DEFAULT_STYLE: Record<TaskType, OutputStyle> = {
-  debug: "concise", // root cause + fix; no padding
-  plan: "balanced", // plans genuinely need a brief rationale per step
-  analyze: "concise", // bullet findings, no narrative
-  documentation: "balanced", // examples + explanation
-  generate: "concise", // code speaks for itself
-  refactor: "concise", // diff is the output
-  general: "concise", // direct answer, no preamble
-};
-
-// PIL-04 Tier 1.1: response-tool gating.
-//
-// Structural enforcement (JSON schema via tool call) is the root-cause fix for
-// cheap-model verbosity: the model CANNOT add preamble/epilogue/padding because
-// there is no surface outside the schema to write into. Text-based OUTPUT RULES
-// rely on instruction compliance which budget models (DeepSeek/Qwen/Llama)
-// frequently ignore.
-//
-// Activation gate: enable when the schema's payload is bounded enough that JSON
-// escaping (\n → \\n, " → \") doesn't outweigh the eliminated padding cost.
-//   - analyze: list of findings — repeated keys, no large strings → big win
-//   - plan:    numbered steps    — same shape → big win
-//   - debug:   {hypothesis, root_cause, fix:{file,diff}, verify} — diff is
-//              normally one small hunk, escaping overhead <5%; structural
-//              enforcement of the hypothesis→root_cause→fix→verify format is
-//              exactly what cheap models drift away from in prose mode
-//   - general: {response, reasoning} — pure text, zero structural overhead;
-//              this task type has the weakest text suffix today and suffers the
-//              most from verbose padding
-//
-// Disabled at task-type level (escaping cost dominates):
-//   - generate (large file contents in JSON strings)
-//   - refactor (multi-file diffs)
-//   - documentation (large markdown blocks)
-// For these, the orchestrator falls back to the markdown OUTPUT RULES suffix.
-// Per-PROVIDER disabling is layered on top via ProviderCapabilities — see
-// `src/providers/capabilities.ts`. A task type listed here may still be
-// dropped at runtime if the active provider's capability returns false.
-const RESPONSE_TOOL_TASK_TYPES = new Set<TaskType>(["analyze", "plan", "debug", "general"]);
-
-// PIL-04 Tier 1.2: per-task output token budget.
-// Hint to model. Empirically derived from interaction_logs avg output sizes;
-// ~20% below mean to nudge brevity without truncating real answers.
-const TASK_OUTPUT_BUDGET: Record<TaskType, number> = {
-  refactor: 800,
-  debug: 500,
-  plan: 700,
-  analyze: 600,
-  documentation: 900,
-  generate: 1200,
-  // general is user-facing prose (not a code artifact). Higher budget + relaxed
-  // style rules so the final answer reads naturally for humans instead of
-  // machine-optimized telegraphic lists. See user report on over-constrained
-  // freetext after Layer 6.
-  general: 650,
-};
-
-// PIL-04 Tier 1.3 + PIL-L6 verbosity fix: ban preamble AND end-of-turn summary.
-// Old rule only covered openers (~30 tok saved). End-of-turn summaries
-// ("In summary...", "I have completed X, Y, Z", "Tóm tắt: ...") cost
-// 100-300 tokens/turn AND give the user nothing they can't read from the
-// diff. Bilingual EN+VN. Skipped for response-tools path (JSON has no
-// freeform surface).
-const NO_PREAMBLE_RULE = `\nFORBIDDEN OPENERS: do not start with "I'll", "I will", "Let me", "Here's", "Sure", "Of course", "Tôi sẽ", "Để tôi", "Vâng". Start directly with the answer content.\nFORBIDDEN END-OF-TURN SUMMARY: do not append a recap section ("In summary", "To summarize", "Tổng kết", "Tóm tắt", "Tóm lại", "Kết luận", "I have done X, Y, Z", "Now you have…", "Đã hoàn thành…"). The diff and command output already show what changed; the user can read them. End the response when the answer is complete.\nFORBIDDEN INTER-TOOL NARRATION: when chaining tool calls, do NOT emit content text between them. Skip phrases like "Now I'll check…", "Let me look at…", "Next, I need to…", "Tiếp theo tôi sẽ…", "Bây giờ tôi cần…". Emit the next tool call directly. Each round-trip of inter-tool narration costs the user ~100 output tokens that they do not need to read — the tool calls themselves are visible in the UI. Only emit content text for the FINAL answer or when surfacing a decision the user must make.`;
-
-const SUFFIXES: Record<string, Record<OutputStyle, string>> = {
-  refactor: {
-    concise: `\nOUTPUT RULES (refactor): Show only changed code. Prefer unified diff or replacement function. No prose unless architecture changes. One sentence max if explanation needed. No preamble.`,
-    balanced: `\nOUTPUT RULES (refactor): Show changed code with brief rationale. Unified diff preferred. Short explanation allowed when architecture changes. Keep prose under 3 sentences.`,
-    detailed: `\nOUTPUT RULES (refactor): Show changed code with full rationale. Explain why each change improves the code. Include before/after comparison when helpful. Unified diff preferred.`,
-  },
-  debug: {
-    concise: `\nOUTPUT RULES (debug): Format = Hypothesis → Root cause (1 line) → Fix (code only) → Verify command. No preamble. No "I think" hedging.`,
-    balanced: `\nOUTPUT RULES (debug): Format = Hypothesis → Root cause → Fix (code) → Verify command. Brief explanation of why the bug occurs. Keep prose minimal.`,
-    detailed: `\nOUTPUT RULES (debug): Format = Hypothesis → Root cause analysis → Fix (code) → Verify command → Prevention. Explain the underlying mechanism and why this fix is correct.`,
-  },
-  plan: {
-    concise: `\nOUTPUT RULES (plan): Numbered steps only. Each step: action verb + acceptance criterion. No prose paragraphs. Add "Assumptions:" section only if needed.`,
-    balanced: `\nOUTPUT RULES (plan): Numbered steps with brief rationale per step. Each step: action verb + acceptance criterion + why. Add "Assumptions:" and "Risks:" sections if applicable.`,
-    detailed: `\nOUTPUT RULES (plan): Numbered steps with full rationale. Each step: action verb + acceptance criterion + why + alternatives considered. Include "Assumptions:", "Risks:", and "Trade-offs:" sections.`,
-  },
-  analyze: {
-    concise: `\nOUTPUT RULES (analyze): Bullet findings with evidence (file:line or direct quote). Add severity label (High/Med/Low) when applicable. No filler sentences.`,
-    balanced: `\nOUTPUT RULES (analyze): Bullet findings with evidence (file:line or direct quote). Add severity label and brief explanation. Context for each finding.`,
-    detailed: `\nOUTPUT RULES (analyze): Bullet findings with evidence (file:line or direct quote). Add severity label, root cause analysis, and recommended action. Provide context and impact assessment.`,
-  },
-  documentation: {
-    concise: `\nOUTPUT RULES (documentation): Markdown only. Lead with a code example, then explanation. No "This function..." openers. All examples in fenced code blocks.`,
-    balanced: `\nOUTPUT RULES (documentation): Markdown only. Lead with a code example, then explanation with context. Cover common use cases. All examples in fenced code blocks.`,
-    detailed: `\nOUTPUT RULES (documentation): Markdown only. Lead with a code example, then thorough explanation. Cover use cases, edge cases, and gotchas. All examples in fenced code blocks. Include parameter descriptions and return value explanations.`,
-  },
-  generate: {
-    concise: `\nOUTPUT RULES (generate): Complete, runnable code only. Include all imports. Brief inline comments for non-obvious logic only. No prose outside code blocks. No partial snippets.`,
-    balanced: `\nOUTPUT RULES (generate): Complete, runnable code with brief explanation. Include all imports. Inline comments for key decisions. Short prose before code block explaining approach.`,
-    detailed: `\nOUTPUT RULES (generate): Complete, runnable code with full explanation. Include all imports. Inline comments for logic and decisions. Explain design choices, alternatives considered, and trade-offs before the code.`,
-  },
-  general: {
-    // General answers should be highly readable. Encourage rich markdown
-    // (bullets, headings, bold text) instead of forcing dense prose.
-    concise: `\nAnswer directly. Use markdown, bullet points, and code blocks to make the output highly readable and scannable. Avoid dense paragraphs.`,
-    balanced: `\nAnswer with helpful context. Structure your response using markdown headings, bullet points, and code blocks for excellent readability. Avoid dense walls of text.`,
-    detailed: `\nAnswer thoroughly. Use rich markdown structure (headings, lists, bold text, code blocks) to organize complex information so it is easy to scan and read.`,
-  },
-};
-
-// TODO(WhoAmI-L6): when EE v4.0 Who Am I profile is available, skip
-// NO_PREAMBLE_RULE for users with feedback_style="explicit" who prefer
-// preamble. Also source TASK_TYPE_DEFAULT_STYLE from profile communication.brevity
-// instead of the hardcoded heuristic map above.
-
-export function applyPilSuffix(systemPrompt: string, ctx: PipelineContext, responseToolsActive = false): string {
-  // Chitchat: layer6Output already skipped suffix work; mirror that here so
-  // direct callers (e.g. orchestrator) don't accidentally re-inject rules.
-  if (ctx.intentKind === "chitchat") return systemPrompt;
-  if (!ctx.taskType || !SUFFIXES[ctx.taskType]) return systemPrompt;
-
-  if (responseToolsActive) {
-    const isMeta = isMetaAnalysisPrompt(ctx.raw);
-    const metaNote = isMeta
-      ? " This is a meta/evaluation question about the system or prior turns — the `response` field MUST contain the complete, unshortened answer with all evidence and detail."
-      : "";
-    const finalAnswerNote =
-      ctx.taskType === "general"
-        ? " Structure your `response` with rich markdown formatting (headings, bullet points, bold text, code blocks). Make it highly readable, scannable, and clearly organized. Avoid dense walls of text (freetext)."
-        : "";
-    // Human-UX guard for question/meta turns. Without it the model narrates its
-    // OWN process into the user-facing answer — a "2-3 line plan" preamble, "per
-    // contract 2/5/7", "emit respond_general", "all facts come only from
-    // this-turn reads" (session 829a83888dd2). The reader is a human who asked a
-    // question, not an agent auditing compliance.
-    const humanNote =
-      isMeta || ctx.taskType === "general"
-        ? " Write the `response` for the HUMAN who asked: lead with the answer. Do NOT include an implementation plan, do NOT narrate your own process, and do NOT restate internal rule/tool/layer names (contract rules, respond_* , layer6, native:NN) as compliance — cite a file:line only where it directly backs a claim. Do NOT append an evidence-provenance footer or a disclaimer that your facts come only from this turn / that you did not infer unopened files — that is internal contract bookkeeping, invisible to the reader: end on the answer's last substantive point."
-        : "";
-    return (
-      systemPrompt +
-      `\nOUTPUT FORMAT: When you finish your work, use the respond_${ctx.taskType} tool to structure your final answer. You may write free-form text to explain your reasoning during the process. Use action tools (bash, read_file, edit_file, etc.) as needed, then deliver the COMPLETE, FULL answer (do not summarize, shorten, or truncate for token budgets) via respond_${ctx.taskType}.${metaNote}${finalAnswerNote}${humanNote}`
-    );
-  }
-
-  // Action tasks (debug/refactor/generate) override "detailed" → "concise" UNLESS
-  // the prompt literally requested detail. Brain/LLM classifiers sometimes
-  // return style=detailed for prompts like "fix CI fail" (Vietnamese: "fix lỗi
-  // CI") just because the prompt looks ambiguous in isolation — session
-  // 127140a47b56 hit this and the model spent 275 LLM calls being "thorough"
-  // about a one-liner CI fix.
-  const ACTION_TASKS = new Set<TaskType>(["debug", "refactor", "generate"]);
-  const DETAIL_KEYWORDS =
-    /\b(explain in detail|thorough analysis|walk me through|in depth|deeply|comprehensive)\b|giải thích chi tiết|phân tích kỹ|cặn kẽ|chi tiết hơn/i;
-  const requestedStyle: OutputStyle = ctx.outputStyle ?? "concise";
-  const style: OutputStyle =
-    requestedStyle === "detailed" && ACTION_TASKS.has(ctx.taskType) && !DETAIL_KEYWORDS.test(ctx.raw)
-      ? "concise"
-      : requestedStyle;
-  const baseSuffix = SUFFIXES[ctx.taskType][style];
-
-  // PIL-04 Tier 1.2: output-budget hint.
-  // For response-tool turns or meta-analysis (self-eval of CLI, prior context, "mù context" etc.),
-  // raise budget and skip aggressive "do not pad / no summary" so full evidence-rich answers
-  // reach the respond_* payload and the user. Before: meta follow-ups often produced only
-  // short "**Đã trả lời xong**" confirmations even when detailed analysis was prepared in the tool.
-  // After: explicit "COMPLETE, FULL" instruction + higher budget + isMetaAnalysisPrompt gate.
-  const isMetaAnalysis = isMetaAnalysisPrompt(ctx.raw);
-  const useHighBudget = responseToolsActive || isMetaAnalysis;
-  const budget = useHighBudget ? 1800 : (TASK_OUTPUT_BUDGET[ctx.taskType as TaskType] ?? 600);
-  const budgetHint = useHighBudget
-    ? `\nOUTPUT BUDGET: provide the complete answer required by the task (analysis/meta may legitimately need 800-1500+ tokens for evidence and bullets). Stop only when the full user-visible content is delivered; do not artificially shorten.`
-    : `\nOUTPUT BUDGET: aim for ≤${budget} tokens. Stop when the answer is complete; do not pad.`;
-
-  // PIL-04 Tier 1.3: ban preamble (~30 tokens saved/turn).
-  // Relax for meta-analysis / evaluation (including follow-ups about previous turns or CLI behavior).
-  // The isMetaAnalysisPrompt (hoisted early) is the single source of truth and already includes
-  // signals like "cli.*bên trong", "mù context", "input.*vừa rồi", "previous turn".
-  const effectiveStyle =
-    isMetaAnalysis && (ctx.taskType === "general" || ctx.taskType === "analyze") ? "balanced" : style;
-  const effectiveSuffix = SUFFIXES[ctx.taskType]?.[effectiveStyle] || baseSuffix;
-
-  let result = systemPrompt + effectiveSuffix + budgetHint;
-  if (!isMetaAnalysis && !responseToolsActive) {
-    result += NO_PREAMBLE_RULE;
-  }
-
-  // T1 behavioral rules (proven-tier EE points set by Layer 3). These are
-  // project-specific reflexes the model MUST follow — injected as instructions,
-  // not as context hints, so they carry imperative weight rather than suggestion weight.
-  if (ctx.t1Rules && ctx.t1Rules.length > 0) {
-    const mandatoryLines = ctx.t1Rules.map((r) => `- ${r}`).join("\n");
-    result += `\nMANDATORY RULES (from experience — must follow):\n${mandatoryLines}`;
-  }
-
-  return result;
-}
-
-/**
- * Detect an explicit IMPLEMENTATION / edit-the-files intent. When the user asks
- * to implement/edit/refactor (the deliverable is file changes, not a structured
- * report), a terminal `respond_<task>` tool is inappropriate: the model can call
- * it mid-task to "state a plan/answer" and the turn then winds down before the
- * edits are finished. Live (grok session 19fa8895c41c): an "Improve … implement
- * these fixes" prompt was classified `debug`, so `respond_debug` was offered; the
- * model called it after reading the files, made a few edits, then stopped — the
- * HTML was never wired. Dropping the tool just falls through to the markdown
- * OUTPUT RULES (graceful — exactly what code-heavy tasks already do), so a false
- * positive on an analysis turn only forgoes structured JSON, never breaks output.
- *
- * High-signal verbs only (implement/edit/wire/rewrite/rename/scaffold/refactor,
- * "make the change", "apply the fix/patch", VI equivalents). Bare "fix"/"replace"
- * are excluded — too common in analysis ("explain the fix") — so pure
- * analyze/plan/debug-investigation turns keep their structured output.
- */
-const IMPLEMENTATION_INTENT_RE =
-  /\b(implement|edit|wire(?:\s+up)?|rewrite|rename|scaffold|refactor)\b|\bmake\s+(the\s+)?(change|edit|modification)s?\b|\bapply\s+(the\s+)?(fix|change|patch|edit|diff)\b|(?:^|\s)(triển\s*khai|trien\s*khai|chỉnh\s*sửa|chinh\s*sua|viết\s*lại|viet\s*lai|đổi\s*tên|doi\s*ten)\b/i;
-
-export function isImplementationIntent(raw: string): boolean {
-  return !!raw && IMPLEMENTATION_INTENT_RE.test(raw);
-}
-
-export function getResponseToolSet(ctx: PipelineContext, providerId?: ProviderId): ToolSet {
-  if (!ctx.taskType) return {};
-  // PIL-04 Tier 1.1: gate JSON-structured output to list-shaped tasks where it
-  // wins on tokens. Code-heavy tasks fall through to markdown OUTPUT RULES.
-  if (!RESPONSE_TOOL_TASK_TYPES.has(ctx.taskType)) return {};
-  // Implementation/edit turns: the deliverable is file changes, not a structured
-  // report. A terminal respond_<task> tool lets the model "answer" (state a plan)
-  // and end the turn before the edits complete — drop it for clear edit intent.
-  if (isImplementationIntent(ctx.raw)) return {};
-  // Provider-aware gating: a provider may report it can't reliably emit
-  // valid JSON tool input for this task type (e.g. DeepSeek leaks special
-  // tokens into `general` responses). Drop the tool to avoid retry storms.
-  if (providerId) {
-    const caps = getProviderCapabilities(providerId);
-    if (!caps.supportsResponseTool(ctx.taskType)) return {};
-  }
-  return buildResponseTools(ctx.taskType);
-}
-
-export async function layer6Output(ctx: PipelineContext): Promise<PipelineContext> {
-  try {
-    // Chitchat short-circuit: greetings/small-talk don't need OUTPUT RULES
-    // suffixes — the model already produces a short reply for "hi" without
-    // being told to. Aligns with L4/L5 chitchat skips so the fast-path stays
-    // clean end-to-end (no GSD, no context, no suffix, no MCP).
-    if (ctx.intentKind === "chitchat") {
-      return {
-        ...ctx,
-        layers: [...ctx.layers, { name: "output-optimization", applied: false, delta: "skip:chitchat" }],
-      };
-    }
-
-    if (ctx.taskType === null) {
-      return {
-        ...ctx,
-        layers: [...ctx.layers, { name: "output-optimization", applied: false, delta: null }],
-      };
-    }
-
-    // PIL-03: rescue outputStyle when L1 couldn't determine it (brain timeout / EE disabled).
-    let outputStyle = ctx.outputStyle;
-    let styleSource = "inherited";
-
-    if (outputStyle === null) {
-      // Pass a: 50ms brain rescue — catches multilingual prompts that regex/L1 missed.
-      // Skip when ctx._brainData is already populated: L1's unified brain call had its
-      // chance to set style; a second classifyViaBrain here is redundant work.
-      if (!ctx._brainData) {
-        try {
-          const brainRaw = await classifyViaBrain(
-            `Task type: ${ctx.taskType}. Reply ONE word only: concise | balanced | detailed\nPrompt: "${ctx.raw.slice(0, 150)}"`,
-            50,
-          );
-          if (brainRaw) {
-            const matched = VALID_STYLES.find((s) => brainRaw.toLowerCase().includes(s));
-            if (matched) {
-              outputStyle = matched;
-              styleSource = "brain-rescue";
-            }
-          }
-        } catch {
-          // fall through to pass b
-        }
-      }
-
-      // Pass b: task-type heuristic — fires when brain is also unavailable
-      if (outputStyle === null) {
-        outputStyle = TASK_TYPE_DEFAULT_STYLE[ctx.taskType as TaskType] ?? "concise";
-        styleSource = "task-heuristic";
-      }
-    }
-
-    const style: OutputStyle = outputStyle;
-    const taskKey = ctx.taskType as string;
-    const suffixEntry = SUFFIXES[taskKey];
-    if (!suffixEntry) {
-      return {
-        ...ctx,
-        outputStyle,
-        layers: [...ctx.layers, { name: "output-optimization", applied: false, delta: "no-suffix-entry" }],
-      };
-    }
-
-    const suffix = suffixEntry[style];
-    return {
-      ...ctx,
-      outputStyle, // propagate resolved style to orchestrator's applyPilSuffix
-      layers: [
-        ...ctx.layers,
-        {
-          name: "output-optimization",
-          applied: true,
-          delta: `suffix=${ctx.taskType},style=${style},src=${styleSource},chars=${suffix.trim().length}`,
-        },
-      ],
-    };
-  } catch {
-    return {
-      ...ctx,
-      layers: [...ctx.layers, { name: "output-optimization", applied: false, delta: null }],
-    };
-  }
-}
-
-// isMetaAnalysisPrompt is defined early (near top) and exported for use by
-// pipeline, orchestrator, and other layers to relax rules for reflective turns.