Omar SobhandClaude Opus 5
2eb0880fc0
Every skill reference in every team template was failing to resolve. The TOMLs used snake_case slugs (`write_rust`, `index_selection`) while the authored skills under `skills/**/*.md` declare kebab-case names (`write-rust-current-edition`, `postgres-index-selection`), so `get_by_name` missed on all of them: 128 skipped bindings across 51 distinct names, and no mission agent received any of its template's skills. The mirror-image half was equally invisible: ten authored skills — including `int-xx-marker-protocol`, whose own `when_to_use` says "pin on every coding role" — were referenced by no role at all, so nothing could ever load them. - Rename the 14 references that have authored skills behind them, and dedupe the two that now collapse onto the commit-protocol skill. - Attach all ten orphaned skills to the roles their `when_to_use` names. All 23 authored skills now reach at least one role. - Aggregate the loader's per-name logging into one line per template. The old per-name spam is why this went unnoticed; a bound/unresolved count is noticeable. References with no authored skill are kept and listed — they record intent for skills not yet written. - Two regression tests: no authored skill may be orphaned, and every authored skill must be referenced by its exact name. Also clears the two standing clippy warnings: group `mint_team_from_template`'s eight positional args into `TeamMint`, and make `provider_alias_for` branch on `is_exact_provider_match` so the helper is live code and the two can't disagree about what counts as an exact family match. Co-Authored-By: Claude Opus 5 <[email protected]>
68 lines
2.3 KiB
TOML
68 lines
2.3 KiB
TOML
key = "mobile"
|
||
name = "Mobile"
|
||
description = "Expo + React Native for iOS/Android — camera, comms, networking, native modules."
|
||
stack = ["typescript", "react-native", "expo", "ios", "android"]
|
||
default_topology = "pipeline"
|
||
risk_profile = "coding_readwrite"
|
||
mcp_bundles = ["clawmates_door", "gitea_forge"]
|
||
version = 1
|
||
|
||
[[roles]]
|
||
slot = "designer"
|
||
order_idx = 0
|
||
skills = ["decompose-int-items", "ios_hig_check", "material_you_check"]
|
||
system_prompt = """
|
||
You are the DESIGNER of a Mobile team.
|
||
|
||
For each INT item: define the screen's state model, navigation
|
||
integration, platform-specific behavior (safe-area on iOS, back-button
|
||
on Android), and any native-module permissions required.
|
||
"""
|
||
brain_seed = """
|
||
# Mobile design seed
|
||
- Safe-area padding is mandatory on every screen.
|
||
- Deep links go through the app's central router — never hand-crafted.
|
||
- Every permission prompt has a pre-prompt explaining WHY.
|
||
"""
|
||
|
||
[[roles]]
|
||
slot = "coder"
|
||
order_idx = 1
|
||
skills = ["write_typescript_react_native", "expo-managed-vs-bare", "workspace-repo-commit-protocol", "int-xx-marker-protocol", "component-4-state-model", "rn-flashlist-perf"]
|
||
system_prompt = """
|
||
You are the CODER of a Mobile team.
|
||
|
||
Working directory /workspace/repo. Prefer Expo's managed workflow;
|
||
justify any drop to bare workflow. All new native modules ship with
|
||
both iOS + Android implementations in the same PR.
|
||
"""
|
||
brain_seed = """
|
||
# RN memory seed
|
||
- Prefer FlashList over FlatList for anything > 20 items.
|
||
- Reanimated for anything animating > 3× per frame.
|
||
- No inline require() — dynamic imports break Metro's tree-shake.
|
||
"""
|
||
|
||
[[roles]]
|
||
slot = "tester"
|
||
order_idx = 2
|
||
skills = ["detox_e2e", "jest_unit", "ios_simulator_check", "android_emulator_check", "tdd-red-green-refactor"]
|
||
system_prompt = """
|
||
You are the TESTER of a Mobile team.
|
||
|
||
Detox e2e on both platforms. Snapshot tests for critical layouts.
|
||
Run on both iPhone 15 Pro simulator + Pixel 8 emulator before signing
|
||
off.
|
||
"""
|
||
brain_seed = ""
|
||
|
||
[[roles]]
|
||
slot = "committer"
|
||
order_idx = 3
|
||
skills = ["workspace-repo-commit-protocol", "small-focused-commits"]
|
||
system_prompt = """
|
||
You are the COMMITTER. Only run on green tests both platforms.
|
||
Emit COMPLETED: INT-<NN>.
|
||
"""
|
||
brain_seed = ""
|