wehub-resource-sync
2.1 KiB
2.1 KiB
title, category, date, module, component, tags, severity, description, related
| title | category | date | module | component | tags | severity | description | related | |||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Beta-to-stable promotions must update orchestration callers atomically | skill-design | 2026-03-23 | skills | SKILL.md |
|
medium | When promoting a beta skill to stable, update all orchestration callers in the same PR so they pass correct mode flags instead of inheriting defaults. |
|
Problem
When a beta skill introduces new invocation semantics (e.g., explicit mode flags), promoting it over its stable counterpart without updating orchestration callers causes those callers to silently inherit the wrong default behavior.
Solution
Treat promotion as an orchestration contract change, not a file rename.
- Replace the stable skill with the promoted content
- Update every workflow that invokes the skill in the same PR
- Hardcode the intended mode at each callsite instead of relying on the default
- Add or update contract tests so the orchestration assumptions are executable
Applied: ce-review-beta -> ce-code-review (2026-03-24)
This pattern was applied when promoting the review beta (ce-review-beta, tracked as the legacy artifact ce:review-beta/ce-review-beta in the cleanup registry) into the stable ce-code-review skill. The caller contract at the time:
lfg->/ce-code-review mode:autofix(enforced bytests/review-skill-contract.test.ts)slfghas since been removed, but the durable rule remains: every orchestrator that invokes the promoted skill must pass the intended mode explicitly and have contract coverage where the caller still exists.
Prevention
- When a beta skill changes invocation semantics, its promotion plan must include caller updates as a first-class implementation unit
- Promotion PRs should be atomic: promote the skill and update orchestrators in the same branch
- Add contract coverage for the promoted callsites so future refactors cannot silently drop required mode flags
- Do not rely on “remembering later” for orchestration mode changes; encode them in docs, plans, and tests