launches/gtm-plan/SKILL.md deviates from the mandatory SKILL.md structure (YAML frontmatter + missing `## When to invoke`)

[ortonom]

Was reading through the skills and spotted one small thing that doesn't quite line up — figured you'd want a heads-up.

ARCHITECTURE.md says the SKILL.md structure is uniform: "Every SKILL.md in this repo follows the same structure." Two small things in launches/gtm-plan/SKILL.md don't quite match it.

1. YAML frontmatter. The spec says "No YAML frontmatter. The file starts with the heading." — but this file opens with a --- / name: / description: block.

2. No ## When to invoke. The spec puts trigger phrases in a ## When to invoke section right after the intro; here they're living inside the frontmatter description: instead. The two go together — dropping the frontmatter just means moving those phrases into the section the spec already expects.

We poked at it to be sure it wasn't an intentional always-on skill — doesn't look like it:

Check	What we saw
The always-on root, foundation/marketing-os (loaded by every skill)	# Marketing OS, no frontmatter, has ## When to invoke
Is gtm-plan always-loaded?	Intro reads "Build the strategy… Use it after the launch brief…"; it's a plain node in the dependency graph, not marked (always)
Frontmatter across all 62 SKILL.md files	gtm-plan is the only one

So it's the lone outlier — small fix either way. Happy to open a PR if that's useful.