@bonesofspring/ai-rules 0.2.26 → 0.2.28

package/presets/claude/svelte/rules/tooling-and-review/preset-twin-sync.md

@@ -1,5 +1,5 @@
 ---
-description: Preset authoring: Cursor SoT → Claude twin same PR. Use when editing preset rules.
+description: "Preset authoring: Cursor SoT → Claude twin same PR. Use when editing preset rules."
 paths:
   - packages/ai-rules/presets/**/*
   - .cursor/rules/**/*

package/presets/claude/svelte/rules/ui-and-accessibility/no-rest-props-spread.md

@@ -1,7 +1,7 @@
 ---
 description: Ban undisciplined rest props / $$restProps bulk spread in Svelte
 paths:
-  - **/*.svelte
+  - "**/*.svelte"
 ---
 
 # No rest-props spread

package/presets/claude/svelte/rules/ui-and-accessibility/svelte-ui.md

@@ -16,6 +16,13 @@ paths:
 - A11y: `svelte-a11y-coding`.
 - UI boundaries: `architecture-boundaries-ui` — no DTO in markup; map in `src/lib/api/**`.
 
+# Conditional rendering
+
+- Prefer **decomposition over conditionals**: markup with several `{#if}` / `{:else if}` / `{:else}` branches should become a thin parent that renders child components, not one growing block.
+- Keep the condition in the parent; children receive resolved props.
+- No nested ternaries in markup. For loading / empty / error, use dedicated `EmptyState` / `ErrorState` components or snippets.
+- A `variant` / `mode` prop that renders two unrelated layouts is **two** components, not one.
+
 ## Incorrect
 
 ```svelte

package/presets/claude/svelte/settings.json

@@ -0,0 +1,16 @@
+{
+  "$schema": "https://json.schemastore.org/claude-code-settings.json",
+  "permissions": {
+    "deny": [
+      "Bash(git commit *)",
+      "Bash(git push *)",
+      "Bash(npx jest*)"
+    ],
+    "ask": [
+      "Bash(bun *)",
+      "Bash(npm *)",
+      "Bash(npx *)",
+      "Bash(pnpm *)"
+    ]
+  }
+}

package/presets/claude/svelte/team/fixtures/metrics-scenarios/tasks/human-gate/metrics.json

@@ -0,0 +1,69 @@
+{
+  "schemaVersion": 1,
+  "task": {
+    "runId": "human-gate-run",
+    "slug": "human-gate",
+    "preset": "ai-rules",
+    "stack": "svelte",
+    "platform": "cursor",
+    "intent": "feature",
+    "profile": "full"
+  },
+  "events": [
+    {
+      "eventId": "human-gate-run:task",
+      "type": "task_started",
+      "at": "2026-07-31T17:00:00.000Z",
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:gate:1",
+      "type": "human_gate_started",
+      "at": "2026-07-31T17:01:00.000Z",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:gate:end",
+      "type": "human_gate_ended",
+      "at": "2026-07-31T17:11:00.000Z",
+      "gateStartedEventId": "human-gate-run:gate:1",
+      "source": "command",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:1:feature-developer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T17:11:00.000Z",
+      "attemptId": "human-gate-run:step-1:feature-developer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "feature-developer",
+      "role": "implementation",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:1:feature-developer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T17:12:00.000Z",
+      "attemptId": "human-gate-run:step-1:feature-developer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "human-gate-run:done",
+      "type": "task_completed",
+      "at": "2026-07-31T17:12:00.000Z",
+      "outcome": "completed",
+      "source": "hook",
+      "attributionConfidence": "high"
+    }
+  ]
+}

package/presets/claude/svelte/team/fixtures/metrics-scenarios/tasks/missing-ledger/status.json

@@ -0,0 +1,5 @@
+{
+  "slug": "missing-ledger",
+  "state": "completed",
+  "note": "Intentional fixture: metrics.json is absent."
+}

package/presets/claude/svelte/team/fixtures/metrics-scenarios/tasks/parallel-receipts/metrics.json

@@ -0,0 +1,79 @@
+{
+  "schemaVersion": 1,
+  "task": {
+    "runId": "parallel-receipts-run",
+    "slug": "parallel-receipts",
+    "preset": "ai-rules",
+    "stack": "svelte",
+    "platform": "cursor",
+    "intent": "feature",
+    "profile": "full"
+  },
+  "events": [
+    {
+      "eventId": "parallel-receipts-run:task",
+      "type": "task_started",
+      "at": "2026-07-31T16:00:00.000Z",
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "parallel-receipts-run:1:code-reviewer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T16:00:00.000Z",
+      "attemptId": "parallel-receipts-run:step-1:code-reviewer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "code-reviewer",
+      "role": "review",
+      "attempt": 1,
+      "parallelGroupId": "parallel-receipts-run:step-1",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "parallel-receipts-run:1:security-reviewer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T16:00:00.000Z",
+      "attemptId": "parallel-receipts-run:step-1:security-reviewer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "security-reviewer",
+      "role": "review",
+      "attempt": 1,
+      "parallelGroupId": "parallel-receipts-run:step-1",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "parallel-receipts-run:1:security-reviewer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T16:01:00.000Z",
+      "attemptId": "parallel-receipts-run:step-1:security-reviewer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "agent",
+      "attributionConfidence": "fallback"
+    },
+    {
+      "eventId": "parallel-receipts-run:1:code-reviewer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T16:02:00.000Z",
+      "attemptId": "parallel-receipts-run:step-1:code-reviewer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 120000,
+      "retryAfterFix": null,
+      "source": "agent",
+      "attributionConfidence": "fallback"
+    },
+    {
+      "eventId": "parallel-receipts-run:done",
+      "type": "task_completed",
+      "at": "2026-07-31T16:02:00.000Z",
+      "outcome": "completed",
+      "source": "hook",
+      "attributionConfidence": "high"
+    }
+  ]
+}

package/presets/claude/svelte/team/fixtures/metrics-scenarios/tasks/serial-retry/metrics.json

@@ -0,0 +1,124 @@
+{
+  "schemaVersion": 1,
+  "task": {
+    "runId": "serial-retry-run",
+    "slug": "serial-retry",
+    "preset": "ai-rules",
+    "stack": "svelte",
+    "platform": "cursor",
+    "intent": "feature",
+    "profile": "full"
+  },
+  "events": [
+    {
+      "eventId": "serial-retry-run:task",
+      "type": "task_started",
+      "at": "2026-07-31T14:00:00.000Z",
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:2:code-reviewer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T14:00:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-1",
+      "stepIndex": 2,
+      "stepLabel": null,
+      "agent": "code-reviewer",
+      "role": "review",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:2:code-reviewer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T14:01:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-1",
+      "outcome": "changes_requested",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:retry:1",
+      "type": "retry_scheduled",
+      "at": "2026-07-31T14:01:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-1",
+      "nextAgent": "feature-developer",
+      "retryAfterFix": "code-reviewer",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:1:feature-developer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T14:01:00.000Z",
+      "attemptId": "serial-retry-run:step-1:feature-developer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "feature-developer",
+      "role": "implementation",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:1:feature-developer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T14:03:00.000Z",
+      "attemptId": "serial-retry-run:step-1:feature-developer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 120000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:retry:2",
+      "type": "retry_scheduled",
+      "at": "2026-07-31T14:03:00.000Z",
+      "attemptId": "serial-retry-run:step-1:feature-developer:attempt-1",
+      "nextAgent": "code-reviewer",
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:2:code-reviewer:2:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T14:03:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-2",
+      "stepIndex": 2,
+      "stepLabel": null,
+      "agent": "code-reviewer",
+      "role": "review",
+      "attempt": 2,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:2:code-reviewer:2:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T14:04:00.000Z",
+      "attemptId": "serial-retry-run:step-2:code-reviewer:attempt-2",
+      "outcome": "completed",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "serial-retry-run:done",
+      "type": "task_completed",
+      "at": "2026-07-31T14:04:00.000Z",
+      "outcome": "completed",
+      "source": "hook",
+      "attributionConfidence": "high"
+    }
+  ]
+}

package/presets/claude/svelte/team/fixtures/metrics-scenarios/tasks/validation-failed/metrics.json

@@ -0,0 +1,124 @@
+{
+  "schemaVersion": 1,
+  "task": {
+    "runId": "validation-failed-run",
+    "slug": "validation-failed",
+    "preset": "ai-rules",
+    "stack": "svelte",
+    "platform": "cursor",
+    "intent": "feature",
+    "profile": "full"
+  },
+  "events": [
+    {
+      "eventId": "validation-failed-run:task",
+      "type": "task_started",
+      "at": "2026-07-31T15:00:00.000Z",
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:2:build-verifier:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T15:00:00.000Z",
+      "attemptId": "validation-failed-run:step-2:build-verifier:attempt-1",
+      "stepIndex": 2,
+      "stepLabel": null,
+      "agent": "build-verifier",
+      "role": "validation",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:2:build-verifier:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T15:02:00.000Z",
+      "attemptId": "validation-failed-run:step-2:build-verifier:attempt-1",
+      "outcome": "validation_failed",
+      "durationMs": 120000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:retry:1",
+      "type": "retry_scheduled",
+      "at": "2026-07-31T15:02:00.000Z",
+      "attemptId": "validation-failed-run:step-2:build-verifier:attempt-1",
+      "nextAgent": "feature-developer",
+      "retryAfterFix": "build-verifier",
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:1:feature-developer:1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T15:02:00.000Z",
+      "attemptId": "validation-failed-run:step-1:feature-developer:attempt-1",
+      "stepIndex": 1,
+      "stepLabel": null,
+      "agent": "feature-developer",
+      "role": "implementation",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:1:feature-developer:1:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T15:05:00.000Z",
+      "attemptId": "validation-failed-run:step-1:feature-developer:attempt-1",
+      "outcome": "completed",
+      "durationMs": 180000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:retry:2",
+      "type": "retry_scheduled",
+      "at": "2026-07-31T15:05:00.000Z",
+      "attemptId": "validation-failed-run:step-1:feature-developer:attempt-1",
+      "nextAgent": "build-verifier",
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:2:build-verifier:2:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T15:05:00.000Z",
+      "attemptId": "validation-failed-run:step-2:build-verifier:attempt-2",
+      "stepIndex": 2,
+      "stepLabel": null,
+      "agent": "build-verifier",
+      "role": "validation",
+      "attempt": 2,
+      "parallelGroupId": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:2:build-verifier:2:finish",
+      "type": "attempt_finished",
+      "at": "2026-07-31T15:06:00.000Z",
+      "attemptId": "validation-failed-run:step-2:build-verifier:attempt-2",
+      "outcome": "completed",
+      "durationMs": 60000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "validation-failed-run:done",
+      "type": "task_completed",
+      "at": "2026-07-31T15:06:00.000Z",
+      "outcome": "completed",
+      "source": "hook",
+      "attributionConfidence": "high"
+    }
+  ]
+}

package/presets/claude/svelte/team/fixtures/metrics.example.json

@@ -0,0 +1,54 @@
+{
+  "schemaVersion": 1,
+  "task": {
+    "runId": "example-run-1",
+    "slug": "example-feature-full",
+    "preset": "ai-rules",
+    "stack": "svelte",
+    "platform": "cursor",
+    "intent": "feature",
+    "profile": "full"
+  },
+  "events": [
+    {
+      "eventId": "example-run-1:task_started",
+      "type": "task_started",
+      "at": "2026-07-31T14:00:00.000Z",
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "example-run-1:attempt-1:start",
+      "type": "attempt_started",
+      "at": "2026-07-31T14:00:00.000Z",
+      "attemptId": "example-run-1:step-0:task-analyst:attempt-1",
+      "stepIndex": 0,
+      "stepLabel": "Clarify and decompose",
+      "agent": "task-analyst",
+      "role": "planning",
+      "attempt": 1,
+      "parallelGroupId": null,
+      "source": "router",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "example-run-1:attempt-1:end",
+      "type": "attempt_finished",
+      "at": "2026-07-31T14:02:00.000Z",
+      "attemptId": "example-run-1:step-0:task-analyst:attempt-1",
+      "outcome": "completed",
+      "durationMs": 120000,
+      "retryAfterFix": null,
+      "source": "hook",
+      "attributionConfidence": "high"
+    },
+    {
+      "eventId": "example-run-1:task_completed",
+      "type": "task_completed",
+      "at": "2026-07-31T14:02:00.000Z",
+      "outcome": "completed",
+      "source": "hook",
+      "attributionConfidence": "high"
+    }
+  ]
+}

package/presets/cursor/android-kotlin/agents/build-verifier.md

@@ -37,7 +37,7 @@ When validating **android-kotlin preset** changes (or `scope: preset-structure-v
 
 ### FAIL (android-kotlin packaging)
 
-1. **Token budget:** Cursor `alwaysApply: true` count ≤ **3**; Claude rules without `paths:` === **3** (**exclude `**/README.md`**). Meta `preset-*` rules must **not** be always-on.
+1. **Token budget:** Cursor `alwaysApply: true` count 3–5; Claude rules without `paths:` 3–5 (**exclude `**/README.md`**). Meta `preset-*` rules must **not** be always-on.
 2. **Chain fork:** `hooks/chain-team-phases.sh` byte-≠ every other stack (Cursor + Claude).
 3. **Forbidden handoff strings** in android-kotlin `chain-team-phases.sh` (Cursor + Claude): `playwright-test-`, `playwright-agents`, `playwright-e2e`, `user-playwright`, `app/__tests__/e2e`, `xcuitest-test-`, positive `xcodebuild` / `SwiftLint` as required handoffs, positive yarn `lint:js` / `lint:css`.
 4. **Required handoff stems:** `AGENT_HANDOFF` includes `instrumentation-test-planner` (or generator/healer); `build-verifier` prose mentions `gradle-tooling` and/or `post-change-build`.
@@ -45,7 +45,7 @@ When validating **android-kotlin preset** changes (or `scope: preset-structure-v
 6. **Twin presence (hard):** Cursor rule → Claude twin + mapping row — **FAIL** if missing.
 7. **Mapping / README:** `claude/android-kotlin/rules/README.md` mapping table present.
 8. **shared-core consumers:** if `_shared/core/**` changed, matching embeds updated (or intentional fork documented).
-9. **Session-start body budget:** sum of the 3 always-on / session-start bodies ≤ **140** lines.
+9. **Session-start body budget:** sum of the intended always-on / session-start bodies ≤ **200** lines.
 10. **Embed drift:** `packages/ai-rules/scripts/check-shared-core-drift.sh` with `AI_RULES_DRIFT_FAIL=1`.
 11. **Root AGENTS template:** ship `REPO_AGENTS.md` → **FAIL** if missing.
 12. **Hooks cookbook:** `hooks.json` must **not** reference `hooks/examples/`.

package/presets/cursor/android-kotlin/commands/task.md

@@ -1,6 +1,6 @@
 ---
 name: task
-description: Route a natural-language task through the agent-team pipeline: task-router, pipeline.json, and the first planned specialist.
+description: "Route a natural-language task through the agent-team pipeline: task-router, pipeline.json, and the first planned specialist."
 ---
 
 # Task — single entry point (router + pipeline)

package/presets/cursor/android-kotlin/hooks/chain-team-phases.sh

@@ -42,7 +42,10 @@ try {
 }
 
 const slug = active.slug;
-if (!slug) {
+// Reject non-kebab / traversal slugs: the runtime writes status.json and
+// metrics.json under tasks/<slug>, so an unchecked slug could escape the tree.
+const SLUG_RE = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
+if (typeof slug !== 'string' || slug.length > 48 || !SLUG_RE.test(slug)) {
   process.stdout.write('{}');
   process.exit(0);
 }
@@ -69,6 +72,9 @@ function waitForMetricsLock() {
   // and never blocked on the lock file, so two concurrent subagentStop events
   // could both acquire the lock and corrupt metrics.json. Retry EEXIST with a
   // 10 ms sleep up to 200 attempts (2 s ceiling). On any other error, throw.
+  // A lock left behind by a killed process is reclaimed once its mtime is older
+  // than STALE_LOCK_MS, so one crash cannot brick the task forever.
+  const STALE_LOCK_MS = 30000;
   for (let attempt = 0; attempt < 200; attempt += 1) {
     try {
       const fd = fs.openSync(METRICS_LOCK, 'wx');
@@ -76,6 +82,13 @@ function waitForMetricsLock() {
       return;
     } catch (error) {
       if (!error || error.code !== 'EEXIST') throw error;
+      try {
+        const lockStat = fs.statSync(METRICS_LOCK);
+        if (Date.now() - lockStat.mtimeMs > STALE_LOCK_MS) {
+          fs.unlinkSync(METRICS_LOCK);
+          continue;
+        }
+      } catch {}
       const until = Date.now() + 10;
       while (Date.now() < until) { /* 10 ms busy-wait */ }
     }
@@ -495,7 +508,10 @@ function legacyFlow() {
 
 
 if (hookInput.action === 'end_human_gate') {
-    status.awaitingHumanGate = false;
+  // Only advance when a gate was actually open. A duplicate or retried
+  // end_human_gate must be a no-op, not a silent pipeline skip.
+  const gateWasOpen = Boolean(status.activeHumanGateEventId) || status.awaitingHumanGate === true;
+  status.awaitingHumanGate = false;
   // A2 (retro §8): pin a single runId per pipeline so subsequent events don't
   // fragment the metrics ledger across multiple runId sequences. Prefer
   // metrics.json.task.runId when metrics already exist, then status.metricsRunId,
@@ -522,7 +538,7 @@ if (hookInput.action === 'end_human_gate') {
     });
     delete status.activeHumanGateEventId;
   }
-  if (hookInput.activateNext && fs.existsSync(pipelinePath)) {
+  if (gateWasOpen && hookInput.activateNext && fs.existsSync(pipelinePath)) {
     let resumePipeline = null;
     try { resumePipeline = JSON.parse(fs.readFileSync(pipelinePath, 'utf8')); } catch {}
     if (resumePipeline) {
@@ -544,6 +560,19 @@ if (hookInput.action === 'end_human_gate') {
         else out(handoffForAgent(resumeAgents[0], resumeNext.step.scope, resumeNext.step));
         process.exit(0);
       }
+      // Gate on the final step: no executable step remains. Complete the
+      // pipeline instead of leaving it in_progress forever.
+      status.state = 'completed';
+      completeTaskMetrics();
+      writeStatus({
+        pipelineIndex: resumeSteps.length,
+        phase: 'done',
+        state: 'completed',
+        currentAgent: null,
+        awaitingHumanGate: false,
+      });
+      out(`Pipeline complete for "${slug}". Run /technical-retro ${slug}.`);
+      process.exit(0);
     }
   }
   // A2 (retro §8): if gate closes the final human gate of the pipeline

package/presets/cursor/android-kotlin/hooks/examples/test-on-save.example.sh

@@ -1,8 +1,10 @@
 #!/usr/bin/env bash
-# Optional: ./gradlew test slice for the active scheme (adjust SCHEME/DEST).
+# Optional: ./gradlew unit-test slice after save.
 # NOT in default hooks.json. No Playwright / yarn lint.
 set -euo pipefail
-SCHEME="${XCODE_SCHEME:-App}"
-DEST="${XCODE_DESTINATION:-platform=Android Simulator,name=iPhone 16}"
-./gradlew test -scheme "$SCHEME" -destination "$DEST" -quiet 2>/dev/null || true
+ROOT="${CURSOR_PROJECT_DIR:-.}"
+cd "$ROOT" 2>/dev/null || exit 0
+[[ -f gradlew ]] || exit 0
+# Prefer a targeted Gradle task; adjust module/task to match the repo.
+./gradlew test --quiet 2>/dev/null || true
 exit 0

package/presets/cursor/android-kotlin/rules/README.md

@@ -6,7 +6,7 @@ Flat Cursor SoT. Claude twins live under `presets/claude/android-kotlin/rules/<t
 
 | Kind | Mechanism | Budget |
 |------|-----------|--------|
-| Always-on trio | `alwaysApply: true` | exactly **3**; bodies ≤140 lines combined |
+| Always-on trio | `alwaysApply: true` | 3–5; bodies ≤200 lines combined |
 | Domain | `globs` / requestable | not always-on |
 | Meta `preset-*` | globs `packages/ai-rules/presets/**` | never always-on |
 

package/presets/cursor/android-kotlin/rules/code-quality-and-refactoring.mdc

@@ -19,6 +19,12 @@ alwaysApply: true
 - **CUPID:** Composable; Unix philosophy (one responsibility); Predictable; Idiomatic (repo); Domain-based.
 - **Composition over inheritance;** Law of Demeter — no `a.b.c.d` chains; via public API / facade.
 
+# Over-abstraction guardrails
+
+- **No unnecessary constants:** do not hoist a one-off literal, string, or boolean into a named constant. Inline it, or use an existing token/enum when the value is meaningful and reused. Named constants are for shared, semantic, or repeated values.
+- **No invented utilities:** do not add a `utils`/helper for a single call site or a thin pass-through wrapper. Inline it or extend the existing helper; a new shared utility is justified only after the Rule of Three.
+- **Decompose over conditionals:** a unit (function, component, hook) that accumulates several branches (`if` / ternary / `switch`) should be split into smaller units with one reason to change instead of piling up conditions.
+
 # Refactoring during changes
 
 - Light refactor is OK if it reduces duplication, improves readability, and does not break public contracts.

package/presets/cursor/android-kotlin/rules/compose-ui.mdc

@@ -30,3 +30,5 @@ alwaysApply: false
 - Render loading / empty / error / content from ViewModel state — no hidden local business rules.
 - Hoist state as the repo does; avoid duplicating ViewModel state in remember blocks.
 - Side-effect APIs (`LaunchedEffect`, etc.) only for UI concerns, not repository calls.
+- Prefer **decomposition over conditionals**: a composable with several `when` / `if` branches that render different UI should delegate to small sub-composables (`LoadingState()`, `EmptyState()`, `Content()`), not nest branches in one body.
+- Keep the condition in the parent; sub-composables receive resolved state. A `mode` / `variant` parameter that switches between unrelated layouts is **two** composables, not one.