docs(droid-control): make single the default layout; gate side-by-side to real comparisons

The skill guidance was tipping droids toward before/after side-by-side as the
canonical video shape — primary JSON example used side-by-side with BEFORE/AFTER
labels, Demo-mode goal was phrased as 'before/after comparison', delegation
table led with 'Capture before branch / Capture after branch'. Result: droids
were reaching for side-by-side even on brand-new features where there is no
meaningful 'before', sometimes synthesizing a fake baseline.

Fix (guidance only, no schema change — layoutSchema already supports both):

- droid-control/SKILL.md: new 'Layout default' table under Workflow shape
  calling out single as the default with the specific cases that warrant
  side-by-side. Explicit 'do not synthesize a before' rule.
- droid-control/SKILL.md: delegation table collapses the two pre-framed
  before/after rows into 'Capture clip' (single) + 'Capture both clips'
  (comparison). Parallel capture section heading/gate updated to read
  'comparison flows only'.
- compose/SKILL.md: primary JSON example now renders a single clip with
  labels: []. One-line note shows how to flip to side-by-side when a real
  comparison exists. Showcase-vs-Demo 'Goal' cell no longer pre-supposes
  before/after.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>

Author: factory-droid[bot]

plugins/droid-control/skills/compose/SKILL.md

@@ -55,7 +55,7 @@ Both use the same Remotion pipeline but target different visual registers.
 
 | | Showcase | Demo |
 |---|---|---|
-| **Goal** | Cinematic, high-polish marketing material | Clear, utilitarian before/after comparison |
+| **Goal** | Cinematic, high-polish marketing material | Clear, utilitarian demonstration — single or comparison, whichever the story calls for |
 | **Preset** | `factory`, `factory-hero`, or `hero` | `macos`, `minimal`, or `presentation` |
 | **Effects tier** | **Full** -- spotlight, zoom, callout, keystroke overlay. Go all out. | **Utilitarian** -- zoom for readability, keystroke overlay for user actions |
 | **Audience** | External — landing pages, social, marketing | Internal — PR reviews, docs, QA |
@@ -146,6 +146,12 @@ This checkpoint is not optional. A video that lands outside the target range fai
 
 ## Step 2: Build props
 
+### Choose layout
+
+**Default: `single`.** One clip of the target/final state. New features, bug-fix proofs, walkthroughs, and README heroes all belong here.
+
+Use `side-by-side` only when the story is fundamentally a comparison: regression (broken vs fixed), behavior-preserving refactor, or an explicit user request. Never fabricate a "before" clip to justify the side-by-side shape.
+
 Save the `showcaseSchema` JSON to a temp file:
 
 ```bash
@@ -154,10 +160,10 @@ PROPS="${DEMO_TMP}/showcase-props.json"
 
 cat > "$PROPS" << 'EOF'
 {
-  "clips": ["before.cast", "after.cast"],
-  "layout": "side-by-side",
+  "clips": ["demo.cast"],
+  "layout": "single",
   "fidelity": "auto",
-  "labels": ["BEFORE (dev)", "AFTER (PR #11621)"],
+  "labels": [],
   "speed": 3,
   "title": "PR #11621 — Prevent session freezes",
   "subtitle": "Bash Mode blocks interactive commands and supports ESC cancellation",
@@ -167,17 +173,16 @@ cat > "$PROPS" << 'EOF'
     {"t": 5.5, "label": "sleep 100"},
     {"t": 8.0, "label": "Esc"}
   ],
-  "sections": [
-    {"t": 2.0, "title": "Testing basic echo"},
-    {"t": 10.0, "title": "Testing long loop"}
-  ],
+  "sections": [],
   "effects": [],
   "speedNote": "3x speed",
   "windowTitle": "droid demo"
 }
 EOF
 ```
 
+For a comparison flow, swap `"clips"` to two paths, `"layout"` to `"side-by-side"`, and populate `"labels"` (e.g., `["BEFORE (main)", "AFTER (PR #11621)"]`).
+
 Use a run-scoped props path like `$PROPS`; do not reuse a global `/tmp/showcase-props.json` across rerenders or concurrent demos.
 
 **CRITICAL: `clipDuration` handling.** The render script auto-detects clip duration via ffprobe when `clipDuration` is omitted from the props. If you set it manually, it **must** match the actual clip duration or you get blank frames (too long) or truncation (too short). When in doubt, omit it and let the script auto-detect.

plugins/droid-control/skills/droid-control/SKILL.md

@@ -71,6 +71,21 @@ Command (intent + commitments)
 
 Commands declare **what** to produce. Atoms own **how**.
 
+### Layout default
+
+**Default: `single`.** One clip showing the target/final state. Pick this unless the deliverable is fundamentally a comparison.
+
+| Case | Layout |
+|---|---|
+| Brand-new feature (no meaningful prior state) | `single` |
+| Bug fix, single-clip proof of the working path | `single` |
+| Walkthrough / tutorial / readme hero | `single` |
+| Regression proof (broken vs fixed) | `side-by-side` |
+| Behavior-preserving refactor (visual parity is the point) | `side-by-side` |
+| User explicitly asks for a comparison | `side-by-side` |
+
+Do not synthesize a "before" state to justify `side-by-side`. If there is no real baseline, use `single`.
+
 ## Delegation
 
 The parent agent plans and orchestrates. Mechanical work runs in **worker subagents** via the Task tool. This keeps the parent's context clean and enables parallelism.
@@ -79,8 +94,8 @@ The parent agent plans and orchestrates. Mechanical work runs in **worker subage
 
 | Task | Delegate? | Why |
 |---|---|---|
-| **Capture before branch** | YES — `run_in_background=true` | Independent from after branch; run both in parallel |
-| **Capture after branch** | YES — `run_in_background=true` | Independent from before branch |
+| **Capture clip** (single layout) | YES | Worker runs the interaction script end-to-end and returns the `.cast` path |
+| **Capture both clips** (comparison layout) | YES — `run_in_background=true` for each | Branches are independent; run in parallel |
 | **Remotion render** | YES | Needs only props JSON, clip paths, output path. Runs `render-showcase.sh` (handles .cast conversion, fidelity profiles, duration detection, cleanup) |
 | Planning, interaction scripting | NO — parent | Requires PR context and editorial judgment |
 | Layout and prop construction | NO — parent | Requires editorial decisions about effects, timing, labels |
@@ -125,7 +140,9 @@ Task prompt for a Remotion render worker:
      /tmp/droid-run-1712345678-42-xxxx/before.cast /tmp/droid-run-1712345678-42-xxxx/after.cast"
 ```
 
-### Parallel capture pattern
+### Parallel capture pattern (comparison flows only)
+
+Only applicable when the Layout default table above selects `side-by-side`. For a `single` layout, launch one capture worker and skip this section.
 
 For before/after comparison demos, launch both capture workers simultaneously: