poe-code 3.0.312 → 3.0.313

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poe-code",
-  "version": "3.0.312",
+  "version": "3.0.313",
   "description": "CLI tool to configure Poe API for developer workflows.",
   "type": "module",
   "main": "./dist/index.js",

package/packages/markdown-reader/dist/core/document.js

@@ -20,6 +20,9 @@ export async function loadMarkdownDocument(file, dependencies = {}) {
     };
 }
 export function resolveMarkdownPath(file, cwd = process.cwd()) {
+    if (file.trim().length === 0) {
+        throw new UserError("invalid file: expected a non-empty path");
+    }
     return path.isAbsolute(file) ? file : path.resolve(cwd, file);
 }
 export function sliceMarkdownBytes(source, start, end) {
@@ -58,6 +61,9 @@ function hasYamlLikeLeadingFrontmatter(source) {
     if (lines[0] !== "---") {
         return false;
     }
+    if (lines[1]?.trim().length === 0) {
+        return false;
+    }
     for (const line of lines.slice(1)) {
         const trimmed = line.trim();
         if (trimmed.length === 0) {

package/packages/markdown-reader/dist/core/read-section.js

@@ -1,7 +1,11 @@
 import { loadMarkdownDocument, sliceMarkdownBytes } from "./document.js";
 import { resolveSection } from "./resolve.js";
+import { UserError } from "toolcraft";
 export function createReadSection(dependencies = {}) {
     return async function readSection(params) {
+        if (params.section.trim().length === 0) {
+            throw new UserError("invalid section: expected a non-empty section id");
+        }
         const { source, sections } = await loadMarkdownDocument(params.file, dependencies);
         const section = resolveSection(sections, params.section);
         const end = params.includeChildren === false ? section.bodyEndNoChildren : section.bodyEnd;

package/packages/markdown-reader/dist/core/resolve.js

@@ -1,14 +1,17 @@
 import { UserError } from "toolcraft";
 export function resolveSection(sections, id) {
     const trimmedId = id.trim();
-    const unnumberedTitleMatch = sections.find((section) => section.number === null && section.title === trimmedId);
-    if (unnumberedTitleMatch !== undefined) {
-        return unnumberedTitleMatch;
+    if (trimmedId.length === 0) {
+        throw new UserError("invalid section: expected a non-empty section id");
     }
     const sectionByNumber = sections.find((section) => section.number === trimmedId);
     if (sectionByNumber !== undefined) {
         return sectionByNumber;
     }
+    const unnumberedTitleMatch = sections.find((section) => section.number === null && section.title === trimmedId);
+    if (unnumberedTitleMatch !== undefined) {
+        return unnumberedTitleMatch;
+    }
     const titleMatches = sections.filter((section) => section.title === trimmedId);
     if (titleMatches.length === 1) {
         return titleMatches[0];

package/packages/markdown-reader/dist/core/scan.js

@@ -4,7 +4,10 @@ export function scanMarkdown(source) {
     if (ast.type !== "root") {
         return [];
     }
-    const headings = ast.children.filter(isHeadingNode);
+    const htmlCommentRanges = collectHtmlCommentRanges(source);
+    const headings = ast.children
+        .filter(isHeadingNode)
+        .filter((heading) => !isInsideHtmlComment(getRequiredRange(heading).start, htmlCommentRanges));
     if (headings.length === 0) {
         return [];
     }
@@ -25,6 +28,30 @@ export function scanMarkdown(source) {
     applyNumbers(sections, baselineDepth);
     return sections;
 }
+function collectHtmlCommentRanges(source) {
+    const ranges = [];
+    let searchStart = 0;
+    while (searchStart < source.length) {
+        const commentStart = source.indexOf("<!--", searchStart);
+        if (commentStart === -1) {
+            return ranges;
+        }
+        const commentEndMarker = source.indexOf("-->", commentStart + "<!--".length);
+        const commentEnd = commentEndMarker === -1 ? source.length : commentEndMarker + "-->".length;
+        ranges.push({
+            start: byteOffset(source, commentStart),
+            end: byteOffset(source, commentEnd)
+        });
+        searchStart = commentEnd;
+    }
+    return ranges;
+}
+function isInsideHtmlComment(offset, ranges) {
+    return ranges.some((range) => offset >= range.start && offset < range.end);
+}
+function byteOffset(source, index) {
+    return Buffer.byteLength(source.slice(0, index), "utf8");
+}
 function isHeadingNode(node) {
     return node.type === "heading";
 }

package/packages/markdown-reader/dist/mcp/tools.js

@@ -3,8 +3,12 @@ import { S } from "toolcraft-schema";
 import { readMarkdown } from "../core/read-markdown.js";
 import { readSection } from "../core/read-section.js";
 const readParams = S.Object({
-    file: S.String({ description: "Path to the markdown file" }),
-    depth: S.Optional(S.Number({ description: "Limit TOC to headings at depth <= n" }))
+    file: S.String({ description: "Path to the markdown file", minLength: 1 }),
+    depth: S.Optional(S.Number({
+        description: "Limit TOC to headings at depth <= n",
+        jsonType: "integer",
+        minimum: 0
+    }))
 });
 const tocEntryResult = S.Object({
     depth: S.Number(),
@@ -24,8 +28,11 @@ export const readTool = defineCommand({
     handler: async ({ params }) => readMarkdown(params)
 });
 const readSectionParams = S.Object({
-    file: S.String({ description: "Path to the markdown file" }),
-    section: S.String({ description: "Numeric path or exact heading text to read" }),
+    file: S.String({ description: "Path to the markdown file", minLength: 1 }),
+    section: S.String({
+        description: "Numeric path or exact heading text to read",
+        minLength: 1
+    }),
     includeChildren: S.Optional(S.Boolean({ description: "Include nested child sections" }))
 });
 export const readSectionTool = defineCommand({