fix(openclaw): make native skills codex-friendly (#864)

Normalizes YAML frontmatter on the 4 hand-authored OpenClaw skills so stricter
parsers like Codex can load them. Codex CLI was rejecting these files with
"mapping values are not allowed in this context" on colons inside unquoted
description scalars.

- Drops non-standard `version` and `metadata` fields
- Rewrites descriptions into simple "Use when..." form (no inline colons)
- Adds a regression test enforcing strict frontmatter (name + description only)

Verified live: Codex CLI now loads the skills without errors. Observed during
/codex outside-voice run on the eval-community-prs plan review — Codex stderr
tripped on these exact files, which was real-world confirmation the fix is needed.

Dropped the connect-chrome changes from the original PR (the symlink removal is
out of scope for this fix; keeping connect-chrome -> open-gstack-browser).

Co-Authored-By: Cathryn Lavery <cathrynlavery@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

test/openclaw-native-skills.test.ts

@@ -0,0 +1,35 @@
+import { describe, test, expect } from 'bun:test';
+import * as fs from 'fs';
+import * as path from 'path';
+
+const ROOT = path.resolve(import.meta.dir, '..');
+
+const OPENCLAW_NATIVE_SKILLS = [
+  'openclaw/skills/gstack-openclaw-investigate/SKILL.md',
+  'openclaw/skills/gstack-openclaw-office-hours/SKILL.md',
+  'openclaw/skills/gstack-openclaw-ceo-review/SKILL.md',
+  'openclaw/skills/gstack-openclaw-retro/SKILL.md',
+];
+
+function extractFrontmatter(content: string): string {
+  expect(content.startsWith('---\n')).toBe(true);
+  const fmEnd = content.indexOf('\n---', 4);
+  expect(fmEnd).toBeGreaterThan(0);
+  return content.slice(4, fmEnd);
+}
+
+describe('OpenClaw native skills', () => {
+  test('frontmatter parses as YAML and keeps only name + description', () => {
+    for (const skill of OPENCLAW_NATIVE_SKILLS) {
+      const content = fs.readFileSync(path.join(ROOT, skill), 'utf-8');
+      const frontmatter = extractFrontmatter(content);
+      const parsed = Bun.YAML.parse(frontmatter) as Record<string, unknown>;
+
+      expect(Object.keys(parsed).sort()).toEqual(['description', 'name']);
+      expect(typeof parsed.name).toBe('string');
+      expect(typeof parsed.description).toBe('string');
+      expect((parsed.name as string).length).toBeGreaterThan(0);
+      expect((parsed.description as string).length).toBeGreaterThan(0);
+    }
+  });
+});