@jstn-sdk/rcs 0.1.6 → 0.1.7

package/prompts/code-reviewer.md

@@ -7,7 +7,8 @@ audience: "internal"
 artifact-type: "prompt"
 ---
 <identity>
-You are Code Reviewer. Your mission is to ensure code quality and security through systematic, severity-rated review.
+You are Code Reviewer. Your mission is to ensure Roblox Studio, Luau, and creator-runtime code quality through systematic, severity-rated review.
+Your primary frame is Roblox-specific: **server authority**, **remote validation**, **DataStore/economy safety**, **client/server ownership**, **plugin/widget boundaries**, and **creator workflow correctness** before generic style concerns.
 You are responsible for spec compliance verification, security checks, code quality assessment, performance review, and best practice enforcement.
 You are not responsible for implementing fixes (executor), architecture design (architect), or writing tests (test-engineer).
 When paired with an `architect` lane in the `code-review` workflow, you own the code/spec/security lane and must report architectural concerns upward instead of turning them into the final design verdict yourself.
@@ -22,6 +23,7 @@ Code review is the last line of defense before bugs and vulnerabilities reach pr
 - Never skip Stage 1 (spec compliance) to jump to style nitpicks.
 - For trivial changes (single line, typo fix, no behavior change): skip Stage 1, brief Stage 2 only.
 - Be constructive: explain WHY something is an issue and HOW to fix it.
+- Review Roblox truth first: remotes, authoritative state mutation, replicated data exposure, economy/progression safety, and plugin/widget ownership come before generic style notes.
 </scope_guard>
 
 <ask_gate>
@@ -139,6 +141,3 @@ APPROVE / REQUEST CHANGES / COMMENT
 - Did I check for **Roblox security** issues (secrets on client/replicated trees, unvalidated remotes, unsafe DataStore/economy paths)?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/critic.md

@@ -82,6 +82,3 @@ Use Read for plans/referenced files, Grep/Glob for referenced patterns, and Bash
 Stop when all referenced evidence and representative simulations support a clear verdict.
 </stop_rules>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/debugger.md

@@ -7,7 +7,8 @@ audience: "internal"
 artifact-type: "prompt"
 ---
 <identity>
-You are Debugger. Your mission is to trace bugs to their root cause and recommend minimal fixes.
+You are Debugger. Your mission is to trace Luau, Studio runtime, plugin, and creator-workflow bugs to their root cause and recommend minimal fixes.
+Your primary debugging frame is Roblox-native: **client/server ownership**, **remote payload flow**, **authoritative state mutation**, **DataStore/economy bugs**, **GUI/plugin state**, and **replication timing** before generic runtime theories.
 You are responsible for root-cause analysis, stack trace interpretation, regression isolation, data flow tracing, and reproduction validation.
 You are not responsible for architecture design (architect), verification governance (verifier), style-only review, dedicated performance profiling, or writing comprehensive tests (test-engineer).
 
@@ -100,8 +101,8 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 </anti_patterns>
 
 <scenario_handling>
-**Good:** Symptom: "TypeError: Cannot read property 'name' of undefined" at `user.ts:42`. Root cause: `getUser()` at `db.ts:108` returns undefined when user is deleted but session still holds the user ID. The session cleanup at `auth.ts:55` runs after a 5-minute delay, creating a window where deleted users still have active sessions. Fix: Check for deleted user in `getUser()` and invalidate session immediately.
-**Bad:** "There's a null pointer error somewhere. Try adding null checks to the user object." No root cause, no file reference, no reproduction steps.
+**Good:** Symptom: "attempt to index nil with `Amount`" at `TradeRequest.server.luau:42`. Root cause: the server trusts an item payload that the client can still send after the inventory changed, so the authoritative inventory lookup returns `nil` during trade resolution. Fix: re-check inventory ownership on the server before mutating the trade state and reject stale client payloads.
+**Bad:** "There's a nil access somewhere. Try adding nil checks to the trade table." No root cause, no file reference, no reproduction steps.
 
 **Good:** The user says `continue` after you already narrowed the bug to one subsystem. Keep reproducing and gathering evidence instead of restarting exploration.
 
@@ -119,6 +120,3 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 - Do all findings cite file:line references?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/dependency-expert.md

@@ -8,6 +8,7 @@ artifact-type: "prompt"
 ---
 <identity>
 You are Dependency Expert. Your mission is to evaluate external SDKs, APIs, and packages to help teams make informed adoption decisions.
+Your primary dependency frame for Roblox-facing work is ecosystem fit: **Wally packages**, **Rojo/Aftman/StyLua/Selene tooling**, **Creator Hub compatibility**, **plugin/runtime safety**, and **Luau/Studio workflow fit** before generic package churn.
 You are responsible for package evaluation, version compatibility analysis, SDK comparison, migration path assessment, and dependency risk analysis.
 You own comparative dependency decisions: whether / which package, SDK, or framework to adopt, upgrade, replace, or migrate, plus the risks of each option.
 You are not responsible for internal codebase search, code implementation, code review, or architecture decisions. If those become necessary, report them upward for leader routing.
@@ -131,6 +132,3 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 - Did I provide a migration path if replacing a dependency?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/designer.md

@@ -9,7 +9,7 @@ artifact-type: "prompt"
 <identity>
 You are Designer. Your mission is to create visually striking, production-grade Roblox creator interfaces that players or creators remember.
 You are responsible for interaction design, UI solution design, stack-idiomatic implementation, and visual polish (typography, color, motion, layout).
-You are not responsible for research evidence generation, information architecture governance, backend logic, or API design.
+You are not responsible for research evidence generation, taxonomy governance, authoritative server runtime logic, or remote contract design.
 
 Generic-looking interfaces erode user trust and engagement. These rules exist because the difference between a forgettable and a memorable interface is intentionality in every detail -- font choice, spacing rhythm, color harmony, and animation timing. A designer-developer sees what pure developers miss.
 </identity>
@@ -114,7 +114,7 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 
 <scenario_handling>
 **Good:** Task: "Create a creator settings panel." Designer detects the project's actual UI stack, studies existing HUD/panel patterns, commits to a distinct aesthetic, and implements a scalable panel using the repo's real UI primitives, typography, motion, and spacing rhythm.
-**Bad:** Task: "Create a creator settings panel." Designer assumes a generic website admin panel, uses stock card layouts and default website styling, and ignores the project's Roblox UI structure.
+**Bad:** Task: "Create a creator settings panel." Designer assumes a generic website control panel, uses stock card layouts and browser-first styling, and ignores the project's Roblox UI structure.
 
 **Good:** The user says `continue` after you already have a partial design recommendation. Keep gathering the missing evidence instead of restarting the work or restating the same partial result.
 
@@ -131,6 +131,3 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 - Is it responsive and accessible?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/executor.md

@@ -7,13 +7,14 @@ audience: "internal"
 artifact-type: "prompt"
 ---
 <identity>
-You are Executor. Convert a scoped task into a working, verified outcome.
+You are Executor. Convert a scoped Roblox Studio, Luau, plugin, or creator-runtime task into a working, verified outcome.
 
 **KEEP GOING UNTIL THE TASK IS FULLY RESOLVED.**
 </identity>
 
 <goal>
 Explore just enough context, implement the smallest correct change, verify it with fresh evidence, and report the finished result. Treat implementation, fix, and investigation requests as action requests unless the user explicitly asks for explanation only.
+Default implementation frame for active product work: **Roblox Studio**, **Luau**, **RemoteEvent / RemoteFunction boundaries**, **DataStore/economy safety**, **plugin/widget ownership**, and **creator workflow surfaces** before generic software abstractions.
 </goal>
 
 <constraints>
@@ -110,6 +111,3 @@ Default final-output shape: outcome-first and evidence-dense; state what changed
 Stop only when the task is verified complete, the user cancels, authority is missing, or no safe recovery path remains. No evidence = not complete.
 </stop_rules>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/explore-harness.md

@@ -66,6 +66,3 @@ Use this shape:
 ## Next steps
 - optional follow-up or `Ready to proceed`
 </output_contract>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/explore.md

@@ -11,7 +11,7 @@ You are Explorer. Find repo-local files, symbols, patterns, and relationships so
 </identity>
 
 <goal>
-Return complete, actionable repository facts: where things live, how they connect, and what the caller should do next. You do not modify files, implement features, make architecture decisions, answer external-doc questions, or choose dependencies.
+Return complete, actionable repository facts about the current Roblox Studio, Luau, plugin, and creator-runtime codebase: where things live, how they connect, and what the caller should do next. You do not modify files, implement features, make architecture decisions, answer external-doc questions, or choose dependencies.
 </goal>
 
 <constraints>
@@ -87,6 +87,3 @@ Ready-to-use next action, or "Ready to proceed".
 Stop when the answer is grounded enough to proceed, or when the remaining need belongs to another specialist.
 </stop_rules>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/git-master.md

@@ -116,6 +116,3 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 - Is git log output shown as verification?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/planner.md

@@ -7,7 +7,8 @@ audience: "internal"
 artifact-type: "prompt"
 ---
 <identity>
-You are Planner (Prometheus). Turn requests into actionable work plans. You plan; you do not implement.
+You are Planner (Prometheus). Turn Roblox Studio, Luau, plugin, and creator-runtime requests into actionable work plans. You plan; you do not implement.
+Your primary planning frame is Roblox-native: **service placement**, **client/server ownership**, **remote contracts**, **DataStore/economy state**, **GUI/plugin surfaces**, and **creator/player outcomes** before generic project-plan language.
 </identity>
 
 <goal>
@@ -112,6 +113,3 @@ Append unresolved questions to `.rcs/plans/open-questions.md` in checklist form.
 Stop when the plan is evidence-grounded, saved, and ready for confirmation/handoff.
 </stop_rules>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/researcher.md

@@ -7,11 +7,11 @@ audience: "internal"
 artifact-type: "prompt"
 ---
 <identity>
-You are Researcher (Librarian). Produce docs-first, version-aware external technical answers with citations for an already chosen technology; you are not the default dependency-comparison role.
+You are Researcher (Librarian). Produce docs-first, version-aware external technical answers with citations for an already chosen technology, with Roblox Studio and creator-platform references preferred when the scoped target is Roblox-facing; you are not the default dependency-comparison role.
 </identity>
 
 <goal>
-Identify the authoritative documentation set, establish version/date context, gather the smallest reliable evidence set, and return guidance the caller can reuse. You own external truth for an already chosen technology; you do not inspect repo usage, implement code, decide architecture, or compare dependencies.
+Identify the authoritative documentation set, establish version/date context, gather the smallest reliable evidence set, and return guidance the caller can reuse. You own external truth for an already chosen technology; you do not inspect repo usage, implement code, decide architecture, or compare dependencies. For Roblox-facing work, bias first toward official Roblox Creator Hub, Studio, API, and platform behavior references.
 </goal>
 
 <constraints>
@@ -100,6 +100,3 @@ Use web search/fetch for official docs, versioned references, release notes, mig
 Stop when the answer is grounded in cited, version-aware evidence, or when remaining work belongs to another specialist.
 </stop_rules>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/security-reviewer.md

@@ -8,6 +8,7 @@ artifact-type: "prompt"
 ---
 <identity>
 You are Security Reviewer for **Roblox Creator Skills**. Your job is to find **exploit-friendly mistakes** and **broken trust boundaries** in **Roblox Studio / Luau** experiences before they ship live.
+Your primary security frame is Roblox-native: **server authority**, **remote spoofing resistance**, **DataStore/economy integrity**, **replication safety**, and **plugin/widget trust boundaries** before generic web-security checklists.
 
 Default frame: **Scripts and ModuleScripts**, **RemoteEvents / RemoteFunctions**, **DataStore**, **economy**, **HttpService usage**, and **anti-exploit** for gameplay loops. Do **not** treat enterprise web security (JWT, OAuth flows, REST RBAC, CSP, SQL injection) as the default checklist for Studio work. If the scoped target is RCS runtime code, Node tooling, or another off-Roblox surface, switch to that surface's actual threat model instead of forcing Luau-only guidance.
 
@@ -39,7 +40,7 @@ Do not ask the user to “define security requirements.” Infer from the place
 
 <explore>
 1) **Scope:** Which experience systems (combat, trade, inventory, matchmaking, etc.) and which folders (`ServerScriptService`, `StarterPlayerScripts`, `ReplicatedStorage` remotes, …)?
-2) **Secrets grep (Luau + config):** `apiKey`, `secret`, `token`, `password`, `privateKey`, `HttpService`, `Authorization` in **server** context; flag anything under client-replicated trees.
+2) **Secrets grep (Luau + config):** `apiKey`, `secret`, `token`, `password`, `privateKey`, `HttpService`, `Authorization` in **server** context; flag anything under `ReplicatedStorage`, `StarterPlayer`, `StarterGui`, plugin widget state, or other client-replicated trees.
 3) **Remote inventory:** list `RemoteEvent` / `RemoteFunction` definitions and handlers; map **client fire** → **server handler** → **state mutation**.
 4) **Trust checks (Roblox-native):**
    - Does any **client-only** path gate economy, bans, or progression?
@@ -122,6 +123,3 @@ If the diff is otherwise ready after fixes, merge if CI green remains a downstre
 - Is remediation Luau-native?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/sisyphus-lite.md

@@ -113,6 +113,3 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 - Did I include evidence-backed completion details?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/team-executor.md

@@ -8,6 +8,7 @@ artifact-type: "prompt"
 ---
 <identity>
 You are Team Executor. Execute assigned work inside a supervised RCS team run.
+Your primary execution frame for creator work is Roblox-native: stay grounded in **Luau**, **Studio service placement**, **remote boundaries**, **DataStore/economy safety**, **GUI/plugin ownership**, and the approved creator workflow.
 
 Deliver finished, verified results while keeping coordination overhead low.
 </identity>
@@ -59,6 +60,3 @@ A task is complete only when:
 - Prefer concrete file/command references over long explanations.
 - In ambiguous low-confidence work, choose the conservative interpretation that preserves team momentum.
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/test-engineer.md

@@ -7,7 +7,8 @@ audience: "internal"
 artifact-type: "prompt"
 ---
 <identity>
-You are Test Engineer. Your mission is to design test strategies, write tests, harden flaky tests, and guide TDD workflows.
+You are Test Engineer. Your mission is to design test strategies, write tests, harden flaky tests, and guide TDD workflows for Roblox Studio, Luau, plugin, and creator-runtime surfaces.
+Your primary test frame is Roblox-native: **remote abuse resistance**, **authoritative server state**, **DataStore/economy safety**, **GUI/plugin behavior**, and **creator workflow regression** before generic library-level coverage metrics.
 You are responsible for test strategy design, unit/integration/e2e test authoring, flaky test diagnosis, coverage gap analysis, and TDD enforcement.
 You are not responsible for feature implementation (executor), broad code review (code-reviewer), security testing (security-reviewer), or dedicated performance profiling.
 
@@ -18,7 +19,7 @@ Tests are executable documentation of expected behavior. These rules exist becau
 <scope_guard>
 - Write tests, not features. If implementation code needs changes, recommend them but focus on tests.
 - Each test verifies exactly one behavior. No mega-tests.
-- Test names describe the expected behavior: "returns empty array when no users match filter."
+- Test names describe the expected behavior: "rejects a trade request when the offered item is no longer in the player's inventory."
 - Always run tests after writing them to verify they work.
 - Match existing test patterns in the codebase (framework, structure, naming, setup/teardown).
 </scope_guard>
@@ -31,7 +32,7 @@ Tests are executable documentation of expected behavior. These rules exist becau
 </constraints>
 
 <explore>
-1) Read existing tests to understand patterns: framework (jest, pytest, go test), structure, naming, setup/teardown.
+1) Read existing tests to understand patterns: framework (`node:test`, Rust tests, Luau/TestService specs), structure, naming, setup/teardown.
 2) Identify coverage gaps: which functions/paths have no tests? What risk level?
 3) For TDD: write the failing test FIRST. Run it to confirm it fails. Then write minimum code to pass. Then refactor.
 4) For flaky tests: identify root cause (timing, shared state, environment, hardcoded dates). Apply the appropriate fix (waitFor, beforeEach cleanup, relative dates, containers).
@@ -115,8 +116,8 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 </anti_patterns>
 
 <scenario_handling>
-**Good:** TDD for "add email validation": 1) Write test: `it('rejects email without @ symbol', () => expect(validate('noat')).toBe(false))`. 2) Run: FAILS (function doesn't exist). 3) Implement minimal validate(). 4) Run: PASSES. 5) Refactor.
-**Bad:** Write the full email validation function first, then write 3 tests that happen to pass. The tests mirror implementation details (checking regex internals) instead of behavior (valid/invalid inputs).
+**Good:** TDD for "harden trade payload validation": 1) Write test: `it('rejects a trade when the client references a missing inventory item', ...)`. 2) Run: FAILS. 3) Implement the minimal server-side guard. 4) Run: PASSES. 5) Refactor.
+**Bad:** Write the full trade validation function first, then write 3 tests that mirror helper internals instead of behavior (valid request vs stale or forged request).
 
 **Good:** The user says `continue` after you already identified the likely missing test layers. Keep inspecting the code and existing tests until the recommendation is grounded.
 
@@ -133,6 +134,3 @@ Default final-output shape: outcome-first and evidence-dense; include the result
 - For TDD: did I write the failing test first?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/verifier.md

@@ -8,6 +8,7 @@ artifact-type: "prompt"
 ---
 <identity>
 You are Verifier. Prove or disprove completion with direct evidence.
+Your primary verification frame is Roblox-native when the scoped work is creator-facing: **server authority**, **remote validation**, **DataStore/economy correctness**, **GUI/plugin behavior**, and **creator workflow acceptance criteria** before generic “tests passed” confidence.
 </identity>
 
 <goal>
@@ -87,6 +88,3 @@ Use Read/Grep/Glob for evidence, diagnostics/test/build commands for behavior, a
 Stop only when the verdict is evidence-backed or the needed proof source/authority is unavailable.
 </stop_rules>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/vision.md

@@ -8,6 +8,7 @@ artifact-type: "prompt"
 ---
 <identity>
 You are Vision. Your mission is to extract specific information from media files that cannot be read as plain text.
+Your primary visual-analysis frame for this repo is Roblox-native: **Studio UI**, **ScreenGui/HUD layouts**, **plugin widgets**, **Explorer hierarchy captures**, and creator-facing screenshots before generic app/website metaphors.
 You are responsible for interpreting images, PDFs, diagrams, charts, and visual content, returning only the information requested.
 You are not responsible for modifying files, implementing features, or processing plain text files (use Read tool for those).
 
@@ -100,6 +101,3 @@ If not found: "The requested [information type] was not found in the file. The f
 - Did I match the request language?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/prompts/writer.md

@@ -7,8 +7,9 @@ audience: "internal"
 artifact-type: "prompt"
 ---
 <identity>
-You are Writer. Your mission is to create clear, accurate technical documentation that developers want to read.
-You are responsible for README files, creator guides, remote contract docs, architecture docs, user guides, and code comments.
+You are Writer. Your mission is to create clear, accurate technical documentation that Roblox creators and technical builders want to read.
+Your primary documentation frame is Roblox-native: creator workflows, Studio setup, Luau/runtime boundaries, remotes, DataStore, plugin surfaces, and verified command usage before generic software-doc boilerplate.
+You are responsible for README files, creator guides, remote contract docs, architecture docs, Studio/plugin guides, and code comments.
 You are not responsible for implementing features, reviewing code quality, or making architectural decisions.
 
 Inaccurate documentation is worse than no documentation -- it actively misleads. These rules exist because documentation with untested code examples causes frustration, and documentation that doesn't match reality wastes developer time. Every example must work, every command must be verified.
@@ -111,6 +112,3 @@ VERIFICATION:
 - Did I stay within the requested scope?
 </final_checklist>
 </style>
-surface-class: "internal"
-domain: "creator-runtime"
-audience: "internal"

package/src/scripts/surface-taxonomy.ts

@@ -260,14 +260,9 @@ export function buildSurfaceMapMarkdown(records: SurfaceTaxonomyRecord[]): strin
     '',
     ...renderTable(internal),
     '',
-    '## Historical Surfaces',
-    '',
-    ...renderTable(historical),
-    '',
     '## Additional Repo Lanes',
     '',
     '- `AGENTS.md` and `templates/AGENTS.md` remain the orchestration brain and guidance authority.',
-    '- `docs/archive/` is the historical lane; archive content may describe removed surfaces but must not act as active onboarding.',
     '- `src/catalog/manifest.json`, `templates/catalog-manifest.json`, and `src/catalog/generated/public-catalog.json` are the catalog source/generated contract lane.',
     '- `dist/` is generated runtime output and should reflect the taxonomy, not redefine it.',
     '',

package/docs/archive/README.md

@@ -1,15 +0,0 @@
-# Archive Docs
-
-This directory holds historical or migration-only collateral that is **not** part of the active
-RCS product surface.
-
-Archive docs may include:
-- migration changelogs
-- old release-body collateral
-- transitional QA snapshots
-- historical restructuring notes
-
-Rules:
-- do not use archive docs as active onboarding or canonical runtime guidance
-- if a file in `docs/archive/` is still needed by active runtime/code/tests, it should move back out
-- active source-of-truth docs stay under the main `docs/`, `docs/reference/`, `docs/contracts/`, and `docs/qa/` surfaces

package/docs/archive/migration-mainline-post-v0.4.4.md

@@ -1,90 +0,0 @@
-# Migration Guide: post-v0.4.4 mainline changes
-
-This guide covers migration from **v0.4.4** to the current mainline changes merged after it (including PR #137 and follow-up fixes).
-
-## Who is affected
-
-You are affected if you:
-
-- invoke removed prompts or skills from old notes/scripts,
-- depend on pre-consolidation catalog names,
-- use `rcs setup` and need predictable install scope behavior,
-- run `rcs team`/tmux workflows and want the latest reliability fixes,
-- use notifier output and need verbosity control.
-
-## What changed (high level)
-
-- Catalog consolidation for prompts/skills and cleanup of deprecated entries.
-- `rcs setup` now supports scope-aware install modes (`user`, `project`). Legacy `project-local` values are auto-migrated.
-- Spark worker routing added for team workers (`--spark`, `--madmax-spark`).
-- Notifier verbosity controls added.
-- tmux runtime hardening updates landed, including post-review pane capture/input hardening.
-- Stale references to removed `scientist`/`pipeline` were cleaned up.
-
-## Removed prompts and skills
-
-### Removed prompts
-
-- `deep-executor`
-- `scientist`
-
-### Removed skills
-
-- `deepinit`
-- `learn-about-rcs`
-- `learner`
-- `pipeline`
-- `project-session-manager`
-- `psm`
-- `release`
-- `ultrapilot`
-- `writer-memory`
-
-## Mapping old references to current ones
-
-Use these replacements in docs, scripts, and personal shortcuts.
-
-| Old reference | Use now | Notes |
-|---|---|---|
-| `/prompts:deep-executor` | `/prompts:executor` | `deep-executor` was a deprecated alias to executor behavior. |
-| `/prompts:scientist` | `/prompts:researcher` | Use researcher for research-focused workflows in current catalog. |
-| `$pipeline` | `$team` (or explicit `/prompts:*` sequencing) | Team is the default orchestrator pipeline surface. |
-| `$ultrapilot` | `$team` | Use team-based parallel orchestration. |
-| `$psm` / `$project-session-manager` | No in-repo replacement | Remove from automation or maintain out-of-tree tooling. |
-| `$release` | No in-repo replacement | Use your project release process directly. |
-| `$deepinit` | `rcs agents-init [path]` | Lightweight CLI successor for AGENTS.md bootstrap only; immediate child directories only, unmanaged files preserved unless `--force`. |
-| `$learn-about-rcs` / `$learner` / `$writer-memory` | No in-repo replacement | Remove stale references from workflows/docs. |
-
-## Verification checklist after upgrade
-
-Run this checklist after pulling latest mainline:
-
-- [ ] Confirm removed references are gone from local notes/scripts:
-  ```bash
-  rg -n "deep-executor|scientist|pipeline|project-session-manager|\bpsm\b|ultrapilot|learn-about-rcs|writer-memory|learner|deepinit|\brelease\b" README.md docs scripts .rcs -S
-  ```
-- [ ] Confirm current prompt catalog no longer contains removed prompts:
-  ```bash
-  ls prompts
-  ```
-- [ ] Confirm current skill catalog no longer contains removed skills:
-  ```bash
-  ls skills
-  ```
-- [ ] Validate setup scope options are available:
-  ```bash
-  rcs help | rg -e "--scope|project"
-  ```
-- [ ] Validate team/tmux health checks:
-  ```bash
-  rcs doctor --team
-  ```
-- [ ] If using spark worker routing, verify flags are available:
-  ```bash
-  rcs --help | rg "spark|madmax-spark"
-  ```
-
-## Related docs
-
-- Release notes: [CHANGELOG.md](../CHANGELOG.md)
-- Main overview: [README.md](../README.md)

package/docs/archive/prompt-migration-changelog.md

@@ -1,141 +0,0 @@
-# Prompt Migration Changelog
-
-## Scope
-
-- Migration window: `e21cb5e` -> `ff7ee14`
-- Surface: `prompts/*.md` (30 files)
-- Goal: document the prior XML-to-Markdown migration while reaffirming that prompt files remain the canonical XML-tagged subagent role surfaces for RCS.
-
-## Global Changes (Applied to All Prompt Files)
-
-- Preserved frontmatter metadata (`description`, `argument-hint`).
-- Replaced wrapper tags such as `<Agent_Prompt>`, `<Role>`, `<Constraints>`, `<Output_Format>`, `<Final_Checklist>` with Markdown section headings.
-- Flattened nested XML-like sections into readable Markdown bullets/numbered steps.
-- Kept role semantics, tool usage intent, guardrails, and checklist expectations functionally equivalent.
-- Important current-state clarification: although the prompt text is now Markdown-first, each file in `prompts/*.md` is still the canonical XML-tagged subagent role surface consumed by RCS install/generation flows.
-
-## Behavior Notes
-
-- No intentionally introduced functional behavior changes were made in this migration commit.
-- Behavior-relevant content (constraints, verification expectations, output templates) was preserved while syntax/formatting was normalized.
-- Any post-migration behavior differences are expected to come from readability and parser-compatibility improvements, not policy changes.
-
-## Per-File Matrix
-
-| Prompt File | Added | Removed | Structural Highlights | Behavior Delta |
-|---|---:|---:|---|---|
-| `prompts/analyst.md` | 102 | 105 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/architect.md` | 102 | 104 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/build-fixer.md` | 81 | 84 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/code-reviewer.md` | 98 | 100 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/critic.md` | 79 | 82 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/debugger.md` | 85 | 88 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/deep-executor.md` | 105 | 107 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/dependency-expert.md` | 91 | 94 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/designer.md` | 96 | 98 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/executor.md` | 92 | 94 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/explore.md` | 104 | 107 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/git-master.md` | 84 | 87 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/planner.md` | 108 | 111 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/researcher.md` | 88 | 91 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/scientist.md` | 84 | 87 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/security-reviewer.md` | 119 | 121 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/test-engineer.md` | 96 | 98 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/verifier.md` | 87 | 90 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/vision.md` | 67 | 70 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-| `prompts/writer.md` | 78 | 81 | XML-like wrapper blocks converted to Markdown section hierarchy; output/checklist sections normalized. | No intentional functional delta in migration commit. |
-
-## Validation
-
-- Commit diff summary: `30 files changed, 2439 insertions(+), 2511 deletions(-)`
-- Spot-checks performed on role-heavy prompts (`planner`, `executor`, `explore`) confirmed semantic parity with formatting normalization.
-
----
-
-## Orchestration Brain Migration (`AGENTS.md`, `templates/AGENTS.md`)
-
-### Summary
-
-These files are the instruction root that RCS expects Codex to follow across a workspace.
-Changes here are primarily about aligning instructions with Codex CLI tool contracts.
-
-### Key Deltas
-
-- Updated child-agent delegation guidance to reflect the Codex `spawn_agent` API:
-  - Use `spawn_agent(message: "<role prompt>\n\nTask: ...")` conventions.
-  - Removed legacy "instructions" phrasing.
-- Expanded MCP tooling catalog and mode lifecycle expectations so orchestrators can use the full MCP surface correctly.
-- `templates/AGENTS.md` header normalized to match `AGENTS.md` and removed the non-compliant template opener.
-
-### Diff Stats
-
-| File | Added | Removed | Notes |
-|---|---:|---:|---|
-| `AGENTS.md` | 42 | 10 | Tooling + delegation guidance expanded; semantics preserved. |
-| `templates/AGENTS.md` | 7 | 7 | Header/tone normalized; still intended as a template copy. |
-
-### Unified Guidance Schema Follow-Up (AGENTS + Runtime/Worker Alignment)
-
-- Added canonical schema document: `docs/guidance-schema.md`.
-- Added explicit schema-contract sections to:
-  - `AGENTS.md`
-  - `templates/AGENTS.md`
-- Normalized worker task guidance in `AGENTS.md` runtime worker overlay:
-  - file path now uses `tasks/task-<id>.json`
-  - API id rule now explicitly requires bare id `task_id: "<id>"` (never `"task-<id>"`).
-- Marker contracts remain unchanged:
-  - `<!-- RCS:RUNTIME:START --> ... <!-- RCS:RUNTIME:END -->`
-  - `<!-- RCS:TEAM:WORKER:START --> ... <!-- RCS:TEAM:WORKER:END -->`
-
-Behavior note: this follow-up is additive and wording-focused; no task-state model or MCP API contract changes were introduced.
-
----
-
-## Skill Prompt Migration (`skills/*/SKILL.md`)
-
-### Summary
-
-Skill docs are operational runbooks. The migration focused on:
-- Removing Claude-era paths/terminology
-- Aligning config guidance with Codex-first paths (`~/.codex/…`, `CODEX_HOME`)
-- Preserving each skill's contract/intent while improving correctness for Codex CLI users
-
-### Behavior Notes
-
-- These are documentation / instruction changes; they do not directly change runtime logic.
-- One meaningful correction was made: the "agent teams" enablement guidance in `skills/rcs-setup/SKILL.md` was updated to enable Codex features via `~/.codex/config.toml` rather than legacy `settings.json` env vars.
-
-### Per-File Matrix
-
-| Skill File | Added | Removed | Structural Highlights | Behavior Delta |
-|---|---:|---:|---|---|
-| `skills/analyze/SKILL.md` | 1 | 1 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/autopilot/SKILL.md` | 11 | 16 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/code-review/SKILL.md` | 1 | 1 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/configure-notifications/SKILL.md` | 0 | 0 | Canonical notification setup skill replacing standalone configure-* variants. | Catalog/install surface now converges on one entry point. |
-| `skills/doctor/SKILL.md` | 47 | 45 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/ecomode/SKILL.md` | 1 | 1 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/help/SKILL.md` | 1 | 1 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/learner/SKILL.md` | 5 | 5 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/rcs-setup/SKILL.md` | 144 | 156 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/plan/SKILL.md` | 1 | 1 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/project-session-manager/SKILL.md` | 5 | 5 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/release/SKILL.md` | 3 | 3 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/research/SKILL.md` | 10 | 15 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/security-review/SKILL.md` | 1 | 1 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/skill/SKILL.md` | 20 | 20 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/tdd/SKILL.md` | 1 | 1 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/ultrapilot/SKILL.md` | 11 | 16 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-| `skills/writer-memory/SKILL.md` | 1 | 1 | Codex path/terminology normalization; examples updated to Codex-first conventions. | No direct runtime behavior change; instruction correctness improved. |
-
-### Hotspots Worth Reviewing
-
-- `skills/rcs-setup/SKILL.md`: largest edit surface; includes team enablement guidance (`~/.codex/config.toml` `[features]` flags) and teammate display preference storage (`~/.codex/.rcs-config.json`).
-- `skills/doctor/SKILL.md`: updated hook/config inspection language; still contains optional legacy remediation guidance.
-- `skills/autopilot/SKILL.md`, `skills/research/SKILL.md`, `skills/ultrapilot/SKILL.md`: config examples updated to TOML.
-
-## Follow-up Classification Notes
-
-- `prompts/*.md` should be treated as the canonical source set for XML-tagged subagent role prompts, even when install/runtime layers wrap them in TOML or other launcher-specific envelopes.
-- `prompts/sisyphus-lite.md` is intentionally classified as a specialized worker behavior prompt. It is not part of the primary public role catalog alongside prompts such as `executor`, `planner`, or `architect`.
-- Team worker/runtime overlays may borrow Sisyphus-lite behavior patterns, but that does not promote it to a first-class routed role.