@selesai/code 0.5.12 → 0.5.13

package/dist/extensions/caveman/caveman-instructions.cjs

@@ -1,39 +1,11 @@
 #!/usr/bin/env node
-// Shared Caveman instruction builder. Caveman is binary on/off (no intensity
-// levels), so the skill body is injected verbatim — no mode filtering needed.
-
-const fs = require('fs');
-const path = require('path');
-
-const SKILL_PATH = path.join(__dirname, '..', '..', 'skills', 'caveman', 'SKILL.md');
-
-const FALLBACK_INSTRUCTIONS =
-  'CAVEMAN MODE ACTIVE.\n\n' +
-  'Respond terse like smart caveman. All technical substance stay. Only fluff die.\n\n' +
-  '## Persistence\n\n' +
-  'ACTIVE EVERY RESPONSE once triggered. No revert after many turns. No filler drift. ' +
-  'Still active if unsure. Off only when user says "stop caveman" or "normal mode".\n\n' +
-  '## Rules\n\n' +
-  'Drop: articles (a/an/the), filler (just/really/basically/actually/simply), ' +
-  'pleasantries (sure/certainly/of course/happy to), hedging. Fragments OK. ' +
-  'Short synonyms. Abbreviate common terms (DB/auth/config/req/res/fn/impl). ' +
-  'Strip conjunctions. Use arrows for causality (X -> Y). One word when one word enough.\n\n' +
-  'Technical terms stay exact. Code blocks unchanged. Errors quoted exact.\n\n' +
-  '## Auto-Clarity Exception\n\n' +
-  'Drop caveman temporarily for: security warnings, irreversible action confirmations, ' +
-  'multi-step sequences where fragment order risks misread, user asks to clarify or ' +
-  'repeats question. Resume caveman after clear part done.';
-
-function stripFrontmatter(body) {
-  return String(body || '').replace(/^---[\s\S]*?---\s*/, '');
-}
+// The skill is user-facing documentation. This short runtime prompt is injected
+// on every turn, while extension state owns activation and deactivation.
+const CAVEMAN_INSTRUCTIONS =
+  'CAVEMAN MODE ACTIVE. Terse, technically complete: omit articles, filler, pleasantries, and hedging; fragments, short terms, standard abbreviations, and -> are OK. Preserve technical terms, code blocks, and quoted errors exactly. Use clear normal prose for security warnings, irreversible confirmations, risky multi-step instructions, or requested/repeated clarification; then resume terse mode.';
 
 function getCavemanInstructions() {
-  try {
-    return 'CAVEMAN MODE ACTIVE.\n\n' + stripFrontmatter(fs.readFileSync(SKILL_PATH, 'utf8'));
-  } catch (e) {
-    return FALLBACK_INSTRUCTIONS;
-  }
+  return CAVEMAN_INSTRUCTIONS;
 }
 
-module.exports = { getCavemanInstructions, stripFrontmatter };
+module.exports = { getCavemanInstructions };

package/dist/extensions/caveman/test/helpers.test.js

@@ -1,6 +1,4 @@
 import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { fileURLToPath } from "node:url";
 import test from "node:test";
 
 import {
@@ -48,17 +46,13 @@ test("isDeactivationCommand: only the whole message, case-insensitive, trailing
   assert.equal(isDeactivationCommand("how do I stop caveman re-rendering"), false);
 });
 
-test("getCavemanInstructions reads SKILL.md and strips frontmatter", () => {
+test("getCavemanInstructions returns the compact runtime prompt", () => {
   const instructions = getCavemanInstructions();
-  assert.ok(instructions.startsWith("CAVEMAN MODE ACTIVE"));
-  assert.ok(!instructions.includes("name: caveman")); // frontmatter gone
-  assert.ok(instructions.includes("Respond terse like smart caveman"));
-});
-
-test("getCavemanInstructions matches src/skills/caveman/SKILL.md body", () => {
-  const skillPath = fileURLToPath(new URL("../../../skills/caveman/SKILL.md", import.meta.url));
-  const raw = readFileSync(skillPath, "utf8");
-  const body = raw.replace(/^---[\s\S]*?---\s*/, "");
 
-  assert.ok(getCavemanInstructions().endsWith(body));
+  assert.equal(instructions,
+    "CAVEMAN MODE ACTIVE. Terse, technically complete: omit articles, filler, pleasantries, and hedging; fragments, short terms, standard abbreviations, and -> are OK. Preserve technical terms, code blocks, and quoted errors exactly. Use clear normal prose for security warnings, irreversible confirmations, risky multi-step instructions, or requested/repeated clarification; then resume terse mode.");
+  assert.ok(!instructions.includes("Persistence"));
+  assert.ok(!instructions.includes("stop caveman"));
+  assert.ok(!instructions.includes("normal mode"));
+  assert.ok(instructions.length < 400);
 });

package/dist/extensions/ponytail/index.js

@@ -184,6 +184,6 @@ export default function ponytailExtension(pi) {
 
   pi.on("before_agent_start", async (event) => {
     if (!currentMode || currentMode === "off") return;
-    return { systemPrompt: `${event.systemPrompt}\n\n${getPonytailInstructions(currentMode)}` };
+    return { systemPrompt: `${event.systemPrompt}\n\n${getPonytailInstructions(currentMode, { compact: true })}` };
   });
 }

package/dist/extensions/ponytail/ponytail-instructions.cjs

@@ -7,6 +7,11 @@ const { DEFAULT_MODE, normalizeMode, normalizePersistedMode } = require('./ponyt
 
 const INDEPENDENT_MODES = new Set(['review']);
 const SKILL_PATH = path.join(__dirname, '..', '..', 'skills', 'ponytail', 'SKILL.md');
+const COMPACT_MODE_RULES = {
+  lite: 'Build the ask; name a lazier alternative.',
+  full: 'Enforce ladder; shortest diff.',
+  ultra: 'YAGNI first; delete; challenge the rest.',
+};
 
 function filterSkillBodyForMode(body, mode) {
   const effectiveMode = normalizeMode(mode) || DEFAULT_MODE;
@@ -70,7 +75,15 @@ function getFallbackInstructions(mode) {
     'Ponytail governs what you build, not how you talk. "stop ponytail" or "normal mode": revert. Level persists until changed or session end.';
 }
 
-function getPonytailInstructions(mode) {
+function getCompactInstructions(mode) {
+  return 'PONYTAIL ' + mode.toUpperCase() + ': persistent every response; off only "stop ponytail"/"normal mode". ' +
+    'Trace first. Ladder: YAGNI -> reuse -> stdlib/native -> installed deps -> minimum diff. ' +
+    'Fix root cause, not caller symptom. No unrequested abstractions/deps/boilerplate; delete first. ' +
+    'Preserve validation, data-loss handling, security, accessibility, explicit asks. ' +
+    'Non-trivial logic: one runnable check. Code first; <=3 lines. ' + COMPACT_MODE_RULES[mode];
+}
+
+function getPonytailInstructions(mode, options) {
   const configuredMode = normalizePersistedMode(mode) || DEFAULT_MODE;
 
   if (INDEPENDENT_MODES.has(configuredMode)) {
@@ -78,6 +91,7 @@ function getPonytailInstructions(mode) {
   }
 
   const effectiveMode = normalizeMode(configuredMode) || DEFAULT_MODE;
+  if (options?.compact === true) return getCompactInstructions(effectiveMode);
 
   try {
     return 'PONYTAIL MODE ACTIVE — level: ' + effectiveMode + '\n\n' +

package/dist/extensions/ponytail/test/extension.test.js

@@ -73,8 +73,8 @@ test("/ponytail updates session mode and injects instructions", async () => with
   });
 
   const result = await events.get("before_agent_start")({ systemPrompt: "BASE" }, ctx);
-  assert.ok(result.systemPrompt.includes("PONYTAIL MODE ACTIVE"));
-  assert.ok(result.systemPrompt.includes("ultra"));
+  assert.match(result.systemPrompt, /^BASE\n\nPONYTAIL ULTRA:/);
+  assert.ok(result.systemPrompt.length - "BASE\n\n".length <= 450);
 }));
 
 test("session_start restores latest persisted mode", async () => withTempConfig(async () => {
@@ -90,7 +90,23 @@ test("session_start restores latest persisted mode", async () => withTempConfig(
   await events.get("session_start")({ reason: "resume" }, ctx);
   const result = await events.get("before_agent_start")({ systemPrompt: "BASE" }, ctx);
 
-  assert.ok(result.systemPrompt.includes("lite"));
+  assert.match(result.systemPrompt, /PONYTAIL LITE:/);
+}));
+
+test("review mode retains its independent skill prompt", async () => withTempConfig(async () => {
+  const { events } = createPiHarness();
+  const ctx = createCommandContext({
+    sessionManager: {
+      getEntries: () => [
+        { type: "custom", customType: "ponytail-mode", data: { mode: "review" } },
+      ],
+    },
+  });
+
+  await events.get("session_start")({ reason: "resume" }, ctx);
+  const result = await events.get("before_agent_start")({ systemPrompt: "BASE" }, ctx);
+
+  assert.equal(result.systemPrompt, "BASE\n\nPONYTAIL MODE ACTIVE — level: review. Behavior defined by /ponytail-review skill.");
 }));
 
 test("skill alias commands delegate to Pi skill commands", async () => {
@@ -133,7 +149,7 @@ test("a request mentioning normal mode stays active", async () => withTempConfig
   await events.get("input")({ text: "add a normal mode toggle next to dark mode", source: "interactive" }, ctx);
 
   const result = await events.get("before_agent_start")({ systemPrompt: "BASE" }, ctx);
-  assert.match(result.systemPrompt, /PONYTAIL MODE ACTIVE/);
+  assert.match(result.systemPrompt, /PONYTAIL ULTRA:/);
 }));
 
 test("status bar renders the mode and flips active on agent_start", async () => withTempConfig(async () => {

package/dist/extensions/ponytail/test/helpers.test.js

@@ -2,8 +2,12 @@ import assert from "node:assert/strict";
 import { existsSync, mkdtempSync, readFileSync, rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
+import { createRequire } from "node:module";
 import test from "node:test";
 
+const require = createRequire(import.meta.url);
+const { getPonytailInstructions } = require("../ponytail-instructions.cjs");
+
 import {
   filterSkillBodyForMode,
   parsePonytailCommand,
@@ -74,6 +78,34 @@ test("filterSkillBodyForMode keeps only requested intensity examples and rows",
   assert.ok(filtered.includes("Other line"));
 });
 
+test("compact Ponytail instructions retain core rules and mode distinctions", () => {
+  const compact = Object.fromEntries(
+    ["lite", "full", "ultra"].map((mode) => [mode, getPonytailInstructions(mode, { compact: true })]),
+  );
+
+  for (const [mode, instructions] of Object.entries(compact)) {
+    assert.match(instructions, new RegExp(`^PONYTAIL ${mode.toUpperCase()}:`));
+    assert.ok(instructions.length >= 300 && instructions.length <= 450, `${mode}: ${instructions.length} characters`);
+    for (const phrase of ["stop ponytail", "Trace first", "root cause", "validation", "data-loss", "security", "accessibility", "runnable check"]) {
+      assert.ok(instructions.includes(phrase), `${mode} lacks ${phrase}`);
+    }
+  }
+
+  assert.match(compact.lite, /Build the ask; name a lazier alternative/);
+  assert.match(compact.full, /Enforce ladder; shortest diff/);
+  assert.match(compact.ultra, /YAGNI first; delete; challenge the rest/);
+
+  const fullSkill = getPonytailInstructions("full");
+  assert.match(fullSkill, /## The ladder/);
+  assert.ok(fullSkill.length > compact.full.length);
+});
+
+test("review instructions remain independent in compact mode", () => {
+  const expected = "PONYTAIL MODE ACTIVE — level: review. Behavior defined by /ponytail-review skill.";
+  assert.equal(getPonytailInstructions("review"), expected);
+  assert.equal(getPonytailInstructions("review", { compact: true }), expected);
+});
+
 test("filterSkillBodyForMode keeps rule bullets that contain a colon", () => {
   // Regression: rule bullets outside the Intensity section (e.g. the
   // "No unrequested abstractions:" rule or the `ponytail:` comment convention)

package/dist/extensions/workflow/adapter.ts

@@ -472,11 +472,6 @@ function registerSharedArtifactWriter(pi: ExtensionAPI): void {
     name: WORKFLOW_ARTIFACT_TOOL,
     label: "Write Workflow Artifact",
     description: "Write the current workflow phase artifact. The workflow enforces the destination path; provide content only.",
-    promptSnippet: `${WORKFLOW_ARTIFACT_TOOL}(content) - write the active workflow artifact; path is chosen by the workflow, not the agent.`,
-    promptGuidelines: [
-      `During any workflow, use ${WORKFLOW_ARTIFACT_TOOL} instead of write/edit for workflow artifacts.`,
-      "Provide artifact content only; do not provide a path.",
-    ],
     parameters: Type.Object({
       content: Type.String({ description: "Artifact markdown/text content to save." }),
     }),
@@ -571,11 +566,6 @@ export function createWorkflowExtension(
       name: start,
       label: options.toolLabels.start,
       description: `Start the ${mode} workflow for the given goal. Sets up ${config.phases[0]} as the first phase and returns its prompt. Do not call if a workflow is already active.`,
-      promptSnippet: `${start}(goal) - start the ${mode} workflow; goal is what the user wants to build.`,
-      promptGuidelines: [
-        `Call ${start} only when the user explicitly asks to start a ${mode} workflow and none is active.`,
-        "Pass the user's full goal as the goal parameter.",
-      ],
       parameters: Type.Object({
         goal: Type.String({
           description: `What the user wants the ${mode} workflow to build or accomplish.`,
@@ -650,11 +640,6 @@ export function createWorkflowExtension(
       name: resume,
       label: options.toolLabels.resume,
       description: `Resume an explicitly selected ${mode} workflow run. Pass a run id, artifact directory, or workflow.json path.`,
-      promptSnippet: `${resume}(run) - resume an explicit ${mode} workflow run; run is its id, artifact directory, or workflow.json path.`,
-      promptGuidelines: [
-        `Call ${resume} only when the user explicitly selects a ${mode} workflow run.`,
-        "Workflow runs never resume automatically after reload.",
-      ],
       parameters: Type.Object({ run: Type.String({ description: "Run id, artifact directory, or workflow.json path." }) }),
       async execute(_id, params, _signal, _onUpdate, ctx) {
         return resumeController(controller, ctx, params.run, end);
@@ -671,10 +656,6 @@ export function createWorkflowExtension(
       name: end,
       label: options.toolLabels.end,
       description: `Close the ${mode} workflow. Must be called when the final phase is complete. Marks the workflow finished and stops the agent loop.`,
-      promptSnippet: `${end}() - close the finished ${mode} workflow; no further phase transitions allowed.`,
-      promptGuidelines: [
-        `Call ${end} exactly once, from the terminal phase, when all close artifacts are complete.`,
-      ],
       parameters: Type.Object({}),
       async execute(_id, _params, _signal, _onUpdate, ctx) {
         const before = checkpoint(controller);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@selesai/code",
-	"version": "0.5.12",
+	"version": "0.5.13",
 	"description": "Selesai coding agent",
 	"type": "module",
 	"piConfig": {