@groeponline/pi-wishcraft 0.27.2 → 1.0.1

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [1.0.1] - 2026-08-20
+
+## [1.0.0] - 2026-08-20
+
+### Added
+- Wishcraft 1.0 cockpit: README lists `/skills doctor`, `/skills new`, `wishcraft.policy`, and idea-review.
+
 ## [0.27.2] - 2026-08-20
 
 ### Added

package/README.md

@@ -34,11 +34,13 @@ Then restart pi or `/reload`. Peer range is `@earendil-works/pi-coding-agent` `>
 | --- | --- |
 | Status bar | Git, TPS (1s window over a 5s ring), context, cost, ports, queue. Default placement is the editor top border; `/powerline placement below` moves it. |
 | `alt+p` | Three overlays: Navigate, Configure, Status. In Navigate, `→` / `tab` opens per-segment detail (ports, git, cost, context). `alt+i` is the ports list. |
-| `# <idea>` | File-backed inbox. Does not send the prompt. `/ideas next` feeds the oldest active idea into the session. |
+| `# <idea>` | File-backed inbox. Does not send the prompt. `/ideas` reviews status, tags, and skill insert. `/ideas next` feeds the oldest active idea into the session. |
 | `alt+s` | Stash the draft, ask something else, get it back when the run finishes. |
-| `/skills` | Substring search on name, description, and path. Enter inserts. |
+| `/skills` | Overlay search on name, description, and path. Enter inserts. `/skills doctor` is the health table. `/skills new` writes a SKILL.md from a template. |
 | `!cmd` / bash mode | Managed shell with ghost suggestions from project history. No shell-native completion probes. |
 | Hooks + repairs | Command hooks on pi events. Custom-tool input repairs before execution. Kill-switch: `wishcraft.hooksEnabled`. |
+| Read hints | Appends a one-line continuation hint after a partial `read`, so the model knows the next offset. Opt-out: `wishcraft.readHints: false`. |
+| Policy | In-process deny/inject rules in global settings. No spawn. Kill-switch: `wishcraft.policyEnabled`. |
 
 Pi owns the footer chrome, feed scrolling, and input. Wishcraft supplies widgets, overlays, and the bash/stash/editor integrations. The bar is not clickable; actions are commands and overlays.
 
@@ -54,6 +56,9 @@ Activates on load. `/powerline` toggles it. `/powerline <preset>` switches look.
 /usage                session / today / week from ~/.pi/agent/wishcraft-usage.json
 /repairs              tool-input repair counters
 /skills               skill manager
+/skills doctor        health table (broken frontmatter, dupes, unused, budget)
+/skills new [name]    write a SKILL.md from a template
+/ideas                idea review overlay (status, tags, skill insert)
 /wishcraft            settings TUI
 /open-ports           listening sockets
 /cd <path>            continue this conversation in another directory
@@ -65,6 +70,7 @@ Queue:
 
 - `# <text>` current project; `# @global`, `# @current`, `# @alias`
 - `/idea`, `/ideas`, `/queue` for capture, send, retry, clear, archive
+- `/ideas` overlay: `reviewStatus` (`idea` / `in-progress` / `done`), tags, Run with skill X
 
 Keybinds (`powerlineShortcuts`, applied after `/reload`; `null` disables):
 

package/ROADMAP.md

@@ -125,13 +125,13 @@ extension from a feature-rich prototype.
 - **English operator UI** — overlays, skill manager, `/wishcraft` TUI
   (GRO-1422, #24, 0.27.1).
 - **`powerline.skills.count` + read hints + Status trim** (GRO-1420, 0.23.0).
+- **1.0.0 cockpit cut** — README lists doctor, templates, policy, and
+  idea-review (GRO-1421). Semver leaves 0.x.
 
 ### Remaining open gaps (maturity)
-P0 — release gate for 1.0:
+P1 — differentiation and quality:
 1. **Settings contract to pi core.** No `contributes.settings`/schema;
    users hand-edit JSON. *Fix: typed settings schema + contributions.*
-
-P1 — differentiation and quality:
 2. **Zero-config first run.** Install still expects JSON edits for the most
    useful features. *Fix: sensible defaults + first-run setup overlay.*
 3. **Perf budget / low-power mode.** Status renders every ~33ms; heavy
@@ -140,15 +140,12 @@ P1 — differentiation and quality:
 4. **Accessibility (no-color / reduced-motion).** Truecolor + animations
    (vibes, rainbow think) break on terminals without truecolor. *Fix:
    `NO_COLOR`/8-color + reduced-motion respect.*
-5. **Semver policy / public API stability.** Still 0.x; top-15 extensions
-   ship a stability contract + deprecation path. *Fix: documented API
-   surface + semver policy. **1.0 cut: GRO-1421 (next).***
 
-P2 — full product, post-1.0 or ride-along on GRO-1415:
-6. **Preset editor in-menu** — custom JSON-only today.
-7. **Skill install from repo/npm** — discovery + doctor exist; install and
+P2 — full product, post-1.0:
+5. **Preset editor in-menu** — custom JSON-only today.
+6. **Skill install from repo/npm** — discovery + doctor exist; install and
    curate missing.
-8. **Host-status integration** — `ctx.ui.setStatus` beside the footer so
+7. **Host-status integration** — `ctx.ui.setStatus` beside the footer so
    status also shows in host UI (`skills.count` is a start; full coverage
    remains).
 
@@ -324,31 +321,26 @@ the same ring as the segment (no second source of truth).
 
 ## 1.0 — Cockpit
 
-Pas na 0.20. Geen parallelle 1.0-tak.
-
-1. **`/skills doctor`** — kapot frontmatter, te lange descriptions
-   (prompt-budget), duplicates global/project, ongebruikte skills.
-   Tabel, geen essay.
-2. **`/skills new` templates** — standaard, browser-workflow,
-   CLI-workflow, review-checklist. Install-van-GitHub/npm is
-   post-1.0; 1.0 is doctor + templates, geen markt.
-3. **Policy engine** — hooks zonder spawn: `deny bash matching
-   "sudo rm"`, `context inject when reading .env`. Command-hooks
-   blijven voor alles wat niet in een regel past.
-4. **Preset editor** — links/rechts segmenten kiezen in het menu,
-   opslaan in settings. Vandaag is custom JSON-only.
-5. **Per-segment detail** (CHE-41): ✅ `→` in de navigator opent
-   ports-lijst, git-samenvatting, cost-breakdown, context-math.
-   Snapshot bij openen, geen live timer.
-6. **Idee-review** — `/ideas` in dezelfde overlay-taal; status
-   idea / in-progress / done; tags; "verwerk met skill X".
-   Welcome queue-widget: item oppakken → prompt.
-7. **ChefGroep-keys** — stabiel `powerline.tps`, `powerline.ports`,
-   `powerline.preset`, `powerline.skills.count` via
-   `ctx.ui.setStatus`.
-8. **Read-tool hints** — `tool_result` op `read` verrijken met
-   "N regels, M–K getoond, volgende offset" alleen als core dat
-   niet zelf al geeft. Eerst meten, dan aanvullen.
+Shipped. The 0.x train ends at this cut. Preset editor and skill
+install-from-repo stay post-1.0 (top-15 P2).
+
+- [x] `/skills doctor` — broken frontmatter, long descriptions, global/project
+      duplicates, unused skills. Table, not an essay. (GRO-1416, #28)
+- [x] `/skills new` templates — standard, browser-workflow, CLI-workflow,
+      review-checklist. No marketplace. (GRO-1417, #32)
+- [x] Policy engine — in-process deny/inject, no spawn. Command hooks remain
+      for anything that needs a process. (GRO-1418, #29)
+- [x] Per-segment detail (CHE-41) — `→` in Navigate, snapshot on open.
+- [x] Idea review — `/ideas` overlay: idea / in-progress / done, tags,
+      Run with skill X. Welcome widget shows next idea + `/ideas next`.
+      (GRO-1419, #31)
+- [x] Status keys — `powerline.tps`, `powerline.ports`, `powerline.preset`,
+      `powerline.skills.count` via `ctx.ui.setStatus`. Not the public pitch.
+      (GRO-1420)
+- [x] Read-tool hints — continuation hint when core omits a range/offset
+      summary. (GRO-1420)
+- [x] English operator UI. (GRO-1422, #24)
+- [x] README lists doctor, new, policy, and idea-review. `banner.png` only.
 
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groeponline/pi-wishcraft",
-  "version": "0.27.2",
+  "version": "1.0.1",
   "description": "Wishcraft cockpit for the pi coding agent — powerline status, vibes, idea inbox, bash mode, and full customization.",
   "type": "module",
   "files": [

package/queue/store.ts

@@ -432,6 +432,10 @@ export class PowerlineQueueStore {
       }
 
       if (archived.length > 0) {
+        // ponytail: commit the inbox (now without the archived items) before
+        // the archive. If the archive write fails, the items stay in the inbox
+        // and are re-archived next run instead of being duplicated.
+        this.writeItems(kept, now);
         mkdirSync(dirname(this.archivePath), { recursive: true });
         const existing = existsSync(this.archivePath)
           ? readFileSync(this.archivePath, "utf-8")
@@ -441,7 +445,6 @@ export class PowerlineQueueStore {
           this.archivePath,
           `${existing.trimEnd() ? `${existing.trimEnd()}\n` : ""}${lines}\n`,
         );
-        this.writeItems(kept, now);
       }
 
       return { archived: archived.length, remainingSent };

package/src/extension/hooks/hooks-runner.ts

@@ -112,18 +112,24 @@ export function runHookCommand(
     child.on("close", (code) => finish(code));
     // Hooks that exit before reading stdin (typical `exit 2` deny scripts)
     // close the pipe; ignore EPIPE so the harness still records the exit code.
-    child.stdin?.on("error", (error: NodeJS.ErrnoException) => {
+    // One idempotent handler for the async stream event, the write callback,
+    // and the synchronous catch: EPIPE is expected (a hook that exits before
+    // reading stdin) and is ignored so its exit code still settles us; any
+    // other failure is recorded and settles the hook once.
+    const onStdinError = (error: NodeJS.ErrnoException): void => {
       if (error.code === "EPIPE") return;
-      stderr += String(error);
-    });
+      stderr += `wishcraft hook stdin error: ${error.message ?? String(error)}\n`;
+      finish(null);
+    };
 
+    child.stdin?.on("error", onStdinError);
     try {
-      child.stdin?.write(JSON.stringify(payload));
+      child.stdin?.write(JSON.stringify(payload), (err) => {
+        if (err) onStdinError(err as NodeJS.ErrnoException);
+      });
       child.stdin?.end();
     } catch (error) {
-      if ((error as NodeJS.ErrnoException).code !== "EPIPE") {
-        stderr += String(error);
-      }
+      onStdinError(error as NodeJS.ErrnoException);
     }
   });
 }

package/src/extension/session/read-hints.ts

@@ -16,14 +16,13 @@ const CORE_SHOWING_LINES =
   /^\[Showing lines \d+[–-]\d+(?: of \d+)?\.(?: Use offset=\d+ to continue\.)?\]$/i;
 const CORE_MORE_LINES =
   /^\[?\d+ more lines in file\. Use offset=\d+ to continue\.?\]?$/i;
-const OWN_READ_HINT = /^\d+ lines, showing \d+[–-]\d+, next offset \d+$/;
+const OWN_READ_HINT = /^\[wishcraft\] \d+ lines, showing \d+[–-]\d+, next offset \d+$/;
 
 function isCoreRangeFooter(line: string): boolean {
   const footer = line.trim();
   return (
     CORE_SHOWING_LINES.test(footer) ||
-    CORE_MORE_LINES.test(footer) ||
-    OWN_READ_HINT.test(footer)
+    CORE_MORE_LINES.test(footer)
   );
 }
 
@@ -33,6 +32,10 @@ function coreFooter(text: string): string {
   return lines[lines.length - 1]?.trim() ?? "";
 }
 
+function hasOwnReadHint(text: string): boolean {
+  return OWN_READ_HINT.test(coreFooter(text));
+}
+
 /** Default on. `wishcraft.readHints: false` is the opt-out. */
 export function readHintsEnabled(wishcraftSettings: unknown): boolean {
   if (
@@ -70,6 +73,7 @@ export function shouldAppendReadHint(
   if (!input) return false;
   if (input.offset === undefined && input.limit === undefined) return false;
   if (coreReadResultHasRangeSummary(text, details)) return false;
+  if (hasOwnReadHint(text)) return false;
 
   const lineCount = countContentLines(text);
   if (lineCount === 0) return false;
@@ -77,10 +81,15 @@ export function shouldAppendReadHint(
   const start = input.offset ?? 1;
   const end = start + lineCount - 1;
   const total = details?.truncation?.totalLines;
-  if (total !== undefined && end >= total) return false;
-  if (input.limit !== undefined && lineCount < input.limit) return false;
-
-  return true;
+  if (total !== undefined) {
+    // With a known file length we can tell whether the window reached EOF.
+    return end < total;
+  }
+  // Without a total we can only infer more content when the window filled the
+  // requested limit exactly; otherwise the read likely reached EOF and we must
+  // not point the model past the end of the file.
+  if (input.limit === undefined) return false;
+  return lineCount >= input.limit;
 }
 
 /** Format the English continuation hint for a partial read window. */
@@ -95,9 +104,9 @@ export function formatReadHint(
   const next = end + 1;
   const total = details?.truncation?.totalLines;
   if (total !== undefined) {
-    return `${total} lines, showing ${start}–${end}, next offset ${next}`;
+    return `[wishcraft] ${total} lines, showing ${start}–${end}, next offset ${next}`;
   }
-  return `${lineCount} lines, showing ${start}–${end}, next offset ${next}`;
+  return `[wishcraft] ${lineCount} lines, showing ${start}–${end}, next offset ${next}`;
 }
 
 export function extractReadToolResultText(content: unknown): string {

package/src/extension/session/session-lifecycle.ts

@@ -70,7 +70,7 @@ import {
 import { CONTEXT_STATUS_RENDER_MS } from "../core/constants.ts";
 import type { RuntimeState } from "../core/types.ts";
 import { isStaleExtensionContextError } from "./stale-context.ts";
-import { dismissWelcome, scheduleDismissWelcome } from "../welcome/welcome-control.ts";
+import { dismissWelcome } from "../welcome/welcome-control.ts";
 import {
   bindSkillsCountPublisher,
   clearSkillsCountPublisher,

package/src/extension/settings/wishcraft-config.ts

@@ -35,6 +35,8 @@ export interface ConfigItem {
   hint?: string;
   /** Detail explanation (optional). */
   description?: string;
+  /** Effective value when the setting is absent (toggles only; defaults to false). */
+  default?: boolean;
 }
 
 export interface ConfigGroup {
@@ -166,7 +168,7 @@ export function buildConfigGroups(settings: Record<string, unknown>): ConfigGrou
       title: "Skills",
       items: [
         { label: "Inline expand /command and $skill", path: "wishcraft.inlineSkills", kind: "toggle", hint: "needs a restart to take effect" },
-        { label: "Read hints", path: "wishcraft.readHints", kind: "toggle", hint: "off = no continuation hint after partial reads" },
+        { label: "Read hints", path: "wishcraft.readHints", kind: "toggle", default: true, hint: "off = no continuation hint after partial reads" },
       ],
     },
     {
@@ -200,12 +202,24 @@ function isPrintable(data: string): boolean {
   return data.length === 1 && data >= " " && data <= "~";
 }
 
-function displayValue(item: ConfigItem, value: ConfigValue): string {
+/** Value shown for a config item given its stored value. */
+export function displayValue(item: ConfigItem, value: ConfigValue): string {
+  // An unset toggle renders as its declared default (read hints default on).
+  if (item.kind === "toggle" && (value === null || value === undefined)) {
+    return (item.default ?? false) ? "on" : "off";
+  }
   if (value === null || value === undefined || value === "") return item.kind === "toggle" ? "off" : "—";
   if (item.kind === "toggle") return value ? "on" : "off";
   return String(value);
 }
 
+/** Next stored boolean after toggling `item` from its current `value`. */
+export function nextToggleValue(item: ConfigItem, value: ConfigValue): boolean {
+  const effective =
+    item.kind === "toggle" && value === null ? (item.default ?? false) : value === true;
+  return !effective;
+}
+
 function coerce(item: ConfigItem, current: ConfigValue, next: string): ConfigValue {
   if (item.kind === "number") {
     const n = Number.parseInt(next, 10);
@@ -288,7 +302,7 @@ export async function showWishcraftConfig(rt: RuntimeState, ctx: any): Promise<v
 
       const toggle = (item: ConfigItem) => {
         const cur = readConfigPath(settings, item.path);
-        const ok = writeConfigPath(cwd, item.path, !(cur === true));
+        const ok = writeConfigPath(cwd, item.path, nextToggleValue(item, cur));
         settings = readSettings(cwd);
         groups = buildConfigGroups(settings);
         ctx.ui.notify(