@pennyfarthing/shared 7.5.0 → 9.3.0

package/dist/marker/constants.js

@@ -0,0 +1,46 @@
+/**
+ * Constants for the Reflector marker system.
+ *
+ * @see docs/adr/0011-reflector-marker-consolidation.md
+ */
+/**
+ * Regex pattern for CYCLIST markers.
+ *
+ * Format: <!-- CYCLIST:TYPE:value --> or <!-- CYCLIST:TYPE --> (for CONTINUE)
+ *
+ * - Case-insensitive for CYCLIST prefix and TYPE
+ * - Preserves value case
+ * - Handles whitespace variations
+ * - Value is optional (CONTINUE marker has no value)
+ *
+ * Groups:
+ * - [1] = TYPE (e.g., HANDOFF, CONTEXT_CLEAR, CONTINUE)
+ * - [2] = value (e.g., /dev, yesno, 1,2,3) - undefined for CONTINUE
+ *
+ * IMPORTANT: Reset lastIndex before each use since this is a global regex.
+ */
+export const MARKER_PATTERN = /<!--\s*CYCLIST:(\w+)(?::([^>]+?))?\s*-->/gi;
+/**
+ * Known marker type constants.
+ * Use these for type-safe comparisons instead of string literals.
+ */
+export const MARKER_TYPES = {
+    HANDOFF: 'handoff',
+    CONTEXT_CLEAR: 'context_clear',
+    INVOKE: 'invoke',
+    QUESTION: 'question',
+    CHOICES: 'choices',
+    CONTINUE: 'continue',
+};
+/**
+ * Valid marker types for validation.
+ */
+export const VALID_MARKER_TYPES = new Set([
+    'handoff',
+    'context_clear',
+    'invoke',
+    'question',
+    'choices',
+    'continue',
+]);
+//# sourceMappingURL=constants.js.map

package/dist/marker/constants.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"constants.js","sourceRoot":"","sources":["../../src/marker/constants.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH;;;;;;;;;;;;;;;GAeG;AACH,MAAM,CAAC,MAAM,cAAc,GAAG,4CAA4C,CAAC;AAE3E;;;GAGG;AACH,MAAM,CAAC,MAAM,YAAY,GAAG;IAC1B,OAAO,EAAE,SAAS;IAClB,aAAa,EAAE,eAAe;IAC9B,MAAM,EAAE,QAAQ;IAChB,QAAQ,EAAE,UAAU;IACpB,OAAO,EAAE,SAAS;IAClB,QAAQ,EAAE,UAAU;CACZ,CAAC;AAEX;;GAEG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAAG,IAAI,GAAG,CAAC;IACxC,SAAS;IACT,eAAe;IACf,QAAQ;IACR,UAAU;IACV,SAAS;IACT,UAAU;CACX,CAAC,CAAC"}

package/dist/marker/continue.test.d.ts

@@ -0,0 +1,12 @@
+/**
+ * Tests for the CONTINUE marker type
+ *
+ * Story: MSSCI-12787 - Implement CYCLIST Marker Parsing and Action Buttons
+ *
+ * The CONTINUE marker signals a status update - the user can continue
+ * or redirect. It has no value parameter.
+ *
+ * Format: <!-- CYCLIST:CONTINUE -->
+ */
+export {};
+//# sourceMappingURL=continue.test.d.ts.map

package/dist/marker/continue.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"continue.test.d.ts","sourceRoot":"","sources":["../../src/marker/continue.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG"}

package/dist/marker/continue.test.js

@@ -0,0 +1,76 @@
+/**
+ * Tests for the CONTINUE marker type
+ *
+ * Story: MSSCI-12787 - Implement CYCLIST Marker Parsing and Action Buttons
+ *
+ * The CONTINUE marker signals a status update - the user can continue
+ * or redirect. It has no value parameter.
+ *
+ * Format: <!-- CYCLIST:CONTINUE -->
+ */
+import { describe, it } from 'node:test';
+import assert from 'node:assert';
+import { detectMarkers, MARKER_TYPES, VALID_MARKER_TYPES, } from './index.js';
+describe('CONTINUE marker type', () => {
+    describe('type definition', () => {
+        it('should include continue in MarkerType', () => {
+            // This test verifies the type includes 'continue'
+            const type = 'continue';
+            assert.strictEqual(type, 'continue');
+        });
+        it('should have CONTINUE constant in MARKER_TYPES', () => {
+            assert.strictEqual(MARKER_TYPES.CONTINUE, 'continue');
+        });
+        it('should include continue in VALID_MARKER_TYPES', () => {
+            assert.ok(VALID_MARKER_TYPES.has('continue'), 'VALID_MARKER_TYPES should include continue');
+        });
+    });
+    describe('detection', () => {
+        it('should detect CONTINUE marker without value', () => {
+            const text = '<!-- CYCLIST:CONTINUE -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect CONTINUE marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'continue');
+            // CONTINUE has no value, should be empty string or null
+            assert.ok(result[0].value === '' || result[0].value === null || result[0].value === undefined, 'CONTINUE marker should have no value');
+        });
+        it('should be case-insensitive for CONTINUE type', () => {
+            const texts = [
+                '<!-- CYCLIST:CONTINUE -->',
+                '<!-- CYCLIST:continue -->',
+                '<!-- CYCLIST:Continue -->',
+            ];
+            for (const text of texts) {
+                const result = detectMarkers(text);
+                assert.ok(result !== null, `Should detect marker in: ${text}`);
+                assert.strictEqual(result[0].type, 'continue', 'Type should be normalized to lowercase');
+            }
+        });
+        it('should detect CONTINUE alongside other markers', () => {
+            const text = `
+Some status update here.
+
+<!-- CYCLIST:CONTINUE -->
+
+Run \`/dev\` to continue
+`;
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect CONTINUE marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'continue');
+        });
+        it('should ignore CONTINUE inside code blocks', () => {
+            const text = `
+Regular text
+\`\`\`
+<!-- CYCLIST:CONTINUE -->
+\`\`\`
+More text
+`;
+            const result = detectMarkers(text);
+            assert.strictEqual(result, null, 'Should not detect marker inside code block');
+        });
+    });
+});
+//# sourceMappingURL=continue.test.js.map

package/dist/marker/continue.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"continue.test.js","sourceRoot":"","sources":["../../src/marker/continue.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,WAAW,CAAC;AACzC,OAAO,MAAM,MAAM,aAAa,CAAC;AACjC,OAAO,EACL,aAAa,EACb,YAAY,EACZ,kBAAkB,GAEnB,MAAM,YAAY,CAAC;AAEpB,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,kDAAkD;YAClD,MAAM,IAAI,GAAe,UAAU,CAAC;YACpC,MAAM,CAAC,WAAW,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;QACvC,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;YACvD,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;QACxD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;YACvD,MAAM,CAAC,EAAE,CAAC,kBAAkB,CAAC,GAAG,CAAC,UAAU,CAAC,EAAE,4CAA4C,CAAC,CAAC;QAC9F,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,WAAW,EAAE,GAAG,EAAE;QACzB,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;YACrD,MAAM,IAAI,GAAG,2BAA2B,CAAC;YACzC,MAAM,MAAM,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC;YAEnC,MAAM,CAAC,EAAE,CAAC,MAAM,KAAK,IAAI,EAAE,+BAA+B,CAAC,CAAC;YAC5D,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YACrC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;YAC/C,wDAAwD;YACxD,MAAM,CAAC,EAAE,CACP,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,KAAK,EAAE,IAAI,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,KAAK,IAAI,IAAI,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,KAAK,SAAS,EACnF,sCAAsC,CACvC,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;YACtD,MAAM,KAAK,GAAG;gBACZ,2BAA2B;gBAC3B,2BAA2B;gBAC3B,2BAA2B;aAC5B,CAAC;YAEF,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;gBACzB,MAAM,MAAM,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC;gBACnC,MAAM,CAAC,EAAE,CAAC,MAAM,KAAK,IAAI,EAAE,4BAA4B,IAAI,EAAE,CAAC,CAAC;gBAC/D,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,UAAU,EAAE,wCAAwC,CAAC,CAAC;YAC3F,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,IAAI,GAAG;;;;;;CAMlB,CAAC;YACI,MAAM,MAAM,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC;YAEnC,MAAM,CAAC,EAAE,CAAC,MAAM,KAAK,IAAI,EAAE,+BAA+B,CAAC,CAAC;YAC5D,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;YACrC,MAAM,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;QACjD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;YACnD,MAAM,IAAI,GAAG;;;;;;CAMlB,CAAC;YACI,MAAM,MAAM,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC;YACnC,MAAM,CAAC,WAAW,CAAC,MAAM,EAAE,IAAI,EAAE,4CAA4C,CAAC,CAAC;QACjF,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/marker/detect.d.ts

@@ -0,0 +1,25 @@
+/**
+ * Marker detection for the Reflector protocol.
+ *
+ * @see docs/adr/0011-reflector-marker-consolidation.md
+ */
+import type { Marker } from './types.js';
+/**
+ * Detect CYCLIST markers in text.
+ *
+ * Scans text for CYCLIST markers and returns an array of parsed markers.
+ * Code blocks are stripped before detection to prevent false positives
+ * from markers appearing inside code examples.
+ *
+ * @param text - Text to scan for markers
+ * @returns Array of markers found, or null if none
+ *
+ * @example
+ * ```typescript
+ * const text = '<!-- CYCLIST:HANDOFF:/dev -->';
+ * const markers = detectMarkers(text);
+ * // markers = [{ type: 'handoff', value: '/dev', source: 'structured_marker' }]
+ * ```
+ */
+export declare function detectMarkers(text: string): Marker[] | null;
+//# sourceMappingURL=detect.d.ts.map

package/dist/marker/detect.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"detect.d.ts","sourceRoot":"","sources":["../../src/marker/detect.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAIH,OAAO,KAAK,EAAE,MAAM,EAAc,MAAM,YAAY,CAAC;AAErD;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,EAAE,GAAG,IAAI,CAkC3D"}

package/dist/marker/detect.js

@@ -0,0 +1,53 @@
+/**
+ * Marker detection for the Reflector protocol.
+ *
+ * @see docs/adr/0011-reflector-marker-consolidation.md
+ */
+import { MARKER_PATTERN, VALID_MARKER_TYPES } from './constants.js';
+import { stripCodeBlocks } from './strip.js';
+/**
+ * Detect CYCLIST markers in text.
+ *
+ * Scans text for CYCLIST markers and returns an array of parsed markers.
+ * Code blocks are stripped before detection to prevent false positives
+ * from markers appearing inside code examples.
+ *
+ * @param text - Text to scan for markers
+ * @returns Array of markers found, or null if none
+ *
+ * @example
+ * ```typescript
+ * const text = '<!-- CYCLIST:HANDOFF:/dev -->';
+ * const markers = detectMarkers(text);
+ * // markers = [{ type: 'handoff', value: '/dev', source: 'structured_marker' }]
+ * ```
+ */
+export function detectMarkers(text) {
+    // Handle null/undefined/empty input
+    if (!text) {
+        return null;
+    }
+    // Strip code blocks first - markers inside code should be ignored
+    const withoutCode = stripCodeBlocks(text);
+    if (!withoutCode.trim()) {
+        return null;
+    }
+    const markers = [];
+    // Reset lastIndex for global regex
+    MARKER_PATTERN.lastIndex = 0;
+    let match;
+    while ((match = MARKER_PATTERN.exec(withoutCode)) !== null) {
+        const rawType = match[1].toLowerCase();
+        // Skip unknown marker types
+        if (!VALID_MARKER_TYPES.has(rawType)) {
+            continue;
+        }
+        markers.push({
+            type: rawType,
+            value: match[2]?.trim() ?? '',
+            source: 'structured_marker',
+        });
+    }
+    return markers.length > 0 ? markers : null;
+}
+//# sourceMappingURL=detect.js.map

package/dist/marker/detect.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"detect.js","sourceRoot":"","sources":["../../src/marker/detect.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,cAAc,EAAE,kBAAkB,EAAE,MAAM,gBAAgB,CAAC;AACpE,OAAO,EAAE,eAAe,EAAE,MAAM,YAAY,CAAC;AAG7C;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,UAAU,aAAa,CAAC,IAAY;IACxC,oCAAoC;IACpC,IAAI,CAAC,IAAI,EAAE,CAAC;QACV,OAAO,IAAI,CAAC;IACd,CAAC;IAED,kEAAkE;IAClE,MAAM,WAAW,GAAG,eAAe,CAAC,IAAI,CAAC,CAAC;IAC1C,IAAI,CAAC,WAAW,CAAC,IAAI,EAAE,EAAE,CAAC;QACxB,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,OAAO,GAAa,EAAE,CAAC;IAE7B,mCAAmC;IACnC,cAAc,CAAC,SAAS,GAAG,CAAC,CAAC;IAE7B,IAAI,KAAK,CAAC;IACV,OAAO,CAAC,KAAK,GAAG,cAAc,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;QAC3D,MAAM,OAAO,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;QAEvC,4BAA4B;QAC5B,IAAI,CAAC,kBAAkB,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC;YACrC,SAAS;QACX,CAAC;QAED,OAAO,CAAC,IAAI,CAAC;YACX,IAAI,EAAE,OAAqB;YAC3B,KAAK,EAAE,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE;YAC7B,MAAM,EAAE,mBAAmB;SAC5B,CAAC,CAAC;IACL,CAAC;IAED,OAAO,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC;AAC7C,CAAC"}

package/dist/marker/detect.test.d.ts

@@ -0,0 +1,10 @@
+/**
+ * Tests for the marker detection module.
+ *
+ * These tests cover the shared marker parsing logic that will be used by
+ * both Cyclist terminal and VS Code extension.
+ *
+ * @see docs/adr/0011-reflector-marker-consolidation.md
+ */
+export {};
+//# sourceMappingURL=detect.test.d.ts.map

package/dist/marker/detect.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"detect.test.d.ts","sourceRoot":"","sources":["../../src/marker/detect.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG"}

package/dist/marker/detect.test.js

@@ -0,0 +1,418 @@
+/**
+ * Tests for the marker detection module.
+ *
+ * These tests cover the shared marker parsing logic that will be used by
+ * both Cyclist terminal and VS Code extension.
+ *
+ * @see docs/adr/0011-reflector-marker-consolidation.md
+ */
+import { describe, it } from 'node:test';
+import assert from 'node:assert';
+import { detectMarkers, stripMarkers, stripCodeBlocks, MARKER_PATTERN, MARKER_TYPES, VALID_MARKER_TYPES, } from './index.js';
+// =============================================================================
+// Type Tests
+// =============================================================================
+describe('marker types', () => {
+    describe('MarkerType', () => {
+        it('should include handoff type', () => {
+            const type = 'handoff';
+            assert.strictEqual(type, 'handoff');
+        });
+        it('should include context_clear type', () => {
+            const type = 'context_clear';
+            assert.strictEqual(type, 'context_clear');
+        });
+        it('should include invoke type', () => {
+            const type = 'invoke';
+            assert.strictEqual(type, 'invoke');
+        });
+        it('should include question type', () => {
+            const type = 'question';
+            assert.strictEqual(type, 'question');
+        });
+        it('should include choices type', () => {
+            const type = 'choices';
+            assert.strictEqual(type, 'choices');
+        });
+    });
+    describe('Marker interface', () => {
+        it('should have type, value, and optional source fields', () => {
+            const marker = {
+                type: 'handoff',
+                value: '/dev',
+                source: 'structured_marker',
+            };
+            assert.strictEqual(marker.type, 'handoff');
+            assert.strictEqual(marker.value, '/dev');
+            assert.strictEqual(marker.source, 'structured_marker');
+        });
+        it('should allow source to be omitted', () => {
+            const marker = {
+                type: 'invoke',
+                value: '/tea',
+            };
+            assert.strictEqual(marker.type, 'invoke');
+            assert.strictEqual(marker.value, '/tea');
+            assert.strictEqual(marker.source, undefined);
+        });
+    });
+});
+// =============================================================================
+// Constants Tests
+// =============================================================================
+describe('marker constants', () => {
+    describe('MARKER_PATTERN', () => {
+        it('should be a global case-insensitive regex', () => {
+            assert.ok(MARKER_PATTERN instanceof RegExp);
+            assert.ok(MARKER_PATTERN.flags.includes('g'), 'Should have global flag');
+            assert.ok(MARKER_PATTERN.flags.includes('i'), 'Should have case-insensitive flag');
+        });
+        it('should match basic CYCLIST marker format', () => {
+            const text = '<!-- CYCLIST:HANDOFF:/dev -->';
+            MARKER_PATTERN.lastIndex = 0;
+            const match = MARKER_PATTERN.exec(text);
+            assert.ok(match !== null, 'Should match basic marker');
+            assert.strictEqual(match[1], 'HANDOFF');
+            assert.strictEqual(match[2], '/dev');
+        });
+        it('should match with whitespace variations', () => {
+            const text = '<!--  CYCLIST:HANDOFF:  /dev   -->';
+            MARKER_PATTERN.lastIndex = 0;
+            const match = MARKER_PATTERN.exec(text);
+            assert.ok(match !== null, 'Should match with extra whitespace');
+        });
+        it('should match lowercase CYCLIST prefix', () => {
+            const text = '<!-- cyclist:handoff:/dev -->';
+            MARKER_PATTERN.lastIndex = 0;
+            const match = MARKER_PATTERN.exec(text);
+            assert.ok(match !== null, 'Should match lowercase prefix');
+        });
+    });
+    describe('MARKER_TYPES', () => {
+        it('should have HANDOFF constant', () => {
+            assert.strictEqual(MARKER_TYPES.HANDOFF, 'handoff');
+        });
+        it('should have CONTEXT_CLEAR constant', () => {
+            assert.strictEqual(MARKER_TYPES.CONTEXT_CLEAR, 'context_clear');
+        });
+        it('should have INVOKE constant', () => {
+            assert.strictEqual(MARKER_TYPES.INVOKE, 'invoke');
+        });
+        it('should have QUESTION constant', () => {
+            assert.strictEqual(MARKER_TYPES.QUESTION, 'question');
+        });
+        it('should have CHOICES constant', () => {
+            assert.strictEqual(MARKER_TYPES.CHOICES, 'choices');
+        });
+    });
+    describe('VALID_MARKER_TYPES', () => {
+        it('should be a Set', () => {
+            assert.ok(VALID_MARKER_TYPES instanceof Set);
+        });
+        it('should contain all 6 marker types', () => {
+            assert.strictEqual(VALID_MARKER_TYPES.size, 6);
+            assert.ok(VALID_MARKER_TYPES.has('handoff'));
+            assert.ok(VALID_MARKER_TYPES.has('context_clear'));
+            assert.ok(VALID_MARKER_TYPES.has('invoke'));
+            assert.ok(VALID_MARKER_TYPES.has('question'));
+            assert.ok(VALID_MARKER_TYPES.has('choices'));
+            assert.ok(VALID_MARKER_TYPES.has('continue'));
+        });
+        it('should match MARKER_TYPES values', () => {
+            // VALID_MARKER_TYPES should contain exactly the values from MARKER_TYPES
+            const typeValues = Object.values(MARKER_TYPES);
+            assert.strictEqual(VALID_MARKER_TYPES.size, typeValues.length);
+            for (const value of typeValues) {
+                assert.ok(VALID_MARKER_TYPES.has(value), `Should contain ${value}`);
+            }
+        });
+        it('should not contain invalid types', () => {
+            assert.ok(!VALID_MARKER_TYPES.has('invalid'));
+            assert.ok(!VALID_MARKER_TYPES.has('HANDOFF')); // Case sensitive - lowercase only
+            assert.ok(!VALID_MARKER_TYPES.has(''));
+        });
+    });
+});
+// =============================================================================
+// Strip Functions Tests
+// =============================================================================
+describe('stripCodeBlocks', () => {
+    it('should remove single code block', () => {
+        const text = 'Before\n```javascript\nconst x = 1;\n```\nAfter';
+        const result = stripCodeBlocks(text);
+        assert.ok(!result.includes('```'), 'Should not contain code fence');
+        assert.ok(!result.includes('const x = 1'), 'Should not contain code content');
+        assert.ok(result.includes('Before'), 'Should preserve text before');
+        assert.ok(result.includes('After'), 'Should preserve text after');
+    });
+    it('should remove multiple code blocks', () => {
+        const text = '```js\ncode1\n```\ntext\n```ts\ncode2\n```';
+        const result = stripCodeBlocks(text);
+        assert.ok(!result.includes('code1'), 'Should remove first code block');
+        assert.ok(!result.includes('code2'), 'Should remove second code block');
+        assert.ok(result.includes('text'), 'Should preserve text between blocks');
+    });
+    it('should handle text without code blocks', () => {
+        const text = 'Just regular text with no code';
+        const result = stripCodeBlocks(text);
+        assert.strictEqual(result, text);
+    });
+    it('should handle empty string', () => {
+        const result = stripCodeBlocks('');
+        assert.strictEqual(result, '');
+    });
+    it('should handle multiline code blocks', () => {
+        const text = '```\nline1\nline2\nline3\n```';
+        const result = stripCodeBlocks(text);
+        assert.ok(!result.includes('line1'), 'Should remove multiline content');
+        assert.ok(!result.includes('line2'), 'Should remove multiline content');
+    });
+});
+describe('stripMarkers', () => {
+    it('should remove HANDOFF marker', () => {
+        const text = 'Some text <!-- CYCLIST:HANDOFF:/dev --> more text';
+        const result = stripMarkers(text);
+        assert.ok(!result.includes('CYCLIST'), 'Should remove marker');
+        assert.ok(!result.includes('<!--'), 'Should remove HTML comment');
+        assert.ok(result.includes('Some text'), 'Should preserve surrounding text');
+        assert.ok(result.includes('more text'), 'Should preserve surrounding text');
+    });
+    it('should remove CONTEXT_CLEAR marker', () => {
+        const text = '<!-- CYCLIST:CONTEXT_CLEAR:/sm -->';
+        const result = stripMarkers(text);
+        assert.strictEqual(result, '');
+    });
+    it('should remove multiple markers', () => {
+        const text = '<!-- CYCLIST:QUESTION:yesno --> text <!-- CYCLIST:CHOICES:1,2,3 -->';
+        const result = stripMarkers(text);
+        assert.ok(!result.includes('QUESTION'), 'Should remove first marker');
+        assert.ok(!result.includes('CHOICES'), 'Should remove second marker');
+        assert.ok(result.includes('text'), 'Should preserve text between markers');
+    });
+    it('should handle text without markers', () => {
+        const text = 'Regular text with no markers';
+        const result = stripMarkers(text);
+        assert.strictEqual(result, text);
+    });
+    it('should handle empty string', () => {
+        const result = stripMarkers('');
+        assert.strictEqual(result, '');
+    });
+    it('should handle null/undefined input', () => {
+        // @ts-expect-error - testing runtime behavior with invalid input
+        const result = stripMarkers(null);
+        assert.strictEqual(result, '');
+    });
+});
+// =============================================================================
+// Detection Tests
+// =============================================================================
+describe('detectMarkers', () => {
+    describe('input handling', () => {
+        it('should return null for empty string', () => {
+            const result = detectMarkers('');
+            assert.strictEqual(result, null);
+        });
+        it('should return null for null input', () => {
+            // @ts-expect-error - testing runtime behavior with invalid input
+            const result = detectMarkers(null);
+            assert.strictEqual(result, null);
+        });
+        it('should return null for undefined input', () => {
+            // @ts-expect-error - testing runtime behavior with invalid input
+            const result = detectMarkers(undefined);
+            assert.strictEqual(result, null);
+        });
+        it('should return null for whitespace-only input', () => {
+            const result = detectMarkers('   \n\t  ');
+            assert.strictEqual(result, null);
+        });
+        it('should return null for text without markers', () => {
+            const result = detectMarkers('Just regular text with no markers');
+            assert.strictEqual(result, null);
+        });
+    });
+    describe('marker type detection', () => {
+        it('should detect HANDOFF marker', () => {
+            const text = '<!-- CYCLIST:HANDOFF:/dev -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'handoff');
+            assert.strictEqual(result[0].value, '/dev');
+        });
+        it('should detect CONTEXT_CLEAR marker', () => {
+            const text = '<!-- CYCLIST:CONTEXT_CLEAR:/reviewer -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'context_clear');
+            assert.strictEqual(result[0].value, '/reviewer');
+        });
+        it('should detect INVOKE marker', () => {
+            const text = '<!-- CYCLIST:INVOKE:/tea -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'invoke');
+            assert.strictEqual(result[0].value, '/tea');
+        });
+        it('should detect QUESTION marker', () => {
+            const text = '<!-- CYCLIST:QUESTION:yesno -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'question');
+            assert.strictEqual(result[0].value, 'yesno');
+        });
+        it('should detect CHOICES marker with numbers', () => {
+            const text = '<!-- CYCLIST:CHOICES:1,2,3 -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'choices');
+            assert.strictEqual(result[0].value, '1,2,3');
+        });
+        it('should detect CHOICES marker with text labels', () => {
+            const text = '<!-- CYCLIST:CHOICES:Option A,Option B,Option C -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'choices');
+            assert.strictEqual(result[0].value, 'Option A,Option B,Option C');
+        });
+    });
+    describe('multiple markers', () => {
+        it('should detect multiple markers in order', () => {
+            const text = `
+        <!-- CYCLIST:QUESTION:yesno -->
+        Some text here
+        <!-- CYCLIST:CHOICES:1,2 -->
+      `;
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect markers');
+            assert.strictEqual(result.length, 2);
+            assert.strictEqual(result[0].type, 'question');
+            assert.strictEqual(result[1].type, 'choices');
+        });
+        it('should preserve order of markers', () => {
+            const text = '<!-- CYCLIST:HANDOFF:/a --><!-- CYCLIST:INVOKE:/b --><!-- CYCLIST:CONTEXT_CLEAR:/c -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect markers');
+            assert.strictEqual(result.length, 3);
+            assert.strictEqual(result[0].value, '/a');
+            assert.strictEqual(result[1].value, '/b');
+            assert.strictEqual(result[2].value, '/c');
+        });
+    });
+    describe('code block handling', () => {
+        it('should ignore markers inside code blocks', () => {
+            const text = `
+        Regular text
+        \`\`\`
+        <!-- CYCLIST:HANDOFF:/dev -->
+        \`\`\`
+        More text
+      `;
+            const result = detectMarkers(text);
+            assert.strictEqual(result, null, 'Should not detect marker inside code block');
+        });
+        it('should detect markers outside code blocks', () => {
+            const text = `
+        \`\`\`
+        code here
+        \`\`\`
+        <!-- CYCLIST:HANDOFF:/dev -->
+      `;
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker outside code block');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'handoff');
+        });
+        it('should handle mixed code blocks and markers', () => {
+            const text = `
+        <!-- CYCLIST:INVOKE:/tea -->
+        \`\`\`javascript
+        <!-- CYCLIST:HANDOFF:/ignored -->
+        \`\`\`
+        <!-- CYCLIST:CONTEXT_CLEAR:/sm -->
+      `;
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect markers');
+            assert.strictEqual(result.length, 2, 'Should only detect markers outside code');
+            assert.strictEqual(result[0].type, 'invoke');
+            assert.strictEqual(result[1].type, 'context_clear');
+        });
+    });
+    describe('case sensitivity', () => {
+        it('should be case-insensitive for CYCLIST prefix', () => {
+            const texts = [
+                '<!-- CYCLIST:HANDOFF:/dev -->',
+                '<!-- cyclist:HANDOFF:/dev -->',
+                '<!-- Cyclist:HANDOFF:/dev -->',
+                '<!-- CyCLiST:HANDOFF:/dev -->',
+            ];
+            for (const text of texts) {
+                const result = detectMarkers(text);
+                assert.ok(result !== null, `Should detect marker in: ${text}`);
+                assert.strictEqual(result[0].type, 'handoff');
+            }
+        });
+        it('should be case-insensitive for marker type', () => {
+            const texts = [
+                '<!-- CYCLIST:HANDOFF:/dev -->',
+                '<!-- CYCLIST:handoff:/dev -->',
+                '<!-- CYCLIST:Handoff:/dev -->',
+                '<!-- CYCLIST:HaNdOfF:/dev -->',
+            ];
+            for (const text of texts) {
+                const result = detectMarkers(text);
+                assert.ok(result !== null, `Should detect marker in: ${text}`);
+                assert.strictEqual(result[0].type, 'handoff', 'Type should be normalized to lowercase');
+            }
+        });
+        it('should preserve value case', () => {
+            const text = '<!-- CYCLIST:HANDOFF:/MyAgent -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null);
+            assert.strictEqual(result[0].value, '/MyAgent', 'Value case should be preserved');
+        });
+    });
+    describe('whitespace handling', () => {
+        it('should handle whitespace inside marker', () => {
+            const text = '<!--  CYCLIST:HANDOFF:  /dev   -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker with whitespace');
+            assert.strictEqual(result[0].value, '/dev', 'Value should be trimmed');
+        });
+        it('should handle newlines around marker', () => {
+            const text = '\n\n<!-- CYCLIST:INVOKE:/tea -->\n\n';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect marker with newlines');
+            assert.strictEqual(result[0].type, 'invoke');
+        });
+    });
+    describe('source field', () => {
+        it('should set source to structured_marker', () => {
+            const text = '<!-- CYCLIST:HANDOFF:/dev -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null);
+            assert.strictEqual(result[0].source, 'structured_marker');
+        });
+    });
+    describe('unknown marker types', () => {
+        it('should skip unknown marker types', () => {
+            const text = '<!-- CYCLIST:UNKNOWN_TYPE:value -->';
+            const result = detectMarkers(text);
+            assert.strictEqual(result, null, 'Should return null for unknown type');
+        });
+        it('should still detect valid markers alongside unknown ones', () => {
+            const text = '<!-- CYCLIST:INVALID:x --> <!-- CYCLIST:HANDOFF:/dev -->';
+            const result = detectMarkers(text);
+            assert.ok(result !== null, 'Should detect valid marker');
+            assert.strictEqual(result.length, 1);
+            assert.strictEqual(result[0].type, 'handoff');
+        });
+    });
+});
+//# sourceMappingURL=detect.test.js.map