@mmnto/cli 1.3.16 → 1.3.18

package/dist/parse-nits.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"parse-nits.d.ts","sourceRoot":"","sources":["../src/parse-nits.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAoDH;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,EAAE,CAmB1D"}

package/dist/parse-nits.js

@@ -0,0 +1,73 @@
+/**
+ * Extract nit content from CodeRabbit review bodies.
+ * Parses <details><summary>... Nitpick ...</summary>...</details> blocks
+ * and returns the inner content as clean text.
+ */
+/**
+ * Given a position right after a `<details>...<summary>...</summary>`,
+ * find the matching `</details>` accounting for nesting, and return
+ * the inner content between the end of the summary and the closing tag.
+ */
+function extractNestedBlock(body, startPos) {
+    let depth = 1;
+    let pos = startPos;
+    while (depth > 0 && pos < body.length) {
+        // Create fresh regexes each iteration to avoid global state corruption
+        const openRe = /<details>/gi;
+        const closeRe = /<\/details>/gi;
+        openRe.lastIndex = pos;
+        closeRe.lastIndex = pos;
+        const openMatch = openRe.exec(body);
+        const closeMatch = closeRe.exec(body);
+        if (!closeMatch) {
+            // No closing tag found — malformed HTML, bail out
+            return null;
+        }
+        if (openMatch && openMatch.index < closeMatch.index) {
+            // Found a nested <details> before the next </details>
+            depth++;
+            pos = openMatch.index + openMatch[0].length;
+        }
+        else {
+            // Found a </details>
+            depth--;
+            if (depth === 0) {
+                return body.slice(startPos, closeMatch.index);
+            }
+            pos = closeMatch.index + closeMatch[0].length;
+        }
+    }
+    return null;
+}
+/** Strip wrapper HTML tags (<details>, <summary>, <blockquote>) but keep inner content. */
+function stripWrapperTags(html) {
+    return html
+        .replace(/<\/?details>/gi, '')
+        .replace(/<summary>[\s\S]*?<\/summary>/gi, '')
+        .replace(/<\/?blockquote>/gi, '')
+        .trim();
+}
+/**
+ * Extract nit content from CodeRabbit review bodies.
+ * Finds all `<details>` blocks whose `<summary>` contains "Nitpick", "nitpick", or the broom emoji.
+ * Returns an array of cleaned nit text strings.
+ */
+export function parseCodeRabbitNits(body) {
+    // Strip fenced code blocks to avoid extracting fake nits from examples
+    const stripped = body.replace(/```[\s\S]*?```/g, '');
+    const nits = [];
+    const nitpickRe = /<details>\s*<summary>[^<]*(?:nitpick|🧹)[^<]*<\/summary>/gi;
+    let match;
+    while ((match = nitpickRe.exec(stripped)) !== null) {
+        const afterSummary = match.index + match[0].length;
+        const innerContent = extractNestedBlock(stripped, afterSummary);
+        if (innerContent) {
+            const cleaned = stripWrapperTags(innerContent);
+            if (cleaned) {
+                nits.push(cleaned);
+            }
+        }
+    }
+    return nits;
+}
+//# sourceMappingURL=parse-nits.js.map

package/dist/parse-nits.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"parse-nits.js","sourceRoot":"","sources":["../src/parse-nits.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH;;;;GAIG;AACH,SAAS,kBAAkB,CAAC,IAAY,EAAE,QAAgB;IACxD,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,GAAG,GAAG,QAAQ,CAAC;IAEnB,OAAO,KAAK,GAAG,CAAC,IAAI,GAAG,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC;QACtC,uEAAuE;QACvE,MAAM,MAAM,GAAG,aAAa,CAAC;QAC7B,MAAM,OAAO,GAAG,eAAe,CAAC;QAChC,MAAM,CAAC,SAAS,GAAG,GAAG,CAAC;QACvB,OAAO,CAAC,SAAS,GAAG,GAAG,CAAC;QAExB,MAAM,SAAS,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACpC,MAAM,UAAU,GAAG,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAEtC,IAAI,CAAC,UAAU,EAAE,CAAC;YAChB,kDAAkD;YAClD,OAAO,IAAI,CAAC;QACd,CAAC;QAED,IAAI,SAAS,IAAI,SAAS,CAAC,KAAK,GAAG,UAAU,CAAC,KAAK,EAAE,CAAC;YACpD,sDAAsD;YACtD,KAAK,EAAE,CAAC;YACR,GAAG,GAAG,SAAS,CAAC,KAAK,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC;QAC9C,CAAC;aAAM,CAAC;YACN,qBAAqB;YACrB,KAAK,EAAE,CAAC;YACR,IAAI,KAAK,KAAK,CAAC,EAAE,CAAC;gBAChB,OAAO,IAAI,CAAC,KAAK,CAAC,QAAQ,EAAE,UAAU,CAAC,KAAK,CAAC,CAAC;YAChD,CAAC;YACD,GAAG,GAAG,UAAU,CAAC,KAAK,GAAG,UAAU,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC;QAChD,CAAC;IACH,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC;AAED,2FAA2F;AAC3F,SAAS,gBAAgB,CAAC,IAAY;IACpC,OAAO,IAAI;SACR,OAAO,CAAC,gBAAgB,EAAE,EAAE,CAAC;SAC7B,OAAO,CAAC,gCAAgC,EAAE,EAAE,CAAC;SAC7C,OAAO,CAAC,mBAAmB,EAAE,EAAE,CAAC;SAChC,IAAI,EAAE,CAAC;AACZ,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,mBAAmB,CAAC,IAAY;IAC9C,uEAAuE;IACvE,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,CAAC,iBAAiB,EAAE,EAAE,CAAC,CAAC;IACrD,MAAM,IAAI,GAAa,EAAE,CAAC;IAC1B,MAAM,SAAS,GAAG,4DAA4D,CAAC;IAC/E,IAAI,KAA6B,CAAC;IAElC,OAAO,CAAC,KAAK,GAAG,SAAS,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;QACnD,MAAM,YAAY,GAAG,KAAK,CAAC,KAAK,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC;QACnD,MAAM,YAAY,GAAG,kBAAkB,CAAC,QAAQ,EAAE,YAAY,CAAC,CAAC;QAChE,IAAI,YAAY,EAAE,CAAC;YACjB,MAAM,OAAO,GAAG,gBAAgB,CAAC,YAAY,CAAC,CAAC;YAC/C,IAAI,OAAO,EAAE,CAAC;gBACZ,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;YACrB,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC"}

package/dist/parse-nits.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=parse-nits.test.d.ts.map

package/dist/parse-nits.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"parse-nits.test.d.ts","sourceRoot":"","sources":["../src/parse-nits.test.ts"],"names":[],"mappings":""}

package/dist/parse-nits.test.js

@@ -0,0 +1,148 @@
+import { describe, expect, it } from 'vitest';
+import { parseCodeRabbitNits } from './parse-nits.js';
+// ─── Sample CodeRabbit nit block ─────────────────────────
+const SAMPLE_NIT_BLOCK = `<details>
+<summary>🧹 Nitpick comments (2)</summary><blockquote>
+
+<details>
+<summary>packages/cli/src/commands/docs.ts (1)</summary><blockquote>
+
+\`194-206\`: **Hardcoded path and fragile counting method.**
+
+Two observations about the baseline counting logic.
+
+</blockquote></details>
+<details>
+<summary>packages/core/src/rule-tester.ts (1)</summary><blockquote>
+
+\`103-103\`: **Consider propagating \`onWarn\` to \`matchAstGrepPattern\` calls.**
+
+Both calls omit the optional \`onWarn\` parameter.
+
+</blockquote></details>
+
+</blockquote></details>`;
+// ─── parseCodeRabbitNits ─────────────────────────────────
+describe('parseCodeRabbitNits', () => {
+    it('extracts nit content from a standard CodeRabbit review body', () => {
+        const nits = parseCodeRabbitNits(SAMPLE_NIT_BLOCK);
+        expect(nits).toHaveLength(1);
+        expect(nits[0]).toContain('Hardcoded path and fragile counting method');
+        expect(nits[0]).toContain('Consider propagating');
+        expect(nits[0]).toContain('Both calls omit the optional');
+    });
+    it('returns empty array when no nit section exists', () => {
+        const body = `## Summary
+
+This PR adds a new feature. No nitpick blocks here.
+
+<details>
+<summary>Some other section</summary>
+Content that is not nits.
+</details>`;
+        const nits = parseCodeRabbitNits(body);
+        expect(nits).toHaveLength(0);
+    });
+    it('handles multiple nit blocks', () => {
+        const body = `<details>
+<summary>🧹 Nitpick comments (1)</summary><blockquote>
+
+First batch of nits.
+
+</blockquote></details>
+
+Some text in between.
+
+<details>
+<summary>🧹 Nitpick comments (1)</summary><blockquote>
+
+Second batch of nits.
+
+</blockquote></details>`;
+        const nits = parseCodeRabbitNits(body);
+        expect(nits).toHaveLength(2);
+        expect(nits[0]).toContain('First batch of nits');
+        expect(nits[1]).toContain('Second batch of nits');
+    });
+    it('strips HTML wrapper tags but preserves content', () => {
+        const nits = parseCodeRabbitNits(SAMPLE_NIT_BLOCK);
+        expect(nits).toHaveLength(1);
+        // Wrapper tags should be stripped
+        expect(nits[0]).not.toMatch(/<\/?details>/);
+        expect(nits[0]).not.toMatch(/<\/?blockquote>/);
+        expect(nits[0]).not.toMatch(/<summary>/);
+        // Content should be preserved
+        expect(nits[0]).toContain('Hardcoded path');
+        expect(nits[0]).toContain('Two observations about the baseline counting logic');
+    });
+    it('matches case-insensitively on "Nitpick"', () => {
+        const body = `<details>
+<summary>nitpick comments (1)</summary>
+
+A lowercase nit.
+
+</details>`;
+        const nits = parseCodeRabbitNits(body);
+        expect(nits).toHaveLength(1);
+        expect(nits[0]).toContain('A lowercase nit');
+    });
+    it('matches on broom emoji without the word "nitpick"', () => {
+        const body = `<details>
+<summary>🧹 Minor comments (1)</summary>
+
+A broom-emoji nit.
+
+</details>`;
+        const nits = parseCodeRabbitNits(body);
+        expect(nits).toHaveLength(1);
+        expect(nits[0]).toContain('A broom-emoji nit');
+    });
+    it('returns empty array for empty string', () => {
+        expect(parseCodeRabbitNits('')).toEqual([]);
+    });
+    it('does not catastrophically backtrack on large inputs', () => {
+        // Build a large body with no matching blocks
+        const large = 'x'.repeat(100_000);
+        const start = performance.now();
+        const nits = parseCodeRabbitNits(large);
+        const elapsed = performance.now() - start;
+        expect(nits).toHaveLength(0);
+        // Should complete in well under 1 second
+        expect(elapsed).toBeLessThan(1000);
+    });
+    it('handles nit block embedded in a larger review body', () => {
+        const body = `## Walkthrough
+
+This PR refactors the compiler module.
+
+## Changes
+
+| File | Summary |
+|------|---------|
+| compiler.ts | Extracted utility |
+
+<details>
+<summary>🧹 Nitpick comments (1)</summary><blockquote>
+
+\`42-50\`: **Consider using a Map instead of a plain object.**
+
+Maps provide better iteration guarantees.
+
+</blockquote></details>
+
+## Assessment
+
+Overall looks good.`;
+        const nits = parseCodeRabbitNits(body);
+        expect(nits).toHaveLength(1);
+        expect(nits[0]).toContain('Consider using a Map');
+    });
+    it('skips blocks with empty content after stripping tags', () => {
+        const body = `<details>
+<summary>🧹 Nitpick comments (0)</summary><blockquote>
+</blockquote></details>`;
+        const nits = parseCodeRabbitNits(body);
+        expect(nits).toHaveLength(0);
+    });
+});
+//# sourceMappingURL=parse-nits.test.js.map

package/dist/parse-nits.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"parse-nits.test.js","sourceRoot":"","sources":["../src/parse-nits.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9C,OAAO,EAAE,mBAAmB,EAAE,MAAM,iBAAiB,CAAC;AAEtD,4DAA4D;AAE5D,MAAM,gBAAgB,GAAG;;;;;;;;;;;;;;;;;;;;wBAoBD,CAAC;AAEzB,4DAA4D;AAE5D,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,MAAM,IAAI,GAAG,mBAAmB,CAAC,gBAAgB,CAAC,CAAC;QACnD,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,4CAA4C,CAAC,CAAC;QACxE,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;QAClD,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,8BAA8B,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,IAAI,GAAG;;;;;;;WAON,CAAC;QACR,MAAM,IAAI,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;QACvC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC/B,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,IAAI,GAAG;;;;;;;;;;;;;;wBAcO,CAAC;QACrB,MAAM,IAAI,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;QACvC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;QACjD,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,MAAM,IAAI,GAAG,mBAAmB,CAAC,gBAAgB,CAAC,CAAC;QACnD,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,kCAAkC;QAClC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,cAAc,CAAC,CAAC;QAC5C,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC;QAC/C,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,WAAW,CAAC,CAAC;QACzC,8BAA8B;QAC9B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC;QAC5C,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,oDAAoD,CAAC,CAAC;IAClF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,IAAI,GAAG;;;;;WAKN,CAAC;QACR,MAAM,IAAI,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;QACvC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC3D,MAAM,IAAI,GAAG;;;;;WAKN,CAAC;QACR,MAAM,IAAI,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;QACvC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sCAAsC,EAAE,GAAG,EAAE;QAC9C,MAAM,CAAC,mBAAmB,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,GAAG,EAAE;QAC7D,6CAA6C;QAC7C,MAAM,KAAK,GAAG,GAAG,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QAClC,MAAM,KAAK,GAAG,WAAW,CAAC,GAAG,EAAE,CAAC;QAChC,MAAM,IAAI,GAAG,mBAAmB,CAAC,KAAK,CAAC,CAAC;QACxC,MAAM,OAAO,GAAG,WAAW,CAAC,GAAG,EAAE,GAAG,KAAK,CAAC;QAC1C,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,yCAAyC;QACzC,MAAM,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC;IACrC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC5D,MAAM,IAAI,GAAG;;;;;;;;;;;;;;;;;;;;;oBAqBG,CAAC;QACjB,MAAM,IAAI,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;QACvC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC7B,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,sBAAsB,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;QAC9D,MAAM,IAAI,GAAG;;wBAEO,CAAC;QACrB,MAAM,IAAI,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;QACvC,MAAM,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;IAC/B,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mmnto/cli",
-  "version": "1.3.16",
+  "version": "1.3.18",
   "description": "CLI for Totem — AI persistent memory and context layer",
   "type": "module",
   "bin": {
@@ -21,7 +21,7 @@
     "ora": "^9.3.0",
     "picocolors": "^1.1.1",
     "zod": "^3.24.0",
-    "@mmnto/totem": "1.3.16"
+    "@mmnto/totem": "1.3.18"
   },
   "devDependencies": {
     "@anthropic-ai/sdk": "^0.78.0",