waypoint-codex 0.10.10 → 0.10.12

package/README.md

@@ -37,7 +37,7 @@ The philosophy is simple:
 - more markdown
 - better continuity for the next agent
 
-By default, Waypoint appends a `.gitignore` snippet that ignores the exact Waypoint-created skill directories and reviewer-agent config files, plus everything under `.waypoint/` except `.waypoint/docs/`, while still ignoring the scaffolded `.waypoint/docs/README.md` and `.waypoint/docs/code-guide.md` assets. User-authored durable docs stay trackable; workspace, context, indexes, and other operational state remain local.
+By default, Waypoint keeps its `.gitignore` rules inside a comment-delimited `# Waypoint state` section. That section ignores the exact Waypoint-created skill directories and reviewer-agent config files, plus everything under `.waypoint/` except `.waypoint/docs/`, while still ignoring the scaffolded `.waypoint/docs/README.md` and `.waypoint/docs/code-guide.md` assets. User-authored durable docs stay trackable; workspace, context, indexes, and other operational state remain local.
 
 ## Best fit
 

package/dist/src/core.js

@@ -10,6 +10,42 @@ const DEFAULT_DOCS_INDEX = ".waypoint/DOCS_INDEX.md";
 const DEFAULT_TRACK_DIR = ".waypoint/track";
 const DEFAULT_TRACKS_INDEX = ".waypoint/TRACKS_INDEX.md";
 const DEFAULT_WORKSPACE = ".waypoint/WORKSPACE.md";
+const GITIGNORE_WAYPOINT_START = "# Waypoint state";
+const GITIGNORE_WAYPOINT_END = "# End Waypoint state";
+const LEGACY_WAYPOINT_GITIGNORE_RULES = new Set([
+    ".codex/",
+    ".codex/config.toml",
+    ".codex/agents/",
+    ".codex/agents/code-reviewer.toml",
+    ".codex/agents/code-health-reviewer.toml",
+    ".codex/agents/plan-reviewer.toml",
+    ".agents/",
+    ".agents/skills/",
+    ".agents/skills/planning/",
+    ".agents/skills/work-tracker/",
+    ".agents/skills/docs-sync/",
+    ".agents/skills/code-guide-audit/",
+    ".agents/skills/adversarial-review/",
+    ".agents/skills/visual-explanations/",
+    ".agents/skills/break-it-qa/",
+    ".agents/skills/frontend-context-interview/",
+    ".agents/skills/backend-context-interview/",
+    ".agents/skills/frontend-ship-audit/",
+    ".agents/skills/backend-ship-audit/",
+    ".agents/skills/conversation-retrospective/",
+    ".agents/skills/workspace-compress/",
+    ".agents/skills/pre-pr-hygiene/",
+    ".agents/skills/pr-review/",
+    ".waypoint/",
+    ".waypoint/DOCS_INDEX.md",
+    ".waypoint/state/",
+    ".waypoint/context/",
+    ".waypoint/*",
+    "!.waypoint/docs/",
+    "!.waypoint/docs/**",
+    ".waypoint/docs/README.md",
+    ".waypoint/docs/code-guide.md",
+]);
 const SHIPPED_SKILL_NAMES = [
     "planning",
     "work-tracker",
@@ -74,7 +110,8 @@ function appendGitignoreSnippet(projectRoot) {
     const content = readFileSync(gitignorePath, "utf8");
     const normalizedLines = content.split(/\r?\n/);
     const normalizedContent = normalizedLines.join("\n");
-    if (normalizedContent.includes(snippet)) {
+    const headerCount = normalizedLines.filter((line) => line === GITIGNORE_WAYPOINT_START).length;
+    if (normalizedContent.includes(snippet) && headerCount <= 1) {
         return;
     }
     const startIndex = normalizedLines.findIndex((line) => line === snippetLines[0]);
@@ -83,31 +120,97 @@ function appendGitignoreSnippet(projectRoot) {
         return;
     }
     const managedLineSet = new Set(snippetLines);
-    const managedEndLine = snippetLines[snippetLines.length - 1];
-    let endIndex = normalizedLines.findIndex((line, index) => index >= startIndex && line === managedEndLine);
+    const endIndex = findWaypointGitignoreBlockEnd(normalizedLines, startIndex);
     if (endIndex === -1) {
         writeText(gitignorePath, `${content.trimEnd()}\n\n${snippet}\n`);
         return;
     }
     const hasForeignLineInsideBlock = normalizedLines
         .slice(startIndex + 1, endIndex)
-        .some((line) => line.length > 0 && !managedLineSet.has(line));
+        .some((line) => line.length > 0 && !isManagedWaypointGitignoreLine(line, managedLineSet));
+    const trailingLines = stripSubsequentWaypointGitignoreBlocks(normalizedLines.slice(endIndex + 1), managedLineSet);
     if (hasForeignLineInsideBlock) {
         const foreignLines = normalizedLines
             .slice(startIndex + 1, endIndex)
-            .filter((line) => line.length > 0 && !managedLineSet.has(line))
+            .filter((line) => line.length > 0 && !isManagedWaypointGitignoreLine(line, managedLineSet))
             .join("\n");
         const before = normalizedLines.slice(0, startIndex).join("\n").trimEnd();
-        const after = normalizedLines.slice(endIndex + 1).join("\n").trimStart();
+        const after = trailingLines.join("\n").trimStart();
         const merged = [before, snippet, foreignLines, after].filter((piece) => piece.length > 0).join("\n\n");
         writeText(gitignorePath, `${merged}\n`);
         return;
     }
     const before = normalizedLines.slice(0, startIndex).join("\n").trimEnd();
-    const after = normalizedLines.slice(endIndex + 1).join("\n").trimStart();
+    const after = trailingLines.join("\n").trimStart();
     const merged = [before, snippet, after].filter((piece) => piece.length > 0).join("\n\n");
     writeText(gitignorePath, `${merged}\n`);
 }
+function findWaypointGitignoreBlockEnd(lines, startIndex) {
+    const explicitEndIndex = lines.findIndex((line, index) => index > startIndex && line === GITIGNORE_WAYPOINT_END);
+    if (explicitEndIndex !== -1) {
+        return explicitEndIndex;
+    }
+    return findLegacyWaypointGitignoreBlockEnd(lines, startIndex);
+}
+function findLegacyWaypointGitignoreBlockEnd(lines, startIndex) {
+    let scanEndExclusive = lines.length;
+    for (let index = startIndex + 1; index < lines.length; index += 1) {
+        const line = lines[index];
+        if (line.length === 0) {
+            scanEndExclusive = index;
+            break;
+        }
+        if (line.startsWith("#") && line !== GITIGNORE_WAYPOINT_START) {
+            scanEndExclusive = index;
+            break;
+        }
+    }
+    let endIndex = -1;
+    for (let index = startIndex + 1; index < scanEndExclusive; index += 1) {
+        if (isLegacyWaypointGitignoreRule(lines[index])) {
+            endIndex = index;
+        }
+    }
+    return endIndex;
+}
+function isLegacyWaypointGitignoreRule(line) {
+    const normalizedLine = line.startsWith("/") ? line.slice(1) : line;
+    return LEGACY_WAYPOINT_GITIGNORE_RULES.has(normalizedLine);
+}
+function isManagedWaypointGitignoreLine(line, managedLineSet) {
+    return managedLineSet.has(line) || isLegacyWaypointGitignoreRule(line);
+}
+function stripSubsequentWaypointGitignoreBlocks(lines, managedLineSet) {
+    const keptLines = [];
+    let index = 0;
+    while (index < lines.length) {
+        if (lines[index] !== GITIGNORE_WAYPOINT_START) {
+            keptLines.push(lines[index]);
+            index += 1;
+            continue;
+        }
+        const endIndex = findWaypointGitignoreBlockEnd(lines, index);
+        if (endIndex === -1) {
+            keptLines.push(lines[index]);
+            index += 1;
+            continue;
+        }
+        const foreignLines = lines
+            .slice(index + 1, endIndex)
+            .filter((line) => line.length > 0 && !isManagedWaypointGitignoreLine(line, managedLineSet));
+        if (foreignLines.length > 0) {
+            if (keptLines.length > 0 && keptLines[keptLines.length - 1] !== "") {
+                keptLines.push("");
+            }
+            keptLines.push(...foreignLines);
+            if (endIndex + 1 < lines.length && lines[endIndex + 1] !== "") {
+                keptLines.push("");
+            }
+        }
+        index = endIndex + 1;
+    }
+    return keptLines;
+}
 function upsertManagedBlock(filePath, block) {
     if (!existsSync(filePath)) {
         writeText(filePath, `${block.trim()}\n`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "waypoint-codex",
-  "version": "0.10.10",
+  "version": "0.10.12",
   "description": "Codex-native repository operating system: scaffolding, docs routing, repo-local skills, doctor, and sync.",
   "license": "MIT",
   "type": "module",

package/templates/.gitignore.snippet

@@ -23,3 +23,4 @@
 !.waypoint/docs/**
 .waypoint/docs/README.md
 .waypoint/docs/code-guide.md
+# End Waypoint state

package/templates/.waypoint/agent-operating-manual.md

@@ -51,6 +51,7 @@ If something important lives only in your head or in the chat transcript, the re
 - Rebuild `.waypoint/TRACKS_INDEX.md` whenever tracker files change.
 - When spawning reviewer agents or other subagents, explicitly set `fork_context: false`, `model` to `gpt-5.4`, and `reasoning_effort` to `high` unless the user explicitly requests a different model or lower reasoning.
 - Use the repo-local skills and reviewer agents instead of improvising from scratch.
+- If you created a PR earlier in the current session and need to push more work, first confirm that PR is still open. If it is closed, create a fresh branch from `origin/main` and open a fresh PR instead of pushing more commits to the old PR branch.
 - Treat reviewer agents as one-shot workers: once a reviewer returns findings, read the result and close it. If another review pass is needed later, spawn a fresh reviewer instead of reusing the same thread.
 - Do not kill long-running subagents or reviewer agents just because they are slow.
 - When waiting on reviewers, subagents, CI, automated review, or external jobs, wait as long as required. There is no fixed timeout where waiting itself becomes the problem.

package/templates/managed-agents-block.md

@@ -100,6 +100,7 @@ Working rules:
 - Before presenting a non-trivial implementation plan to the user, run `plan-reviewer` and iterate on the plan until it has no meaningful review findings left
 - Treat `plan-reviewer`, `code-reviewer`, and `code-health-reviewer` as one-shot agents: once a reviewer returns findings, close it; if another pass is needed later, spawn a fresh reviewer instead of reusing the old thread
 - Before pushing or opening/updating a PR for substantial work, use `pre-pr-hygiene`
+- If you created a PR earlier in the current session and need to push more work, first confirm that PR is still open. If it is closed, create a fresh branch from `origin/main` and open a fresh PR instead of pushing more commits to the old PR branch
 - Use `pr-review` once a PR has active review comments or automated review in progress
 - Treat the generated context bundle as required session bootstrap, not optional reference material
 - After plan approval, own the execution through implementation, verification, review, and repo-memory updates before surfacing a final completion report