@bonesofspring/ai-rules 0.2.10 → 0.2.11

package/presets/cursor/mcp-ts/team/conventions.md

@@ -0,0 +1,14 @@
+# Team conventions (mcp-ts)
+
+Fill after init with living gotchas for this MCP server repo.
+
+## Suggested topics
+
+- Package manager + test/typecheck scripts actually used
+- SDK version pin and VERIFY notes
+- stdio entry / `bin` packaging
+- Capability allowlists (FS roots, outbound hosts)
+
+## Fixture policy
+
+Phase 1 fixtures: `skipped[]` may name only Phase 1 roster agents (see `team/README.md`).

package/presets/cursor/mcp-ts/team/fixtures/bugfix-standard.json

@@ -0,0 +1,32 @@
+{
+  "slug": "example-schema-bugfix",
+  "intent": "bugfix",
+  "profile": "standard",
+  "summary": "Fix schema validation rejecting valid tool input",
+  "steps": [
+    {
+      "agent": "task-analyst",
+      "label": "Reproduce + AC"
+    },
+    {
+      "agent": "feature-developer",
+      "label": "Fix schema/handler + tests"
+    },
+    {
+      "agent": "build-verifier",
+      "label": "typecheck + tests"
+    }
+  ],
+  "humanGates": [],
+  "autoChain": true,
+  "skipped": [
+    {
+      "agent": "security-reviewer",
+      "reason": "Proven schema/style-only bugfix — no auth/**, secrets, or FS/net capability change (document skip; otherwise route security-reviewer)"
+    },
+    {
+      "agent": "solution-architect",
+      "reason": "Localized schema bugfix"
+    }
+  ]
+}

package/presets/cursor/mcp-ts/team/fixtures/feature-full.json

@@ -0,0 +1,33 @@
+{
+  "slug": "example-add-tool-full",
+  "intent": "feature",
+  "profile": "full",
+  "summary": "Add MCP tool with schema, handler, registration, and tests",
+  "steps": [
+    {
+      "agent": "task-analyst",
+      "label": "Brief + decomposition"
+    },
+    {
+      "agent": "feature-developer",
+      "label": "contract→schema→handler→register→test→package"
+    },
+    {
+      "agent": "build-verifier",
+      "label": "typecheck + tests"
+    },
+    {
+      "agent": "security-reviewer",
+      "label": "Capability / secrets review",
+      "model": "strong"
+    }
+  ],
+  "humanGates": ["after:task-analyst"],
+  "autoChain": true,
+  "skipped": [
+    {
+      "agent": "solution-architect",
+      "reason": "Straightforward tool add; architecture spike not required"
+    }
+  ]
+}

package/presets/cursor/mcp-ts/team/fixtures/feature-light.json

@@ -0,0 +1,34 @@
+{
+  "slug": "example-add-tool-light",
+  "intent": "feature",
+  "profile": "light",
+  "summary": "Add a single MCP tool with explicit AC",
+  "steps": [
+    {
+      "agent": "feature-developer",
+      "label": "Implement tool schema/handler/register",
+      "model": "cheap"
+    },
+    {
+      "agent": "build-verifier",
+      "label": "typecheck + tests",
+      "model": "cheap"
+    }
+  ],
+  "humanGates": [],
+  "autoChain": true,
+  "skipped": [
+    {
+      "agent": "task-analyst",
+      "reason": "Explicit AC and single-tool scope"
+    },
+    {
+      "agent": "security-reviewer",
+      "reason": "Light profile valid only with no FS/net/auth expansion — otherwise use standard/full and include security-reviewer"
+    },
+    {
+      "agent": "solution-architect",
+      "reason": "Light profile — no architecture spike"
+    }
+  ]
+}

package/presets/cursor/next/rules/preset-no-cross-stack-leakage.mdc

@@ -17,7 +17,8 @@ alwaysApply: false
 
 - `next` must not require iOS-only agents/tooling (`xcuitest-*`, `xcodebuild` as required handoff, Keychain-only rules).
 - `ios-swift` must not ship Playwright handoffs or Next lint recipes as required commands.
-- `go` / `java` must not ship Playwright, XCUITest, Next lint recipes, or `xcodebuild`/`SwiftLint` as required handoffs.
+- `go` / `java` / `mcp-ts` must not ship Playwright, XCUITest, Next lint recipes, or `xcodebuild`/`SwiftLint` as required handoffs.
+- `mcp-ts` must not ship Go tool commands (`go test`, `go vet`, `golangci-lint`) or `integration-test-*` agents as required delivery gates.
 - Shared core must not embed stack paths (`app/src/**`, `Features/**`, `playwright`, `xcodebuild`).
 
 ## Forbidden in ios-swift hooks/agents (FAIL)
@@ -46,6 +47,18 @@ Shared backend-stack bans (both `go` and `java`):
 
 Required instead: `integration-test-*`; build-verifier / `post-change-test` plus stack tooling — `go-tooling` (`go test`, `go vet`) or `java-tooling` (`mvn test` / `./gradlew test`).
 
+## Forbidden in mcp-ts hooks/agents (FAIL)
+
+Same UI/mobile bans as backend stacks, plus:
+
+- `playwright-test-`, `playwright-agents`, `playwright-e2e`, `user-playwright`, `app/__tests__/e2e`
+- Positive yarn recipes `lint:js` / `lint:css` as required gates
+- `xcuitest-test-`, `xcodebuild`, `SwiftLint` as required handoffs
+- Go tool commands as required MCP delivery gates: `go test`, `go vet`, `golangci-lint`
+- Agent files: `playwright-test-*`, `xcuitest-test-*`, `integration-test-*`, `accessibility-reviewer`
+
+Required instead: package-manager–discovered typecheck/test (`mcp-ts-tooling`, `post-change-test`); Phase 1 gates `build-verifier` + `security-reviewer`.
+
 ## Soft / document
 
 - Mentioning the other stack in **docs** comparing presets is OK.
@@ -53,4 +66,4 @@ Required instead: `integration-test-*`; build-verifier / `post-change-test` plus
 
 ## Chain fork
 
-`hooks/chain-team-phases.sh` must differ **byte-wise** pairwise among `next`, `ios-swift`, `go`, and `java` (per platform).
+`hooks/chain-team-phases.sh` must differ **byte-wise** pairwise among `next`, `ios-swift`, `go`, `java`, and `mcp-ts` (per platform).

package/presets/cursor/next/rules/preset-twin-sync.mdc

@@ -21,6 +21,7 @@ Cursor `.mdc` — **SoT per stack**. Claude topic `.md` — derived twin.
 | `ios-swift` | `presets/cursor/ios-swift/rules/*.mdc` | `presets/claude/ios-swift/rules/<topic>/` | Keep in `claude/ios-swift/rules/README.md` |
 | `go` | `presets/cursor/go/rules/*.mdc` | `presets/claude/go/rules/<topic>/` | Keep in `claude/go/rules/README.md` |
 | `java` | `presets/cursor/java/rules/*.mdc` | `presets/claude/java/rules/<topic>/` | Keep in `claude/java/rules/README.md` |
+| `mcp-ts` | `presets/cursor/mcp-ts/rules/*.mdc` | `presets/claude/mcp-ts/rules/<topic>/` | Keep in `claude/mcp-ts/rules/README.md` |
 
 ## Same-PR checklist
 
@@ -40,5 +41,6 @@ Cursor `.mdc` — **SoT per stack**. Claude topic `.md` — derived twin.
 - **ios-swift:** missing Claude twin for a changed Cursor rule → **FAIL**.
 - **go:** missing Claude twin for a changed Cursor rule → **FAIL** (same as next / ios-swift; mapping table required).
 - **java:** missing Claude twin for a changed Cursor rule → **FAIL** (same as next / ios-swift / go; mapping table required).
+- **mcp-ts:** missing Claude twin for a changed Cursor rule → **FAIL** (same as next / ios-swift / go / java; mapping table required).
 
 Dogfood: sync packages → root `.cursor/` after Cursor SoT changes (not the reverse). Use `packages/ai-rules/scripts/sync-dogfood-cursor.sh`.