feat(skill): consolidate canonical SKILL.md — ADR Pre-Check, OpenClaw metadata, root SKILL.md

[benw5483]

Summary

Consolidates the actual skill into a single canonical SKILL.md that serves Claude Code, Codex, skills.sh, and ClawHub. This PR started as the ADR Pre-Check fold-in and now carries the remaining consolidation pieces, so the superseded OpenClaw variant no longer holds anything this repository lacks.

Four changes, all additive against main:

• ADR Pre-Check section in skills/actual/SKILL.md:56, unchanged from the version already reviewed on this branch.
• Namespaced OpenClaw frontmatter at skills/actual/SKILL.md:14, the last thing the variant carried that this repository did not.
• Root SKILL.md, a symlink to the canonical file, so a ClawHub publish can run from the repository root without restructuring.
• README gains the ClawHub install path and records the variant repository as superseded.

Conflict resolution

The branch was opened 07-24 and main moved on 07-28, which left this PR CONFLICTING. Nothing semantic went wrong; it's purely positional. Both sides appended immediately after the "After install, verify" line, so both are kept: the flag-verification paragraph from main stays with the install context it belongs to, and ADR Pre-Check keeps its place ahead of ## Commands.

The ADR Pre-Check body is byte-identical to the reviewed version, including the --force --no-tui adaptation that Non-Interactive Environments requires.

Please read: argument-hint was deliberately not restored

The task for this PR asked for an argument-hint frontmatter key alongside the metadata block. I didn't add it, because doing so would break skill validation.

PR #3 removed that key hours before this work started, and skill-creator's quick_validate.py explains why: it enforces an allowlist of name, description, license, allowed-tools, and metadata. That's easy to get wrong from memory, so I confirmed it directly rather than inferring it:

$ quick_validate.py skills/actual          # as shipped in this PR
Skill is valid!                            # exit 0

$ quick_validate.py <same file + argument-hint>
Unexpected key(s) in SKILL.md frontmatter: argument-hint.
Allowed properties are: allowed-tools, description, license, metadata, name
                                           # exit 1

Since PR #3 gates on that validator and has an OpenAI plugin submission pending behind it, restoring the key would have failed validation and put the submission at risk. metadata is on the allowlist, so the OpenClaw block lands cleanly and the "one file, every surface" goal still holds. If the key is wanted for a surface I have not accounted for, it needs a different home than this frontmatter.

Why a symlink for the root file

A symlink keeps exactly one copy of the body on disk, so the two paths can't drift. That's the failure this consolidation exists to end. A pointer file was the alternative, and it loses either way: it publishes a stub with no skill content, or it repeats name and description at the root and reintroduces the drift in the frontmatter.

One thing worth knowing before publishing. ClawHub stores its own copy of the SKILL.md body and doesn't carry references/, so the relative reference links behave from the root exactly as they already do for the published variant. Nothing here changes that, and no clawhub command was run.

Test plan

This repository has no CI, so everything below was checked by hand and is reproducible from the branch.

• PR is MERGEABLE / CLEAN again, still a draft, still one PR.
• quick_validate.py skills/actual passes, and quick_validate.py . passes through the root symlink.
• Frontmatter parses as YAML with keys description, metadata, name; both name and description are non-empty.
• metadata.openclaw carries requires.bins: [actual], primaryEnv, and homepage; the canonical description is untouched.
• ADR Pre-Check survives the merge byte-identical, nested fenced blocks included, and still sits before ## Commands.
• Root SKILL.md resolves to the same bytes as skills/actual/SKILL.md, verified with cmp; git records it as mode 120000.
• No conflict markers remain, and all three plugin manifests still parse as JSON.
• git diff origin/main --stat reports 3 files and 41 insertions with no deletions, so nothing from main was reverted.

---

Generated by the operator's software factory.
• City: factory-main · Agent: local-core.builder-2
• On behalf of: @benw5483