fix(daemon): parse sequence mappings by unquoted colon

Detect single-line sequence mappings only when the colon is outside
quotes and followed by whitespace or end-of-string. This keeps quoted
trigger phrases like "1:1 figma" as scalars while preserving quoted-key
mappings such as - "name": "foo".

Author: saidyildiz

apps/daemon/src/design-systems/frontmatter.ts

@@ -79,12 +79,13 @@ function parseYamlSubset(src: string): FrontmatterObject {
           continue;
         }
       }
-      // Quoted scalars keep their colons (e.g. `- "1:1 figma"`). Only unquoted
-      // `key: value` items are single-line mapping entries in a sequence.
-      if (value.includes(':') && !isQuotedScalar(value)) {
+      // Single-line sequence mappings use a colon outside quotes (e.g. `- k: v`,
+      // `- "name": "foo"`). Colons inside quoted scalars (e.g. `- "1:1 figma"`)
+      // must not open a mapping — skills catalogue triggers rely on that.
+      const colonIdx = findSequenceMappingColon(value);
+      if (colonIdx !== -1) {
         const obj: FrontmatterObject = {};
-        const colonIdx = value.indexOf(':');
-        const key = value.slice(0, colonIdx).trim();
+        const key = unquoteScalar(value.slice(0, colonIdx).trim());
         const valRaw = value.slice(colonIdx + 1).trim();
         if (valRaw) obj[key] = coerce(valRaw);
         if (!Array.isArray(container)) throw new Error('frontmatter array container expected');
@@ -205,6 +206,37 @@ function isQuotedScalar(raw: string): boolean {
   );
 }
 
+/** Strip surrounding quotes from a scalar; leave other text unchanged. */
+function unquoteScalar(raw: string): string {
+  const v = raw.trim();
+  return isQuotedScalar(v) ? v.slice(1, -1) : v;
+}
+
+/**
+ * Index of a single-line mapping separator in a sequence item, or -1.
+ * Only colons outside quotes count, and the colon must be followed by
+ * whitespace or end-of-string (this parser's dash-prefixed mapping subset).
+ */
+function findSequenceMappingColon(raw: string): number {
+  let quote: '"' | "'" | null = null;
+  for (let i = 0; i < raw.length; i++) {
+    const ch = raw[i] ?? '';
+    if (quote) {
+      if (ch === quote) quote = null;
+      continue;
+    }
+    if (ch === '"' || ch === "'") {
+      quote = ch;
+      continue;
+    }
+    if (ch === ':') {
+      const next = raw[i + 1];
+      if (next === undefined || /\s/.test(next)) return i;
+    }
+  }
+  return -1;
+}
+
 function coerce(raw: string | undefined): FrontmatterValue {
   if (raw === undefined) return '';
   let v = raw.trim();

apps/daemon/tests/frontmatter.test.ts

@@ -100,6 +100,16 @@ describe('parseFrontmatter block sequences and arrays', () => {
     ]);
   });
 
+  // Quoted keys must still form mappings (`- "name": "foo"`). A whole-value
+  // isQuotedScalar gate would mis-classify this as a scalar because the item
+  // both starts and ends with a quote.
+  it('parses quoted-key sequence mappings', () => {
+    expect(data('items:\n  - "name": "foo"\n  - \'id\': 1').items).toEqual([
+      { name: 'foo' },
+      { id: 1 },
+    ]);
+  });
+
   it('does not split inline-array elements on commas inside quotes', () => {
     expect(data('a: ["a,b", "c"]').a).toEqual(['a,b', 'c']);
     expect(data("a: ['x, y', z]").a).toEqual(['x, y', 'z']);

packages/plugin-runtime/src/parsers/frontmatter.ts

@@ -119,12 +119,13 @@ function parseYamlSubset(src: string): FrontmatterObject {
           continue;
         }
       }
-      // Quoted scalars keep their colons (e.g. `- "1:1 figma"`). Only unquoted
-      // `key: value` items are single-line mapping entries in a sequence.
-      if (value.includes(':') && !isQuotedScalar(value)) {
+      // Single-line sequence mappings use a colon outside quotes (e.g. `- k: v`,
+      // `- "name": "foo"`). Colons inside quoted scalars (e.g. `- "1:1 figma"`)
+      // must not open a mapping — skills catalogue triggers rely on that.
+      const colonIdx = findSequenceMappingColon(value);
+      if (colonIdx !== -1) {
         const obj: FrontmatterObject = {};
-        const colonIdx = value.indexOf(':');
-        const key = value.slice(0, colonIdx).trim();
+        const key = unquoteScalar(value.slice(0, colonIdx).trim());
         const valRaw = value.slice(colonIdx + 1).trim();
         if (valRaw) obj[key] = coerce(valRaw);
         if (!Array.isArray(container)) throw new Error('frontmatter array container expected');
@@ -244,6 +245,37 @@ function isQuotedScalar(raw: string): boolean {
   );
 }
 
+/** Strip surrounding quotes from a scalar; leave other text unchanged. */
+function unquoteScalar(raw: string): string {
+  const v = raw.trim();
+  return isQuotedScalar(v) ? v.slice(1, -1) : v;
+}
+
+/**
+ * Index of a single-line mapping separator in a sequence item, or -1.
+ * Only colons outside quotes count, and the colon must be followed by
+ * whitespace or end-of-string (this parser's dash-prefixed mapping subset).
+ */
+function findSequenceMappingColon(raw: string): number {
+  let quote: '"' | "'" | null = null;
+  for (let i = 0; i < raw.length; i++) {
+    const ch = raw[i] ?? '';
+    if (quote) {
+      if (ch === quote) quote = null;
+      continue;
+    }
+    if (ch === '"' || ch === "'") {
+      quote = ch;
+      continue;
+    }
+    if (ch === ':') {
+      const next = raw[i + 1];
+      if (next === undefined || /\s/.test(next)) return i;
+    }
+  }
+  return -1;
+}
+
 function coerce(raw: string | undefined): FrontmatterValue {
   if (raw === undefined) return '';
   const v = raw.trim();

packages/plugin-runtime/tests/parsers.test.ts

@@ -189,6 +189,13 @@ describe('parseFrontmatter', () => {
     expect(data['items']).toEqual([{ k: 'v' }, { name: 'foo' }]);
   });
 
+  it('parses quoted-key sequence mappings', () => {
+    const { data } = parseFrontmatter(
+      '---\nitems:\n  - "name": "foo"\n  - \'id\': 1\n---\n',
+    );
+    expect(data['items']).toEqual([{ name: 'foo' }, { id: 1 }]);
+  });
+
   it('does not split inline-array elements on commas inside quotes', () => {
     const { data } = parseFrontmatter('---\na: ["a,b", "c"]\nb: [\'x, y\', z]\n---\n');
     expect(data['a']).toEqual(['a,b', 'c']);