@jwilger/pi-development-system 0.3.7 → 0.3.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jwilger/pi-development-system",
-  "version": "0.3.7",
+  "version": "0.3.8",
   "description": "A pi extension package representing a seasoned approach to software development using a full AI SDLC.",
   "keywords": [
     "pi-package"

package/src/core/git-intent.ts

@@ -30,7 +30,10 @@ function splitLines(command: string): string[] {
   let quote: "'" | '"' | undefined;
   for (let i = 0; i < command.length; i++) {
     const ch = command.charAt(i);
-    if (quote !== undefined) {
+    if (quote === '"' && ch === "\\") {
+      current += ch + command.charAt(i + 1);
+      i++;
+    } else if (quote !== undefined) {
       if (ch === quote) quote = undefined;
       current += ch;
     } else if (ch === "\\" && command.charAt(i + 1) === "\n") {

package/src/core/test-paths.ts

@@ -49,6 +49,15 @@ export function isTestPath(path: string, profile?: TestPathProfile): boolean {
   return (profile?.testGlobs ?? []).some((g) => matchGlob(g, normalized));
 }
 
+/** A malformed `file://` URL is treated as the raw path rather than throwing out of a guard. */
+function decodeFileUrl(url: string): string {
+  try {
+    return fileURLToPath(url);
+  } catch {
+    return url;
+  }
+}
+
 /**
  * Repo-relative form of a path exactly as pi would resolve it: a leading `@` is dropped, `~` expands
  * to the home directory and `file://` URLs are decoded, so guards see the file that is really touched.
@@ -57,6 +66,6 @@ export function normalizeRepoPath(cwd: string, input: string, home: string): str
   let path = input.startsWith("@") ? input.slice(1) : input;
   if (path === "~") path = home;
   else if (path.startsWith("~/")) path = join(home, path.slice(2));
-  else if (/^file:\/\//.test(path)) path = fileURLToPath(path);
+  else if (/^file:\/\//.test(path)) path = decodeFileUrl(path);
   return isAbsolute(path) ? relative(cwd, path) : relative(cwd, resolve(cwd, path)) || ".";
 }

package/src/core/test-weakening.ts

@@ -6,7 +6,7 @@ export type WeakeningSignals = { addsSkip: boolean; emptied: boolean; commentedO
 const SKIP_MARKERS: readonly RegExp[] = [
   /\b(?:it|test|describe|context)\.(?:skip|only)\b/g,
   /\b(?:this|t|self|ctx)\.skip\(/g,
-  /\bskip:\s*true\b/g,
+  /\bskip:\s*(?:true\b|["'`])/g,
   /\bskipTest\(/g,
   /(?<![.\w])f(?:it|describe)\(/g,
   /(?<![.\w])x(?:it|describe|test)\(/g,
@@ -130,7 +130,13 @@ function splitLines(command: string): string {
   let quote: "'" | '"' | undefined;
   for (let i = 0; i < command.length; i++) {
     const c = command.charAt(i);
-    if (quote === undefined && c === "\\" && command.charAt(i + 1) === "\n") {
+    if (quote === undefined && c === "#" && (out === "" || /[\s;&|(]$/.test(out))) {
+      // A comment runs to the end of the line; keep the newline so the next command is still seen.
+      while (i + 1 < command.length && command.charAt(i + 1) !== "\n") i++;
+    } else if (quote === '"' && c === "\\") {
+      out += c + command.charAt(i + 1);
+      i++;
+    } else if (quote === undefined && c === "\\" && command.charAt(i + 1) === "\n") {
       out += " ";
       i++;
     } else if (quote === undefined && c === "\\") {