fix: Register repoman plugin and refine entrypoint UX

[rubambiza]

Summary

Two fixes to make the /repoman entrypoint installable and the setup flow smoother, found while dogfooding the merged skill.

1. Register the repoman plugin (demo blocker). The marketplace manifest had no repoman entry, so the merged skill could not be installed or invoked as a plugin. The manifest also carried a broken repo-sync skills array (missing ]},) introduced by an earlier merge (064eb0b), which made the whole file invalid JSON. This adds the repoman stanza and repairs the repo-sync break so the manifest parses.

2. Refine the entrypoint SKILL.md from dogfooding the first-time setup flow.

Changes

• .claude-plugin/marketplace.json: add the repoman plugin stanza; close the unterminated repo-sync skills array (pre-existing merge regression from 064eb0b). Manifest is valid JSON again.
• skills/repoman/SKILL.md:
  • Requirements parser is a sourced library — state source-then-call explicitly in both "Where the scripts live" and the Q3 requirement wiring, so a driver never executes it directly and silently captures empty requirements (which would defeat the capability gate). Pairs with fix: Fail loud when repoman-requirements.sh is executed automation#113, which makes the script fail loud on direct execution.
  • Presentation section — friendly program display names (Link-Health-Manager, …) with fuzzy-match-to-canonical resolution before any writer call; pacing guidance (terse "missing → input → next" prompts, internals surfaced only on error); choice-UI note (prefer a host structured-select widget, mind its option cap).
  • Note that program descriptions in SKILL.md frontmatter are YAML folded block scalars, not inline values.

Testing

• Manifest validates as JSON; all nine skills paths resolve to real dirs; repoman present.
• skills/repoman/SKILL.md still parses via repoman_parse_requirements (its own Requirements block unchanged).
• Verified end-to-end by dogfooding: fresh /repoman install from this branch drove a full first-time setup (enroll repos, enable programs with correct requirement wiring) and run-path reconciliation.

Scope

The pr-review description still leads with an adjective rather than a verb; standardizing descriptions touches other programs' frontmatter and is deferred to its own PR. The bundle-vs-automation-checkout question for the parser is deferred to the run-path/architecture work.

Assisted-By: Claude Code

[clawgenti]

Repairs the marketplace manifest (verified: main fails JSON parsing on the unterminated repo-sync array; at HEAD it is valid JSON, 9 plugins, all 17 skill paths resolving) and adds the missing repoman stanza, plus solid SKILL.md refinements — source-then-call parser guidance and a friendly-name→canonical mapping consistent with KNOWN_PROGRAMS (Q3/Q4 references check out). CI green, DCO signed.

Findings

• suggestion skills/repoman/SKILL.md:57 — the "fails loud on direct execution" claim depends on the still-open rossoctl/automation#113; on current automation main, direct execution is silent (see inline comment).

---

Reviewed by clawgenti using the github-pr-review skill

[clawgenti]

Follow-up commit 735fe8d resolves the prior review's only finding: the parser guidance now accurately describes both merge-order states of rossoctl/automation#113 (verified — #113 still open; automation main currently exits 0 silently on direct execution; the #113 patch fails loud with matching guidance). Manifest repair, repoman registration, and the friendly-name → KNOWN_PROGRAMS mapping re-verified at HEAD (valid JSON, 9 plugins, all 17 skill paths resolving; main confirmed broken).

All checks pass. Ready for human review.

---

Reviewed by clawgenti using the github-pr-review skill

[clawgenti]

Commit 09fddbd adds the README row for the repoman entrypoint — description matches the manifest stanza and the skills/repoman/ link resolves. Re-verified at HEAD: manifest is valid JSON with 9 plugins and all 17 skill paths resolving (main still fails to parse on the unterminated repo-sync array), DCO 3/3, CI green, and the sourced-library parser guidance still matches automation main (repoman-requirements.sh defines the function only; rossoctl/automation#113 remains open).

All checks pass. Ready for human review.

---

Reviewed by clawgenti using the github-pr-review skill

[mrsabath]

Verified end-to-end at HEAD (09fddbd):

• Manifest repair: main's .claude-plugin/marketplace.json fails JSON parsing (unterminated repo-sync array); at HEAD it parses cleanly — 9 plugins, repoman stanza present, and all 17 skills paths resolve to real directories.
• README: the new repoman row links to skills/repoman/, which exists; description matches the manifest stanza.
• Parser guidance: accurate against automation main — repoman-requirements.sh defines repoman_parse_requirements with no top-level execution, so a direct run exits 0 with empty output exactly as described; rossoctl/automation#113 is still open, and the doc honestly frames both merge-order states.
• Friendly-name table: matches the KNOWN_PROGRAMS allowlist in repoman-setup.sh exactly (all 5 canonical tokens), and the writer validates canonical tokens, so the resolve-before-writer-call rule is the right guard.
• Folded-scalar note: confirmed — program SKILL.md descriptions use description: >- blocks.
• SKILL.md Requirements block at HEAD is unchanged and still valid.

Commits 3/3 signed-off with Assisted-By; DCO and title checks green. Clean review — approving.