@groeponline/pi-wishcraft 1.0.0 → 1.0.2

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [1.0.2] - 2026-08-22
+
+### Changed
+- Release workflow pins `actions/checkout` and `actions/setup-node` to commit SHAs, so the publish lane cannot pick up a moved tag. Publish stays provenance-free by design: the contract test keeps `id-token: write` out of `release.yml`.
+
+## [1.0.1] - 2026-08-20
+
 ## [1.0.0] - 2026-08-20
 
 ### Added

package/README.md

@@ -39,6 +39,7 @@ Then restart pi or `/reload`. Peer range is `@earendil-works/pi-coding-agent` `>
 | `/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.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groeponline/pi-wishcraft",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "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(