gsd-pi 2.50.0-dev.d210a87 → 2.50.0

package/README.md

@@ -19,8 +19,6 @@ One command. Walk away. Come back to a built project with clean git history.
 
 <pre><code>npm install -g gsd-pi@latest</code></pre>
 
-> GSD now provisions a managed [RTK](https://github.com/rtk-ai/rtk) binary on supported macOS, Linux, and Windows installs to compress shell-command output in `bash`, `async_bash`, `bg_shell`, and verification flows. GSD forces `RTK_TELEMETRY_DISABLED=1` for all managed invocations. Set `GSD_RTK_DISABLED=1` to disable the integration.
-
 > **📋 NOTICE: New to Node on Mac?** If you installed Node.js via Homebrew, you may be running a development release instead of LTS. **[Read this guide](./docs/node-lts-macos.md)** to pin Node 24 LTS and avoid compatibility issues.
 
 </div>

package/dist/cli.js

@@ -14,8 +14,6 @@ import { parseCliArgs as parseWebCliArgs, runWebCliBranch, migrateLegacyFlatSess
 import { stopWebMode } from './web-mode.js';
 import { getProjectSessionsDir } from './project-sessions.js';
 import { markStartup, printStartupTimings } from './startup-timings.js';
-import { bootstrapRtk, GSD_RTK_DISABLED_ENV } from './rtk.js';
-import { loadEffectiveGSDPreferences } from './resources/extensions/gsd/preferences.js';
 // ---------------------------------------------------------------------------
 // V8 compile cache — Node 22+ can cache compiled bytecode across runs,
 // eliminating repeated parse/compile overhead for unchanged modules.
@@ -116,26 +114,6 @@ if (!process.stdin.isTTY && !isPrintMode && !hasSubcommand && !cliFlags.listMode
     process.stderr.write('[gsd]   gsd --mode text "message"      Text output mode\n');
     process.exit(1);
 }
-async function ensureRtkBootstrap() {
-    if (ensureRtkBootstrap._done)
-        return;
-    // RTK is opt-in via experimental.rtk preference. Default: disabled.
-    // Honor GSD_RTK_DISABLED if already explicitly set in the environment
-    // (env var takes precedence over preferences for manual override).
-    if (!process.env[GSD_RTK_DISABLED_ENV]) {
-        const prefs = loadEffectiveGSDPreferences();
-        const rtkEnabled = prefs?.preferences.experimental?.rtk === true;
-        if (!rtkEnabled) {
-            process.env[GSD_RTK_DISABLED_ENV] = "1";
-        }
-    }
-    const rtkStatus = await bootstrapRtk();
-    ensureRtkBootstrap._done = true;
-    markStartup('bootstrapRtk');
-    if (!rtkStatus.available && rtkStatus.supported && rtkStatus.enabled && rtkStatus.reason) {
-        process.stderr.write(`[gsd] Warning: RTK unavailable — continuing without shell-command compression (${rtkStatus.reason}).\n`);
-    }
-}
 // `gsd <subcommand> --help` — show subcommand-specific help
 const subcommand = cliFlags.messages[0];
 if (subcommand && process.argv.includes('--help')) {
@@ -183,7 +161,6 @@ if (cliFlags.messages[0] === 'web' && cliFlags.messages[1] === 'stop') {
 }
 // `gsd --web [path]` or `gsd web [start] [path]` — launch browser-only web mode
 if (cliFlags.web || (cliFlags.messages[0] === 'web' && cliFlags.messages[1] !== 'stop')) {
-    await ensureRtkBootstrap();
     const webFlags = parseWebCliArgs(process.argv);
     const webBranch = await runWebCliBranch(webFlags, {
         stderr: process.stderr,
@@ -243,7 +220,6 @@ if (cliFlags.messages[0] === 'sessions') {
 }
 // `gsd headless` — run auto-mode without TUI
 if (cliFlags.messages[0] === 'headless') {
-    await ensureRtkBootstrap();
     const { runHeadless, parseHeadlessArgs } = await import('./headless.js');
     await runHeadless(parseHeadlessArgs(process.argv));
     process.exit(0);
@@ -372,7 +348,6 @@ if (!settingsManager.getCollapseChangelog()) {
 // Print / subagent mode — single-shot execution, no TTY required
 // ---------------------------------------------------------------------------
 if (isPrintMode) {
-    await ensureRtkBootstrap();
     const sessionManager = cliFlags.noSession
         ? SessionManager.inMemory()
         : SessionManager.create(process.cwd());
@@ -490,7 +465,6 @@ if (!cliFlags.worktree && !isPrintMode) {
 // ---------------------------------------------------------------------------
 // Interactive mode — normal TTY session
 // ---------------------------------------------------------------------------
-await ensureRtkBootstrap();
 // Per-directory session storage — same encoding as the upstream SDK so that
 // /resume only shows sessions from the current working directory.
 const cwd = process.cwd();

package/dist/loader.js

@@ -61,7 +61,6 @@ if (firstArg === '--help' || firstArg === '-h') {
     }
 }
 import { agentDir, appRoot } from './app-paths.js';
-import { applyRtkProcessEnv } from './rtk.js';
 import { serializeBundledExtensionPaths } from './bundled-extension-paths.js';
 import { discoverExtensionEntryPaths } from './extension-discovery.js';
 import { loadRegistry, readManifestFromEntryPath, isExtensionEnabled } from './extension-registry.js';
@@ -93,9 +92,6 @@ if (!existsSync(appRoot)) {
 }
 // GSD_CODING_AGENT_DIR — tells pi's getAgentDir() to return ~/.gsd/agent/ instead of ~/.gsd/agent/
 process.env.GSD_CODING_AGENT_DIR = agentDir;
-// RTK environment — make ~/.gsd/agent/bin visible to all child-process paths,
-// not just the bash tool, and force-disable RTK telemetry for GSD-managed use.
-applyRtkProcessEnv(process.env);
 // NODE_PATH — make gsd's own node_modules available to extensions loaded via jiti.
 // Without this, extensions (e.g. browser-tools) can't resolve dependencies like
 // `playwright` because jiti resolves modules from pi-coding-agent's location, not gsd's.

package/dist/resource-loader.d.ts

@@ -8,12 +8,9 @@ export declare function getNewerManagedResourceVersion(agentDir: string, current
  *
  * - extensions/ → ~/.gsd/agent/extensions/   (overwrite when version changes)
  * - agents/     → ~/.gsd/agent/agents/        (overwrite when version changes)
+ * - skills/     → ~/.gsd/agent/skills/        (overwrite when version changes)
  * - GSD-WORKFLOW.md → ~/.gsd/agent/GSD-WORKFLOW.md (fallback for env var miss)
  *
- * Skills are NOT synced here. They are installed by the user via the
- * skills.sh CLI (`npx skills add <repo>`) into ~/.agents/skills/ — the
- * industry-standard Agent Skills ecosystem directory.
- *
  * Skips the copy when the managed-resources.json version matches the current
  * GSD version, avoiding ~128ms of synchronous cpSync on every startup.
  * After `npm update -g @glittercowboy/gsd`, versions will differ and the

package/dist/resource-loader.js

@@ -1,7 +1,7 @@
 import { DefaultResourceLoader } from '@gsd/pi-coding-agent';
 import { createHash } from 'node:crypto';
 import { homedir } from 'node:os';
-import { chmodSync, copyFileSync, cpSync, existsSync, lstatSync, mkdirSync, openSync, closeSync, readFileSync, readlinkSync, readdirSync, rmSync, statSync, symlinkSync, unlinkSync, writeFileSync } from 'node:fs';
+import { chmodSync, copyFileSync, cpSync, existsSync, lstatSync, mkdirSync, readFileSync, readlinkSync, readdirSync, rmSync, statSync, symlinkSync, unlinkSync, writeFileSync } from 'node:fs';
 import { dirname, join, relative, resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { compareSemver } from './update-check.js';
@@ -352,12 +352,9 @@ function pruneRemovedBundledExtensions(manifest, agentDir) {
  *
  * - extensions/ → ~/.gsd/agent/extensions/   (overwrite when version changes)
  * - agents/     → ~/.gsd/agent/agents/        (overwrite when version changes)
+ * - skills/     → ~/.gsd/agent/skills/        (overwrite when version changes)
  * - GSD-WORKFLOW.md → ~/.gsd/agent/GSD-WORKFLOW.md (fallback for env var miss)
  *
- * Skills are NOT synced here. They are installed by the user via the
- * skills.sh CLI (`npx skills add <repo>`) into ~/.agents/skills/ — the
- * industry-standard Agent Skills ecosystem directory.
- *
  * Skips the copy when the managed-resources.json version matches the current
  * GSD version, avoiding ~128ms of synchronous cpSync on every startup.
  * After `npm update -g @glittercowboy/gsd`, versions will differ and the
@@ -378,9 +375,6 @@ export function initResources(agentDir) {
     // launch, not just during resource syncs. A stale/broken symlink makes ALL
     // extensions fail to resolve @gsd/* packages, rendering GSD non-functional.
     ensureNodeModulesSymlink(agentDir);
-    // Migrate legacy skills on every launch (not gated by manifest) so that
-    // partial-failure retries don't wait for a version bump.
-    migrateSkillsToEcosystemDir(agentDir);
     // Skip the full copy when both version AND content fingerprint match.
     // Version-only checks miss same-version content changes (npm link dev workflow,
     // hotfixes within a release). The content hash catches those at ~1ms cost.
@@ -395,13 +389,7 @@ export function initResources(agentDir) {
     // Sync bundled resources — overwrite so updates land on next launch.
     syncResourceDir(bundledExtensionsDir, join(agentDir, 'extensions'));
     syncResourceDir(join(resourcesDir, 'agents'), join(agentDir, 'agents'));
-    // Skills are no longer force-synced here. Users install skills via the
-    // skills.sh CLI (`npx skills add <repo>`) into ~/.agents/skills/ which
-    // is the industry-standard Agent Skills ecosystem directory.
-    //
-    // Migration from the legacy ~/.gsd/agent/skills/ directory is handled
-    // above the manifest check so it runs on every launch (including retries
-    // after partial copy failures).
+    syncResourceDir(join(resourcesDir, 'skills'), join(agentDir, 'skills'));
     // Sync GSD-WORKFLOW.md to agentDir as a fallback for when GSD_WORKFLOW_PATH
     // env var is not set (e.g. fork/dev builds, alternative entry points).
     const workflowSrc = join(resourcesDir, 'GSD-WORKFLOW.md');
@@ -417,129 +405,6 @@ export function initResources(agentDir) {
     writeManagedResourceManifest(agentDir);
     ensureRegistryEntries(join(agentDir, 'extensions'));
 }
-// ─── Legacy Skill Migration ──────────────────────────────────────────────────────
-/**
- * One-time migration: copy user-customized skills from the old
- * ~/.gsd/agent/skills/ directory into ~/.agents/skills/.
- *
- * The migration is conservative:
- *  - Only skill directories containing a SKILL.md are considered.
- *  - Copies, does not move — the old directory stays intact so downgrading
- *    to a pre-migration GSD version still works.
- *  - Collision-safe — if a skill name already exists in the target, the
- *    existing ecosystem skill wins (user may have already installed a newer
- *    version via skills.sh).
- *  - Writes a `.migrated-to-agents` marker inside the legacy directory so
- *    the migration runs at most once.
- */
-function migrateSkillsToEcosystemDir(agentDir) {
-    const legacyDir = join(agentDir, 'skills');
-    const markerPath = join(legacyDir, '.migrated-to-agents');
-    // Already migrated or no legacy dir — nothing to do
-    if (!existsSync(legacyDir))
-        return;
-    // Atomic marker check — 'wx' fails if file already exists, preventing races
-    // when two GSD processes start simultaneously.
-    let markerFd;
-    try {
-        markerFd = openSync(markerPath, 'wx');
-    }
-    catch {
-        return; // marker already exists (another process won the race, or already migrated)
-    }
-    try {
-        const ecosystemDir = join(homedir(), '.agents', 'skills');
-        mkdirSync(ecosystemDir, { recursive: true });
-        const entries = readdirSync(legacyDir, { withFileTypes: true });
-        let migrated = 0;
-        let candidates = 0;
-        for (const entry of entries) {
-            // Handle both real directories and symlinks pointing to directories
-            const isDir = entry.isDirectory();
-            const isSymlink = entry.isSymbolicLink();
-            if (!isDir && !isSymlink)
-                continue;
-            const sourcePath = join(legacyDir, entry.name);
-            // For symlinks, verify the target is a directory
-            if (isSymlink) {
-                try {
-                    const stat = statSync(sourcePath);
-                    if (!stat.isDirectory())
-                        continue;
-                }
-                catch {
-                    continue; // broken symlink — skip
-                }
-            }
-            const skillMd = join(sourcePath, 'SKILL.md');
-            if (!existsSync(skillMd))
-                continue;
-            const target = join(ecosystemDir, entry.name);
-            if (existsSync(target))
-                continue; // ecosystem version wins
-            candidates++;
-            try {
-                if (isSymlink) {
-                    // Recreate the symlink in the ecosystem directory using an absolute
-                    // target. Relative symlinks would resolve from the new parent dir
-                    // (~/.agents/skills/) instead of the original (~/.gsd/agent/skills/),
-                    // pointing to the wrong location.
-                    const rawTarget = readlinkSync(sourcePath);
-                    const absTarget = resolve(dirname(sourcePath), rawTarget);
-                    symlinkSync(absTarget, target);
-                }
-                else {
-                    cpSync(sourcePath, target, { recursive: true });
-                }
-                migrated++;
-            }
-            catch {
-                // non-fatal — skip this skill
-            }
-        }
-        // If any skills failed to copy, remove the marker so migration retries
-        // on the next launch.  This keeps the legacy dir as fallback until every
-        // skill has been successfully migrated.
-        if (migrated < candidates) {
-            try {
-                closeSync(markerFd);
-                markerFd = -1;
-            }
-            catch { /* non-fatal */ }
-            try {
-                unlinkSync(markerPath);
-            }
-            catch { /* non-fatal */ }
-            return;
-        }
-        // Write migration info to the marker
-        try {
-            writeFileSync(markerFd, `Migrated ${migrated} skill(s) to ${ecosystemDir} on ${new Date().toISOString()}\n`);
-        }
-        catch { /* non-fatal */ }
-    }
-    catch {
-        // can't create ecosystem dir or read legacy dir — close fd first (required on Windows
-        // where unlinkSync fails on open handles), then remove marker so we retry next launch
-        try {
-            closeSync(markerFd);
-            markerFd = -1;
-        }
-        catch { /* non-fatal */ }
-        try {
-            unlinkSync(markerPath);
-        }
-        catch { /* non-fatal */ }
-    }
-    finally {
-        if (markerFd !== -1) {
-            try {
-                closeSync(markerFd);
-            }
-            catch { /* non-fatal */ }
-        }
-    }
-}
 export function hasStaleCompiledExtensionSiblings(extensionsDir) {
     if (!existsSync(extensionsDir))
         return false;

package/dist/resources/extensions/async-jobs/async-bash-tool.js

@@ -12,7 +12,6 @@ import { createWriteStream } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { randomBytes } from "node:crypto";
-import { rewriteCommandWithRtk } from "../shared/rtk.js";
 const schema = Type.Object({
     command: Type.String({ description: "Bash command to execute in the background" }),
     timeout: Type.Optional(Type.Number({ description: "Timeout in seconds (optional)" })),
@@ -94,8 +93,7 @@ function executeBashInBackground(command, cwd, signal, timeout) {
             reject(err);
         } };
         const { shell, args } = getShellConfig();
-        const rewrittenCommand = rewriteCommandWithRtk(command);
-        const resolvedCommand = sanitizeCommand(rewrittenCommand);
+        const resolvedCommand = sanitizeCommand(command);
         const child = spawn(shell, [...args, resolvedCommand], {
             cwd,
             detached: true,

package/dist/resources/extensions/bg-shell/interaction.js

@@ -2,7 +2,6 @@
  * Expect-style interactions: send_and_wait, run on session, query shell environment.
  */
 import { randomUUID } from "node:crypto";
-import { rewriteCommandWithRtk } from "../shared/rtk.js";
 // ── Query Shell Environment ────────────────────────────────────────────────
 export async function queryShellEnv(bg, timeout, signal) {
     const sentinel = `__GSD_ENV_${randomUUID().slice(0, 8)}__`;
@@ -98,10 +97,9 @@ export async function runOnSession(bg, command, timeout, signal) {
     // Snapshot current output buffer position
     const startIndex = bg.output.length;
     // Write the sentinel-wrapped command to stdin
-    const rewrittenCommand = rewriteCommandWithRtk(command);
     const wrappedCommand = [
         `echo ${startMarker}`,
-        rewrittenCommand,
+        command,
         `${exitVar}=$?`,
         `echo ${endMarker} $${exitVar}`,
     ].join("\n");

package/dist/resources/extensions/bg-shell/process-manager.js

@@ -7,7 +7,6 @@ import { randomUUID } from "node:crypto";
 import { writeFileSync, readFileSync, existsSync, mkdirSync } from "node:fs";
 import { join } from "node:path";
 import { getShellConfig, sanitizeCommand } from "@gsd/pi-coding-agent";
-import { rewriteCommandWithRtk } from "../shared/rtk.js";
 import { MAX_BUFFER_LINES, MAX_EVENTS, DEAD_PROCESS_TTL, } from "./types.js";
 import { restoreWindowsVTInput, formatUptime } from "./utilities.js";
 import { analyzeLine } from "./output-formatter.js";
@@ -102,9 +101,7 @@ export function startProcess(opts) {
     const env = { ...process.env, ...(opts.env || {}) };
     const { shell, args: shellArgs } = getShellConfig();
     // Shell sessions default to the user's shell if no command specified
-    const command = processType === "shell" && !opts.command
-        ? shell
-        : rewriteCommandWithRtk(opts.command);
+    const command = processType === "shell" && !opts.command ? shell : opts.command;
     const proc = spawn(shell, [...shellArgs, sanitizeCommand(command)], {
         cwd: opts.cwd,
         stdio: ["pipe", "pipe", "pipe"],

package/dist/resources/extensions/claude-code-cli/partial-builder.js

@@ -210,12 +210,7 @@ export class PartialMessageBuilder {
                         block.arguments = JSON.parse(jsonStr);
                     }
                     catch {
-                        // Stream was truncated mid-tool-call — JSON is garbage.
-                        // Preserve the raw string for diagnostics but signal the
-                        // malformation explicitly so downstream consumers can
-                        // distinguish this from a healthy tool completion (#2574).
                         block.arguments = { _raw: jsonStr };
-                        return { type: "toolcall_end", contentIndex, toolCall: block, partial: this.partial, malformedArguments: true };
                     }
                     return { type: "toolcall_end", contentIndex, toolCall: block, partial: this.partial };
                 }

package/dist/resources/extensions/gsd/auto/phases.js

@@ -219,8 +219,6 @@ export async function runPreDispatch(ic, loopState) {
             if (s.currentMilestoneId) {
                 try {
                     deps.resolver.mergeAndExit(s.currentMilestoneId, ctx.ui);
-                    // Prevent stopAuto from attempting the same merge (#2645)
-                    s.milestoneMergedInPhases = true;
                 }
                 catch (mergeErr) {
                     if (mergeErr instanceof MergeConflictError) {
@@ -286,8 +284,6 @@ export async function runPreDispatch(ic, loopState) {
         if (s.currentMilestoneId) {
             try {
                 deps.resolver.mergeAndExit(s.currentMilestoneId, ctx.ui);
-                // Prevent stopAuto from attempting the same merge (#2645)
-                s.milestoneMergedInPhases = true;
             }
             catch (mergeErr) {
                 if (mergeErr instanceof MergeConflictError) {
@@ -338,18 +334,7 @@ export async function runDispatch(ic, preData, loopState) {
     });
     if (dispatchResult.action === "stop") {
         deps.emitJournalEvent({ ts: new Date().toISOString(), flowId: ic.flowId, seq: ic.nextSeq(), eventType: "dispatch-stop", rule: dispatchResult.matchedRule, data: { reason: dispatchResult.reason } });
-        // Warning-level stops are recoverable human checkpoints (e.g. UAT verdict
-        // gate) — pause instead of hard-stopping so the session is resumable with
-        // `/gsd auto`. Error/info-level stops remain hard stops for infrastructure
-        // failures and terminal conditions respectively.
-        // See: https://github.com/gsd-build/gsd-2/issues/2474
-        if (dispatchResult.level === "warning") {
-            ctx.ui.notify(dispatchResult.reason, "warning");
-            await deps.pauseAuto(ctx, pi);
-        }
-        else {
-            await closeoutAndStop(ctx, pi, s, deps, dispatchResult.reason);
-        }
+        await closeoutAndStop(ctx, pi, s, deps, dispatchResult.reason);
         debugLog("autoLoop", { phase: "exit", reason: "dispatch-stop" });
         return { action: "break", reason: "dispatch-stop" };
     }

package/dist/resources/extensions/gsd/auto/session.js

@@ -64,10 +64,6 @@ export class AutoSession {
     // ── Isolation degradation ────────────────────────────────────────────
     /** Set to true when worktree creation fails; prevents merge of nonexistent branch. */
     isolationDegraded = false;
-    // ── Merge guard ──────────────────────────────────────────────────────
-    /** Set to true after phases.ts successfully calls mergeAndExit, so that
-     *  stopAuto does not attempt the same merge a second time (#2645). */
-    milestoneMergedInPhases = false;
     // ── Dispatch circuit breakers ──────────────────────────────────────
     rewriteAttemptCount = 0;
     // ── Metrics ──────────────────────────────────────────────────────────────
@@ -148,7 +144,6 @@ export class AutoSession {
         this.sidecarQueue = [];
         this.rewriteAttemptCount = 0;
         this.isolationDegraded = false;
-        this.milestoneMergedInPhases = false;
         // Signal handler
         this.sigtermHandler = null;
         // Loop promise state lives in auto-loop.ts module scope

package/dist/resources/extensions/gsd/auto-dashboard.js

@@ -19,7 +19,6 @@ import { getActiveWorktreeName } from "./worktree-command.js";
 import { loadEffectiveGSDPreferences, getGlobalGSDPreferencesPath } from "./preferences.js";
 import { resolveServiceTierIcon, getEffectiveServiceTier } from "./service-tier.js";
 import { parseUnitId } from "./unit-id.js";
-import { formatRtkSavingsLabel, getRtkSessionSavings, } from "../shared/rtk-session-stats.js";
 // ─── UAT Slice Extraction ─────────────────────────────────────────────────────
 /**
  * Extract the target slice ID from a run-uat unit ID (e.g. "M001/S01" → "S01").
@@ -384,18 +383,6 @@ export function updateProgressWidget(ctx, unitType, unitId, state, accessors, ti
         let pulseBright = true;
         let cachedLines;
         let cachedWidth;
-        let cachedRtkLabel;
-        const refreshRtkLabel = () => {
-            try {
-                const sessionId = ctx.sessionManager.getSessionId();
-                const savings = sessionId ? getRtkSessionSavings(accessors.getBasePath(), sessionId) : null;
-                cachedRtkLabel = formatRtkSavingsLabel(savings);
-            }
-            catch {
-                cachedRtkLabel = null;
-            }
-        };
-        refreshRtkLabel();
         const pulseTimer = setInterval(() => {
             pulseBright = !pulseBright;
             cachedLines = undefined;
@@ -405,16 +392,13 @@ export function updateProgressWidget(ctx, unitType, unitId, state, accessors, ti
         // task/slice completion mid-unit. Without this, the progress bar only
         // updates at dispatch time, appearing frozen during long-running units.
         // 15s (vs 5s) reduces synchronous file I/O on the hot path.
-        const progressRefreshTimer = setInterval(() => {
+        const progressRefreshTimer = mid ? setInterval(() => {
             try {
-                if (mid) {
-                    updateSliceProgressCache(accessors.getBasePath(), mid.id, slice?.id);
-                }
-                refreshRtkLabel();
+                updateSliceProgressCache(accessors.getBasePath(), mid.id, slice?.id);
                 cachedLines = undefined;
             }
             catch { /* non-fatal */ }
-        }, 15_000);
+        }, 15_000) : null;
         return {
             render(width) {
                 if (cachedLines && cachedWidth === width)
@@ -675,9 +659,6 @@ export function updateProgressWidget(ctx, unitType, unitId, state, accessors, ti
                     if (statsLine) {
                         lines.push(rightAlign("", statsLine, width));
                     }
-                    if (cachedRtkLabel) {
-                        lines.push(rightAlign("", theme.fg("dim", cachedRtkLabel), width));
-                    }
                 }
                 // PWD line with last commit info right-aligned
                 const lastCommit = getLastCommit(accessors.getBasePath());