@sous-io/sous 0.2.36 → 0.2.37

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sous-io/sous",
-  "version": "0.2.36",
+  "version": "0.2.37",
   "description": "Compiles AI coding agent configuration (CLAUDE.md, skills, memories) from LiquidJS templates",
   "type": "module",
   "license": "Apache-2.0",

package/recipes/core/sous-skills/sous.recipe.yaml

@@ -11,7 +11,7 @@ formatVersion: 1
 
 namespace: core
 name: sous-skills
-version: 0.2.36
+version: 0.2.37
 
 description: >-
   The skills that teach an agent what sous is and how it works: which files sous

package/src/lib/repos/providers/git.ts

@@ -28,7 +28,14 @@ export type CommandResult = {
 export type CommandRunner = (
   command: string,
   args: string[],
-  options: { cwd?: string }
+  options: {
+    cwd?: string;
+    /**
+     * Start the command through the shell. Only for a fixed command line: it
+     * is how Windows finds a `.cmd` shim such as npm's.
+     */
+    shell?: boolean;
+  }
 ) => Promise<CommandResult>;
 
 /**
@@ -38,13 +45,14 @@ export type CommandRunner = (
  *
  * @param command - The executable to run.
  * @param args - Its arguments, already split.
- * @param options - Optional working directory.
+ * @param options - Optional working directory, and whether to start it through the shell.
  */
 export const spawnCommand: CommandRunner = (command, args, options = {}) =>
   new Promise<CommandResult>((resolve) => {
     const child = spawn(command, args, {
       cwd: options.cwd,
       stdio: ["ignore", "pipe", "pipe"],
+      shell: options.shell === true,
     });
 
     let stdout = "";

package/src/lib/repos/providers/local.ts

@@ -83,6 +83,25 @@ export function localRepoPath(url: string): string | undefined {
   return path.isAbsolute(trimmed) ? path.normalize(trimmed) : undefined;
 }
 
+/**
+ * A directory spelled as an identity's owner path: its segments joined with `/`.
+ * The identity becomes directories in the store and the index cache, split on
+ * `/`, so a Windows path would otherwise land there as one segment holding its
+ * drive colon and backslashes. A drive (`C:`) or UNC (`\\server\share`) root
+ * becomes plain segments. A POSIX path comes back unchanged.
+ *
+ * identityOwnerPath("/home/me");        // -> "/home/me"
+ * identityOwnerPath("C:\\Users\\me");  // -> "/C/Users/me" (on Windows)
+ *
+ * @param directory - An absolute, normalized directory.
+ */
+function identityOwnerPath(directory: string): string {
+  const { root } = path.parse(directory);
+  const rootSegments = root.split(/[\\/]/).map((segment) => segment.replace(/:$/, ""));
+  const segments = [...rootSegments, ...directory.slice(root.length).split(path.sep)];
+  return "/" + segments.filter((segment) => segment.length > 0).join("/");
+}
+
 /** A URL that names its scheme, such as `https://` or `ssh://`. */
 const SCHEME_PATTERN = /^[a-z][a-z0-9+.-]*:\/\//i;
 
@@ -290,7 +309,7 @@ export class LocalProvider extends ProviderBase {
 
     return {
       host: "localhost",
-      owner: path.dirname(directory),
+      owner: identityOwnerPath(path.dirname(directory)),
       name: path.basename(directory),
       httpsUrl: directory,
       sshUrl: pathToFileURL(directory).href,

package/src/lib/self-update/installs.ts

@@ -361,7 +361,8 @@ async function globalRoot(manager: GlobalManager, run: CommandRunner): Promise<s
   const query = GLOBAL_ROOT_QUERIES[manager];
   let output: string;
   try {
-    const result = await run(manager, query.args, {});
+    // On Windows the managers are `.cmd` shims, which only the shell finds.
+    const result = await run(manager, query.args, { shell: process.platform === "win32" });
     if (result.code !== 0) return undefined;
     output = result.stdout.trim().split(/\r?\n/).pop()?.trim() ?? "";
   } catch {

package/src/lib/settings.ts

@@ -2,7 +2,7 @@ import fs from "node:fs";
 import path from "node:path";
 import { spawnSync } from "node:child_process";
 import { createRequire } from "node:module";
-import { fileURLToPath } from "node:url";
+import { fileURLToPath, pathToFileURL } from "node:url";
 import { globSync } from "glob";
 import { inferGlobBase, type CompilationConfig, type CompilationTarget, type ResolvedRuntimeContext } from "./markdown-compiler.js";
 import { buildAliasMap, resolveAliasPrefix, type AliasMap } from "./include-resolver.js";
@@ -329,7 +329,9 @@ export async function loadSettingsWithLayers(
 
   const attempts: { label: string; args: string[] }[] = [
     { label: "node", args: [CONFIG_KERNEL_PATH] },
-    { label: "tsx", args: ["--import", tsxPath, CONFIG_KERNEL_PATH] },
+    // `--import` takes a module specifier, so the path goes as a file:// URL:
+    // a bare Windows path reads as a URL whose scheme is the drive letter.
+    { label: "tsx", args: ["--import", pathToFileURL(tsxPath).href, CONFIG_KERNEL_PATH] },
   ];
 
   const failures: { label: string; message: string }[] = [];