waypoint-codex 1.0.17 → 1.0.19

package/dist/src/upgrade.js

@@ -76,18 +76,90 @@ export function compareVersions(left, right) {
     }
     return 0;
 }
-function latestWaypointVersion(options) {
-    const npmBinary = options.npmBinary ?? process.env.WAYPOINT_NPM_COMMAND ?? npmBinaryForPlatform();
-    const latest = spawnSync(npmBinary, ["view", "waypoint-codex", "version"], {
+function spawnNpmPipe(npmBinary, args) {
+    const result = spawnSync(npmBinary, args, {
         stdio: "pipe",
         encoding: "utf8",
     });
-    if ((latest.status ?? 1) !== 0) {
+    return {
+        status: result.status ?? 1,
+        stdout: result.stdout ?? "",
+        stderr: result.stderr ?? "",
+    };
+}
+function writePipedOutput(result, stdio) {
+    if (stdio !== "inherit") {
+        return;
+    }
+    if (result.stdout.length > 0) {
+        process.stdout.write(result.stdout);
+    }
+    if (result.stderr.length > 0) {
+        process.stderr.write(result.stderr);
+    }
+}
+function latestWaypointVersion(options) {
+    const npmBinary = options.npmBinary ?? process.env.WAYPOINT_NPM_COMMAND ?? npmBinaryForPlatform();
+    const latest = spawnNpmPipe(npmBinary, ["view", "waypoint-codex", "version"]);
+    if (latest.status !== 0) {
         return null;
     }
-    const version = latest.stdout?.trim();
+    const version = latest.stdout.trim();
     return version ? version : null;
 }
+function latestWaypointTarballUrl(options) {
+    const tarball = spawnNpmPipe(options.npmBinary, ["view", `waypoint-codex@${options.version}`, "dist.tarball"]);
+    if (tarball.status !== 0) {
+        return null;
+    }
+    const tarballUrl = tarball.stdout.trim();
+    return tarballUrl.length > 0 ? tarballUrl : null;
+}
+function appendCacheBust(url) {
+    try {
+        const parsed = new URL(url);
+        parsed.searchParams.set("waypointCacheBust", `${Date.now()}`);
+        return parsed.toString();
+    }
+    catch {
+        const separator = url.includes("?") ? "&" : "?";
+        return `${url}${separator}waypointCacheBust=${Date.now()}`;
+    }
+}
+function isTransientTarball404(result) {
+    const combined = `${result.stdout}\n${result.stderr}`;
+    return /E404/.test(combined) && /waypoint-codex-.*\.tgz/i.test(combined);
+}
+function installLatestWaypointCli(options) {
+    const primary = spawnNpmPipe(options.npmBinary, ["install", "-g", "waypoint-codex@latest"]);
+    if (primary.status === 0) {
+        writePipedOutput(primary, options.stdio);
+        return 0;
+    }
+    if (!isTransientTarball404(primary)) {
+        writePipedOutput(primary, options.stdio);
+        return primary.status;
+    }
+    const latestVersion = options.knownLatestVersion ?? latestWaypointVersion({ npmBinary: options.npmBinary });
+    if (!latestVersion) {
+        writePipedOutput(primary, options.stdio);
+        return primary.status;
+    }
+    const tarballUrl = latestWaypointTarballUrl({
+        npmBinary: options.npmBinary,
+        version: latestVersion,
+    });
+    if (!tarballUrl) {
+        writePipedOutput(primary, options.stdio);
+        return primary.status;
+    }
+    if (options.stdio === "inherit") {
+        console.log(`Waypoint npm install hit a transient tarball 404 for ${latestVersion}. Retrying with cache-busted URL...`);
+    }
+    const fallback = spawnNpmPipe(options.npmBinary, ["install", "-g", appendCacheBust(tarballUrl)]);
+    writePipedOutput(fallback, options.stdio);
+    return fallback.status;
+}
 function hasWaypointConfig(projectRoot) {
     return existsSync(path.join(projectRoot, ".waypoint/config.toml"));
 }
@@ -95,11 +167,12 @@ export function upgradeWaypoint(options) {
     const nodeBinary = options.nodeBinary ?? process.execPath;
     const npmBinary = options.npmBinary ?? process.env.WAYPOINT_NPM_COMMAND ?? npmBinaryForPlatform();
     const stdio = options.stdio ?? "inherit";
-    const update = spawnSync(npmBinary, ["install", "-g", "waypoint-codex@latest"], {
+    const updateStatus = installLatestWaypointCli({
+        npmBinary,
         stdio,
     });
-    if ((update.status ?? 1) !== 0) {
-        return update.status ?? 1;
+    if (updateStatus !== 0) {
+        return updateStatus;
     }
     if (options.skipRepoRefresh) {
         console.log("Waypoint CLI updated. Skipped repo refresh.");
@@ -129,11 +202,13 @@ export function maybeUpgradeWaypointBeforeInit(options) {
         return null;
     }
     console.log(`Waypoint CLI ${options.currentVersion} is older than latest ${latestVersion}. Updating before init...`);
-    const update = spawnSync(npmBinary, ["install", "-g", "waypoint-codex@latest"], {
+    const updateStatus = installLatestWaypointCli({
+        npmBinary,
         stdio,
+        knownLatestVersion: latestVersion,
     });
-    if ((update.status ?? 1) !== 0) {
-        return update.status ?? 1;
+    if (updateStatus !== 0) {
+        return updateStatus;
     }
     const reexecArgs = options.initArgs.includes("--skip-cli-update")
         ? options.initArgs

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "waypoint-codex",
-  "version": "1.0.17",
+  "version": "1.0.19",
   "description": "Make Codex better by default with stronger planning, code quality, reviews, tracking, and repo guidance.",
   "license": "MIT",
   "type": "module",

package/templates/.agents/skills/pr-review/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: pr-review
-description: Run and close the full PR review loop with CodeRabbit and Codex reviewers. Use when a PR already has review activity or when you need to request, wait for, triage, fix, and re-request automated review until no meaningful issues remain and CI/CD is green.
+description: Run and close the full PR review loop with CodeRabbit and Codex reviewers. Use when a PR already has review activity or when you need to request, wait for, triage, fix, and re-request automated review until no major (`P1+`) issues remain and CI/CD is green.
 ---
 
 # PR review
@@ -22,8 +22,10 @@ Run this loop until exit criteria are satisfied.
 - collect CI/CD status for required checks
 
 2. Triage and act:
-- treat each reviewer finding as actionable unless it is clearly a false positive
-- fix all non-false-positive findings in code/docs/tests
+- classify each reviewer finding as either major (`P1+`) or minor/nitpick
+- treat major (`P1+`) findings as mandatory unless they are clearly false positives
+- fix all non-false-positive major (`P1+`) findings in code/docs/tests
+- minor/nitpick findings may be accepted without code changes, but must still be replied to inline and resolved
 - if CI/CD has failures, fix those failures as part of the same loop
 
 3. Thread discipline for every addressed or skipped finding:
@@ -39,17 +41,17 @@ Run this loop until exit criteria are satisfied.
 - wait up to 30 minutes total
 - check every 5 minutes using a sleep interval (`sleep 300`)
 - on each check, re-read both review and CI/CD status
-- if meaningful findings or CI/CD failures appear, continue the loop immediately
+- if major (`P1+`) findings or CI/CD failures appear, continue the loop immediately
 
 ## Exit Criteria
 
 You may end the loop only when all are true:
 
-- no unresolved meaningful CodeRabbit findings remain
-- no unresolved meaningful Codex findings remain
+- no unresolved major (`P1+`) CodeRabbit findings remain
+- no unresolved major (`P1+`) Codex findings remain
 - every addressed or skipped finding has an inline reply and is resolved
 - CI/CD is green (or explicitly non-blocking per repo policy)
-- the latest reviewer rounds contain no meaningful new issues
+- the latest reviewer rounds contain only nitpicks/minor issues (no major `P1+` issues)
 
 ## Required Behavior
 

package/templates/.agents/skills/pr-review/agents/openai.yaml

@@ -1,4 +1,4 @@
 interface:
   display_name: "PR Review"
   short_description: "Close the review loop with CodeRabbit, Codex, and CI/CD"
-  default_prompt: "Use $pr-review: address all existing PR review findings, fix actionable CI/CD failures, reply inline and resolve each handled thread, push fixes, comment '@coderabbitai review' and '@codex review', then poll every 5 minutes for up to 30 minutes per round until no meaningful new issues remain."
+  default_prompt: "Use $pr-review: address all existing PR review findings, fix actionable CI/CD failures, reply inline and resolve each handled thread, push fixes, comment '@coderabbitai review' and '@codex review', then poll every 5 minutes for up to 30 minutes per round until no major (P1+) issues remain and latest comments are only minor or nitpicks."