@hanzlaa/rcode 2.3.1 → 2.3.3

package/rihal/team.yaml

@@ -6,6 +6,7 @@ agents:
   - id: rihal-sadiq
     name: Sadiq (صادق)
     file_path: rihal/agents/rihal-sadiq.md
+    skill_path: rihal/skills/agents/sadiq-analyst
     role: Director of Strategy
     authority_level: strategic
     domain_keywords:
@@ -21,6 +22,7 @@ agents:
   - id: rihal-waleed
     name: Waleed (وليد)
     file_path: rihal/agents/rihal-waleed.md
+    skill_path: rihal/skills/agents/waleed-architect
     role: Chief Technology Officer
     authority_level: technical
     domain_keywords:
@@ -38,6 +40,7 @@ agents:
   - id: rihal-fatima
     name: Fatima (فاطمة)
     file_path: rihal/agents/rihal-fatima.md
+    skill_path: rihal/skills/agents/fatima-qa
     role: QA Lead
     authority_level: quality
     domain_keywords:
@@ -54,6 +57,7 @@ agents:
   - id: rihal-mariam
     name: Mariam (مريم)
     file_path: rihal/agents/rihal-mariam.md
+    skill_path: rihal/skills/agents/mariam-marketing
     role: Marketing & Growth Lead
     authority_level: growth
     domain_keywords:
@@ -69,6 +73,7 @@ agents:
   - id: rihal-hussain-pm
     name: Hussain (حسين)
     file_path: rihal/agents/rihal-hussain-pm.md
+    skill_path: rihal/skills/agents/hussain-pm
     role: Product Manager
     authority_level: product
     domain_keywords:
@@ -97,6 +102,7 @@ agents:
   - id: rihal-yousef
     name: Yousef (يوسف)
     file_path: rihal/agents/rihal-yousef.md
+    skill_path: rihal/skills/agents/yousef-backend
     role: Senior Backend Engineer
     authority_level: technical
     domain_keywords:
@@ -115,6 +121,7 @@ agents:
   - id: rihal-haitham
     name: Haitham (هيثم)
     file_path: rihal/agents/rihal-haitham.md
+    skill_path: rihal/skills/agents/haitham-frontend
     role: Senior Frontend Engineer
     authority_level: technical
     domain_keywords:
@@ -132,6 +139,7 @@ agents:
   - id: rihal-zayd
     name: Zayd (زيد)
     file_path: rihal/agents/rihal-zayd.md
+    skill_path: rihal/skills/agents/zayd-ml
     role: Senior ML Engineer
     authority_level: technical
     domain_keywords:
@@ -150,6 +158,7 @@ agents:
   - id: rihal-layla
     name: Layla (ليلى)
     file_path: rihal/agents/rihal-layla.md
+    skill_path: rihal/skills/agents/layla-designer
     role: UX Designer
     authority_level: design
     domain_keywords:
@@ -166,6 +175,7 @@ agents:
   - id: rihal-zahra
     name: Zahra (زهرة)
     file_path: rihal/agents/rihal-zahra.md
+    skill_path: rihal/skills/agents/zahra-branding
     role: Branding & Creative Director
     authority_level: design
     domain_keywords:
@@ -182,6 +192,7 @@ agents:
   - id: rihal-nasser
     name: Nasser (ناصر)
     file_path: rihal/agents/rihal-nasser.md
+    skill_path: rihal/skills/agents/nasser-eng-manager
     role: Software Engineering Manager
     authority_level: management
     domain_keywords:
@@ -215,6 +226,7 @@ agents:
   - id: rihal-noor
     name: Noor (نور)
     file_path: rihal/agents/rihal-noor.md
+    skill_path: rihal/skills/agents/noor-writer
     role: Technical Writer & Presentation Lead
     authority_level: operational
     domain_keywords:
@@ -231,6 +243,7 @@ agents:
   - id: rihal-ahmed
     name: Ahmed Al Hassani (أحمد الحسني)
     file_path: rihal/agents/rihal-ahmed.md
+    skill_path: rihal/skills/agents/ahmed-hassani-director
     role: Technology & Development Director
     authority_level: strategic
     domain_keywords:
@@ -246,6 +259,7 @@ agents:
   - id: rihal-hanzla
     name: Hanzla (حنظلة)
     file_path: rihal/agents/rihal-hanzla.md
+    skill_path: rihal/skills/agents/hanzla-engineer
     role: Senior Full-Stack Engineer
     authority_level: technical
     domain_keywords:
@@ -341,3 +355,194 @@ routing:
     - rihal-hussain-pm  # scope
     - rihal-waleed      # feasibility
     - rihal-sadiq       # strategic fit
+
+# Tactical / Workflow Agents (spawned by workflows)
+tactical_agents:
+  - id: rihal-advisor-researcher
+    name: Advisor Researcher
+    file_path: rihal/agents/rihal-advisor-researcher.md
+    role: Advisor Researcher
+    authority_level: research
+    description: Advisor-level deep research for gray-area decisions
+
+  - id: rihal-architect
+    name: Architect
+    file_path: rihal/agents/rihal-architect.md
+    role: Architect
+    authority_level: technical
+    description: Enterprise architecture and system design reviews
+
+  - id: rihal-assumptions-analyzer
+    name: Assumptions Analyzer
+    file_path: rihal/agents/rihal-assumptions-analyzer.md
+    role: Assumptions Analyzer
+    authority_level: analysis
+    description: Analyzes phase assumptions with evidence from codebase
+
+  - id: rihal-code-fixer
+    name: Code Fixer
+    file_path: rihal/agents/rihal-code-fixer.md
+    role: Code Fixer
+    authority_level: implementation
+    description: Applies code review findings and implements fixes
+
+  - id: rihal-code-reviewer
+    name: Code Reviewer
+    file_path: rihal/agents/rihal-code-reviewer.md
+    role: Code Reviewer
+    authority_level: quality
+    description: Reviews code for bugs, security issues, quality problems
+
+  - id: rihal-codebase-mapper
+    name: Codebase Mapper
+    file_path: rihal/agents/rihal-codebase-mapper.md
+    role: Codebase Mapper
+    authority_level: analysis
+    description: Explores and documents codebase architecture
+
+  - id: rihal-debugger
+    name: Debugger
+    file_path: rihal/agents/rihal-debugger.md
+    role: Debugger
+    authority_level: implementation
+    description: Investigates bugs using scientific method
+
+  - id: rihal-deviation-analyzer
+    name: Deviation Analyzer
+    file_path: rihal/agents/rihal-deviation-analyzer.md
+    role: Deviation Analyzer
+    authority_level: analysis
+    description: Analyzes plan deviations and root causes
+
+  - id: rihal-docs-auditor
+    name: Docs Auditor
+    file_path: rihal/agents/rihal-docs-auditor.md
+    role: Docs Auditor
+    authority_level: quality
+    description: Audits documentation completeness and accuracy
+
+  - id: rihal-edge-case-hunter
+    name: Edge Case Hunter
+    file_path: rihal/agents/rihal-edge-case-hunter.md
+    role: Edge Case Hunter
+    authority_level: quality
+    description: Enumerates edge cases and boundary conditions
+
+  - id: rihal-executor
+    name: Executor
+    file_path: rihal/agents/rihal-executor.md
+    role: Executor
+    authority_level: implementation
+    description: Executes sprint plans with atomic commits
+
+  - id: rihal-integration-checker
+    name: Integration Checker
+    file_path: rihal/agents/rihal-integration-checker.md
+    role: Integration Checker
+    authority_level: quality
+    description: Verifies cross-phase integration and E2E flows
+
+  - id: rihal-nyquist-auditor
+    name: Nyquist Auditor
+    file_path: rihal/agents/rihal-nyquist-auditor.md
+    role: Nyquist Auditor
+    authority_level: quality
+    description: Fills test coverage gaps for phase requirements
+
+  - id: rihal-phase-researcher
+    name: Phase Researcher
+    file_path: rihal/agents/rihal-phase-researcher.md
+    role: Phase Researcher
+    authority_level: research
+    description: Researches how to implement a phase before planning
+
+  - id: rihal-planner
+    name: Planner
+    file_path: rihal/agents/rihal-planner.md
+    role: Planner
+    authority_level: planning
+    description: Creates executable phase plans with task breakdown
+
+  - id: rihal-profiler
+    name: Profiler
+    file_path: rihal/agents/rihal-profiler.md
+    role: Profiler
+    authority_level: analysis
+    description: Analyzes user behavior patterns and personas
+
+  - id: rihal-project-researcher
+    name: Project Researcher
+    file_path: rihal/agents/rihal-project-researcher.md
+    role: Project Researcher
+    authority_level: research
+    description: Researches domain ecosystem before roadmap creation
+
+  - id: rihal-remediation-planner
+    name: Remediation Planner
+    file_path: rihal/agents/rihal-remediation-planner.md
+    role: Remediation Planner
+    authority_level: planning
+    description: Plans remediation for issues and blockers
+
+  - id: rihal-research-synthesizer
+    name: Research Synthesizer
+    file_path: rihal/agents/rihal-research-synthesizer.md
+    role: Research Synthesizer
+    authority_level: research
+    description: Synthesizes parallel research outputs into summary
+
+  - id: rihal-roadmapper
+    name: Roadmapper
+    file_path: rihal/agents/rihal-roadmapper.md
+    role: Roadmapper
+    authority_level: planning
+    description: Creates project roadmaps with phase breakdown
+
+  - id: rihal-security-adversary
+    name: Security Adversary
+    file_path: rihal/agents/rihal-security-adversary.md
+    role: Security Adversary
+    authority_level: security
+    description: Adversarial security review and threat modeling
+
+  - id: rihal-security-auditor
+    name: Security Auditor
+    file_path: rihal/agents/rihal-security-auditor.md
+    role: Security Auditor
+    authority_level: security
+    description: Comprehensive security audit and compliance verification
+
+  - id: rihal-sprint-checker
+    name: Sprint Checker
+    file_path: rihal/agents/rihal-sprint-checker.md
+    role: Sprint Checker
+    authority_level: quality
+    description: Verifies sprint plans before execution
+
+  - id: rihal-ui-auditor
+    name: UI Auditor
+    file_path: rihal/agents/rihal-ui-auditor.md
+    role: UI Auditor
+    authority_level: quality
+    description: Audits UI for usability and design quality
+
+  - id: rihal-ux-designer
+    name: UX Designer
+    file_path: rihal/agents/rihal-ux-designer.md
+    role: UX Designer
+    authority_level: design
+    description: UX design, accessibility, and design system work
+
+  - id: rihal-plan-checker
+    name: Plan Checker
+    file_path: rihal/agents/rihal-sprint-checker.md
+    role: Plan Checker
+    authority_level: quality
+    description: Validates PLAN.md/SPRINT.md quality before execution (alias for sprint-checker)
+
+  - id: rihal-verifier
+    name: Verifier
+    file_path: rihal/agents/rihal-verifier.md
+    role: Verifier
+    authority_level: quality
+    description: Verifies phase goal achievement goal-backward

package/rihal/workflows/autonomous.md

@@ -40,8 +40,8 @@ interpos audit (issue #221) — DO NOT regress.
 @.rihal/references/output-format.md
 @.rihal/references/workstream-flag.md
 @.rihal/references/output-realism.md
-@.rihal/skills/_shared/no-autonomous-bypass.md
-@.rihal/skills/_shared/state-sync-rule.md
+@rihal/brain/best-practices/no-autonomous-bypass.md
+@rihal/brain/best-practices/state-sync-rule.md
 
 Read all files referenced by the invoking prompt's execution_context before starting.
 

package/rihal/workflows/chain.md

@@ -185,4 +185,4 @@ Silent on failure — state tracking is optional.
 - **state.json missing or corrupted:** continue without error — chain artifacts are mandatory, state tracking is optional.
 - **Stage fails to produce artifact:** print stage number, allow `/rihal:chain --continue` to resume.
 - **Agent returns empty output:** print "Agent produced no output. Check input and retry."
-- **`rihal-tools.cjs` missing:** tell user to run `rihal-code install-v2`.
+- **`rihal-tools.cjs` missing:** tell user to run `npx @hanzlaa/rcode install` (or `rcode install` if installed globally).

package/rihal/workflows/checkpoint-preview.md

@@ -4,4 +4,4 @@
 Human-in-the-loop change review. Make sense of a diff, focus attention where it matters, and walk through testing. Delegates to the rihal-checkpoint-preview skill for the full review protocol.
 </purpose>
 
-@.rihal/skills/rihal-checkpoint-preview/SKILL.md
+@rihal/skills/actions/4-implementation/rihal-checkpoint-preview/SKILL.md

package/rihal/workflows/council.md

@@ -558,7 +558,7 @@ node .rihal/bin/rihal-tools.cjs state record-session
 
 - **Empty arguments or --help:** print usage block (Step 0), stop.
 - **Single-agent question detected:** redirect to `/rihal:discuss` (Step 0.5).
-- **`rihal-tools.cjs` not found:** user has v1 installed or package broken. Tell user to run `rihal-code install-v2`.
+- **`rihal-tools.cjs` not found:** user has v1 installed or package broken. Tell user to run `npx @hanzlaa/rcode install` (or `rcode install` if installed globally).
 - **Panel contains unknown agent:** print the installed-agent list and exit.
 - **state.json missing or corrupted:** continue without error — session artifact is mandatory, state tracking is optional.
 - **All panelists return empty responses:** likely subagents were spawned without proper prompts. Re-check Step 4 prompt construction.

package/rihal/workflows/discuss.md

@@ -219,8 +219,8 @@ This records the discuss session in `.rihal/state.json` as `last_session`.
 - **Empty arguments:** print usage block and stop (Step 0).
 - **Question redirects to council:** print redirect message (Step 0.5).
 - **state.json missing or corrupted:** continue without error — session artifact is mandatory, state tracking is optional.
-- **`rihal-tools.cjs` not found:** tell the user to run `rihal-code install-v2`.
-- **No installed agents:** print "No agents installed. Run `rihal-code install-v2`."
+- **`rihal-tools.cjs` not found:** tell the user to run `npx @hanzlaa/rcode install` (or `rcode install` if installed globally).
+- **No installed agents:** print "No agents installed. Run `npx @hanzlaa/rcode install`."
 - **Agent id not in installed list:** print available agents and ask the user to pick one.
 - **Agent returns empty response:** print "Agent responded empty. Check question clarity and retry."
 

package/rihal/workflows/document-project.md

@@ -162,7 +162,7 @@ After audit, if resume-work.md exists, prepend to Step 2:
 ```
 If .rihal/DOCS-AUDIT.md exists, check for missing/stale docs:
 
-@.rihal/DOCS-AUDIT.md
+<!-- DOCS-AUDIT.md is generated at runtime by rihal-tools.cjs docs-audit — not a tracked file -->
 ```
 
 ## Success Criteria

package/rihal/workflows/enable-hooks.md

@@ -30,7 +30,7 @@ cat .rihal/templates/settings-hooks.json
 Parse the JSON. If file does not exist, print error and stop:
 
 ```
-Error: .rihal/templates/settings-hooks.json not found. Run 'rihal-code install-v2' to reinstall.
+Error: .rihal/templates/settings-hooks.json not found. Run 'npx @hanzlaa/rcode install' to reinstall.
 ```
 
 ## Step 2 — Load or create settings.json

package/rihal/workflows/help.md

@@ -109,7 +109,7 @@ init → new-project → plan → execute → next → status → ship
 | `/rihal:code-review-fix` | Auto-apply fixes from a code-review report. |
 | `/rihal:verify-phase <n>` | Goal-backward audit — does the codebase deliver what the phase promised? |
 | `/rihal:verify-work` | Conversational UAT against acceptance criteria. |
-| `/rihal:ship` | Create PR, run review, prepare for merge after verification passes. |
+| `/rihal:ship [phase] [--draft]` | Push feature branch + open PR with auto-generated body (goal, changes, requirements, verification status). Use AFTER `/rihal:verify-phase` passes. **Not for npm publish or tagging releases.** |
 
 ## Phase & plan management
 

package/rihal/workflows/prfaq.md

@@ -4,4 +4,4 @@
 Working Backwards PRFAQ challenge. Stress-test a product concept by writing the press release before building it. Produces a battle-hardened PRFAQ document + PRD distillate. Delegates to the rihal-prfaq skill for the full interview and generation protocol.
 </purpose>
 
-@.rihal/skills/rihal-prfaq/SKILL.md
+@rihal/skills/actions/1-analysis/rihal-prfaq/SKILL.md

package/rihal/workflows/settings.md

@@ -124,38 +124,39 @@ STOP.
 
 ## Step 2 — Interactive edit
 
-After Step 1.5 prints the table, ask:
+After Step 1.5 prints the table, use the **`AskUserQuestion` tool** to prompt:
 
 ```
-Question:
 Which setting would you like to change?
 
-Options:
   1. mode (guided / yolo)
-  2. model_profile
-  3. workflow.research_by_default
-  4. workflow.plan_checker
-  5. workflow.post_execute_gates
-  6. workflow.ui_safety_gate
-  7. workflow.discuss_mode
-  8. git.branching_strategy
-  9. git.commit_docs
+  2. model_profile (quality / balanced / budget / inherit)
+  3. workflow.research_by_default (true / false)
+  4. workflow.plan_checker (true / false)
+  5. workflow.post_execute_gates (true / false)
+  6. workflow.ui_safety_gate (true / false)
+  7. workflow.discuss_mode (adaptive / discuss / skip)
+  8. git.branching_strategy (none / feature-branch / worktree-isolation)
+  9. git.commit_docs (true / false)
   10. communication_language
-  11. output.verbose (slim output / full detail)
+  11. output.verbose (true / false)
   0. Done — exit
 ```
 
-For each pick, present the constrained option set from Step 1.8, accept the
-choice, then call:
+When the user picks a number, use **`AskUserQuestion` tool** again to ask for
+the new value, showing the allowed values for that key.
+
+Then call:
 
 ```bash
 node .rihal/bin/rihal-tools.cjs config-set "{dotted.key}" "{value}"
 ```
 
-After each successful write, re-display the affected row from Step 1.5 so the
-user sees the change took effect.
+After each successful write, re-display the affected row so the user sees the
+change took effect.
 
-Loop until the user picks `0. Done`.
+Use **`AskUserQuestion`** again to loop — ask which setting to change next —
+until the user picks `0` or `done`.
 
 ## Step 3 — Closing summary
 

package/rihal/workflows/ship.md

@@ -1,5 +1,35 @@
 <purpose>
-Create a pull request from completed phase/milestone work, generate a rich PR body from planning artifacts, optionally run code review, and prepare for merge. Closes the plan → execute → verify → ship loop.
+Push a verified feature branch and open a pull request with an auto-generated
+body drawn from planning artifacts (ROADMAP, VERIFICATION, SUMMARY). Closes
+the plan → execute → verify → **ship** loop.
+
+**What this command does:**
+1. Runs preflight: clean tree, on a feature branch, VERIFICATION.md passed, gh CLI authenticated
+2. Pushes the branch to origin
+3. Generates a rich PR body — phase goal, list of changes, requirements addressed, verification status
+4. Creates the PR via `gh pr create`
+5. Optionally requests a reviewer
+6. Updates STATE.md with shipping status
+
+**Preconditions (all must be true before running):**
+- `/rihal:execute <phase>` completed
+- `/rihal:verify-phase <phase>` passed (VERIFICATION.md exists with `status: passed`)
+- You are on a feature branch (not main/develop directly)
+- `gh` CLI is authenticated (`gh auth status`)
+
+**This command is NOT for:**
+- Publishing npm packages → use `npm publish`
+- Creating git release tags → use `git tag -a vX.Y.Z && git push --tags`
+- Repos that commit directly to main (`git.branching_strategy: none`)
+- The rihal-code framework repo itself (no phases exist there)
+
+**Typical usage:**
+```
+/rihal:plan 1          → plan the phase
+/rihal:execute 1       → build it
+/rihal:verify-phase 1  → prove it works
+/rihal:ship 1          → PR it ← you are here
+```
 </purpose>
 
 <required_reading>

package/rihal/workflows/sprint-planning.md

@@ -37,8 +37,8 @@ they bypass the capacity gate.
 
 <required_reading>
 @.rihal/references/output-format.md
-@.rihal/skills/_shared/no-autonomous-bypass.md
-@.rihal/skills/_shared/state-sync-rule.md
+@rihal/brain/best-practices/no-autonomous-bypass.md
+@rihal/brain/best-practices/state-sync-rule.md
 </required_reading>
 
 <output_format>