@selesai/code 0.13.30-beta.0 → 0.13.31

package/CHANGELOG.md

@@ -2,9 +2,15 @@
 
 All notable changes to `@selesai/code` will be documented in this file.
 
-## [0.13.30-beta.0] - 2026-09-23
+## [0.13.31] - 2026-09-23
+
+### Changed
+- **Jev capability tie-breaking is now on by default.** Toggle ambiguous-tool tie-breaking in `/settings`. It requires Token-In credentials; without them, Jev is not called and Selesai prompts you to run `/tokenin add`. Deterministic routing remains the fallback.
+
+## [0.13.30] - 2026-09-23
 
 ### Added
+- **Four default skills from `addyosmani/agent-skills`.** `planning-and-task-breakdown`, `incremental-implementation`, `code-review-and-quality`, and `code-simplification` now ship as bundled skills and seed into the agent directory like the rest. Their shared reference files (`definition-of-done.md`, `security-checklist.md`, `performance-checklist.md`) ship in `src/skills/references/` with the upstream MIT license, and the links in the skills that cite them were rebased to that location.
 - **Jev advisory routing (opt-in beta).** The new bundled `jev-advisory-routing` extension can make a bounded, read-only durable-memory lookup after an explicit memory cue, and suggest one discovered skill/workflow or a proportionate verification level. Both routes are disabled by default; neither writes memory, loads a skill, starts a workflow, runs a command, or weakens required checks.
 - **Jev capability tie-breaking (opt-in beta).** The capability gateway can use Jev only to break an ambiguous lexical hint between two or three optional tools. It never receives the full catalog, tool schemas, or conversation history; unavailable, malformed, low-confidence, and `none` decisions preserve deterministic behavior.
 

package/README.md

@@ -13,13 +13,13 @@
 
 </div>
 
-Selesai is a terminal-first **AI coding agent** that takes software tasks from prompt to verified result. It is a maintained, extension-first fork of the [Pi coding agent](https://github.com/earendil-works/pi), with subagents, adaptive workflows, live research, recovery tools, a richer TUI, and **28 built-in skills** shipped together in one install.
+Selesai is a terminal-first **AI coding agent** that takes software tasks from prompt to verified result. It is a maintained, extension-first fork of the [Pi coding agent](https://github.com/earendil-works/pi), with subagents, adaptive workflows, live research, recovery tools, a richer TUI, and **32 built-in skills** shipped together in one install.
 
 Use your preferred model provider, connect a local or OpenAI-compatible endpoint, or get a ready-to-use token from **[token.in](https://token.selesai.in)**. Selesai keeps Pi's small, hackable core while reducing the setup and extension work needed before an agent can be productive.
 
 | | |
 | --- | --- |
-| **Ready out of the box** | One package bundles coordinated extensions, tools, themes, and 28 skills across 9 meaningful categories. |
+| **Ready out of the box** | One package bundles coordinated extensions, tools, themes, and 32 skills across 9 meaningful categories. |
 | **Delegates and parallelizes** | Focused subagents can research, review, build, or run independent workstreams with foreground, background, parallel, and chained execution. |
 | **Adapts to the task** | The workflow chooses the smallest useful path—from a direct edit to reconnaissance, implementation, review, fixes, and validation. |
 | **Researches with current sources** | Built-in web research, public GitHub code search, and interactive questions help the agent resolve uncertainty before it acts. |
@@ -113,6 +113,8 @@ Skills ship inside `@selesai/code` and are discovered when Selesai starts. Invok
 
 ### 📋 Planning & orchestration
 
+- `planning-and-task-breakdown` — turn a spec into ordered, implementable tasks with acceptance criteria.
+- `incremental-implementation` — land work in thin, verifiable slices with per-increment checks.
 - `pi-subagents` — delegation patterns for parallel, asynchronous, and chained work.
 
 ### 🧹 Code simplicity — the ponytail family
@@ -127,6 +129,8 @@ Skills ship inside `@selesai/code` and are discovered when Selesai starts. Invok
 ### 🛠️ Codebase quality
 
 - `improve-codebase` — inspect and improve architecture, maintainability, and overall codebase quality.
+- `code-review-and-quality` — five-axis review, severity labels, and change sizing before merge.
+- `code-simplification` — remove accumulated complexity without changing behavior.
 
 ### ✅ Completion & output discipline
 
@@ -146,6 +150,8 @@ Skills ship inside `@selesai/code` and are discovered when Selesai starts. Invok
 
 - `agent-browser` — browser and Electron automation, scraping, and QA workflows.
 
+`planning-and-task-breakdown`, `incremental-implementation`, `code-review-and-quality`, and `code-simplification` are adapted from [addyosmani/agent-skills](https://github.com/addyosmani/agent-skills) (MIT). Their shared reference material ships in `src/skills/references/`, with the upstream license alongside it.
+
 ---
 
 ## What Selesai adds to Pi

package/dist/core/settings-manager-auto-handoff.test.js

@@ -39,4 +39,15 @@ describe("auto handoff settings", () => {
         await manager.flush();
         expect(manager.getAutoHandoffThresholdTokens()).toBe(128_500);
     });
+    it("defaults Jev capability routing on and persists the /settings toggle", async () => {
+        const manager = SettingsManager.inMemory();
+        expect(manager.getCapabilityGatewayJevEnabled()).toBe(true);
+        manager.setCapabilityGatewayJevEnabled(false);
+        await manager.flush();
+        expect(manager.getCapabilityGatewayJevEnabled()).toBe(false);
+        expect(manager.getGlobalSettings().capabilityGateway?.routing?.jev?.enabled).toBe(false);
+        manager.setCapabilityGatewayJevEnabled(true);
+        await manager.flush();
+        expect(manager.getCapabilityGatewayJevEnabled()).toBe(true);
+    });
 });

package/dist/core/settings-manager.d.ts

@@ -64,6 +64,13 @@ export interface AutoHandoffSettings {
     enabled?: boolean;
     thresholdTokens?: number;
 }
+export interface CapabilityGatewaySettings {
+    routing?: {
+        jev?: {
+            enabled?: boolean;
+        };
+    };
+}
 export type DefaultProjectTrust = "ask" | "always" | "never";
 export type TransportSetting = Transport;
 /**
@@ -130,6 +137,7 @@ export interface Settings {
     markdown?: MarkdownSettings;
     warnings?: WarningSettings;
     autoHandoff?: AutoHandoffSettings;
+    capabilityGateway?: CapabilityGatewaySettings;
     sessionDir?: string;
     extensionHost?: Record<string, "selesai" | "pi">;
     httpProxy?: string;
@@ -361,6 +369,8 @@ export declare class SettingsManager {
     getWarnings(): WarningSettings;
     setWarnings(warnings: WarningSettings): void;
     getAutoHandoffEnabled(): boolean;
+    getCapabilityGatewayJevEnabled(): boolean;
+    setCapabilityGatewayJevEnabled(enabled: boolean): void;
     setAutoHandoffEnabled(enabled: boolean): void;
     getAutoHandoffThresholdTokens(): number;
     setAutoHandoffThresholdTokens(tokens: number): void;

package/dist/core/settings-manager.js

@@ -1087,6 +1087,12 @@ export class SettingsManager {
     getAutoHandoffEnabled() {
         return this.settings.autoHandoff?.enabled ?? DEFAULT_AUTO_HANDOFF_SETTINGS.enabled;
     }
+    getCapabilityGatewayJevEnabled() {
+        return this.settings.capabilityGateway?.routing?.jev?.enabled ?? true;
+    }
+    setCapabilityGatewayJevEnabled(enabled) {
+        this.setGlobalSettings({ capabilityGateway: { routing: { jev: { enabled } } } });
+    }
     setAutoHandoffEnabled(enabled) {
         if (!this.globalSettings.autoHandoff) {
             this.globalSettings.autoHandoff = {};

package/dist/defaults/settings.json

@@ -11,6 +11,11 @@
       "recommendations": { "enabled": false }
     }
   },
+  "capabilityGateway": {
+    "routing": {
+      "jev": { "enabled": true }
+    }
+  },
   "collapseChangelog": true,
   "compaction": {
     "enabled": true,

package/dist/extensions/capability-gateway/index.ts

@@ -16,7 +16,7 @@
  * - A deterministic router activates a uniquely matched tool before the run.
  *   Skills and ambiguous matches remain discoverable through the catalog
  *   without injecting fuzzy hints into the model context.
- * - Opt-in Jev-assisted routing (capabilityGateway.routing.jev in settings.json)
+ * - Default-on Jev-assisted routing (capabilityGateway.routing.jev in settings.json)
  *   is only a bounded tie-breaker: when the deterministic router returns an
  *   ambiguous lexical hint among optional tools, the gateway offers just those
  *   hinted tools (two or three) and the current prompt to the Jev decisions
@@ -24,8 +24,9 @@
  *   hinted canonical tool name; the gateway revalidates the choice against the
  *   live catalog and activates it for the current run only. No hint, a unique
  *   activation, a skill match, or an already-activated tool never reaches Jev.
- *   Every Jev failure is an ordinary abstention that leaves the deterministic
- *   behavior in place, so a missing Token-In subscription is invisible.
+ *   Every Jev failure is an ordinary abstention that leaves deterministic
+ *   behavior in place. Without Token-In credentials, no Jev request is sent and
+ *   the user is prompted to add an account with `/tokenin add`.
  * - Temporary activations reset at agent_settled, restoring the baseline
  *   active-tool set; a tool_execution_start for one of them records whether the
  *   activation was actually used (content-free tool name + source only).
@@ -146,6 +147,7 @@ export default function capabilityGatewayExtension(pi: ExtensionAPI): void {
 	// Tools this gateway activated for the current run, and whether they were invoked.
 	// Cleared at agent_settled with the activations themselves.
 	const activations = new Map<string, { source: ActivationSource; used: boolean }>();
+	let tokenInSetupPrompted = false;
 
 	/** Activate one tool for the current run, keeping the rest of the loadout untouched. */
 	function activateTool(name: string, source: ActivationSource): void {
@@ -313,7 +315,7 @@ export default function capabilityGatewayExtension(pi: ExtensionAPI): void {
 	});
 
 	// ------------------------------------------------------------------
-	// Routing: the deterministic catalog router first; opt-in Jev routing is
+	// Routing: the deterministic catalog router first; default-on Jev routing is
 	// only a bounded tie-breaker for its ambiguous/hint result.
 	// ------------------------------------------------------------------
 	pi.on("before_agent_start", async (event, ctx) => {
@@ -344,6 +346,17 @@ export default function capabilityGatewayExtension(pi: ExtensionAPI): void {
 		emitTelemetry(pi, "route", { source: "jev", outcome: "attempt", candidates: candidates.length });
 		const jevRoute = await routeToJevTool(candidates, event.prompt, ctx, config);
 		if (!jevRoute.selected) {
+			if (
+				!tokenInSetupPrompted &&
+				config.provider === "tokenin" &&
+				(jevRoute.reason === "no-credential" || jevRoute.reason === "no-template")
+			) {
+				tokenInSetupPrompted = true;
+				ctx.ui.notify(
+					"Jev tool tie-breaking needs a Token-In account. Add one with /tokenin add; deterministic routing will keep working meanwhile.",
+					"warning",
+				);
+			}
 			emitTelemetry(pi, "route", {
 				source: "jev",
 				outcome: JEV_UNAVAILABLE_REASONS.has(jevRoute.reason) ? "unavailable" : "abstained",

package/dist/extensions/capability-gateway/integration.test.ts

@@ -333,7 +333,7 @@ describe("capability gateway integration", () => {
 });
 
 // ---------------------------------------------------------------------------
-// Jev-assisted routing (opt-in): capabilityGateway.routing.jev.
+// Default-on Jev-assisted routing: capabilityGateway.routing.jev.
 // ---------------------------------------------------------------------------
 
 describe("capability gateway Jev routing", () => {
@@ -518,14 +518,37 @@ describe("capability gateway Jev routing", () => {
 		);
 	});
 
-	it("leaves routing deterministic when the Jev route is not configured", async () => {
+	it("tries Jev by default but abstains safely without Token-In", async () => {
 		allowNetwork();
-		const disabled = stubJev("grep_app_search");
+		const unavailable = stubJev("grep_app_search");
 		const h = await createGatewaySession({ enabled: true, extensions: [GATEWAY_DIR, GREP_APP_DIR] });
 		harnesses.push(h);
 
 		await route(h, HINT_PROMPT);
 
+		expect(unavailable.fetchMock).not.toHaveBeenCalled();
+		expect(h.session.getActiveToolNames()).not.toContain("grep_app_search");
+		expect(routeEvents(h)).toContainEqual(expect.objectContaining({ source: "jev", outcome: "attempt" }));
+		expect(routeEvents(h)).toContainEqual(
+			expect.objectContaining({
+				source: "jev",
+				outcome: "unavailable",
+				reason: expect.stringMatching(/^no-(template|credential)$/),
+			}),
+		);
+	});
+
+	it("does not attempt Jev when explicitly disabled", async () => {
+		const disabled = stubJev("grep_app_search");
+		const h = await createGatewaySession({
+			enabled: true,
+			extensions: [GATEWAY_DIR, GREP_APP_DIR],
+			jev: { enabled: false },
+		});
+		harnesses.push(h);
+
+		await route(h, HINT_PROMPT);
+
 		expect(disabled.fetchMock).not.toHaveBeenCalled();
 		expect(h.session.getActiveToolNames()).not.toContain("grep_app_search");
 		expect(routeEvents(h)).toEqual([]);

package/dist/extensions/capability-gateway/routing.test.ts

@@ -115,10 +115,10 @@ describe("gateway Jev configuration", () => {
 		return path;
 	}
 
-	it("stays disabled with its own short pre-turn defaults when settings are missing or malformed", () => {
+	it("defaults Jev routing on with short pre-turn settings when settings are missing or malformed", () => {
 		expect(readGatewayJevConfig(join(settingsDir, "absent.json"))).toEqual(DEFAULT_GATEWAY_JEV_CONFIG);
 		expect(DEFAULT_GATEWAY_JEV_CONFIG).toMatchObject({
-			enabled: false,
+			enabled: true,
 			provider: "tokenin",
 			model: "jev-1.13",
 			timeoutMs: DEFAULT_GATEWAY_JEV_TIMEOUT_MS,
@@ -133,7 +133,7 @@ describe("gateway Jev configuration", () => {
 		);
 	});
 
-	it("reads the opt-in route settings over the gateway defaults", () => {
+	it("reads explicit route settings over the default-on gateway config", () => {
 		const path = write({
 			capabilityGateway: {
 				routing: {
@@ -174,7 +174,7 @@ describe("gateway Jev configuration", () => {
 		expect(read).not.toHaveProperty("contextChars");
 	});
 
-	it("rejects wrong types and never enables the route implicitly", () => {
+	it("rejects wrong types while allowing an omitted enabled flag to inherit true", () => {
 		const path = write({
 			capabilityGateway: {
 				routing: {
@@ -190,7 +190,13 @@ describe("gateway Jev configuration", () => {
 				},
 			},
 		});
-		expect(readGatewayJevConfig(path)).toEqual(DEFAULT_GATEWAY_JEV_CONFIG);
+		expect(readGatewayJevConfig(path)).toEqual({ ...DEFAULT_GATEWAY_JEV_CONFIG, enabled: false });
+
+		const omittedEnabled = write({ capabilityGateway: { routing: { jev: { timeoutMs: 500 } } } });
+		expect(readGatewayJevConfig(omittedEnabled).enabled).toBe(true);
+
+		const disabled = write({ capabilityGateway: { routing: { jev: { enabled: false } } } });
+		expect(readGatewayJevConfig(disabled).enabled).toBe(false);
 	});
 
 	it("derives the shared connection settings and hard-caps the pre-turn timeout", () => {

package/dist/extensions/capability-gateway/routing.ts

@@ -12,9 +12,9 @@
  * an abstention that leaves the deterministic behavior in place. The transport and validation half
  * lives in the shared `../jev/decisions.ts` client.
  *
- * Configuration (opt-in, disabled by default). The gateway reads its own settings area — it does
- * not inherit `jevAdvisory` route policy; only the Jev provider/model identity is shared so every
- * Jev consumer defaults to the same deployment:
+ * Configuration (enabled by default, but requires Token-In credentials). The gateway reads its
+ * own settings area — it does not inherit `jevAdvisory` route policy; only the Jev provider/model
+ * identity is shared so every Jev consumer defaults to the same deployment:
  *
  *   "capabilityGateway": {
  *     "routing": { "jev": { "enabled": true, "timeoutMs": 1000, "minConfidence": 0.6 } }
@@ -68,7 +68,7 @@ export interface GatewayJevConfig {
 }
 
 export const DEFAULT_GATEWAY_JEV_CONFIG: GatewayJevConfig = {
-	enabled: false,
+	enabled: true,
 	// The Jev deployment identity, shared with every other Jev consumer.
 	provider: DEFAULT_JEV_ADVISORY_CONFIG.provider,
 	model: DEFAULT_JEV_ADVISORY_CONFIG.model,
@@ -99,13 +99,13 @@ export function readGatewayJevConfig(settingsPath: string = getSettingsPath()):
 			if (isRecord(jev)) raw = jev;
 		}
 	} catch {
-		// Missing or malformed settings: the feature stays disabled.
+		// Missing or malformed settings: use the default-on route preference.
 	}
 	if (!raw) return DEFAULT_GATEWAY_JEV_CONFIG;
 
 	const baseUrl = raw.baseUrl;
 	return {
-		enabled: raw.enabled === true,
+		enabled: raw.enabled === undefined ? DEFAULT_GATEWAY_JEV_CONFIG.enabled : raw.enabled === true,
 		provider: stringOr(raw.provider, DEFAULT_GATEWAY_JEV_CONFIG.provider),
 		model: stringOr(raw.model, DEFAULT_GATEWAY_JEV_CONFIG.model),
 		baseUrl: typeof baseUrl === "string" && baseUrl.trim() !== "" ? baseUrl : undefined,

package/dist/modes/interactive/components/settings-selector.d.ts

@@ -15,6 +15,7 @@ export interface SettingsConfig {
     autoCompact: boolean;
     autoHandoffEnabled: boolean;
     autoHandoffThresholdTokens: number;
+    capabilityGatewayJevEnabled: boolean;
     showImages: boolean;
     imageWidthCells: number;
     autoResizeImages: boolean;
@@ -62,6 +63,7 @@ export interface SettingsCallbacks {
     onAutoCompactChange: (enabled: boolean) => void;
     onAutoHandoffEnabledChange: (enabled: boolean) => void;
     onAutoHandoffThresholdTokensChange: (tokens: number) => void;
+    onCapabilityGatewayJevChange: (enabled: boolean) => boolean;
     onShowImagesChange: (enabled: boolean) => void;
     onImageWidthCellsChange: (width: number) => void;
     onAutoResizeImagesChange: (enabled: boolean) => void;

package/dist/modes/interactive/components/settings-selector.js

@@ -363,6 +363,13 @@ export class SettingsSelectorComponent extends Container {
                 currentValue: String(config.autoHandoffThresholdTokens),
                 values: ["64000", "96000", "128000", "160000", "192000", "256000", "512000"],
             },
+            {
+                id: "capability-gateway-jev",
+                label: "Jev tool tie-breaking",
+                description: "Use Token-In Jev only for ambiguous optional-tool matches. Requires Token-In credentials; add one with /tokenin add.",
+                currentValue: config.capabilityGatewayJevEnabled ? "true" : "false",
+                values: ["true", "false"],
+            },
             {
                 id: "steering-mode",
                 label: "Steering mode",
@@ -731,6 +738,11 @@ export class SettingsSelectorComponent extends Container {
                 case "auto-handoff-threshold":
                     callbacks.onAutoHandoffThresholdTokensChange(parseInt(newValue, 10));
                     break;
+                case "capability-gateway-jev":
+                    if (!callbacks.onCapabilityGatewayJevChange(newValue === "true")) {
+                        this.settingsList.updateValue(id, config.capabilityGatewayJevEnabled ? "true" : "false");
+                    }
+                    break;
                 case "show-images":
                     callbacks.onShowImagesChange(newValue === "true");
                     break;

package/dist/modes/interactive/interactive-mode.js

@@ -3785,6 +3785,7 @@ export class InteractiveMode {
                 autoCompact: this.session.autoCompactionEnabled,
                 autoHandoffEnabled: this.settingsManager.getAutoHandoffEnabled(),
                 autoHandoffThresholdTokens: this.settingsManager.getAutoHandoffThresholdTokens(),
+                capabilityGatewayJevEnabled: this.settingsManager.getCapabilityGatewayJevEnabled(),
                 defaultModel: (() => {
                     const defaultProvider = this.settingsManager.getDefaultProvider();
                     const defaultModelId = this.settingsManager.getDefaultModel();
@@ -3850,6 +3851,23 @@ export class InteractiveMode {
                 onAutoHandoffThresholdTokensChange: (tokens) => {
                     this.settingsManager.setAutoHandoffThresholdTokens(tokens);
                 },
+                onCapabilityGatewayJevChange: (enabled) => {
+                    if (enabled && !this.session.modelRuntime.getProviderAuthStatus("tokenin").configured) {
+                        this.showStatus("Token-In is required for Jev routing; starting /tokenin add");
+                        done();
+                        void this.session
+                            .prompt("/tokenin add")
+                            .then(() => {
+                            if (this.session.modelRuntime.getProviderAuthStatus("tokenin").configured) {
+                                this.settingsManager.setCapabilityGatewayJevEnabled(true);
+                            }
+                        })
+                            .catch((error) => this.showError(`Could not start Token-In setup: ${String(error)}`));
+                        return false;
+                    }
+                    this.settingsManager.setCapabilityGatewayJevEnabled(enabled);
+                    return true;
+                },
                 onShowImagesChange: (enabled) => {
                     this.settingsManager.setShowImages(enabled);
                     for (const child of this.chatContainer.children) {