@d3ara1n/pi-scout 1.0.2 → 1.1.1

package/src/skill-inject.ts

@@ -10,14 +10,15 @@
  */
 
 /** Match pi's entire skills section: intro paragraph + XML block. */
-const SKILLS_SECTION_RE = /\n\nThe following skills provide specialized instructions[\s\S]*?<\/available_skills>/;
+const SKILLS_SECTION_RE =
+  /\n\nThe following skills provide specialized instructions[\s\S]*?<\/available_skills>/;
 
 /** Track skill names already shown to the LLM in this session. */
 let shownSkills: Set<string> = new Set();
 
 /** Reset the cache — called on session_start. */
 export function resetSkillCache(): void {
-	shownSkills = new Set();
+  shownSkills = new Set();
 }
 
 /**
@@ -33,51 +34,53 @@ export function resetSkillCache(): void {
  * @returns Modified system prompt
  */
 export function filterSkillsBlock(
-	systemPrompt: string,
-	selectedSkills: string[],
-	allSkills: Array<{ name: string; description: string; filePath: string }>,
+  systemPrompt: string,
+  selectedSkills: string[],
+  allSkills: Array<{ name: string; description: string; filePath: string }>,
 ): string {
-	if (selectedSkills.length === 0) {
-		return systemPrompt.replace(SKILLS_SECTION_RE, "");
-	}
+  if (selectedSkills.length === 0) {
+    return systemPrompt.replace(SKILLS_SECTION_RE, "");
+  }
 
-	const skillMap = new Map(allSkills.map((s) => [s.name, s]));
-	const entries: string[] = [];
-	const newlyShown: string[] = [];
+  const skillMap = new Map(allSkills.map((s) => [s.name, s]));
+  const entries: string[] = [];
+  const newlyShown: string[] = [];
 
-	for (const name of selectedSkills) {
-		const skill = skillMap.get(name);
-		if (!skill) continue;
+  for (const name of selectedSkills) {
+    const skill = skillMap.get(name);
+    if (!skill) continue;
 
-		if (shownSkills.has(name)) {
-			// Already introduced — compact form
-			entries.push(`  <skill name="${esc(skill.name)}" location="${esc(skill.filePath)}" />`);
-		} else {
-			// First time — include description
-			entries.push(`  <skill name="${esc(skill.name)}" location="${esc(skill.filePath)}">${esc(skill.description)}</skill>`);
-			newlyShown.push(name);
-		}
-	}
+    if (shownSkills.has(name)) {
+      // Already introduced — compact form
+      entries.push(`  <skill name="${esc(skill.name)}" location="${esc(skill.filePath)}" />`);
+    } else {
+      // First time — include description
+      entries.push(
+        `  <skill name="${esc(skill.name)}" location="${esc(skill.filePath)}">${esc(skill.description)}</skill>`,
+      );
+      newlyShown.push(name);
+    }
+  }
 
-	if (entries.length === 0) {
-		return systemPrompt.replace(SKILLS_SECTION_RE, "");
-	}
+  if (entries.length === 0) {
+    return systemPrompt.replace(SKILLS_SECTION_RE, "");
+  }
 
-	// Update cache
-	for (const name of newlyShown) {
-		shownSkills.add(name);
-	}
+  // Update cache
+  for (const name of newlyShown) {
+    shownSkills.add(name);
+  }
 
-	const compact = `\n\nActive skills (use \`read\` to load a skill's file):\n<available_skills>\n${entries.join("\n")}\n</available_skills>`;
+  const compact = `\n\nActive skills (use \`read\` to load a skill's file):\n<available_skills>\n${entries.join("\n")}\n</available_skills>`;
 
-	return systemPrompt.replace(SKILLS_SECTION_RE, compact);
+  return systemPrompt.replace(SKILLS_SECTION_RE, compact);
 }
 
 function esc(str: string): string {
-	return str
-		.replace(/&/g, "&amp;")
-		.replace(/</g, "&lt;")
-		.replace(/>/g, "&gt;")
-		.replace(/"/g, "&quot;")
-		.replace(/'/g, "&apos;");
+  return str
+    .replace(/&/g, "&amp;")
+    .replace(/</g, "&lt;")
+    .replace(/>/g, "&gt;")
+    .replace(/"/g, "&quot;")
+    .replace(/'/g, "&apos;");
 }

package/src/types.ts

@@ -14,10 +14,35 @@ export interface ScoutConfig {
   modules: {
     skillRouter: boolean;
     modelRouter: boolean;
+    /** Short-circuit layer: skip the side LLM on high-confidence prompts */
+    shortCircuit: boolean;
   };
+  /** Tuning for the short-circuit module */
+  shortCircuit: ShortCircuitConfig;
 }
 
-/** Decision returned by the side agent. */
+/**
+ * Tuning for the short-circuit module.
+ *
+ * The short-circuit layer lets scout skip the side model entirely on
+ * trivial acknowledgments ("好的" / "ok" / "はい"). A trivial ack means
+ * "no skills, don't switch models" — both module answers are certain, so
+ * skipping the side model is always safe, even with model-router on.
+ * Ambiguous prompts always fall through to the side model, so there is no
+ * quality loss on hard cases.
+ */
+export interface ShortCircuitConfig {
+  /** Enable the trivial-acknowledgment rule */
+  trivialAck: boolean;
+  /** Max prompt length (chars) for the trivial-ack rule. Longer prompts are
+   *  never short-circuited as acks even if they begin with an ack word. */
+  maxAckLength: number;
+  /** Additional ack phrases merged on top of the built-in 中/英/日/韓 table.
+   *  Provide raw strings; they are normalized (trimmed, lowercased) at runtime. */
+  ackPhrases: string[];
+}
+
+/** Decision returned by the side agent or the short-circuit layer. */
 export interface ScoutDecision {
   /** Selected skill names */
   skills: string[];
@@ -25,6 +50,8 @@ export interface ScoutDecision {
   role: string | null;
   /** Brief reasoning */
   reasoning: string;
+  /** Where the decision came from — controls status-bar presentation */
+  source?: "side-agent" | "short-circuit";
 }
 
 export const DEFAULT_CONFIG: ScoutConfig = {
@@ -34,5 +61,11 @@ export const DEFAULT_CONFIG: ScoutConfig = {
   modules: {
     skillRouter: true,
     modelRouter: false,
+    shortCircuit: true,
+  },
+  shortCircuit: {
+    trivialAck: true,
+    maxAckLength: 12,
+    ackPhrases: [],
   },
 };