valent-pipeline 0.2.19 → 0.2.21

package/pipeline/templates/mobile-handoff.template.md

@@ -0,0 +1,122 @@
+# mobile-handoff
+<!-- Template version: 1.0 | Used by: MOBILE | Read by: CRITIC, QA-B -->
+
+---
+agent: {agent-name}
+story: {story-id}
+status: {in_progress | completed}
+stepsCompleted: []
+pendingSteps: []
+lastCheckpoint: {ISO-8601 timestamp}
+inputsRead: []
+outputsWritten: []
+blockers: []
+ios_deferred: {true | false}
+---
+
+## Orchestrator Summary -- required
+- **Agent:** {agent-name}
+- **Story:** {story-id}
+- **Verdict:** {pass | fail | needs-review}
+- **State transition:** {from-phase} -> {to-phase}
+- **Files created/modified:** {list of file paths}
+- **Flags:** {alerts for downstream agents, or "none"}
+
+## Files Created/Modified -- required
+<!-- All production and test files created or modified, with purpose. -->
+
+| File | Action | Purpose |
+|------|--------|---------|
+| {file-path} | {created \| modified} | {one-line description} |
+
+## Screens Implemented -- required
+<!-- Mobile screens implemented in this story. -->
+
+| Screen | File Path | Description | Navigation |
+|--------|-----------|-------------|------------|
+| {screen-name} | {file-path} | {one-line description} | {stack \| tab \| drawer \| modal \| deep-link} |
+
+## Components Created -- required
+<!-- UI components implemented in this story. -->
+
+| Component | File Path | Description | Props |
+|-----------|-----------|-------------|-------|
+| {component-name} | {file-path} | {one-line description} | {key props and types} |
+
+## Navigation Structure -- conditional
+<!-- Only include if new navigation routes were added -->
+
+| Route / Deep Link | Screen | Description | Auth Required |
+|-------------------|--------|-------------|---------------|
+| {deep-link-uri or nav-route} | {screen-name} | {one-line description} | {yes \| no} |
+
+## API Dependencies on BEND -- required
+<!-- Every backend endpoint consumed by the mobile app, with integration details. -->
+
+| Method | Endpoint | Used By | Purpose | Error Handling |
+|--------|----------|---------|---------|----------------|
+| {HTTP method} | {route path} | {screen-name} | {one-line description} | {how errors are displayed to user} |
+
+## Emulator Configuration -- required
+<!-- Emulator/simulator used for testing. -->
+
+| Platform | Device | OS Version | Status |
+|----------|--------|------------|--------|
+| Android | {AVD name or device} | {API level} | {booted \| failed \| skipped} |
+| iOS | {simulator name} | {iOS version} | {booted \| failed \| skipped \| deferred} |
+
+## Maestro Flow Files -- required
+<!-- E2E test flows written and executed. -->
+
+| Flow File | AC Coverage | Platform | Status |
+|-----------|-------------|----------|--------|
+| {file-path} | {AC-ids} | {both \| android \| ios} | {passed \| failed \| deferred} |
+
+## Test Files Written -- required
+<!-- All test files created or modified, mapped to the spec cases they satisfy. -->
+
+| Test File | Test Cases | Spec Reference |
+|-----------|-----------|----------------|
+| {file-path} | {list of test case names} | `qa-test-spec.md#{ac-id}` |
+
+## Test Results Summary -- required
+<!-- Aggregate results from running the test suite, separated by platform. -->
+
+### Android
+| Suite | Total | Passed | Failed | Skipped | Duration |
+|-------|-------|--------|--------|---------|----------|
+| {suite-name} | {count} | {count} | {count} | {count} | {duration} |
+
+### iOS
+| Suite | Total | Passed | Failed | Skipped | Duration |
+|-------|-------|--------|--------|---------|----------|
+| {suite-name or "Deferred"} | {count} | {count} | {count} | {count} | {duration} |
+
+## Platform Coverage -- required
+<!-- Summary of platform test coverage for this story. -->
+
+| Platform | Tests Run | Tests Passed | Deferred | Reason |
+|----------|-----------|--------------|----------|--------|
+| Android | {count} | {count} | 0 | N/A |
+| iOS | {count} | {count} | {count} | {reason or "N/A"} |
+
+## Deferred iOS Tests -- conditional
+<!-- Only include when ios_deferred is true. Lists Maestro flows that could not run on this host. -->
+
+| Flow File | AC Coverage | Reason |
+|-----------|-------------|--------|
+| {file-path} | {AC-ids} | {Host OS lacks iOS simulator} |
+
+## Implementation Decisions -- required
+<!-- Key decisions made during implementation. Each entry: decision, rationale, alternatives rejected. -->
+
+- **Decision:** {what was decided}
+  - **Rationale:** {why}
+  - **Rejected:** {alternatives considered and why they lost}
+
+## Cross-References -- required
+<!-- Explicit pointers to upstream artifacts consumed. -->
+- Reqs brief: `reqs-brief.md#{section}`
+- UXA spec: `uxa-spec.md#{section}`
+- Test spec: `qa-test-spec.md#{section}`
+- BEND handoff: `bend-handoff.md#{section}`

package/pipeline/templates/reqs-brief.template.md

@@ -1,5 +1,5 @@
 # reqs-brief
-<!-- Template version: 1.0 | Used by: REQS | Read by: UXA, QA-A, BEND, FEND, CRITIC, READINESS -->
+<!-- Template version: 1.0 | Used by: REQS | Read by: UXA, QA-A, BEND, FEND, DATA, MCP-DEV, LIBDEV, DOCGEN, IAC, CRITIC, READINESS -->
 
 ---
 agent: reqs
@@ -35,7 +35,8 @@ blockers: []
 |---------|------|-------------|------------|
 | {br-id} | {rule statement} | {ac-id list} | {edge case notes or "none"} |
 
-## Database Changes -- required
+## Database Changes -- conditional
+<!-- Only include if api or data-pipeline in testing_profiles -->
 <!-- Tables to create/modify. Each entry: table name, action (create/alter/drop), columns with types, constraints, indexes, relationships. -->
 
 ```yaml
@@ -48,14 +49,16 @@ database_changes:
     relationships: [{relationship}]
 ```
 
-## API Endpoints -- required
+## API Endpoints -- conditional
+<!-- Only include if api in testing_profiles -->
 <!-- Each endpoint: method, path, request body, response codes, response body shape, auth requirements. -->
 
 | Method | Path | Request Body | Response Codes | Response Shape | Auth |
 |--------|------|-------------|---------------|----------------|------|
 | {method} | {path} | {body or "none"} | {codes} | {shape} | {auth requirement} |
 
-## Authentication and Authorization -- required
+## Authentication and Authorization -- conditional
+<!-- Only include if api or ui in testing_profiles -->
 <!-- Auth flow, token handling, role/permission requirements, tenant isolation rules. -->
 - **Auth mechanism:** {mechanism}
 - **Roles:** {role list with permissions}
@@ -71,6 +74,59 @@ database_changes:
 - **Pagination:** {strategy or "n/a"}
 - **Rate limiting:** {strategy or "n/a"}
 
+## Data Pipeline Specification -- conditional
+<!-- Only include if data-pipeline in testing_profiles -->
+
+| Stage | Input | Transform | Output | Quality Rules | Error Handling |
+|-------|-------|-----------|--------|---------------|----------------|
+| {stage-name} | {input source/format} | {transformation logic} | {output target/format} | {quality checks} | {error strategy} |
+
+- **Idempotency strategy:** {approach — natural keys, upsert, dedup}
+- **Checkpoint/resume:** {what state is persisted, recovery granularity}
+- **Schema evolution:** {strategy — additive-only, migration scripts, etc.}
+
+## Tool Definitions -- conditional
+<!-- Only include if mcp-server in testing_profiles -->
+
+| Tool Name | Description | inputSchema | Output Types | Error Behavior |
+|-----------|-------------|-------------|-------------|----------------|
+| {name} | {purpose} | {JSON Schema summary} | {text/image/audio/resource} | {JSON-RPC error code or isError:true scenario} |
+
+- **Capabilities:** {what server advertises at initialize}
+- **Transport:** {stdio / SSE / streamable HTTP}
+
+## Public API Surface -- conditional
+<!-- Only include if library in testing_profiles -->
+
+| Export | Signature | Return Type | Module Path | Breaking Change? |
+|--------|-----------|-------------|-------------|-----------------|
+| {symbol} | {function signature or type} | {return type} | {path in exports map} | {yes/no — vs prior version} |
+
+- **Semver policy:** {what constitutes major/minor/patch}
+- **Module system:** {CJS / ESM / dual}
+- **Peer dependencies:** {required peer deps and version ranges}
+
+## Template Specifications -- conditional
+<!-- Only include if document-generation in testing_profiles -->
+
+| Template | Variables (required) | Variables (optional) | Output Format | Conditional Sections |
+|----------|---------------------|---------------------|---------------|---------------------|
+| {template-name} | {var: type, var: type} | {var: type (default)} | {PDF/HTML/Markdown} | {section: condition} |
+
+- **Encoding:** {UTF-8, unicode support, RTL}
+- **Asset dependencies:** {fonts, images, logos — embedded or referenced}
+
+## Infrastructure Resources -- conditional
+<!-- Only include if iac in testing_profiles -->
+
+| Resource | Provider | Type | Purpose | Tags | Destroy Protection |
+|----------|----------|------|---------|------|--------------------|
+| {name} | {AWS/Azure/GCP} | {resource type} | {purpose} | {required tags} | {yes/no} |
+
+- **State management:** {remote backend, locking mechanism}
+- **IAM policy:** {roles/permissions created, least-privilege verification}
+- **Environment topology:** {dev/staging/prod differences}
+
 ## NFR Targets -- required
 <!-- Non-functional requirement targets. Each entry: category, metric, target value, measurement method. -->
 

package/skills/valent-run-deferred-tests/SKILL.md

@@ -0,0 +1,109 @@
+---
+name: valent-run-deferred-tests
+description: 'Run deferred iOS tests on a Mac host. Use when the user says "run deferred tests", "run ios tests", or "complete deferred"'
+argument-hint: '<story-id>'
+---
+
+# valent-run-deferred-tests
+
+Run deferred iOS Maestro flows for stories that shipped as SHIP-PARTIAL on a non-Mac host. Requires a Mac with Xcode and iOS Simulator installed.
+
+## Arguments
+
+The user provides a story ID: `STORY-042`
+
+If no argument is provided, scan `{story_output_directory}` for all stories with `ios_deferred: true` in their `mobile-handoff.md` and process them all.
+
+## Execution Steps
+
+### Step 0: Pre-Flight Checks
+
+1. Verify host is Mac: `uname -s` must return `Darwin`
+   - If not Mac: **STOP** with: `This skill requires a Mac with Xcode and iOS Simulator. Current host: {uname output}.`
+2. Verify Xcode tools: `xcrun simctl list devices` must succeed
+   - If fails: **STOP** with: `Xcode command line tools not installed. Run: xcode-select --install`
+3. Verify Maestro installed: `maestro --version` must succeed
+   - If fails: **STOP** with: `Maestro not installed. Install with: curl -Ls "https://get.maestro.mobile.dev" | bash`
+
+### Step 1: Load Pipeline Config
+
+Read `.valent-pipeline/pipeline-config.yaml` for tech stack, project config, and story output directory.
+
+### Step 2: Load Story Artifacts
+
+Read `{story_output_dir}/mobile-handoff.md`. Verify:
+- `ios_deferred: true` in frontmatter
+- `Deferred iOS Tests` section exists with at least one flow listed
+
+If `ios_deferred` is not `true`, **STOP** with: `Story {story_id} does not have deferred iOS tests.`
+
+Read `{story_output_dir}/qa-test-spec.md` for the original test specifications. If ACs have changed since the original run (compare against current repo state), flag the drift and spec against current reality.
+
+### Step 3: Clean Install Dependencies
+
+From repo root:
+- `npm ci` or `yarn install --frozen-lockfile` (Node dependencies)
+- If React Native: `cd ios && pod install && cd ..` (CocoaPods)
+- If Flutter: `flutter pub get`
+
+This is done once, shared across all deferred stories.
+
+### Step 4: Build iOS App
+
+Build the iOS binary from HEAD of `main` (one build, shared across all stories):
+- React Native: `npx react-native build-ios --scheme {scheme} --configuration Debug --simulator`
+- Flutter: `flutter build ios --debug --simulator`
+
+If build fails, **STOP** with build output. This is a blocking issue.
+
+### Step 5: Boot iOS Simulator
+
+Follow emulator lifecycle procedure from `.valent-pipeline/steps/mobile/emulator-lifecycle.md` (iOS section):
+1. `xcrun simctl list devices available` — pick appropriate device
+2. `xcrun simctl boot {device_udid}`
+3. Wait for boot confirmation
+
+Install the built app on the simulator.
+
+### Step 6: Execute Deferred Maestro Flows (Per Story)
+
+For each deferred flow from `mobile-handoff.md#deferred-ios-tests`:
+1. **State isolation:** `xcrun simctl terminate {device_udid} {bundle_id}` + `xcrun simctl privacy {device_udid} reset all {bundle_id}`
+2. **Pre-grant permissions:** per flow requirements
+3. **Execute:** `maestro test {flow_file} --device {ios_simulator_name}`
+4. **Record result:** pass/fail with output
+
+### Step 7: Update Story Artifacts
+
+- Update `mobile-handoff.md`:
+  - Move flows from `Deferred iOS Tests` to `Maestro Flow Files` with iOS results
+  - Update `Platform Coverage` table with iOS results
+  - Update `Test Results Summary` iOS section
+  - Set `ios_deferred: false` in frontmatter if all flows passed
+
+- Update `execution-report.md`:
+  - Add iOS results to the `Mobile Platform Coverage Audit` section
+  - Update platform coverage from `PARTIAL-IOS-DEFERRED` to `FULL` (if all pass)
+
+- If **all deferred flows pass:**
+  - Update `judge-decision.md`: change verdict from `SHIP-PARTIAL` to `SHIP`, update Platform Coverage evidence item to `PASS`, add `SHIP-PARTIAL Detail` noting "Resolved via /run-deferred-tests"
+  - Story transitions from `android-only-verified` → `verified`
+
+- If **any deferred flows fail:**
+  - File bugs in `bugs.md` against current code (routed to next sprint)
+  - Story remains `SHIP-PARTIAL` until bugs are fixed and deferred tests re-run
+  - Update `judge-decision.md` with failure details but do NOT change verdict
+
+### Step 8: Report
+
+Output summary:
+- Stories processed: {count}
+- Per story: flows executed, pass/fail counts, updated verdict
+- If failures: bug IDs filed, recommended next steps
+
+## Notes
+
+- This skill is idempotent — re-running picks up any remaining deferred flows.
+- Deferred flows are the same YAML files MOBILE wrote; they just weren't executed on the original host.
+- The iOS binary is built once from HEAD and shared across all stories. If code has changed significantly since the original run, test failures may reflect current code state, not the original implementation.
+- If processing multiple stories, the simulator stays booted between stories — only app state is cleared.

package/skills/valent-run-epic/SKILL.md

@@ -97,7 +97,7 @@ Phase 1 agents (REQS, UXA, QA-A, READINESS) stay alive across the grooming batch
 
 Read and follow `.valent-pipeline/steps/orchestration/sprint-size.md`.
 
-BEND/FEND estimate Fibonacci points per story. Kill estimation agents after sizing.
+BEND/FEND estimate Fibonacci points per story (filtered by `testing_profiles`). Agents persist into story 1 execution.
 
 #### 4f. Sprint Planning
 

package/skills/valent-run-project/SKILL.md

@@ -111,7 +111,7 @@ Phase 1 agents (REQS, UXA, QA-A, READINESS) stay alive across the grooming batch
 
 Read and follow `.valent-pipeline/steps/orchestration/sprint-size.md`.
 
-BEND/FEND estimate Fibonacci points per story. Kill estimation agents after sizing.
+BEND/FEND estimate Fibonacci points per story (filtered by `testing_profiles`). Agents persist into story 1 execution.
 
 #### 4f. Sprint Planning
 

package/src/commands/db-rebuild.js

@@ -9,6 +9,11 @@ const ARTIFACT_MAP = {
   'qa-test-spec.md': { type: 'qa-test-spec', agent: 'QA-A' },
   'bend-handoff.md': { type: 'bend-handoff', agent: 'BEND' },
   'fend-handoff.md': { type: 'fend-handoff', agent: 'FEND' },
+  'data-handoff.md': { type: 'data-handoff', agent: 'DATA' },
+  'mcp-dev-handoff.md': { type: 'mcp-dev-handoff', agent: 'MCP-DEV' },
+  'libdev-handoff.md': { type: 'libdev-handoff', agent: 'LIBDEV' },
+  'docgen-handoff.md': { type: 'docgen-handoff', agent: 'DOCGEN' },
+  'iac-handoff.md': { type: 'iac-handoff', agent: 'IAC' },
   'critic-review.md': { type: 'critic-review', agent: 'CRITIC' },
   'execution-report.md': { type: 'execution-report', agent: 'QA-B' },
   'bugs.md': { type: 'bugs', agent: 'QA-B' },

package/src/lib/config-schema.js

@@ -116,7 +116,7 @@ export const defaults = {
   },
   models: {
     opus: ['BEND', 'FEND', 'CRITIC'],
-    sonnet: ['REQS', 'UXA', 'QA-A', 'QA-B', 'READINESS', 'JUDGE', 'PMCP', 'Retrospective'],
+    sonnet: ['REQS', 'UXA', 'QA-A', 'QA-B', 'READINESS', 'JUDGE', 'PMCP', 'Retrospective', 'DATA', 'MCP-DEV', 'LIBDEV', 'DOCGEN', 'IAC'],
     haiku: ['Knowledge', 'Embed', 'Help'],
   },
   quality: {

package/src/lib/db.js

@@ -12,7 +12,7 @@ import { existsSync, readFileSync } from 'fs';
 
 // ---------------------------------------------------------------------------
 // Schema DDL — authoritative source. Keep pipeline/scripts/db-bootstrap.ts
-// in sync when modifying (see docs/design/refactor-checklist.md).
+// in sync when modifying (see pipeline/docs/design/refactor-checklist.md).
 // ---------------------------------------------------------------------------
 
 export const SCHEMA_DDL = `