issy 0.10.1 → 0.11.0

package/README.md

@@ -187,6 +187,8 @@ Issues can declare blockers with `depends_on`:
 depends_on: 0012, 0035
 ```
 
+Roadmap placement is validated against open dependencies. An open issue must appear after every open issue it depends on, and `create`, `update`, or `reopen` fails if the requested position would put a blocked issue before one of its blockers.
+
 ### Hooks
 
 issy supports optional hook files in `.issy/` that inject context into stdout after successful operations. The file contents are printed directly, making them visible to AI agents in their command output.

package/dist/cli.js

@@ -404,6 +404,41 @@ function getBlockingIssues(issue, issues) {
 function isIssueUnblocked(issue, issues) {
   return issue.frontmatter.status === "open" && getBlockingIssues(issue, issues).length === 0;
 }
+function compareIssuesByRoadmapOrder(a, b) {
+  const orderA = a.frontmatter.order;
+  const orderB = b.frontmatter.order;
+  if (orderA && orderB) {
+    if (orderA < orderB)
+      return -1;
+    if (orderA > orderB)
+      return 1;
+  }
+  if (orderA && !orderB)
+    return -1;
+  if (!orderA && orderB)
+    return 1;
+  return a.id < b.id ? -1 : a.id > b.id ? 1 : 0;
+}
+function validateRoadmapDependencyOrder(issues) {
+  const openIssues = issues.filter((issue) => issue.frontmatter.status === "open").sort(compareIssuesByRoadmapOrder);
+  const positionById = new Map(openIssues.map((issue, index) => [issue.id, index]));
+  const violations = [];
+  for (const issue of openIssues) {
+    const issuePosition = positionById.get(issue.id);
+    if (issuePosition === undefined)
+      continue;
+    for (const dependency of getBlockingIssues(issue, issues)) {
+      const dependencyPosition = positionById.get(dependency.id);
+      if (dependencyPosition === undefined || dependencyPosition < issuePosition) {
+        continue;
+      }
+      violations.push(`#${issue.id} depends on #${dependency.id}`);
+    }
+  }
+  if (violations.length > 0) {
+    throw new Error(`Roadmap dependency order invalid: ${violations.join("; ")}. Issues must be placed after all open issues they depend on.`);
+  }
+}
 function getIssueIdFromFilename(filename) {
   const match = filename.match(/^(\d+)-/);
   return match ? match[1] : filename.replace(".md", "");
@@ -460,17 +495,7 @@ async function getAllIssues() {
       content: body
     });
   }
-  return issues.sort((a, b) => {
-    const orderA = a.frontmatter.order;
-    const orderB = b.frontmatter.order;
-    if (orderA && orderB)
-      return orderA < orderB ? -1 : orderA > orderB ? 1 : 0;
-    if (orderA && !orderB)
-      return -1;
-    if (!orderA && orderB)
-      return 1;
-    return a.id < b.id ? -1 : a.id > b.id ? 1 : 0;
-  });
+  return issues.sort(compareIssuesByRoadmapOrder);
 }
 async function getOpenIssuesByOrder() {
   const allIssues = await getAllIssues();
@@ -558,8 +583,7 @@ async function createIssue(input) {
   const content = `${generateFrontmatter(frontmatter)}
 ${body}
 `;
-  await writeFile(join(getIssuesDir(), filename), content);
-  return {
+  const newIssue = {
     id: issueNumber,
     filename,
     frontmatter,
@@ -567,6 +591,9 @@ ${body}
 ${body}
 `
   };
+  validateRoadmapDependencyOrder([...existingIssues, newIssue]);
+  await writeFile(join(getIssuesDir(), filename), content);
+  return newIssue;
 }
 async function updateIssue(id, input) {
   const issue = await getIssue(id);
@@ -596,12 +623,16 @@ ${input.body}
 ` : issue.content;
   const content = `${generateFrontmatter(updatedFrontmatter)}
 ${updatedContent}`;
-  await writeFile(join(getIssuesDir(), issue.filename), content);
-  return {
+  const updatedIssue = {
     ...issue,
     frontmatter: updatedFrontmatter,
     content: updatedContent
   };
+  if (input.order !== undefined || input.depends_on !== undefined || input.status === "open") {
+    validateRoadmapDependencyOrder(allIssues.map((current) => current.id === issue.id ? updatedIssue : current));
+  }
+  await writeFile(join(getIssuesDir(), issue.filename), content);
+  return updatedIssue;
 }
 async function closeIssue(id) {
   return updateIssue(id, { status: "closed" });
@@ -2002,17 +2033,7 @@ function createSearchIndex(issues) {
 function sortIssues(issues, sortBy) {
   const sortOption = sortBy.toLowerCase();
   if (sortOption === "roadmap") {
-    issues.sort((a, b) => {
-      const orderA = a.frontmatter.order;
-      const orderB = b.frontmatter.order;
-      if (orderA && orderB)
-        return orderA < orderB ? -1 : orderA > orderB ? 1 : 0;
-      if (orderA && !orderB)
-        return -1;
-      if (!orderA && orderB)
-        return 1;
-      return a.id < b.id ? -1 : a.id > b.id ? 1 : 0;
-    });
+    issues.sort(compareIssuesByRoadmapOrder);
   } else if (sortOption === "priority") {
     const priorityOrder = {
       high: 0,
@@ -2066,17 +2087,7 @@ function sortIssues(issues, sortBy) {
   } else if (sortOption === "id") {
     issues.sort((a, b) => b.id.localeCompare(a.id));
   } else {
-    issues.sort((a, b) => {
-      const orderA = a.frontmatter.order;
-      const orderB = b.frontmatter.order;
-      if (orderA && orderB)
-        return orderA < orderB ? -1 : orderA > orderB ? 1 : 0;
-      if (orderA && !orderB)
-        return -1;
-      if (!orderA && orderB)
-        return 1;
-      return a.id < b.id ? -1 : a.id > b.id ? 1 : 0;
-    });
+    issues.sort(compareIssuesByRoadmapOrder);
   }
 }
 function filterByQuery(issues, query) {

package/dist/main.js

@@ -415,6 +415,41 @@ function getBlockingIssues(issue, issues) {
 function isIssueUnblocked(issue, issues) {
   return issue.frontmatter.status === "open" && getBlockingIssues(issue, issues).length === 0;
 }
+function compareIssuesByRoadmapOrder(a, b) {
+  const orderA = a.frontmatter.order;
+  const orderB = b.frontmatter.order;
+  if (orderA && orderB) {
+    if (orderA < orderB)
+      return -1;
+    if (orderA > orderB)
+      return 1;
+  }
+  if (orderA && !orderB)
+    return -1;
+  if (!orderA && orderB)
+    return 1;
+  return a.id < b.id ? -1 : a.id > b.id ? 1 : 0;
+}
+function validateRoadmapDependencyOrder(issues) {
+  const openIssues = issues.filter((issue) => issue.frontmatter.status === "open").sort(compareIssuesByRoadmapOrder);
+  const positionById = new Map(openIssues.map((issue, index) => [issue.id, index]));
+  const violations = [];
+  for (const issue of openIssues) {
+    const issuePosition = positionById.get(issue.id);
+    if (issuePosition === undefined)
+      continue;
+    for (const dependency of getBlockingIssues(issue, issues)) {
+      const dependencyPosition = positionById.get(dependency.id);
+      if (dependencyPosition === undefined || dependencyPosition < issuePosition) {
+        continue;
+      }
+      violations.push(`#${issue.id} depends on #${dependency.id}`);
+    }
+  }
+  if (violations.length > 0) {
+    throw new Error(`Roadmap dependency order invalid: ${violations.join("; ")}. Issues must be placed after all open issues they depend on.`);
+  }
+}
 function getIssueIdFromFilename(filename) {
   const match = filename.match(/^(\d+)-/);
   return match ? match[1] : filename.replace(".md", "");
@@ -471,17 +506,7 @@ async function getAllIssues() {
       content: body
     });
   }
-  return issues.sort((a, b) => {
-    const orderA = a.frontmatter.order;
-    const orderB = b.frontmatter.order;
-    if (orderA && orderB)
-      return orderA < orderB ? -1 : orderA > orderB ? 1 : 0;
-    if (orderA && !orderB)
-      return -1;
-    if (!orderA && orderB)
-      return 1;
-    return a.id < b.id ? -1 : a.id > b.id ? 1 : 0;
-  });
+  return issues.sort(compareIssuesByRoadmapOrder);
 }
 async function getOpenIssuesByOrder() {
   const allIssues = await getAllIssues();
@@ -569,8 +594,7 @@ async function createIssue(input) {
   const content = `${generateFrontmatter(frontmatter)}
 ${body}
 `;
-  await writeFile(join(getIssuesDir(), filename), content);
-  return {
+  const newIssue = {
     id: issueNumber,
     filename,
     frontmatter,
@@ -578,6 +602,9 @@ ${body}
 ${body}
 `
   };
+  validateRoadmapDependencyOrder([...existingIssues, newIssue]);
+  await writeFile(join(getIssuesDir(), filename), content);
+  return newIssue;
 }
 async function updateIssue(id, input) {
   const issue = await getIssue(id);
@@ -607,12 +634,16 @@ ${input.body}
 ` : issue.content;
   const content = `${generateFrontmatter(updatedFrontmatter)}
 ${updatedContent}`;
-  await writeFile(join(getIssuesDir(), issue.filename), content);
-  return {
+  const updatedIssue = {
     ...issue,
     frontmatter: updatedFrontmatter,
     content: updatedContent
   };
+  if (input.order !== undefined || input.depends_on !== undefined || input.status === "open") {
+    validateRoadmapDependencyOrder(allIssues.map((current) => current.id === issue.id ? updatedIssue : current));
+  }
+  await writeFile(join(getIssuesDir(), issue.filename), content);
+  return updatedIssue;
 }
 async function closeIssue(id) {
   return updateIssue(id, { status: "closed" });
@@ -2013,17 +2044,7 @@ function createSearchIndex(issues) {
 function sortIssues(issues, sortBy) {
   const sortOption = sortBy.toLowerCase();
   if (sortOption === "roadmap") {
-    issues.sort((a, b) => {
-      const orderA = a.frontmatter.order;
-      const orderB = b.frontmatter.order;
-      if (orderA && orderB)
-        return orderA < orderB ? -1 : orderA > orderB ? 1 : 0;
-      if (orderA && !orderB)
-        return -1;
-      if (!orderA && orderB)
-        return 1;
-      return a.id < b.id ? -1 : a.id > b.id ? 1 : 0;
-    });
+    issues.sort(compareIssuesByRoadmapOrder);
   } else if (sortOption === "priority") {
     const priorityOrder = {
       high: 0,
@@ -2077,17 +2098,7 @@ function sortIssues(issues, sortBy) {
   } else if (sortOption === "id") {
     issues.sort((a, b) => b.id.localeCompare(a.id));
   } else {
-    issues.sort((a, b) => {
-      const orderA = a.frontmatter.order;
-      const orderB = b.frontmatter.order;
-      if (orderA && orderB)
-        return orderA < orderB ? -1 : orderA > orderB ? 1 : 0;
-      if (orderA && !orderB)
-        return -1;
-      if (!orderA && orderB)
-        return 1;
-      return a.id < b.id ? -1 : a.id > b.id ? 1 : 0;
-    });
+    issues.sort(compareIssuesByRoadmapOrder);
   }
 }
 function filterByQuery(issues, query) {
@@ -2218,6 +2229,7 @@ var compact = markdown([
   "- When reopening an issue and other open issues exist, include exactly one position flag.",
   "- Use dependency order: prerequisites first, dependent/user-facing work later. Use `--last` when placement is unclear.",
   "- Use `depends_on` / `--depends-on` when an issue cannot start until specific blocking issues are closed.",
+  "- Roadmap placement is enforced against open dependencies: an open issue cannot be placed before an open issue it depends on.",
   "- `issy list` shows a `Blk` column with open blocker counts; `-` means unblocked.",
   "- `issy list --unblocked` shows only open issues with no open blockers.",
   "",
@@ -2310,6 +2322,7 @@ var topics = [
       "- Replace blockers with `issy update <id> --depends-on 0001,0003`.",
       '- Clear blockers with `issy update <id> --depends-on ""`.',
       "- Missing or malformed dependency IDs are ignored.",
+      "- Roadmap placement is enforced against open dependencies: an open issue cannot be placed before an open issue it depends on, and a blocker cannot be moved after open issues that depend on it.",
       "- `issy list` shows a compact `Blk` column. `-` means unblocked; otherwise the value is the number of open blockers.",
       "- `issy list --unblocked` shows only open issues with no open blockers.",
       "",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "issy",
-  "version": "0.10.1",
+  "version": "0.11.0",
   "description": "AI-native issue tracking. Markdown files in .issues/, managed by your coding assistant.",
   "type": "module",
   "license": "MIT",
@@ -35,8 +35,8 @@
     "lint": "biome check src bin"
   },
   "dependencies": {
-    "@miketromba/issy-app": "^0.10.1",
-    "@miketromba/issy-core": "^0.10.1",
+    "@miketromba/issy-app": "^0.11.0",
+    "@miketromba/issy-core": "^0.11.0",
     "update-notifier": "^7.3.1"
   }
 }