gsd-pi 2.50.0 → 2.51.0-dev.7d435fe

package/dist/resources/extensions/gsd/dispatch-guard.js

@@ -1,6 +1,7 @@
 // GSD Dispatch Guard — prevents out-of-order slice dispatch
 import { resolveMilestoneFile } from "./paths.js";
 import { findMilestoneIds } from "./guided-flow.js";
+import { parseUnitId } from "./unit-id.js";
 import { isDbAvailable, getMilestoneSlices } from "./gsd-db.js";
 import { parseRoadmap } from "./parsers-legacy.js";
 import { readFileSync } from "node:fs";
@@ -14,7 +15,7 @@ const SLICE_DISPATCH_TYPES = new Set([
 export function getPriorSliceCompletionBlocker(base, _mainBranch, unitType, unitId) {
     if (!SLICE_DISPATCH_TYPES.has(unitType))
         return null;
-    const [targetMid, targetSid] = unitId.split("/");
+    const { milestone: targetMid, slice: targetSid } = parseUnitId(unitId);
     if (!targetMid || !targetSid)
         return null;
     // Use findMilestoneIds to respect custom queue order.

package/dist/resources/extensions/gsd/docs/preferences-reference.md

@@ -1,6 +1,6 @@
 # GSD Preferences Reference
 
-Full documentation for `~/.gsd/preferences.md` (global) and `.gsd/preferences.md` (project).
+Full documentation for `~/.gsd/PREFERENCES.md` (global) and `.gsd/PREFERENCES.md` (project).
 
 ---
 
@@ -51,8 +51,8 @@ skill_rules: []
 
 Preferences are loaded from two locations and merged:
 
-1. **Global:** `~/.gsd/preferences.md` — applies to all projects
-2. **Project:** `.gsd/preferences.md` — applies to the current project only
+1. **Global:** `~/.gsd/PREFERENCES.md` — applies to all projects
+2. **Project:** `.gsd/PREFERENCES.md` — applies to the current project only
 
 **Merge behavior** (see `mergePreferences()` in `preferences.ts`):
 
@@ -241,6 +241,9 @@ Setting `prefer_skills: []` does **not** disable skill discovery — it just mea
 
   **Known unit types for `before`/`after`:** `research-milestone`, `plan-milestone`, `research-slice`, `plan-slice`, `execute-task`, `complete-slice`, `replan-slice`, `reassess-roadmap`, `run-uat`.
 
+- `experimental`: opt-in experimental features. All features here are **off by default** — you must explicitly set each one to `true` to enable it. Features in this block may change or be removed without a deprecation cycle while in experimental status. Keys:
+  - `rtk`: boolean — enable RTK (Real-Time Kompression) shell-command compression. When enabled, GSD wraps shell commands through the RTK binary to reduce token usage during command execution. RTK is downloaded automatically on first use if not already installed. **Default: `false`** (opt-in required). Set `GSD_RTK_DISABLED=1` in the environment to force-disable regardless of this preference.
+
 ---
 
 ## Best Practices
@@ -652,3 +655,15 @@ verification_max_retries: 2
 ```
 
 Runs test, lint, and typecheck after each task. On failure, auto-fix is attempted up to 2 times before reporting the issue.
+
+## Experimental Features Example
+
+```yaml
+---
+version: 1
+experimental:
+  rtk: true
+---
+```
+
+Opts in to RTK shell-command compression. RTK is downloaded automatically on first use. Set `GSD_RTK_DISABLED=1` to force-disable at the environment level regardless of this setting.

package/dist/resources/extensions/gsd/error-classifier.js

@@ -0,0 +1,105 @@
+/**
+ * Unified error classifier for provider/network/server errors.
+ *
+ * Consolidates patterns from:
+ *  - isTransientNetworkError()  in preferences-models.ts
+ *  - classifyProviderError()    in provider-error-pause.ts
+ *
+ * Single entry point: classifyError(errorMsg, retryAfterMs?)
+ *
+ * @see https://github.com/gsd-build/gsd/issues/2577
+ */
+export function createRetryState() {
+    return { networkRetryCount: 0, consecutiveTransientCount: 0, currentRetryModelId: undefined };
+}
+export function resetRetryState(state) {
+    state.networkRetryCount = 0;
+    state.consecutiveTransientCount = 0;
+    state.currentRetryModelId = undefined;
+}
+// ── Classification ──────────────────────────────────────────────────────────
+const PERMANENT_RE = /auth|unauthorized|forbidden|invalid.*key|invalid.*api|billing|quota exceeded|account/i;
+const RATE_LIMIT_RE = /rate.?limit|too many requests|429/i;
+const NETWORK_RE = /network|ECONNRESET|ETIMEDOUT|ECONNREFUSED|socket hang up|fetch failed|connection.*reset|dns/i;
+const SERVER_RE = /internal server error|500|502|503|overloaded|server_error|api_error|service.?unavailable/i;
+// ECONNRESET/ECONNREFUSED are in NETWORK_RE (same-model retry first).
+const CONNECTION_RE = /terminated|connection.?refused|other side closed|EPIPE|network.?(?:is\s+)?unavailable|stream_exhausted(?:_without_result)?/i;
+const STREAM_RE = /Unexpected end of JSON|Unexpected token.*JSON|Expected double-quoted property name|SyntaxError.*JSON/i;
+const RESET_DELAY_RE = /reset in (\d+)s/i;
+/**
+ * Classify an error message into one of the ErrorClass kinds.
+ *
+ * Classification order:
+ *  1. Permanent (auth/billing/quota) — unless also rate-limited
+ *  2. Rate limit (429, rate.?limit, too many requests)
+ *  3. Network (ECONNRESET, ETIMEDOUT, socket hang up, fetch failed, dns)
+ *  4. Server (500/502/503, overloaded, server_error)
+ *  5. Connection (terminated, ECONNREFUSED, EPIPE, other side closed)
+ *  6. Stream truncation (malformed JSON from mid-stream cut)
+ *  7. Unknown
+ */
+export function classifyError(errorMsg, retryAfterMs) {
+    const isPermanent = PERMANENT_RE.test(errorMsg);
+    const isRateLimit = RATE_LIMIT_RE.test(errorMsg);
+    // 1. Permanent — but rate limit takes precedence
+    if (isPermanent && !isRateLimit) {
+        return { kind: "permanent" };
+    }
+    // 2. Rate limit
+    if (isRateLimit) {
+        if (retryAfterMs != null && retryAfterMs > 0) {
+            return { kind: "rate-limit", retryAfterMs };
+        }
+        const resetMatch = errorMsg.match(RESET_DELAY_RE);
+        const delayMs = resetMatch ? Number(resetMatch[1]) * 1000 : 60_000;
+        return { kind: "rate-limit", retryAfterMs: delayMs };
+    }
+    // 3. Network errors — same-model retry candidate
+    if (NETWORK_RE.test(errorMsg)) {
+        // Exclude if also matches permanent signals (already handled above for
+        // rate-limit, but double-check for non-rate-limit permanent overlap like
+        // "billing" appearing alongside "network").
+        return { kind: "network", retryAfterMs: retryAfterMs ?? 3_000 };
+    }
+    // 4. Server errors — try fallback model
+    if (SERVER_RE.test(errorMsg)) {
+        return { kind: "server", retryAfterMs: retryAfterMs ?? 30_000 };
+    }
+    // 5. Connection errors — try fallback model
+    if (CONNECTION_RE.test(errorMsg)) {
+        return { kind: "connection", retryAfterMs: retryAfterMs ?? 15_000 };
+    }
+    // 6. Stream truncation — downstream symptom of connection drop
+    if (STREAM_RE.test(errorMsg)) {
+        return { kind: "stream", retryAfterMs: retryAfterMs ?? 15_000 };
+    }
+    // 7. Unknown
+    return { kind: "unknown" };
+}
+// ── Helpers ─────────────────────────────────────────────────────────────────
+/** Returns true for all transient (auto-resumable) error kinds. */
+export function isTransient(cls) {
+    switch (cls.kind) {
+        case "network":
+        case "rate-limit":
+        case "server":
+        case "stream":
+        case "connection":
+            return true;
+        default:
+            return false;
+    }
+}
+/**
+ * Backward-compatible thin wrapper.
+ *
+ * Returns true when the error is a transient *network* error specifically
+ * (worth retrying the same model). Permanent signals (auth, billing, quota)
+ * cause this to return false even if a network keyword is present.
+ */
+export function isTransientNetworkError(errorMsg) {
+    if (!errorMsg)
+        return false;
+    const cls = classifyError(errorMsg);
+    return cls.kind === "network";
+}

package/dist/resources/extensions/gsd/files.js

@@ -222,6 +222,8 @@ function _parseSummaryImpl(content) {
             whatHappened: nativeResult.whatHappened,
             deviations: nativeResult.deviations,
             filesModified: nativeResult.filesModified,
+            followUps: extractSection(content, 'Follow-ups') ?? '',
+            knownLimitations: extractSection(content, 'Known Limitations') ?? '',
         };
     }
     const [fmLines, body] = splitFrontmatter(content);
@@ -276,7 +278,9 @@ function _parseSummaryImpl(content) {
             }
         }
     }
-    return { frontmatter, title, oneLiner, whatHappened, deviations, filesModified };
+    const followUps = extractSection(body, 'Follow-ups') ?? '';
+    const knownLimitations = extractSection(body, 'Known Limitations') ?? '';
+    return { frontmatter, title, oneLiner, whatHappened, deviations, filesModified, followUps, knownLimitations };
 }
 // ─── Continue Parser ───────────────────────────────────────────────────────
 export function parseContinue(content) {

package/dist/resources/extensions/gsd/gitignore.js

@@ -1,8 +1,8 @@
 /**
- * GSD bootstrappers for .gitignore and preferences.md
+ * GSD bootstrappers for .gitignore and PREFERENCES.md
  *
  * Ensures baseline .gitignore exists with universally-correct patterns.
- * Creates an empty preferences.md template if it doesn't exist.
+ * Creates an empty PREFERENCES.md template if it doesn't exist.
  * Both idempotent — non-destructive if already present.
  */
 import { join } from "node:path";
@@ -190,16 +190,16 @@ export function untrackRuntimeFiles(basePath) {
     }
 }
 /**
- * Ensure basePath/.gsd/preferences.md exists as an empty template.
+ * Ensure basePath/.gsd/PREFERENCES.md exists as an empty template.
  * Creates the file with frontmatter only if it doesn't exist.
  * Returns true if created, false if already exists.
  *
- * Checks both lowercase (canonical) and uppercase (legacy) to avoid
- * creating a duplicate when an uppercase file already exists.
+ * Checks both uppercase (canonical) and lowercase (legacy) to avoid
+ * creating a duplicate when a lowercase file already exists.
  */
 export function ensurePreferences(basePath) {
-    const preferencesPath = join(gsdRoot(basePath), "preferences.md");
-    const legacyPath = join(gsdRoot(basePath), "PREFERENCES.md");
+    const preferencesPath = join(gsdRoot(basePath), "PREFERENCES.md");
+    const legacyPath = join(gsdRoot(basePath), "preferences.md");
     if (existsSync(preferencesPath) || existsSync(legacyPath)) {
         return false;
     }