From e2462592447a7699e1205ef9be14f74636bf294b Mon Sep 17 00:00:00 2001
From: Phil Haack <haacked@gmail.com>
Date: Thu, 23 Jul 2026 16:38:05 -0700
Subject: [PATCH 1/3] feat(skills): support disable-model-invocation skill
 frontmatter

Parse the disable-model-invocation frontmatter field into SkillInfo,
preserve it through manifest saves (previously editing a skill silently
stripped it), emit it from serializeSkillMarkdown, and surface it in the
skills UI as a badge plus an edit toggle.

Generated-By: PostHog Code
Task-Id: b9824961-d68a-4ac2-b272-7053ecc0e497
---
 packages/shared/src/skills.ts                 | 13 ++++-
 packages/ui/src/features/skills/SkillCard.tsx |  7 +++
 .../src/features/skills/SkillDetailPanel.tsx  |  9 ++++
 .../features/skills/SkillManifestEditor.tsx   | 30 +++++++++++-
 .../skills/parse-skill-frontmatter.test.ts    | 40 +++++++++++++++-
 .../skills/parse-skill-frontmatter.ts         | 20 ++++++--
 .../src/services/skills/schemas.ts            |  4 ++
 .../src/services/skills/skill-discovery.ts    |  5 +-
 .../src/services/skills/skills.test.ts        | 47 +++++++++++++++++++
 .../src/services/skills/skills.ts             | 13 ++++-
 .../skills/write-skill-frontmatter.test.ts    | 32 ++++++++++++-
 11 files changed, 208 insertions(+), 12 deletions(-)

diff --git a/packages/shared/src/skills.ts b/packages/shared/src/skills.ts
index 540b92eba5..08be3205ae 100644
--- a/packages/shared/src/skills.ts
+++ b/packages/shared/src/skills.ts
@@ -11,6 +11,12 @@ export interface SkillInfo {
   editable: boolean;
   /** Size of SKILL.md in bytes (context-cost signal). */
   skillMdBytes: number;
+  /**
+   * Frontmatter `disable-model-invocation: true`: the agent never invokes the
+   * skill on its own — only an explicit user invocation (slash command or
+   * composer skill tag) runs it.
+   */
+  disableModelInvocation?: boolean;
 }
 
 export interface SkillFileEntry {
@@ -44,13 +50,18 @@ export interface ExportedSkill {
  * sandbox, so it must not drift between hosts.
  */
 export function serializeSkillMarkdown(
-  meta: { name: string; description: string },
+  meta: {
+    name: string;
+    description: string;
+    disableModelInvocation?: boolean;
+  },
   body: string,
 ): string {
   const frontmatter = [
     "---",
     `name: ${serializeSkillScalar(meta.name)}`,
     `description: ${serializeSkillScalar(meta.description)}`,
+    ...(meta.disableModelInvocation ? ["disable-model-invocation: true"] : []),
     "---",
   ].join("\n");
 
diff --git a/packages/ui/src/features/skills/SkillCard.tsx b/packages/ui/src/features/skills/SkillCard.tsx
index 5f896404a9..f3cd49f79f 100644
--- a/packages/ui/src/features/skills/SkillCard.tsx
+++ b/packages/ui/src/features/skills/SkillCard.tsx
@@ -92,6 +92,13 @@ export function SkillCard({
               {skill.repoName}
             </Badge>
           )}
+          {skill.disableModelInvocation && (
+            <Tooltip content="The agent won't use this skill on its own — only when you invoke it">
+              <Badge size="1" variant="soft" color="gray" className="shrink-0">
+                Manual
+              </Badge>
+            </Tooltip>
+          )}
         </>
       }
     />
diff --git a/packages/ui/src/features/skills/SkillDetailPanel.tsx b/packages/ui/src/features/skills/SkillDetailPanel.tsx
index 6be4fd58e3..0011942ee8 100644
--- a/packages/ui/src/features/skills/SkillDetailPanel.tsx
+++ b/packages/ui/src/features/skills/SkillDetailPanel.tsx
@@ -3,6 +3,7 @@ import {
   DownloadSimple,
   FilePlus,
   Folder,
+  HandTap,
   LockSimple,
   PencilSimple,
   Trash,
@@ -287,6 +288,14 @@ export function SkillDetailPanel({
               Read-only
             </Badge>
           )}
+          {skill.disableModelInvocation && (
+            <Tooltip content="The agent won't use this skill on its own — only when you invoke it">
+              <Badge size="1" variant="soft" color="gray">
+                <HandTap size={10} className="text-gray-9" />
+                Manual-only
+              </Badge>
+            </Tooltip>
+          )}
           {skill.source === "codex" && (
             <Button
               size="1"
diff --git a/packages/ui/src/features/skills/SkillManifestEditor.tsx b/packages/ui/src/features/skills/SkillManifestEditor.tsx
index f048a1858e..b4d6fd005f 100644
--- a/packages/ui/src/features/skills/SkillManifestEditor.tsx
+++ b/packages/ui/src/features/skills/SkillManifestEditor.tsx
@@ -1,6 +1,14 @@
 import type { SkillInfo } from "@posthog/shared";
 import { toast } from "@posthog/ui/primitives/toast";
-import { Box, Button, Flex, Text, TextArea, TextField } from "@radix-ui/themes";
+import {
+  Box,
+  Button,
+  Flex,
+  Switch,
+  Text,
+  TextArea,
+  TextField,
+} from "@radix-ui/themes";
 import { useRef, useState } from "react";
 import { SkillCodeEditor } from "./SkillCodeEditor";
 import { skillErrorDescription } from "./skillErrors";
@@ -25,6 +33,9 @@ export function SkillManifestEditor({
 }: SkillManifestEditorProps) {
   const [name, setName] = useState(skill.name);
   const [description, setDescription] = useState(skill.description);
+  const [disableModelInvocation, setDisableModelInvocation] = useState(
+    skill.disableModelInvocation ?? false,
+  );
   // Captured at mount: background refetches must not reset in-flight edits.
   const [mountedBody] = useState(initialBody);
   const bodyRef = useRef(mountedBody);
@@ -37,6 +48,7 @@ export function SkillManifestEditor({
         name,
         description,
         body: bodyRef.current,
+        disableModelInvocation,
       });
       onSaved();
     } catch (error) {
@@ -72,6 +84,22 @@ export function SkillManifestEditor({
             placeholder="When should an agent use this skill?"
           />
         </Box>
+        <Flex align="center" justify="between" gap="2">
+          <Box>
+            <Text className="block text-[12px] text-gray-12">
+              Manual invocation only
+            </Text>
+            <Text className="block text-[11px] text-gray-10">
+              The agent won't use this skill on its own — only when you invoke
+              it
+            </Text>
+          </Box>
+          <Switch
+            size="1"
+            checked={disableModelInvocation}
+            onCheckedChange={setDisableModelInvocation}
+          />
+        </Flex>
       </Flex>
 
       <Box className="min-h-0 flex-1 border-t border-t-(--gray-5)">
diff --git a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
index 43b336bf51..2baafb758e 100644
--- a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
+++ b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
@@ -1,5 +1,43 @@
 import { describe, expect, it } from "vitest";
-import { parseSkillDependencies } from "./parse-skill-frontmatter";
+import {
+  parseSkillDependencies,
+  parseSkillFrontmatter,
+} from "./parse-skill-frontmatter";
+
+describe("parseSkillFrontmatter disable-model-invocation", () => {
+  it.each([
+    ["absent", `---\nname: a\ndescription: d\n---\nbody`, false],
+    [
+      "true",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: true\n---\nbody`,
+      true,
+    ],
+    [
+      "capitalized True",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: True\n---\nbody`,
+      true,
+    ],
+    [
+      "quoted true",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: "true"\n---\nbody`,
+      true,
+    ],
+    [
+      "false",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: false\n---\nbody`,
+      false,
+    ],
+    [
+      "non-boolean value",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: maybe\n---\nbody`,
+      false,
+    ],
+  ])("parses %s", (_label, content, expected) => {
+    expect(parseSkillFrontmatter(content)?.disableModelInvocation).toBe(
+      expected,
+    );
+  });
+});
 
 describe("parseSkillDependencies", () => {
   it.each([
diff --git a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
index 0704a3cb0a..5fa4344983 100644
--- a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
+++ b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
@@ -1,15 +1,17 @@
 /**
  * Parses YAML frontmatter from a SKILL.md file.
- * Extracts `name` and `description` fields.
+ * Extracts `name`, `description`, and `disable-model-invocation` fields.
  *
  * Handles:
  * - Simple values: `name: my-skill`
  * - Quoted strings: `description: 'Some text'` or `description: "Some text"`
  * - Multi-line folded: `description: >-\n  line1\n  line2`
  */
-export function parseSkillFrontmatter(
-  content: string,
-): { name: string; description: string } | null {
+export function parseSkillFrontmatter(content: string): {
+  name: string;
+  description: string;
+  disableModelInvocation: boolean;
+} | null {
   const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
   if (!match) return null;
 
@@ -18,7 +20,15 @@ export function parseSkillFrontmatter(
   if (!name) return null;
 
   const description = extractYamlValue(yaml, "description") ?? "";
-  return { name, description };
+  const disableModelInvocation = parseYamlBoolean(
+    extractYamlValue(yaml, "disable-model-invocation"),
+  );
+  return { name, description, disableModelInvocation };
+}
+
+/** YAML 1.2 core-schema booleans: `true`/`True`/`TRUE` (quoted forms too). */
+function parseYamlBoolean(value: string | null): boolean {
+  return value !== null && value.toLowerCase() === "true";
 }
 
 /**
diff --git a/packages/workspace-server/src/services/skills/schemas.ts b/packages/workspace-server/src/services/skills/schemas.ts
index 7c08afadd5..9a4e0cbd83 100644
--- a/packages/workspace-server/src/services/skills/schemas.ts
+++ b/packages/workspace-server/src/services/skills/schemas.ts
@@ -17,6 +17,9 @@ export const skillInfo = z.object({
   repoName: z.string().optional(),
   editable: z.boolean(),
   skillMdBytes: z.number(),
+  // Frontmatter `disable-model-invocation: true`: only explicit user
+  // invocation runs the skill; the agent never picks it on its own.
+  disableModelInvocation: z.boolean().optional(),
 });
 
 export const listSkillsOutput = z.array(skillInfo);
@@ -59,6 +62,7 @@ export const saveSkillManifestInput = z.object({
   name: z.string(),
   description: z.string(),
   body: z.string(),
+  disableModelInvocation: z.boolean().optional(),
 });
 
 export const saveSkillFileInput = z.object({
diff --git a/packages/workspace-server/src/services/skills/skill-discovery.ts b/packages/workspace-server/src/services/skills/skill-discovery.ts
index 7bb0f7c679..72ea3fcd95 100644
--- a/packages/workspace-server/src/services/skills/skill-discovery.ts
+++ b/packages/workspace-server/src/services/skills/skill-discovery.ts
@@ -176,7 +176,7 @@ export async function readSkillMetadataFromDir(
   if (skillNames.length === 0) return [];
 
   const results = await Promise.all(
-    skillNames.map(async (skillName) => {
+    skillNames.map(async (skillName): Promise<SkillInfo | null> => {
       const skillPath = path.join(skillsDir, skillName);
       try {
         const content = await fs.promises.readFile(
@@ -192,6 +192,9 @@ export async function readSkillMetadataFromDir(
           ...(repoName ? { repoName } : {}),
           editable: isEditableSource(source),
           skillMdBytes: Buffer.byteLength(content, "utf-8"),
+          ...(frontmatter?.disableModelInvocation
+            ? { disableModelInvocation: true }
+            : {}),
         } satisfies SkillInfo;
       } catch {
         return null;
diff --git a/packages/workspace-server/src/services/skills/skills.test.ts b/packages/workspace-server/src/services/skills/skills.test.ts
index dc06bf1b91..de26712bcb 100644
--- a/packages/workspace-server/src/services/skills/skills.test.ts
+++ b/packages/workspace-server/src/services/skills/skills.test.ts
@@ -75,6 +75,24 @@ describe("listSkills", () => {
     expect(repoSkill?.editable).toBe(true);
     expect(bundledSkill?.editable).toBe(false);
   });
+
+  it("surfaces disable-model-invocation from frontmatter", async () => {
+    await createSkill(
+      repoSkillsDir,
+      "manual-skill",
+      `---\nname: manual-skill\ndescription: d\ndisable-model-invocation: true\n---\nbody`,
+    );
+    await createSkill(repoSkillsDir, "auto-skill");
+
+    const skills = await makeService().listSkills();
+
+    expect(
+      skills.find((s) => s.name === "manual-skill")?.disableModelInvocation,
+    ).toBe(true);
+    expect(
+      skills.find((s) => s.name === "auto-skill")?.disableModelInvocation,
+    ).toBeUndefined();
+  });
 });
 
 describe("getSkillContents", () => {
@@ -555,6 +573,35 @@ describe("skill mutations", () => {
     expect(content).toContain("# Alpha");
   });
 
+  it("writes and clears disable-model-invocation through manifest saves", async () => {
+    const skillPath = await createSkill(repoSkillsDir, "alpha");
+    const service = makeService();
+
+    await service.saveSkillManifest(skillPath, {
+      name: "alpha",
+      description: "d",
+      body: "body",
+      disableModelInvocation: true,
+    });
+    let skills = await service.listSkills();
+    expect(
+      skills.find((s) => s.path === skillPath)?.disableModelInvocation,
+    ).toBe(true);
+
+    await service.saveSkillManifest(skillPath, {
+      name: "alpha",
+      description: "d",
+      body: "body",
+      disableModelInvocation: false,
+    });
+    skills = await service.listSkills();
+    expect(
+      skills.find((s) => s.path === skillPath)?.disableModelInvocation,
+    ).toBeUndefined();
+    const content = await service.readSkillFile(skillPath, "SKILL.md");
+    expect(content).not.toContain("disable-model-invocation");
+  });
+
   it("rejects manifest saves without a name", async () => {
     const skillPath = await createSkill(repoSkillsDir, "alpha");
 
diff --git a/packages/workspace-server/src/services/skills/skills.ts b/packages/workspace-server/src/services/skills/skills.ts
index d5de57be0c..86472181f6 100644
--- a/packages/workspace-server/src/services/skills/skills.ts
+++ b/packages/workspace-server/src/services/skills/skills.ts
@@ -133,11 +133,20 @@ export class SkillsService {
 
   async saveSkillManifest(
     skillPath: string,
-    manifest: { name: string; description: string; body: string },
+    manifest: {
+      name: string;
+      description: string;
+      body: string;
+      disableModelInvocation?: boolean;
+    },
   ): Promise<void> {
     const skillDir = await this.resolveWritableSkillDir(skillPath);
     const content = serializeSkillMarkdown(
-      { name: manifest.name.trim(), description: manifest.description.trim() },
+      {
+        name: manifest.name.trim(),
+        description: manifest.description.trim(),
+        disableModelInvocation: manifest.disableModelInvocation ?? false,
+      },
       manifest.body,
     );
     // The writer and parser must agree, or the skill vanishes from the list.
diff --git a/packages/workspace-server/src/services/skills/write-skill-frontmatter.test.ts b/packages/workspace-server/src/services/skills/write-skill-frontmatter.test.ts
index ed4b6eae46..7f1967edd0 100644
--- a/packages/workspace-server/src/services/skills/write-skill-frontmatter.test.ts
+++ b/packages/workspace-server/src/services/skills/write-skill-frontmatter.test.ts
@@ -28,10 +28,40 @@ describe("serializeSkillMarkdown", () => {
 
       const parsed = parseSkillFrontmatter(content);
 
-      expect(parsed).toEqual({ name, description });
+      expect(parsed).toEqual({
+        name,
+        description,
+        disableModelInvocation: false,
+      });
     },
   );
 
+  it("emits disable-model-invocation and round-trips it", () => {
+    const content = serializeSkillMarkdown(
+      { name: "my-skill", description: "d", disableModelInvocation: true },
+      "The body",
+    );
+
+    expect(content).toBe(
+      "---\nname: my-skill\ndescription: d\ndisable-model-invocation: true\n---\n\nThe body\n",
+    );
+    expect(parseSkillFrontmatter(content)).toEqual({
+      name: "my-skill",
+      description: "d",
+      disableModelInvocation: true,
+    });
+  });
+
+  it("omits disable-model-invocation when false", () => {
+    const content = serializeSkillMarkdown(
+      { name: "my-skill", description: "d", disableModelInvocation: false },
+      "The body",
+    );
+
+    expect(content).not.toContain("disable-model-invocation");
+    expect(parseSkillFrontmatter(content)?.disableModelInvocation).toBe(false);
+  });
+
   it("appends the body after the frontmatter with a trailing newline", () => {
     const content = serializeSkillMarkdown(
       { name: "my-skill", description: "d" },

From e9f211d189d495fc7f188360874a0c4bf5fcd684 Mon Sep 17 00:00:00 2001
From: Phil Haack <haacked@gmail.com>
Date: Thu, 23 Jul 2026 17:19:27 -0700
Subject: [PATCH 2/3] fix(skills): strip trailing YAML comments when parsing
 disable-model-invocation

A value like `disable-model-invocation: true  # manual only` misparsed
to false because extractYamlValue keeps trailing comments, and a later
manifest save would then silently drop the field from disk. Strip an
unquoted trailing comment before the boolean comparison and cover the
commented forms with tests.

Generated-By: PostHog Code
Task-Id: b9824961-d68a-4ac2-b272-7053ecc0e497
---
 .../skills/parse-skill-frontmatter.test.ts        | 15 +++++++++++++++
 .../services/skills/parse-skill-frontmatter.ts    |  8 +++++++-
 2 files changed, 22 insertions(+), 1 deletion(-)

diff --git a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
index 2baafb758e..fd749fdc74 100644
--- a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
+++ b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
@@ -32,6 +32,21 @@ describe("parseSkillFrontmatter disable-model-invocation", () => {
       `---\nname: a\ndescription: d\ndisable-model-invocation: maybe\n---\nbody`,
       false,
     ],
+    [
+      "true with trailing comment",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: true  # manual only\n---\nbody`,
+      true,
+    ],
+    [
+      "quoted true with trailing comment",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: "true" # manual only\n---\nbody`,
+      true,
+    ],
+    [
+      "false with trailing comment",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: false # keep automatic\n---\nbody`,
+      false,
+    ],
   ])("parses %s", (_label, content, expected) => {
     expect(parseSkillFrontmatter(content)?.disableModelInvocation).toBe(
       expected,
diff --git a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
index 5fa4344983..419b1dd4b5 100644
--- a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
+++ b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
@@ -28,7 +28,13 @@ export function parseSkillFrontmatter(content: string): {
 
 /** YAML 1.2 core-schema booleans: `true`/`True`/`TRUE` (quoted forms too). */
 function parseYamlBoolean(value: string | null): boolean {
-  return value !== null && value.toLowerCase() === "true";
+  if (value === null) return false;
+  // extractYamlValue's plain-scalar path keeps trailing `# ...` comments
+  // (YAML only treats `#` as a comment after whitespace), so strip one here —
+  // otherwise `true  # manual only` misparses to false and a later manifest
+  // save would silently drop the field from disk.
+  const withoutComment = value.replace(/\s+#.*$/, "").trim();
+  return unquoteYamlScalar(withoutComment).toLowerCase() === "true";
 }
 
 /**

From ad3a9703e37a22e45eb7faf0680c25e7a352d104 Mon Sep 17 00:00:00 2001
From: Phil Haack <haacked@gmail.com>
Date: Thu, 23 Jul 2026 17:36:16 -0700
Subject: [PATCH 3/3] fix(skills): strip trailing YAML comments before
 unquoting, not after

Comment stripping in parseYamlBoolean ran on the already-unquoted value,
so a quoted non-boolean like "true # manual only" was mangled into
boolean true. Move comment handling into extractYamlValue's plain-scalar
path (quoted content is literal in YAML and never contains a comment)
and add a regression test.

Generated-By: PostHog Code
Task-Id: b9824961-d68a-4ac2-b272-7053ecc0e497
---
 .../skills/parse-skill-frontmatter.test.ts      |  5 +++++
 .../services/skills/parse-skill-frontmatter.ts  | 17 +++++++----------
 2 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
index fd749fdc74..448d2eab04 100644
--- a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
+++ b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.test.ts
@@ -47,6 +47,11 @@ describe("parseSkillFrontmatter disable-model-invocation", () => {
       `---\nname: a\ndescription: d\ndisable-model-invocation: false # keep automatic\n---\nbody`,
       false,
     ],
+    [
+      "quoted string that only starts with true",
+      `---\nname: a\ndescription: d\ndisable-model-invocation: "true # manual only"\n---\nbody`,
+      false,
+    ],
   ])("parses %s", (_label, content, expected) => {
     expect(parseSkillFrontmatter(content)?.disableModelInvocation).toBe(
       expected,
diff --git a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
index 419b1dd4b5..caebb4c743 100644
--- a/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
+++ b/packages/workspace-server/src/services/skills/parse-skill-frontmatter.ts
@@ -28,13 +28,7 @@ export function parseSkillFrontmatter(content: string): {
 
 /** YAML 1.2 core-schema booleans: `true`/`True`/`TRUE` (quoted forms too). */
 function parseYamlBoolean(value: string | null): boolean {
-  if (value === null) return false;
-  // extractYamlValue's plain-scalar path keeps trailing `# ...` comments
-  // (YAML only treats `#` as a comment after whitespace), so strip one here —
-  // otherwise `true  # manual only` misparses to false and a later manifest
-  // save would silently drop the field from disk.
-  const withoutComment = value.replace(/\s+#.*$/, "").trim();
-  return unquoteYamlScalar(withoutComment).toLowerCase() === "true";
+  return value !== null && value.toLowerCase() === "true";
 }
 
 /**
@@ -116,7 +110,8 @@ function extractYamlValue(yaml: string, key: string): string | null {
       return collectIndentedLines(lines, i + 1).join("\n");
     }
 
-    // Quoted string (single or double)
+    // Quoted string (single or double). Quoted content is literal in YAML —
+    // a `#` inside never starts a comment, so no comment stripping here.
     if (
       (rawValue.startsWith("'") && rawValue.endsWith("'")) ||
       (rawValue.startsWith('"') && rawValue.endsWith('"'))
@@ -124,8 +119,10 @@ function extractYamlValue(yaml: string, key: string): string | null {
       return rawValue.slice(1, -1);
     }
 
-    // Plain scalar
-    return rawValue;
+    // Plain scalar: a `#` preceded by whitespace starts a trailing comment.
+    // Stripping it may uncover a quoted scalar (`"true" # manual only`).
+    const withoutComment = rawValue.replace(/\s+#.*$/, "").trim();
+    return unquoteYamlScalar(withoutComment);
   }
 
   return null;