summer-engine 1.3.2 → 1.4.0

package/hooks/pre-commit-doctor.sh

@@ -0,0 +1,123 @@
+#!/usr/bin/env bash
+# Summer Engine — Pre-commit doctor hook (OPT-IN)
+#
+# Event:    PreToolUse (matcher: Bash, if: "Bash(git commit*)")
+# Purpose:  Run `summer doctor` before any git commit Claude is about to make.
+#           Block the commit on FAIL. Allow with warnings on WARN.
+#
+# Status:   OPT-IN. Not registered in the Summer plugin manifest by default.
+#
+# To enable, add to ~/.claude/settings.json (or .claude/settings.json):
+#
+#   {
+#     "hooks": {
+#       "PreToolUse": [
+#         {
+#           "matcher": "Bash",
+#           "hooks": [
+#             {
+#               "type": "command",
+#               "command": "${CLAUDE_PLUGIN_ROOT}/hooks/pre-commit-doctor.sh",
+#               "if": "Bash(git commit*)",
+#               "timeout": 15
+#             }
+#           ]
+#         }
+#       ]
+#     }
+#   }
+#
+# Then: chmod +x ${CLAUDE_PLUGIN_ROOT}/hooks/pre-commit-doctor.sh
+#
+# Input:    JSON on stdin: { tool_name, tool_input: { command, ... }, ... }
+# Output:   On FAIL: stderr explanation, exit 2 (blocks commit).
+#           On WARN: stderr warnings, exit 0 (allows commit).
+#           On PASS or no `summer` CLI: silent, exit 0.
+#
+# Portability: bash 3.2+, POSIX-friendly. jq optional.
+
+set +e
+
+INPUT=$(cat)
+
+# ---------- Parse the command ----------
+COMMAND=""
+if command -v jq >/dev/null 2>&1; then
+    COMMAND=$(printf '%s' "$INPUT" | jq -r '.tool_input.command // empty' 2>/dev/null)
+fi
+if [ -z "$COMMAND" ]; then
+    COMMAND=$(printf '%s' "$INPUT" \
+        | grep -oE '"command"[[:space:]]*:[[:space:]]*"[^"]*"' \
+        | head -1 \
+        | sed 's/^"command"[[:space:]]*:[[:space:]]*"//;s/"$//')
+fi
+
+# ---------- Bail early if not a git commit ----------
+if ! printf '%s' "$COMMAND" | grep -qE '^[[:space:]]*git[[:space:]]+commit'; then
+    exit 0
+fi
+
+# ---------- Bail if `summer` CLI missing ----------
+if ! command -v summer >/dev/null 2>&1; then
+    # Don't block — user may have installed via plugin only.
+    exit 0
+fi
+
+# ---------- Run doctor ----------
+DOCTOR_OUT=$(summer doctor --json 2>/dev/null)
+if [ -z "$DOCTOR_OUT" ]; then
+    exit 0
+fi
+
+# ---------- Parse status ----------
+HAS_FAIL=0
+HAS_WARN=0
+FAIL_MSGS=""
+WARN_MSGS=""
+
+if command -v jq >/dev/null 2>&1; then
+    FAIL_MSGS=$(printf '%s' "$DOCTOR_OUT" \
+        | jq -r '.checks[]? | select(.status == "FAIL") | "  - " + (.name // "?") + ": " + (.message // "")' 2>/dev/null)
+    WARN_MSGS=$(printf '%s' "$DOCTOR_OUT" \
+        | jq -r '.checks[]? | select(.status == "WARN") | "  - " + (.name // "?") + ": " + (.message // "")' 2>/dev/null)
+    [ -n "$FAIL_MSGS" ] && HAS_FAIL=1
+    [ -n "$WARN_MSGS" ] && HAS_WARN=1
+else
+    # Fallback: substring sniffing
+    if printf '%s' "$DOCTOR_OUT" | grep -q '"status"[[:space:]]*:[[:space:]]*"FAIL"'; then
+        HAS_FAIL=1
+        FAIL_MSGS=$(printf '%s' "$DOCTOR_OUT" \
+            | grep -oE '"status"[[:space:]]*:[[:space:]]*"FAIL"[^}]*"message"[[:space:]]*:[[:space:]]*"[^"]*"' \
+            | sed -E 's/.*"message"[[:space:]]*:[[:space:]]*"([^"]*)"/  - \1/')
+    fi
+    if printf '%s' "$DOCTOR_OUT" | grep -q '"status"[[:space:]]*:[[:space:]]*"WARN"'; then
+        HAS_WARN=1
+        WARN_MSGS=$(printf '%s' "$DOCTOR_OUT" \
+            | grep -oE '"status"[[:space:]]*:[[:space:]]*"WARN"[^}]*"message"[[:space:]]*:[[:space:]]*"[^"]*"' \
+            | sed -E 's/.*"message"[[:space:]]*:[[:space:]]*"([^"]*)"/  - \1/')
+    fi
+fi
+
+# ---------- FAIL: block ----------
+if [ "$HAS_FAIL" -eq 1 ]; then
+    {
+        echo "=== Summer doctor: FAIL — commit blocked ==="
+        echo "$FAIL_MSGS"
+        echo ""
+        echo "Fix the failures above, or run: summer doctor"
+        echo "To bypass once: disable the pre-commit-doctor hook in settings.json."
+        echo "============================================"
+    } >&2
+    exit 2
+fi
+
+# ---------- WARN: allow with notice ----------
+if [ "$HAS_WARN" -eq 1 ]; then
+    {
+        echo "=== Summer doctor: WARN — commit allowed ==="
+        echo "$WARN_MSGS"
+        echo "============================================"
+    } >&2
+fi
+
+exit 0

package/hooks/run-hook.cmd

@@ -0,0 +1,23 @@
+#!/usr/bin/env bash
+# Cross-platform hook dispatcher.
+# Usage: run-hook.cmd <hook-name>
+# Reads stdin, forwards to <plugin-root>/hooks/<hook-name>.sh
+# On Windows, Claude Code / Cursor / Codex invoke this via Git-Bash.
+
+HOOK_NAME="$1"
+shift || true
+
+HOOK_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+HOOK_SCRIPT="$HOOK_DIR/$HOOK_NAME.sh"
+
+if [ ! -f "$HOOK_SCRIPT" ]; then
+  # Missing hook = no-op (don't break the host).
+  exit 0
+fi
+
+if [ ! -x "$HOOK_SCRIPT" ]; then
+  # Not executable yet (likely fresh checkout). Fall back to bash invocation.
+  exec bash "$HOOK_SCRIPT" "$@"
+fi
+
+exec "$HOOK_SCRIPT" "$@"

package/hooks/session-start.sh

@@ -0,0 +1,120 @@
+#!/usr/bin/env bash
+# Summer Engine — SessionStart hook
+#
+# Event:    SessionStart (matcher: startup|clear|compact)
+# Purpose:  Orient Claude on every fresh Summer-enabled session.
+#           Detect Godot project + GameSoul + doctor health, surface a one-line
+#           Bob-style greeting as additional context.
+#
+# Input:    JSON on stdin with { session_id, source, model, cwd, ... }
+#           Common Claude Code env vars: $CLAUDE_PROJECT_DIR, $CLAUDE_PLUGIN_ROOT
+# Output:   JSON on stdout with hookSpecificOutput.additionalContext
+#           (also emits Cursor / Copilot CLI variants based on host env vars)
+# Exit:     Always 0 — never block session start.
+#
+# Portability: bash 3.2+ (macOS default), POSIX-friendly, works on Git Bash.
+#              Uses `grep -E` not `grep -P`. Falls back gracefully when jq
+#              and `summer` are missing.
+
+set +e  # Hooks must never kill the session.
+
+# ---------- Resolve project dir ----------
+PROJECT_DIR="${CLAUDE_PROJECT_DIR:-$(pwd)}"
+
+# ---------- Build the orientation lines (max 3) ----------
+LINES=()
+
+# Line 1: Summer ready + skill count
+SKILL_COUNT=""
+if command -v summer >/dev/null 2>&1; then
+    # `summer skills count` is best-effort; ignore failure
+    SKILL_COUNT=$(summer skills count 2>/dev/null | grep -oE '[0-9]+' | head -1)
+fi
+if [ -n "$SKILL_COUNT" ]; then
+    LINES+=("Summer ready. ${SKILL_COUNT} skills.")
+else
+    LINES+=("Summer ready.")
+fi
+
+# Line 2: Godot project + GameSoul one-liner (only if project.godot exists)
+if [ -f "${PROJECT_DIR}/project.godot" ]; then
+    PROJECT_NAME=$(grep -E '^config/name=' "${PROJECT_DIR}/project.godot" 2>/dev/null \
+        | head -1 | sed 's/^config\/name=//;s/^"//;s/"$//')
+    [ -z "$PROJECT_NAME" ] && PROJECT_NAME=$(basename "$PROJECT_DIR")
+
+    SOUL_FILE="${PROJECT_DIR}/.summer/GameSoul.md"
+    if [ -f "$SOUL_FILE" ]; then
+        # First non-empty, non-heading paragraph (one line)
+        SOUL_LINE=$(awk 'BEGIN{p=""} /^[^#[:space:]]/ && NF>0 {p=$0; exit} END{print p}' \
+            "$SOUL_FILE" 2>/dev/null | cut -c1-120)
+        if [ -n "$SOUL_LINE" ]; then
+            LINES+=("Godot project: ${PROJECT_NAME}. GameSoul: ${SOUL_LINE}")
+        else
+            LINES+=("Godot project: ${PROJECT_NAME}.")
+        fi
+    else
+        LINES+=("Godot project: ${PROJECT_NAME}.")
+    fi
+fi
+
+# Line 3: doctor FAIL one-liner (only on FAIL — silent on PASS/WARN)
+if command -v summer >/dev/null 2>&1; then
+    DOCTOR_OUT=$(summer doctor --json 2>/dev/null)
+    if [ -n "$DOCTOR_OUT" ]; then
+        FAIL_REASON=""
+        if command -v jq >/dev/null 2>&1; then
+            FAIL_REASON=$(printf '%s' "$DOCTOR_OUT" \
+                | jq -r 'first(.checks[]? | select(.status == "FAIL") | .message) // empty' 2>/dev/null)
+        else
+            # Crude fallback: first "message" near a "FAIL"
+            FAIL_REASON=$(printf '%s' "$DOCTOR_OUT" \
+                | grep -oE '"status"[[:space:]]*:[[:space:]]*"FAIL"[^}]*"message"[[:space:]]*:[[:space:]]*"[^"]*"' \
+                | head -1 \
+                | grep -oE '"message"[[:space:]]*:[[:space:]]*"[^"]*"' \
+                | sed 's/.*"message"[[:space:]]*:[[:space:]]*"//;s/"$//')
+        fi
+        if [ -n "$FAIL_REASON" ]; then
+            LINES+=("Summer setup needs attention: ${FAIL_REASON}. Run: summer doctor")
+        fi
+    fi
+fi
+
+# ---------- Compose context ----------
+CONTEXT=""
+for line in "${LINES[@]}"; do
+    if [ -z "$CONTEXT" ]; then CONTEXT="$line"
+    else CONTEXT="${CONTEXT}"$'\n'"$line"
+    fi
+done
+
+# Bail with empty JSON if we somehow have no lines
+if [ -z "$CONTEXT" ]; then
+    printf '{}\n'
+    exit 0
+fi
+
+# ---------- JSON-escape for embedding ----------
+escape_for_json() {
+    local s="$1"
+    s="${s//\\/\\\\}"
+    s="${s//\"/\\\"}"
+    s="${s//$'\n'/\\n}"
+    s="${s//$'\r'/\\r}"
+    s="${s//$'\t'/\\t}"
+    printf '%s' "$s"
+}
+CONTEXT_ESC=$(escape_for_json "$CONTEXT")
+
+# ---------- Emit per-host shape ----------
+# Cursor: top-level additional_context (snake_case)
+# Claude Code: nested hookSpecificOutput.additionalContext
+# Copilot CLI / SDK standard: top-level additionalContext (camelCase)
+if [ -n "${CURSOR_PLUGIN_ROOT:-}" ]; then
+    printf '{"additional_context":"%s"}\n' "$CONTEXT_ESC"
+elif [ -n "${CLAUDE_PLUGIN_ROOT:-}" ] && [ -z "${COPILOT_CLI:-}" ]; then
+    printf '{"hookSpecificOutput":{"hookEventName":"SessionStart","additionalContext":"%s"}}\n' "$CONTEXT_ESC"
+else
+    printf '{"additionalContext":"%s"}\n' "$CONTEXT_ESC"
+fi
+
+exit 0

package/marketplace.json

@@ -0,0 +1,24 @@
+{
+  "$schema": "https://json.schemastore.org/claude-code-marketplace.json",
+  "name": "summer",
+  "description": "Summer Engine plugins for Claude Code",
+  "owner": { "name": "Summer Engine", "email": "founders@summerengine.com" },
+  "plugins": [
+    {
+      "name": "summer",
+      "source": "./",
+      "version": "1.4.0",
+      "description": "AI game-dev studio: 19 skills under summer: namespace, MCP bridge, session-start + pre-commit hooks.",
+      "author": { "name": "Summer Engine", "email": "founders@summerengine.com" },
+      "category": "game-development",
+      "tags": ["game-dev", "godot", "ai-game-engine"],
+      "keywords": ["game-dev", "godot", "ai-game-engine", "skills"],
+      "brandColor": "#FF8F17",
+      "capabilities": ["Interactive", "Read", "Write"],
+      "defaultPrompt": [
+        "I want to make a game.",
+        "/summer:brainstorm-game"
+      ]
+    }
+  ]
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "summer-engine",
-  "version": "1.3.2",
+  "version": "1.4.0",
   "description": "CLI and MCP tools for Summer Engine — the AI-native game engine",
   "keywords": [
     "game-engine",
@@ -25,6 +25,14 @@
   "files": [
     "dist/",
     "skills/",
+    "hooks/",
+    ".claude-plugin/",
+    ".cursor-plugin/",
+    ".codex-plugin/",
+    ".opencode/",
+    "gemini-extension.json",
+    "marketplace.json",
+    ".mcp.json",
     "README.md",
     "LICENSE",
     "CHANGELOG.md"

package/skills/_tests/specs/peer-to-peer-multiplayer.md

@@ -0,0 +1,99 @@
+# Skill Spec: /peer-to-peer-multiplayer
+
+## Fixture
+
+- A Godot 4.5 / Summer Engine project that does NOT yet have multiplayer.
+- Project may be empty or may have an existing single-player scene.
+- Summer MCP tools available; engine running on localhost:6550.
+
+## Case 1: Happy Path — fresh multiplayer architecture from scratch
+
+**Input:** "I want to add 2-player co-op to my game. Players will connect through Steam friend invites."
+
+**Expected behavior:**
+
+1. Skill confirms P2P is the right architecture (Steam friend invite ≤8 players → P2P, not dedicated server).
+2. Skill walks the four layers in order, checkpointing between each.
+3. Layer 1: asks "May I create `autoloads/network_manager.gd`?" — creates it on user OK, calls `summer_project_setting` to register the autoload.
+4. Layer 2: asks "May I create `autoloads/game_state.gd` and walk through the authority decision matrix?" — creates it, prompts user for game-specific state fields and proposes ownership.
+5. Layer 3: explains the three RPC patterns with examples; doesn't auto-apply (this is reference material until concrete game logic exists).
+6. Layer 4: defers prediction/interpolation until there's a player scene to apply it to.
+
+**Expected MCP tool sequence (in order, partial):**
+
+1. `summer_get_scene_tree` — see what already exists.
+2. `summer_get_project_context` — read autoload list to avoid name collisions.
+3. (User approves Layer 1) `Write autoloads/network_manager.gd`.
+4. `summer_project_setting(key="autoload/NetworkManager", value="*res://autoloads/network_manager.gd")`.
+5. (User approves Layer 2) `Write autoloads/game_state.gd`.
+6. `summer_project_setting(key="autoload/GameState", value="*res://autoloads/game_state.gd")`.
+7. `summer_save_scene` and `summer_get_script_errors` to verify clean.
+
+**Assertions:**
+
+- [ ] Skill explicitly checks Step 0 (P2P vs. client-server) before any code is written.
+- [ ] Skill creates Layer 1 (NetworkManager) before Layer 2 (GameState).
+- [ ] NetworkManager is the only place that calls `multiplayer.peer_connected.connect()` directly. GameState listens to NetworkManager signals, NOT to the multiplayer API directly.
+- [ ] GameState uses `@rpc("authority", "call_remote", "reliable")` for host-broadcast functions and `@rpc("any_peer", "call_remote", "reliable")` with a `is_host` guard for client-request functions.
+- [ ] Skill walks the user through the authority decision matrix before writing GameState. Doesn't dump generic state fields without asking what the game has.
+- [ ] Skill asks "May I…" before each new file creation and before the autoload registration.
+- [ ] Skill does NOT apply prediction/interpolation in Layer 4 unless there's an existing player scene to wire into.
+- [ ] At any layer, `summer_get_script_errors` is called after the file write to confirm clean compile.
+
+## Case 2: Failure Path — user wants matchmaking at scale
+
+**Input:** "I'm building a competitive shooter. I want matchmaking with anti-cheat for 10,000 concurrent matches."
+
+**Expected behavior:**
+
+- Skill recognizes Step 0: this is dedicated client-server territory, not P2P.
+- Skill explains why P2P is wrong (host can cheat, no anti-cheat hooks, scale concerns).
+- Skill defers to `/summer:client-server-multiplayer` (when shipped) OR explains the user needs to roll a custom server.
+- Does NOT start writing P2P NetworkManager code anyway.
+
+**Assertions:**
+
+- [ ] Skill stops at Step 0 — does NOT proceed to Layer 1.
+- [ ] Skill explains the architecture mismatch in one sentence.
+- [ ] Skill names the right alternative (client-server) and which skill to use instead.
+
+## Case 3: Edge Case — retrofitting multiplayer to a single-player game
+
+**Input:** "I have a single-player platformer that's mostly done. Can you add multiplayer?"
+
+**Expected behavior:**
+
+- Skill is honest about the cost: retrofitting multiplayer is 5× harder than starting MP-first.
+- Skill asks: "Are you committed? It will require auditing every state mutation in your existing code. Time estimate: significant."
+- If user proceeds: skill walks the same 4 layers but adds an AUDIT phase before Layer 2: identify every place existing code writes to state that should be host-authoritative.
+- If user balks: skill points to alternatives (release MP as a sequel; ship single-player; add async leaderboards instead).
+
+**Assertions:**
+
+- [ ] Skill warns about retrofit cost BEFORE starting Layer 1.
+- [ ] Skill adds an AUDIT step that doesn't exist in the happy-path workflow.
+- [ ] Skill never silently proceeds as if the game were greenfield.
+
+## Case 4: No Summer MCP — fallback path
+
+**Fixture:** Same as Case 1, but Summer MCP unavailable.
+
+**Input:** "Add multiplayer to my Godot game (P2P, friend invite)."
+
+**Expected behavior:**
+
+- Skill detects MCP unavailable.
+- Walks the same 4 layers.
+- Instead of `summer_project_setting`, asks the user to manually edit `project.godot`'s `[autoload]` section — provides the exact lines to paste.
+- Instead of `summer_save_scene` / `summer_get_script_errors`, asks the user to save and check errors via the Godot editor.
+- All file writes still go through `Write` host tool with "May I…" approval per file.
+
+**Assertions:**
+
+- [ ] Skill detects MCP unavailable; doesn't blindly call `summer_*` tools and fail.
+- [ ] Project.godot edits are provided as exact paste-able lines.
+- [ ] Workflow shape (4 layers, checkpoints between) is identical regardless of MCP availability.
+
+---
+
+This spec runs via `/skill-test peer-to-peer-multiplayer spec` (see `_meta/skill-test/SKILL.md`).

package/skills/asset-pipeline/asset-strategy/SKILL.md

@@ -233,3 +233,11 @@ summer_generate_audio(capability="text_to_speech", text="You shall not pass!", v
 - **Extremely thin objects** (paper, cloth, flags): 3D gen produces thick meshes. Better to use MeshInstance3D with a plane + texture + shader.
 - **Emissive/glowing objects**: Generate the mesh, add emissive material in engine. Don't rely on the glow being in the mesh.
 - **LOD (Level of Detail)**: Generate one version. If perf needs LOD, use Godot's LOD system or generate a second low-poly version.
+
+## Fallback
+
+No fallback for this — Summer MCP required for the generation pipelines (`summer_generate_image`, `summer_generate_3d`, `summer_generate_audio`, `summer_search_assets`, `summer_import_from_url`). If MCP isn't connected, the user must source assets externally (Sketchfab, Quaternius, Kenney, AmbientCG) and import via Godot's import system.
+
+## Collaborative protocol
+
+This skill triggers metered generation (image / 3D / audio) and mutates the scene with imported assets. Always ask before generating ("May I generate a 3D treasure chest via image-to-3D? This is one metered generation.") and before importing ("May I import the result to `res://assets/models/chest.glb`?"). See `../../_shared/collaborative-protocol.md`.

package/skills/catalog.yaml

@@ -48,6 +48,13 @@ skills:
     public: true
 
   # ── gameplay-mechanics ──
+  - name: design-mechanic
+    category: gameplay-mechanics
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: melee-combat
     category: gameplay-mechanics
     status: NEXT
@@ -132,6 +139,13 @@ skills:
     public: true
 
   # ── scene-and-project ──
+  - name: brainstorm-game
+    category: scene-and-project
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: scene-composition
     category: scene-and-project
     status: HAVE
@@ -177,6 +191,13 @@ skills:
     public: true
 
   # ── rendering-and-lighting ──
+  - name: art-direction
+    category: rendering-and-lighting
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: 3d-lighting
     category: rendering-and-lighting
     status: HAVE
@@ -255,6 +276,13 @@ skills:
     public: true
 
   # ── visual-effects ──
+  - name: vfx
+    category: visual-effects
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: gpuparticles-3d-basics
     category: visual-effects
     status: NEXT
@@ -333,6 +361,13 @@ skills:
     public: true
 
   # ── audio ──
+  - name: audio-direction
+    category: audio
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: audio-bus-setup
     category: audio
     status: NEXT
@@ -393,6 +428,25 @@ skills:
     public: true
 
   # ── multiplayer-and-networking ──
+  - name: setup-multiplayer
+    category: multiplayer-and-networking
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
+  - name: peer-to-peer-multiplayer
+    category: multiplayer-and-networking
+    status: NEXT
+    public: true
+    recommended: true
+    user-invocable: true
+  - name: host-authoritative-state
+    category: multiplayer-and-networking
+    status: NEXT
+    public: true
+    recommended: true
+    user-invocable: true
   - name: multiplayerapi-basics
     category: multiplayer-and-networking
     status: NEXT
@@ -423,6 +477,13 @@ skills:
     public: true
 
   # ── ai-and-npcs ──
+  - name: design-npc
+    category: ai-and-npcs
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: navmesh-pathfinding
     category: ai-and-npcs
     status: NEXT
@@ -453,6 +514,13 @@ skills:
     public: true
 
   # ── level-design ──
+  - name: design-level
+    category: level-design
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: gridmap-building
     category: level-design
     status: NEXT
@@ -479,6 +547,13 @@ skills:
     public: true
 
   # ── performance ──
+  - name: tune-performance
+    category: performance
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: profiling-godot
     category: performance
     status: NEXT
@@ -624,6 +699,13 @@ skills:
     public: true
 
   # ── deployment ──
+  - name: export-and-ship
+    category: deployment
+    status: HAVE
+    public: true
+    recommended: true
+    user-invocable: true
+    last-tested: 2026-05-05
   - name: export-presets
     category: deployment
     status: NEXT

package/skills/character-controllers/fps-controller/SKILL.md

@@ -131,3 +131,30 @@ For `summer_set_resource_property` on CapsuleShape3D:
 - **Falling through floor:** Ensure the floor has a StaticBody3D or similar with a CollisionShape3D
 - **Camera clipping:** If the camera goes inside geometry, enable `near` on the Camera3D or add collision exception
 - **Mouse capture:** For FPS, you typically need `Input.mouse_mode = Input.MOUSE_MODE_CAPTURED` in `_ready()` and a way to release (e.g., Escape)
+
+## Fallback (no MCP — edit `.tscn` directly)
+
+If Summer MCP isn't connected, write the scene by hand. Save as `res://player.tscn`:
+
+```
+[gd_scene load_steps=2 format=3]
+
+[sub_resource type="CapsuleShape3D" id="1"]
+radius = 0.4
+height = 1.6
+
+[node name="Player" type="CharacterBody3D"]
+position = Vector3(0, 1, 0)
+
+[node name="CollisionShape3D" type="CollisionShape3D" parent="."]
+shape = SubResource("1")
+
+[node name="Camera3D" type="Camera3D" parent="."]
+position = Vector3(0, 1.6, 0)
+```
+
+Then attach `player.gd` (script body above) via the inspector.
+
+## Collaborative protocol
+
+This skill writes scene nodes and a `.gd` script. Always ask before applying. Group writes per phase: "May I add the Player + CollisionShape3D + Camera3D, bind WASD/jump InputMap actions, and attach `player.gd`?" See `../../_shared/collaborative-protocol.md`.