@llblab/pi-kit 0.5.1 → 0.6.0

package/node_modules/@llblab/skills/abcd-context/scripts/_self-test.mjs

@@ -23,6 +23,7 @@ const missingPathOutput = run([path.join(fixtureRoot, "missing")], {
 const jsonOptionOutput = run(["--json"]);
 const invalidLineRefRoot = createInvalidLineRefFixture();
 const tableRoot = createTableFixture();
+const listRoot = createListFixture();
 try {
   const invalidLineRefOutput = run([invalidLineRefRoot], {
     withoutRootEnv: true,
@@ -49,6 +50,12 @@ try {
       .replace("|------------|---:|", "| --- | ---: |"),
   );
   const wideTableOutput = run([tableRoot], { withoutRootEnv: true });
+  const tightListOutput = run([listRoot], { withoutRootEnv: true });
+  fs.appendFileSync(
+    path.join(listRoot, "docs/lists.md"),
+    "\n- First loose item.\n  Continued detail.\n\n- Second loose item.\n  - Nested item.\n\n  - Nested sibling.\n",
+  );
+  const looseListOutput = run([listRoot], { withoutRootEnv: true });
 
   checkSuccess("default fixture", defaultOutput);
   checkSuccess("fixture path arg", pathOutput);
@@ -59,9 +66,12 @@ try {
   checkCompactTableAndLatex(compactTableOutput);
   checkNonCompactTable(nonCompactTableOutput);
   checkWideTable(wideTableOutput);
+  checkTightLists(tightListOutput);
+  checkLooseLists(looseListOutput);
 } finally {
   fs.rmSync(invalidLineRefRoot, { recursive: true, force: true });
   fs.rmSync(tableRoot, { recursive: true, force: true });
+  fs.rmSync(listRoot, { recursive: true, force: true });
 }
 
 console.log("PASS: validate-context fixture + self-reference regression");
@@ -146,6 +156,57 @@ function checkWideTable(result) {
   );
 }
 
+function checkTightLists(result) {
+  assert(result.status === 0, "contiguous lists and fenced examples pass");
+  assert(
+    result.stdout.includes("Markdown list checks passed"),
+    "contiguous list check reports success",
+  );
+}
+
+function checkLooseLists(result) {
+  assert(result.status !== 0, "blank lines inside lists fail validation");
+  assert(
+    result.stdout.includes("Blank line inside Markdown list: docs/lists.md"),
+    "loose list failure identifies its file",
+  );
+}
+
+function createListFixture() {
+  const root = fs.mkdtempSync(path.join(os.tmpdir(), "abcd-context-list-"));
+  fs.mkdirSync(path.join(root, "docs"));
+  fs.writeFileSync(
+    path.join(root, "README.md"),
+    "# Probe\n\n[Context](./AGENTS.md) [Backlog](./BACKLOG.md) [History](./CHANGELOG.md) [Docs](./docs/README.md)\n",
+  );
+  fs.writeFileSync(
+    path.join(root, "AGENTS.md"),
+    "# Context\n\n## Meta-Protocol Principles\n\n- First rule.\n- Second rule.\n\n## Operating Principles\n\n- Rule.\n",
+  );
+  fs.writeFileSync(path.join(root, "BACKLOG.md"), "# Backlog\n\nNo open work.\n");
+  fs.writeFileSync(path.join(root, "CHANGELOG.md"), "# Changelog\n\nNo releases.\n");
+  fs.writeFileSync(path.join(root, "docs/README.md"), "# Docs\n\n- [Lists](./lists.md)\n");
+  fs.writeFileSync(
+    path.join(root, "docs/lists.md"),
+    [
+      "# Lists",
+      "",
+      "1. First item.",
+      "2. Second item.",
+      "   - Nested item.",
+      "   - Nested sibling.",
+      "",
+      "```markdown",
+      "- Fenced item.",
+      "",
+      "- Fenced loose item.",
+      "```",
+      "",
+    ].join("\n"),
+  );
+  return root;
+}
+
 function createTableFixture() {
   const root = fs.mkdtempSync(path.join(os.tmpdir(), "abcd-context-table-"));
   fs.mkdirSync(path.join(root, "docs"));

package/node_modules/@llblab/skills/abcd-context/scripts/validate-context.mjs

@@ -16,6 +16,7 @@ for (let index = 0; index < args.length; index += 1) {
         "Validates the current directory by default, VALIDATE_CONTEXT_ROOT when set,",
         "or the explicit project-root argument when provided.",
         "",
+        "Markdown list items must be contiguous; blank separators fail validation.",
         "Markdown table formatting is always checked. Rows over 120 characters warn.",
       ].join("\n"),
     );
@@ -167,6 +168,53 @@ function markdownLines(text) {
   });
 }
 
+function markdownIndent(line) {
+  const leading = line.match(/^[ \t]*/)?.[0] || "";
+  return [...leading].reduce((width, char) => width + (char === "\t" ? 4 : 1), 0);
+}
+
+function findLooseListBlankLines(text) {
+  const lines = markdownLines(text);
+  const listIndents = [];
+  let pendingBlankLine;
+  const violations = [];
+  for (let index = 0; index < lines.length; index += 1) {
+    const { line, outsideFence } = lines[index];
+    if (!outsideFence) {
+      listIndents.length = 0;
+      pendingBlankLine = undefined;
+      continue;
+    }
+    if (/^\s*$/.test(line)) {
+      if (listIndents.length > 0 && pendingBlankLine === undefined) {
+        pendingBlankLine = index + 1;
+      }
+      continue;
+    }
+    const item = line.match(/^([ \t]*)(?:[-+*]|\d+[.)])[ \t]+\S/);
+    if (item) {
+      const indent = markdownIndent(item[1]);
+      if (pendingBlankLine !== undefined) violations.push(pendingBlankLine);
+      while (listIndents.length > 0 && listIndents.at(-1) > indent) listIndents.pop();
+      if (listIndents.at(-1) !== indent) listIndents.push(indent);
+      pendingBlankLine = undefined;
+      continue;
+    }
+    if (pendingBlankLine !== undefined) {
+      if (markdownIndent(line) > (listIndents.at(-1) ?? 0)) {
+        violations.push(pendingBlankLine);
+        pendingBlankLine = undefined;
+        continue;
+      }
+      listIndents.length = 0;
+      pendingBlankLine = undefined;
+      continue;
+    }
+    if (/^ {0,3}(?:#{1,6}\s|>|\|)/.test(line)) listIndents.length = 0;
+  }
+  return violations;
+}
+
 function stripFences(text) {
   return markdownLines(text)
     .filter(({ outsideFence }) => outsideFence)
@@ -392,6 +440,25 @@ if (contextFile) {
 
 const docsDir = path.join(root, "docs");
 
+progress("Checking Markdown lists...");
+let listIssue = false;
+for (const file of mdFiles) {
+  const sourceSize = fs.statSync(file).size;
+  if (sourceSize > markdownLinkScanMaxBytes) {
+    info(
+      `Skipped list validation for large Markdown file: ${rel(file)} (${sourceSize} bytes > ${markdownLinkScanMaxBytes})`,
+    );
+    continue;
+  }
+  for (const line of findLooseListBlankLines(read(file))) {
+    fail(
+      `Blank line inside Markdown list: ${rel(file)}:${line} (keep adjacent list items contiguous)`,
+    );
+    listIssue = true;
+  }
+}
+if (!listIssue) pass("Markdown list checks passed");
+
 progress("Checking Markdown tables...");
 let tableIssue = false;
 for (const file of mdFiles) {

package/node_modules/@llblab/skills/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llblab/skills",
-  "version": "1.14.0",
+  "version": "1.14.1",
   "private": false,
   "description": "Dream Skills",
   "keywords": [

package/node_modules/@llblab/skills/release-flow/SKILL.md

@@ -210,7 +210,6 @@ After the hard gate passes:
 1.  Reconfirm immediately before push that local `dev` and `origin/dev` remain absent. Stop if either now exists or the remote check fails.
 2.  Push the release commit from local `main` to `origin/main` without creating a pull request.
 3.  Verify that `origin/main` resolves to the pushed local `main` commit.
-
 4.  Confirm the version on `main` matches the intended release version and that the current `main` commit is the released commit on `origin/main`.
 5.  Create and push exactly one release tag for the confirmed version on the current `main` commit:
 
@@ -268,9 +267,8 @@ npm view <package-name> version
 Treat an explicit npm not-found response as package absence. Authentication, authorization, network, registry-resolution, and other lookup failures are not absence; stop and report them. Automation-owned npm publication already performed this lookup before the hard gate and must not defer it until after tagging.
 
 15. Publish only when both conditions are true:
-
-- The package already exists on npm.
-- The `main` version matches the intended release version and is newer than the npm version established before publication.
+    - The package already exists on npm.
+    - The `main` version matches the intended release version and is newer than the npm version established before publication.
 
 When repository automation owns npm publication, never run `npm publish` manually. After all owning tag workflows succeed, require `npm view <package-name>@<version> version` to resolve to the intended version; stop if the package remains absent or mismatched.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@llblab/pi-kit",
-  "version": "0.5.1",
+  "version": "0.6.0",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -43,10 +43,10 @@
     "@llblab/pi-actors": "0.52.1",
     "@llblab/pi-clean-room": "0.1.1",
     "@llblab/pi-codex-usage": "0.9.4",
-    "@llblab/pi-grow-loop": "0.7.4",
+    "@llblab/pi-grow-loop": "0.7.5",
     "@llblab/pi-state-flow": "0.3.0",
-    "@llblab/pi-telegram": "0.43.1",
-    "@llblab/skills": "1.14.0"
+    "@llblab/pi-telegram": "0.44.0",
+    "@llblab/skills": "1.14.1"
   },
   "bundledDependencies": [
     "@llblab/pi-actors",