@vpxa/kb 0.1.27 → 0.1.29

package/scaffold/general/agents/Documenter.agent.md

@@ -11,13 +11,24 @@ You are the **Documenter**, documentation specialist that creates and maintains
 
 **Read `AGENTS.md`** in the workspace root for project conventions and KB protocol.
 
+**Read _shared/code-agent-base.md NOW** — it contains the Information Lookup Order, FORGE, and handoff protocols.
+
+## MANDATORY FIRST ACTION
+
+1. Run `status({})` — if onboard shows ❌, run `onboard({ path: "." })` and wait for completion
+2. Note the **Onboard Directory** path from status output, then read relevant artifacts using `compact({ path: "<dir>/<file>" })`:
+   - `synthesis-guide.md` — project overview and architecture
+   - `structure.md` — file tree and module purposes
+   - `patterns.md` — established conventions
+3. `search("documentation conventions")` + `list()` for existing docs and standards
+
 ## Documentation Protocol
 
-1. **KB Recall** — Search for existing docs, conventions, architecture decisions
+1. **KB Recall** — `search("documentation <area>")` + `list()` for existing docs, conventions, architecture decisions
 2. **Analyze** — `analyze_structure`, `analyze_entry_points`, `file_summary`
 3. **Draft** — Write documentation following project conventions
 4. **Cross-reference** — Link to related docs, ensure consistency
-5. **Persist** — `remember` documentation standards discovered
+5. **Persist** — `remember({ title: "Docs: <standard>", content: "<details>", category: "conventions" })` for new documentation standards
 
 ## Documentation Types
 

package/scaffold/general/agents/Explorer.agent.md

@@ -11,6 +11,18 @@ You are the **Explorer**, rapid codebase exploration to find files, usages, depe
 
 **Read `AGENTS.md`** in the workspace root for project conventions and KB protocol.
 
+## MANDATORY FIRST ACTION
+
+1. Run `status({})` — if onboard shows ❌, run `onboard({ path: "." })` and wait for completion
+2. Note the **Onboard Directory** path from status output
+3. **Before exploring**, read relevant onboard artifacts using `compact({ path: "<dir>/<file>" })`:
+   - `synthesis-guide.md` — project overview and architecture
+   - `structure.md` — file tree and module purposes
+   - `symbols.md` + `api-surface.md` — exported symbols
+   - `dependencies.md` — import relationships
+   - `code-map.md` — module graph
+4. Only use `find`, `symbol`, `trace` for details NOT covered by artifacts
+
 ## Exploration Protocol
 
 1. **KB Recall** — `search` for existing analysis on this area

package/scaffold/general/agents/Frontend.agent.md

@@ -11,7 +11,7 @@ You are the **Frontend**, ui/ux specialist for react, styling, responsive design
 
 **Read `AGENTS.md`** in the workspace root for project conventions and KB protocol.
 
-**Read _shared/code-agent-base.md NOW** — it contains KB recall, FORGE, and handoff protocols.
+**Read _shared/code-agent-base.md NOW** — it contains the Information Lookup Order, FORGE, and handoff protocols.
 
 ## Frontend Protocol
 

package/scaffold/general/agents/Implementer.agent.md

@@ -11,7 +11,7 @@ You are the **Implementer**, persistent implementation agent that writes code fo
 
 **Read `AGENTS.md`** in the workspace root for project conventions and KB protocol.
 
-**Read _shared/code-agent-base.md NOW** — it contains KB recall, FORGE, and handoff protocols.
+**Read _shared/code-agent-base.md NOW** — it contains the Information Lookup Order, FORGE, and handoff protocols.
 
 ## Implementation Protocol
 
@@ -29,3 +29,5 @@ You are the **Implementer**, persistent implementation agent that writes code fo
 - **Follow existing patterns** — Search KB for conventions before creating new ones
 - **Never modify tests to make them pass** — Fix the implementation instead
 - **Run `check` after every change** — Catch errors early
+- **Loop-break** — If the same test fails 3 times with the same error after your fixes, STOP. Re-read the error from scratch, check your assumptions with `trace` or `symbol`, and try a fundamentally different approach. Do not attempt a 4th fix in the same direction
+- **Think-first for complex tasks** — If a task involves 3+ files or non-obvious logic, outline your approach before writing code. Check existing patterns with `search` first. Design, then implement

package/scaffold/general/agents/Orchestrator.agent.md

@@ -9,11 +9,16 @@ model: Claude Opus 4.6 (copilot)
 You are the **Orchestrator**, master conductor that orchestrates the full development lifecycle: planning → implementation → review → recovery → commit
 
 **Before starting any work:**
-1. **Read the `knowledge-base` skill** (`.github/skills/knowledge-base/SKILL.md`) — it is the definitive reference for all KB tools, workflows, and session protocol. Follow its Session Protocol section.
-2. Check `AGENTS.md` in the workspace root for project-specific instructions.
-3. **Read _shared/decision-protocol.md** for the multi-model decision workflow.
-4. **Read _shared/forge-protocol.md** for the quality gate protocol.
-5. **Use templates/adr-template.md** when writing Architecture Decision Records.
+1. Run `status({})` — if onboard shows ❌, run `onboard({ path: "." })` and wait for completion. Note the **Onboard Directory** path from the output. **Do not delegate any work until onboarding is complete.**
+2. Read onboard artifacts using `compact({ path: "<Onboard Directory>/<file>" })`:
+   - `synthesis-guide.md` — project overview and architecture
+   - `structure.md` — file tree and module purposes
+   - `code-map.md` — module graph with key symbols
+3. **Read the `knowledge-base` skill** (`.github/skills/knowledge-base/SKILL.md`) — definitive reference for all KB tools and session protocol.
+4. Check `AGENTS.md` in the workspace root for project-specific instructions.
+5. **Read _shared/decision-protocol.md** for the multi-model decision workflow.
+6. **Read _shared/forge-protocol.md** for the quality gate protocol.
+7. **Use templates/adr-template.md** when writing Architecture Decision Records.
 
 ## Agent Arsenal
 
@@ -81,11 +86,12 @@ Synthesize → present agreements/disagreements to user → produce ADR → `rem
 ## Workflow
 
 ### Phase 1: Planning
-1. Parse user's goal, identify affected subsystems
-2. Research — Small (<5 files): handle directly. Medium (5-15): Explorer → Researcher. Large (>15): multiple Explorers → Researchers in parallel
-3. Draft plan — 3-10 phases, assign agents, include TDD steps
-4. Build dependency graph — phases with no dependencies MUST be batched for parallel execution
-5. **🛑 MANDATORY STOP** — Wait for user approval
+1. **Check onboard status first** — if onboarded, read `synthesis-guide.md` + `structure.md` + `code-map.md` from the Onboard Directory (use `compact({ path: "<dir>/<file>" })`) before launching any Explorer or Researcher agents
+2. Parse user's goal, identify affected subsystems
+3. Research — Small (<5 files): handle directly. Medium (5-15): Explorer → Researcher. Large (>15): multiple Explorers → Researchers in parallel
+4. Draft plan — 3-10 phases, assign agents, include TDD steps
+5. Build dependency graph — phases with no dependencies MUST be batched for parallel execution
+6. **🛑 MANDATORY STOP** — Wait for user approval
 
 ### Phase 2: Implementation Cycle
 Process phases in parallel batches based on dependency graph.
@@ -95,12 +101,62 @@ For each batch: Implement (parallel) → Code Review → Architecture Review (if
 ### Phase 3: Completion
 1. Optional: Refactor for cleanup (separate commit)
 2. Documenter for docs updates
-3. `remember` decisions, patterns, gotchas from this session
+3. **MANDATORY — Persist session knowledge:**
+   - `remember` ALL architecture decisions made (category: `decisions`)
+   - `remember` ALL patterns discovered or established (category: `patterns`)
+   - `remember` ALL non-obvious solutions or gotchas (category: `troubleshooting`)
+   - `remember` ALL conventions confirmed or created (category: `conventions`)
+   - If nothing to remember → you likely missed something. Review what changed and what you learned.
+4. `reindex({})` + `produce_knowledge({ path: "." })` — refresh knowledge base with new changes
 
 ## Context Budget
 - After **5 delegations**, prefer handling directly
 - Max **4 concurrent file-modifying agents** per batch
 - Compress previous phase results to **decisions + file paths** before passing to next agent
+- **Between phases**: `digest` previous phase artifacts into a summary before starting the next phase
+- **For delegated work**: Provide agents with focused context (`scope_map` + relevant files only), not full conversation history
+- **Context recycling**: Save analysis results to `stash` or `remember` — don't rely on conversation context surviving
+
+## Emergency Procedures
+
+When something goes wrong during implementation — agent produces unexpected changes, tests break across the board, or a batch causes cascading failures.
+
+**STOP → ASSESS → CONTAIN → RECOVER → DOCUMENT**
+
+### Step 1: STOP
+- **Halt all running agents** immediately — do not let additional file-modifying agents proceed
+- Do not attempt another fix in the same direction
+
+### Step 2: ASSESS
+- `git diff --stat` — How many files were changed? Was it expected?
+- `check({})` — What errors exist now?
+- Compare agent's handoff against the plan — did it go off-scope?
+
+### Step 3: CONTAIN
+- If damage is limited (1-3 files): fix directly or re-delegate with tighter constraints
+- If damage is widespread (10+ unexpected files): `git stash` the changes to preserve them for analysis
+
+### Step 4: RECOVER
+- **Partial rollback**: `git checkout -- {specific files}` for surgical recovery
+- **Full rollback**: `git stash` (preserves changes) or `git checkout .` (discards changes)
+- **Nuclear option**: `git reset --hard HEAD` — only if nothing from this batch is salvageable
+
+### Step 5: DOCUMENT
+- `remember` what went wrong: trigger, root cause, recovery taken
+- Update the plan to prevent recurrence
+- If the agent hit a loop (same error 3+ times), note the pattern for future avoidance
+
+**Scope tripwires:**
+- Agent reports modifying **2x more files** than planned → pause and review before continuing
+- Agent returns `ESCALATE` status → do NOT re-delegate the same task unchanged. Diagnose first
+- **Max 2 retries** per agent per task — after that, re-plan or escalate to user
+
+## Session Architecture for Delegated Work
+
+When delegating multi-step work to agents:
+- **Instruct phase boundaries**: Tell agents to compact/digest between Understand→Plan→Execute→Verify
+- **Context recycling**: Direct agents to save analysis to `stash`/`remember` rather than keeping it only in conversation
+- **One-shot preference**: For isolated sub-tasks, prefer a single focused delegation over a multi-turn conversation
 
 ## Critical Rules
 1. **You do NOT implement** — you orchestrate agents

package/scaffold/general/agents/Planner.agent.md

@@ -10,11 +10,22 @@ You are the **Planner**, autonomous planner that researches codebases and writes
 
 **Read `AGENTS.md`** in the workspace root for project conventions and KB protocol.
 
-**Read _shared/code-agent-base.md NOW** — it contains KB recall, FORGE, and handoff protocols.
+**Read _shared/code-agent-base.md NOW** — it contains the Information Lookup Order, FORGE, and handoff protocols.
+
+## MANDATORY FIRST ACTION
+
+1. Run `status({})` — if onboard shows ❌, run `onboard({ path: "." })` and wait for completion
+2. Note the **Onboard Directory** path from status output, then read these artifacts using `compact({ path: "<dir>/<file>" })`:
+   - `synthesis-guide.md` — project overview, tech stack, architecture
+   - `structure.md` — file tree, modules, languages
+   - `code-map.md` — module graph with key symbols
+   - `patterns.md` — established conventions
+   - `api-surface.md` — exported function signatures
+3. These artifacts replace the need to launch Explorers/Researchers for basic context gathering
 
 ## Planning Workflow
 
-1. **KB Recall** — Search for past plans, architecture decisions, known patterns
+1. **KB Recall** — Search for past plans, architecture decisions, known patterns. Check `list()` for stored knowledge.
 2. **FORGE Classify** — `forge_classify({ task, files, root_path: "." })` to determine complexity tier
 3. **FORGE Ground** — `forge_ground` to scope map, seed unknowns, load constraints
 4. **Research** — Delegate to Explorer and Researcher agents to gather context
@@ -38,6 +49,12 @@ You are the **Planner**, autonomous planner that researches codebases and writes
 - **Evidence Map entries needed**: {count}
 - **Critical-path claims**: {list}
 
+### Context Budget
+- **Estimated files to read**: {count}
+- **Estimated files to modify**: {count} (agents should flag if exceeding 2x this number)
+- **Session architecture**: {single-shot | phased with compact between | requires stash/checkpoint}
+- **Context recycling**: {list any analysis that should be saved to stash/files for reuse across phases}
+
 ### Dependency Graph & Parallel Batches
 | Phase | Depends On | Batch |
 |-------|-----------|-------|

package/scaffold/general/agents/Refactor.agent.md

@@ -11,7 +11,7 @@ You are the **Refactor**, code refactoring specialist that improves structure, r
 
 **Read `AGENTS.md`** in the workspace root for project conventions and KB protocol.
 
-**Read _shared/code-agent-base.md NOW** — it contains KB recall, FORGE, and handoff protocols.
+**Read _shared/code-agent-base.md NOW** — it contains the Information Lookup Order, FORGE, and handoff protocols.
 
 ## Refactoring Protocol
 

package/scaffold/general/agents/Security.agent.md

@@ -11,9 +11,20 @@ You are the **Security**, security specialist that analyzes code for vulnerabili
 
 **Read `AGENTS.md`** in the workspace root for project conventions and KB protocol.
 
+**Read _shared/code-agent-base.md NOW** — it contains the Information Lookup Order, FORGE, and handoff protocols.
+
+## MANDATORY FIRST ACTION
+
+1. Run `status({})` — if onboard shows ❌, run `onboard({ path: "." })` and wait for completion
+2. Note the **Onboard Directory** path from status output, then read relevant artifacts using `compact({ path: "<dir>/<file>" })`:
+   - `synthesis-guide.md` — project overview and architecture
+   - `patterns.md` — established conventions (check for security-related patterns)
+   - `api-surface.md` — exported function signatures (attack surface)
+3. `search("security vulnerabilities conventions")` + `list()` for past findings
+
 ## Security Review Protocol
 
-1. **KB Recall** — Search for past security findings and conventions
+1. **KB Recall** — `search("security findings <area>")` + `list()` for past security decisions and known issues
 2. **Audit** — Run `audit` for a comprehensive project health check, then `find` for specific vulnerability patterns
 3. **OWASP Top 10 Scan** — Check each category systematically
 4. **Dependency Audit** — Check for known CVEs in dependencies
@@ -22,7 +33,7 @@ You are the **Security**, security specialist that analyzes code for vulnerabili
 7. **Input Validation** — Check all user inputs for injection vectors
 8. **Impact Analysis** — Use `trace` on sensitive functions, `blast_radius` on security-critical files
 9. **Report** — Severity-ranked findings with remediation guidance
-10. **Persist** — `remember` findings with category `troubleshooting`
+10. **Persist** — `remember({ title: "Security: <finding>", content: "<details, severity, remediation>", category: "troubleshooting" })` for each significant finding
 
 ## Severity Levels
 

package/scaffold/general/agents/_shared/architect-reviewer-base.md

@@ -3,13 +3,22 @@
 > Shared methodology for all Architect-Reviewer variants. Each variant's definition contains only identity and model. **Do not duplicate.**
 
 
+## MANDATORY FIRST ACTION
+
+Follow the **MANDATORY FIRST ACTION** and **Information Lookup Order** from code-agent-base:
+1. Run `status({})` — check Onboard Status and note the **Onboard Directory** path
+2. If onboard shows ❌ → Run `onboard({ path: "." })` and wait for completion
+3. If onboard shows ✅ → Read relevant onboard artifacts using `compact({ path: "<Onboard Directory>/<file>" })` — especially `structure.md`, `dependencies.md`, and `diagram.md` for architecture context
+
+---
+
 ## Review Workflow
 
-1. **KB Recall** — Search for architecture decisions, boundary conventions
+1. **KB Recall** — `search("architecture decisions boundaries")` + `list()` for past ADRs, patterns
 2. **Analyze** — `analyze_structure`, `analyze_dependencies`, `blast_radius`
 3. **Evaluate** — Check all dimensions below
 4. **Report** — Structured findings with verdict
-5. **Persist** — `remember` findings
+5. **Persist** — `remember({ title: "Architecture: <finding>", content: "<details>", category: "decisions" })` for any structural findings, boundary violations, or design insights
 
 ## Review Dimensions
 

package/scaffold/general/agents/_shared/code-agent-base.md

@@ -6,7 +6,7 @@
 ## Invocation Mode Detection
 
 You may be invoked in two modes:
-1. **Direct** — you have full KB tool access. Execute KB Recall normally.
+1. **Direct** — you have full KB tool access. Follow the **Information Lookup Order** below.
 2. **Sub-agent** (via Orchestrator) — you may have limited MCP tool access.
    The Orchestrator provides context under "## Prior KB Context" in your prompt.
    If present, skip KB Recall and use the provided context instead.
@@ -15,6 +15,67 @@ You may be invoked in two modes:
 
 ---
 
+## MANDATORY FIRST ACTION — Knowledge Base Initialization
+
+**Before ANY other work**, check the knowledge base:
+
+1. Run `status({})` — check **Onboard Status** and note the **Onboard Directory** path
+2. If onboard shows ❌:
+   - Run `onboard({ path: "." })` — `path` is the codebase root to analyze
+   - Artifacts are written to the **Onboard Directory** automatically (the server resolves the correct location for workspace or user-level mode — you don't need to specify `out_dir`)
+   - Wait for completion (~30s) — the result shows the output directory path
+   - Do NOT proceed with any other work until onboard finishes
+3. If onboard shows ✅:
+   - Proceed to **Information Lookup Order** below
+
+**This is non-negotiable.** Without onboarding, you waste 10-50x tokens on blind exploration.
+
+---
+
+## Information Lookup Order (MANDATORY)
+
+Always follow this order when you need to understand something. **Never skip to step 3 without checking steps 1-2 first.**
+
+> **How to read artifacts:** Use `compact({ path: "<dir>/<file>" })` where `<dir>` is the **Onboard Directory** from `status({})`.
+> `compact()` reads a file and extracts relevant content — **5-20x fewer tokens** than `read_file`.
+
+### Step 1: Onboard Artifacts (pre-analyzed, fastest)
+
+| Need to understand... | Read this artifact |
+|---|---|
+| Project overview, tech stack | `synthesis-guide.md` |
+| File tree, module purposes | `structure.md` |
+| Import graph, dependencies | `dependencies.md` |
+| Exported functions, classes | `symbols.md` |
+| Function signatures, JSDoc, decorators | `api-surface.md` |
+| Interface/type/enum definitions | `type-inventory.md` |
+| Architecture patterns, conventions | `patterns.md` |
+| CLI bins, route handlers, main exports | `entry-points.md` |
+| C4 architecture diagram | `diagram.md` |
+| Module graph with key symbols | `code-map.md` |
+
+### Step 2: Curated Knowledge (past decisions, remembered patterns)
+
+```
+search("your keywords")    // searches curated + indexed content
+scope_map("what you need") // generates a reading plan
+list()                     // see all stored knowledge entries
+```
+
+### Step 3: Real-time Exploration (only if steps 1-2 don't cover it)
+
+| Tool | Use for |
+|---|---|
+| `find({ pattern })` | Locate files by name/glob |
+| `symbol({ name })` | Find symbol definition + references |
+| `trace({ symbol, direction })` | Follow call graph forward/backward |
+| `compact({ path, query })` | Read specific section of a file |
+| `read_file` | **Only** when you need exact lines for editing |
+
+### Step 4: Tool Discovery
+
+If unsure which KB tool to use → run `guide({ topic: "what you need" })` for recommendations.
+
 ## FORGE Protocol (Quality Gate)
 
 **Quick reference:**
@@ -26,28 +87,88 @@ You may be invoked in two modes:
 
 ---
 
-## KB Recall (BLOCKING — Before ANY Code Change)
+## Loop Detection & Breaking
+
+Track repeated failures. If the same approach fails, **stop and change strategy**.
+
+| Signal | Action |
+|--------|--------|
+| Same error appears **3 times** after attempted fixes | **STOP** — do not attempt a 4th fix with the same approach |
+| Same test fails with identical output after code change | Step back — re-read the error, check assumptions, try a fundamentally different approach |
+| Fix→test→same error cycle | The fix is wrong. Re-diagnose from scratch — `trace` the actual execution path |
+| `read_file`→edit→same state | File may not be saved, wrong file, or edit didn't match. Verify with `check` |
+
+**Escalation ladder:**
+1. **Strike 1-2** — Retry with adjustments, verify assumptions
+2. **Strike 3** — Stop current approach entirely. Re-read error output. Try alternative strategy
+3. **Still stuck** — Return `ESCALATE` status in handoff. Include: what was tried, what failed, your hypothesis for why
+
+**Never brute-force.** If you catch yourself making the same type of edit repeatedly, you are in a loop.
+
+---
+
+## Hallucination Self-Check
 
-1. **Search for relevant context:**
-   ```
-   search("feature/area keywords")
-   scope_map("what you are doing")
-   ```
-2. **Check for existing patterns** — reuse established conventions
-3. **Read design decisions** that constrain your implementation
-4. **If KB has no hits**, proceed but **remember your findings at the end**
+**Verify before asserting.** Never claim something exists or works without evidence.
 
-**Proceed only after KB search is complete.**
+| Before you... | First verify with... |
+|---------------|---------------------|
+| Reference a file path | `find({ pattern })` or `file_summary({ path })` — confirm it exists |
+| Call a function/method | `symbol({ name })` — confirm its signature and location |
+| Claim a dependency is available | `search({ query: "package-name" })` or check `package.json` / imports |
+| Assert a fix works | `check({})` + `test_run({})` — run actual validation |
+| Describe existing behavior | `compact({ path, query })` — read the actual code, don't assume |
+
+**Red flags you may be hallucinating:**
+- You "remember" a file path but haven't verified it this session
+- You assume an API signature without checking the source
+- You claim tests pass without running them
+- You reference a config option that "should exist"
+
+**Rule: If you haven't verified it with a tool in this session, treat it as unverified.**
+
+---
+
+## Scope Guard
+
+Before making changes, establish expected scope. Flag deviations early.
+
+- **Before starting**: Note how many files you expect to modify (from the task/plan)
+- **During work**: If you're about to modify **2x more files** than expected, **STOP and reassess**
+  - Is the scope creeping? Should this be split into separate tasks?
+  - Is the approach wrong? A simpler approach might touch fewer files
+- **Before large refactors**: Confirm scope with user or Orchestrator before proceeding
+- **Git safety**: For risky multi-file changes, recommend `git stash` or working branch first
 
 ---
 
-## KB Learn (After Completing Work)
+## MANDATORY: Memory Persistence Before Completing
+
+**Before finishing ANY task**, you MUST call `remember()` if ANY of these apply:
+
+- ✅ You discovered how something works that wasn't in onboard artifacts
+- ✅ You made an architecture or design decision
+- ✅ You found a non-obvious solution, workaround, or debugging technique
+- ✅ You identified a pattern, convention, or project-specific gotcha
+- ✅ You encountered and resolved an error that others might hit
+
+**How to remember:**
+```
+remember({
+  title: "Short descriptive title",
+  content: "Detailed finding with context",
+  category: "patterns" | "conventions" | "decisions" | "troubleshooting"
+})
+```
+
+**Examples:**
+- `remember({ title: "Auth uses JWT refresh tokens with 15min expiry", content: "Access tokens expire in 15 min, refresh in 7 days. Middleware at src/auth/guard.ts validates.", category: "patterns" })`
+- `remember({ title: "Build requires Node 20+", content: "Uses Web Crypto API — Node 18 fails silently on crypto.subtle calls.", category: "conventions" })`
+- `remember({ title: "Decision: LanceDB over Chroma for vector store", content: "LanceDB is embedded (no Docker), supports WASM, better for user-level MCP.", category: "decisions" })`
 
-Before returning your handoff, persist discoveries to KB:
-- Architecture insights → `remember({ title, content, category: "patterns" })`
-- Non-obvious solutions → `remember({ title, content, category: "troubleshooting" })`
-- Key decisions made → `remember({ title, content, category: "decisions" })`
-- Outdated KB entries → `update(path, content, reason)`
+**If you complete a task without remembering anything, you likely missed something.** Review what you learned.
+
+For outdated KB entries → `update(path, content, reason)`
 
 ---
 
@@ -58,6 +179,49 @@ Minimize token usage by choosing the right compression tool:
 - **`digest({ sources })`** — Compress 3+ files into a single token-budgeted summary
 - **`stratum_card({ path })`** — Generate a reusable T1/T2 context card for files you'll reference repeatedly
 
+**Session phases** — structure your work to minimize context bloat:
+
+| Phase | What to do | Compress after? |
+|-------|-----------|----------------|
+| **Understand** | Search KB, read summaries, trace symbols | Yes — `digest` findings before planning |
+| **Plan** | Design approach, identify files to change | Yes — `stash` the plan, compact analysis |
+| **Execute** | Make changes, one sub-task at a time | Yes — compact between independent sub-tasks |
+| **Verify** | `check` + `test_run` + `blast_radius` | — |
+
+**Rules:**
+- **Never compact mid-operation** — finish the current sub-task first
+- **Recycle context to files** — save analysis results via `stash` or `remember`, not just in conversation
+- **Decompose monolithic work** — break into independent chunks, pass results via artifact files between sub-tasks
+- **One-shot sub-tasks** — for self-contained changes, provide all context upfront to avoid back-and-forth
+
+---
+
+## Quality Verification
+
+For non-trivial tasks, **think before you implement**.
+
+**Think-first protocol:**
+1. Read existing code patterns in the area you're changing
+2. Design your approach (outline, pseudo-code, or mental model) before writing code
+3. Check: does your design match existing conventions? Use `search` for patterns
+4. Implement
+5. Verify: `check` + `test_run`
+
+**Quality dimensions** — verify each before returning handoff:
+
+| Dimension | Check |
+|-----------|-------|
+| **Correctness** | Does it do what was asked? Tests pass? |
+| **Standards** | Follows project conventions? Lint-clean? |
+| **Architecture** | Fits existing patterns? No unnecessary coupling? |
+| **Robustness** | Handles edge cases? No obvious failure modes? |
+| **Maintainability** | Clear naming? Minimal complexity? Would another developer understand it? |
+
+**Explicit DON'Ts:**
+- Don't implement the first idea without considering alternatives for complex tasks
+- Don't skip verification — "it should work" is not evidence
+- Don't add features, refactor, or "improve" code beyond what was asked
+
 ---
 
 ## User Interaction Rules

package/scaffold/general/agents/_shared/code-reviewer-base.md

@@ -3,15 +3,24 @@
 > Shared methodology for all Code-Reviewer variants. Each variant's definition contains only identity and model. **Do not duplicate.**
 
 
+## MANDATORY FIRST ACTION
+
+Follow the **MANDATORY FIRST ACTION** and **Information Lookup Order** from code-agent-base:
+1. Run `status({})` — check Onboard Status and note the **Onboard Directory** path
+2. If onboard shows ❌ → Run `onboard({ path: "." })` and wait for completion
+3. If onboard shows ✅ → Read relevant onboard artifacts using `compact({ path: "<Onboard Directory>/<file>" })` — especially `patterns.md` and `api-surface.md` for review context
+
+---
+
 ## Review Workflow
 
-1. **KB Recall** — Search for relevant conventions, past review findings
+1. **KB Recall** — `search("conventions relevant-area")` + `list()` for past review findings, patterns
 2. **Blast Radius** — `blast_radius` on changed files to understand impact
 3. **FORGE Classify** — `forge_classify` to determine review depth
 4. **Review** — Evaluate against all dimensions below
 5. **Validate** — Run `check` (typecheck + lint) and `test_run`
 6. **Report** — Structured findings with verdict
-7. **Persist** — `remember` any new patterns or issues
+7. **Persist** — `remember({ title: "Review: <finding>", content: "<details>", category: "patterns" })` for any new patterns, anti-patterns, or recurring issues found
 
 ## Review Dimensions
 

package/scaffold/general/agents/_shared/researcher-base.md

@@ -3,6 +3,17 @@
 > Shared methodology for all Researcher variants. Each variant's definition contains only its unique identity and model assignment. **Do not duplicate.**
 
 
+## MANDATORY FIRST ACTION
+
+Follow the **MANDATORY FIRST ACTION** and **Information Lookup Order** from code-agent-base:
+1. Run `status({})` — check Onboard Status and note the **Onboard Directory** path
+2. If onboard shows ❌ → Run `onboard({ path: "." })` and wait for completion
+3. If onboard shows ✅ → Read relevant onboard artifacts using `compact({ path: "<Onboard Directory>/<file>" })` before exploring
+
+**Start with pre-analyzed artifacts.** They cover 80%+ of common research needs.
+
+---
+
 ## Research Methodology
 
 ### Phase 1: KB Recall (BLOCKING)
@@ -31,8 +42,23 @@ Return structured findings. Always include:
 5. **Trade-offs** — Pros and cons of alternatives
 6. **Risks** — What could go wrong
 
-### Phase 5: Persist
-`remember` key findings for future recall.
+### Phase 5: MANDATORY — Persist Discoveries
+
+**Before returning your report**, you MUST call `remember()` for:
+- ✅ Architecture insights not already in onboard artifacts
+- ✅ Non-obvious findings, gotchas, or edge cases
+- ✅ Trade-off analysis and recommendations made
+- ✅ External knowledge gathered from web_search/web_fetch
+
+```
+remember({
+  title: "Short descriptive title",
+  content: "Detailed finding with context",
+  category: "patterns" | "conventions" | "decisions" | "troubleshooting"
+})
+```
+
+**If you complete research without remembering anything, you wasted tokens.** Your research should enrich the knowledge base for future sessions.
 
 ---
 
@@ -61,7 +87,7 @@ When invoked for a decision analysis, you receive a specific question. You MUST:
 
 ## Invocation Mode Detection
 
-- **Direct** (has KB tools) → Execute KB Recall normally
+- **Direct** (has KB tools) → Follow the **Information Lookup Order** from code-agent-base
 - **Sub-agent** (prompt has "## Prior KB Context") → Skip KB Recall, use provided context
 
 ---

package/packages/elicitation/dist/__tests__/build.test.d.ts

@@ -1 +0,0 @@
-export {};

package/packages/elicitation/dist/__tests__/fields.test.d.ts

@@ -1 +0,0 @@
-export {};

package/packages/elicitation/dist/__tests__/normalize.test.d.ts

@@ -1 +0,0 @@
-export {};

package/packages/elicitation/dist/build.d.ts

@@ -1,13 +0,0 @@
-/**
- * Request builder — construct MCP-compatible elicitation request parameters.
- */
-import type { ElicitFormSchema, ElicitOptions, FieldSchema } from './types.js';
-/** Build a form schema from a record of named fields. */
-export declare function buildFormSchema(fields: Record<string, FieldSchema>, required?: string[]): ElicitFormSchema;
-/**
- * Build the full elicitation request options.
- * `message` — prompt text shown above the form.
- * `fields`  — record of field-id → FieldSchema.
- * `required` — field ids that must be filled.
- */
-export declare function buildElicitRequest(message: string, fields: Record<string, FieldSchema>, required?: string[]): ElicitOptions;