groundswell 0.0.1 → 0.0.2

package/plan/001_d3bb02af4886/bugfix/001_e8e04329daf3/docs/P1M4T2S1/PRP.md

@@ -0,0 +1,717 @@
+# Product Requirement Prompt (PRP): Bug Fix Summary Documentation
+
+**PRP ID**: P1.M4.T2.S1
+**Work Item**: Document critical and major bug fixes
+**Created**: 2026-01-12
+**Confidence Score**: 10/10
+
+---
+
+## Goal
+
+**Feature Goal**: Create a comprehensive bug fix summary document that enables users and developers to understand all bug fixes from P1.M1 (Critical), P1.M2 (Major), and P1.M3 (Minor) with clear descriptions, code examples, and migration guidance.
+
+**Deliverable**: Bug fix summary document at `plan/bugfix/BUG_FIX_SUMMARY.md` with:
+- All critical and major bug fixes fully documented
+- Code examples showing before/after for each fix
+- Migration steps for any breaking changes
+- Severity classification and impact analysis
+
+**Success Definition**:
+- Document follows the project's existing CHANGELOG.md patterns
+- All fixes have clear before/after code examples
+- Severity is properly justified for each fix
+- Migration steps are actionable where needed
+- Document passes markdown linting and all links work
+
+---
+
+## User Persona
+
+**Target User**: Developers and technical users who need to:
+- Understand what bugs were fixed in the current release
+- Migrate their code if breaking changes exist
+- Assess whether to upgrade based on bug fixes
+- Debug issues that may be related to fixed bugs
+
+**Use Case**: Primary scenario when reviewing release notes before upgrading the Groundswell workflow engine.
+
+**User Journey**:
+1. User opens BUG_FIX_SUMMARY.md to review changes
+2. User scans Critical and Major sections for relevant fixes
+3. User reads before/after code examples for fixes affecting their code
+4. User follows migration steps if breaking changes exist
+5. User verifies upgrade path by running provided validation commands
+
+**Pain Points Addressed**:
+- **Unclear impact**: Users don't know what was fixed and how it affects them
+- **Missing context**: Traditional changelogs lack code examples
+- **Upgrade fear**: Users hesitate to upgrade without understanding breaking changes
+- **Debugging difficulty**: Users can't tell if a bug they're experiencing was already fixed
+
+---
+
+## Why
+
+**Business value and user impact**:
+- Comprehensive bug fix documentation builds user trust in the project
+- Clear migration guidance reduces support burden and upgrade friction
+- Code examples enable users to quickly assess relevance to their codebase
+- Proper severity classification helps users prioritize upgrades
+
+**Integration with existing features**:
+- Builds upon existing CHANGELOG.md patterns and formatting
+- Complements PRD.md by documenting implemented fixes
+- Supports test suite validation from P1.M4.T1
+
+**Problems this solves and for whom**:
+- **For developers**: Clear understanding of what changed and how to migrate
+- **For maintainers**: Standardized documentation format for all future bug fixes
+- **For users**: Actionable information to make upgrade decisions
+
+---
+
+## What
+
+Create a markdown document at `plan/bugfix/BUG_FIX_SUMMARY.md` with the following structure:
+
+### Document Structure
+
+```markdown
+# Bug Fix Summary
+
+## Executive Summary
+[Brief overview of all fixes, severity distribution, test coverage]
+
+## Critical Fixes (P1.M1)
+### WorkflowLogger.child() Signature Fix
+[Description, severity justification, before/after code, migration steps]
+
+## Major Fixes (P1.M2)
+### Promise.allSettled for Concurrent Tasks
+[Description, severity justification, before/after code, migration steps]
+
+### ErrorMergeStrategy Implementation
+[Description, severity justification, code examples, usage guide]
+
+### trackTiming Default Documentation
+[Description, severity justification]
+
+## Minor Fixes (P1.M3)
+### Console.error to Logger Replacement
+### Tree Debugger Optimization
+### Workflow Name Validation
+### isDescendantOf Public API
+
+## Breaking Changes & Migration
+[Summary of any breaking changes and migration steps]
+
+## Testing & Validation
+[Test coverage summary, validation commands]
+
+## References
+[Links to PRD, implementation files, test files]
+```
+
+### Success Criteria
+
+- [ ] Document created at `plan/bugfix/BUG_FIX_SUMMARY.md`
+- [ ] All P1.M1 (Critical) fixes documented with code examples
+- [ ] All P1.M2 (Major) fixes documented with code examples
+- [ ] All P1.M3 (Minor) fixes summarized
+- [ ] Severity properly justified for each fix
+- [ ] Before/after code examples for behavior-changing fixes
+- [ ] Migration steps provided where applicable
+- [ ] All file references use absolute paths with line numbers
+- [ ] All external URLs include section anchors
+- [ ] Follows CHANGELOG.md formatting conventions
+
+---
+
+## All Needed Context
+
+### Context Completeness Check
+
+**"No Prior Knowledge" test passed**: This PRP provides all necessary context including:
+- Exact file paths and line numbers for every bug fix
+- Code examples extracted from actual source files
+- Severity classification standards with decision trees
+- Documentation patterns from the project's own CHANGELOG.md
+- External best practices with specific URLs
+
+### Documentation & References
+
+```yaml
+# MUST READ - Project's documentation patterns
+- file: /home/dustin/projects/groundswell/CHANGELOG.md
+  why: Primary template for bug fix documentation patterns, migration guide format, and markdown conventions
+  pattern: Bold method names, detailed "Before (Buggy Pattern)" sections, numbered migration steps
+  critical: The project follows this exact pattern - replicate it
+
+- file: /home/dustin/projects/groundswell/PRD.md
+  why: Source of truth for original specifications that bugs violated
+  section: Section 12.1 for WorkflowLogger.child() specification
+  gotcha: PRD specifies `child(meta: Partial<LogEntry>)` not `child(parentLogId: string)`
+
+# CRITICAL - Bug fix implementations to document
+- file: /home/dustin/projects/groundswell/src/core/logger.ts
+  why: WorkflowLogger.child() signature fix implementation
+  lines: 98-111
+  pattern: Function overloads with backward compatibility
+
+- file: /home/dustin/projects/groundswell/src/decorators/task.ts
+  why: Promise.allSettled and error merge strategy implementation
+  lines: 112-142
+  pattern: Error collection and aggregation logic
+
+- file: /home/dustin/projects/groundswell/src/types/decorators.ts
+  why: TaskOptions interface with errorMergeStrategy
+  lines: 25-32
+  pattern: Optional configuration interface pattern
+
+- file: /home/dustin/projects/groundswell/src/utils/workflow-error-utils.ts
+  why: Default error merger implementation
+  lines: 23-56
+  pattern: Error aggregation with WorkflowError construction
+
+- file: /home/dustin/projects/groundswell/src/decorators/step.ts
+  why: trackTiming default implementation
+  lines: 94-101
+  pattern: Conditional check `!== false` for default true behavior
+
+- file: /home/dustin/projects/groundswell/src/core/workflow.ts
+  why: Multiple fixes - observer logging, name validation, isDescendantOf public API
+  lines: 98-107, 201-219, 426, 444
+  pattern: Constructor validation, JSDoc documentation, logger usage
+
+- file: /home/dustin/projects/groundswell/src/debugger/tree-debugger.ts
+  why: Tree debugger incremental update optimization
+  lines: 65-84, 92-117
+  pattern: Event-driven incremental updates with BFS traversal
+
+# TEST FILES - For coverage documentation
+- file: /home/dustin/projects/groundswell/src/__tests__/unit/logger.test.ts
+  why: Comprehensive test coverage for child() signature fix
+  lines: 7-293 (294 lines of tests)
+
+- file: /home/dustin/projects/groundswell/src/__tests__/adversarial/concurrent-task-failures.test.ts
+  why: Tests for Promise.allSettled concurrent failure scenarios
+
+- file: /home/dustin/projects/groundswell/src/__tests__/adversarial/error-merge-strategy.test.ts
+  why: Tests for error merge strategy functionality
+
+- file: /home/dustin/projects/groundswell/src/__tests__/unit/tree-debugger-incremental.test.ts
+  why: Tests for incremental node map updates
+
+- file: /home/dustin/projects/groundswell/src/__tests__/unit/workflow-isDescendantOf.test.ts
+  why: Tests for public isDescendantOf API
+
+# EXTERNAL STANDARDS - Best practices
+- url: https://keepachangelog.com/en/1.1.0/
+  why: Industry standard changelog format (Added, Changed, Fixed categories)
+  critical: Use exact category names: "Fixed", "Added", "Changed"
+
+- url: https://keepachangelog.com/en/1.1.0/#example
+  why: Example showing proper version links, ISO date format, categorization
+
+- url: https://semver.org/spec/v2.0.0.html
+  why: Semantic versioning for determining version bump (PATCH for bug fixes)
+  section: Summary section for MAJOR.MINOR.PATCH definitions
+
+# PROJECT STRUCTURE - For creating directory
+- file: /home/dustin/projects/groundswell/plan/001_d3bb02af4886
+  why: Parent directory for bug fix documentation
+  pattern: Create `plan/bugfix/` subdirectory if it doesn't exist
+```
+
+### Current Codebase tree
+
+```bash
+# Relevant portion of tree (output from tree -L 3)
+plan/
+├── 001_d3bb02af4886/
+│   ├── backlog.json
+│   ├── bugfix/
+│   │   └── 001_e8e04329daf3/
+│   │       ├── P1M4T2S1/           # PRP will be stored here
+│   │       └── docs/
+│   ├── bug_fix_tasks.json
+│   ├── docs/
+│   │   ├── architecture/
+│   │   ├── bugfix/
+│   │   └── research/
+│   ├── prd_snapshot.md
+│   └── TEST_RESULTS.md
+└── bugfix/                         # CREATE THIS DIRECTORY
+    └── BUG_FIX_SUMMARY.md          # OUTPUT FILE
+```
+
+### Desired Codebase tree with files to be added
+
+```bash
+plan/
+└── bugfix/
+    └── BUG_FIX_SUMMARY.md          # Main output - comprehensive bug fix documentation
+
+# NOTE: Also update CHANGELOG.md (existing file) if not already updated
+```
+
+### Known Gotchas of our codebase & Library Quirks
+
+```markdown
+# CRITICAL: Project-specific markdown conventions
+# 1. Use bold for method names: **methodName()**
+# 2. Include file:line references in implementation details
+# 3. Use absolute file paths with #L##-L## anchor syntax for links
+# 4. Date format: YYYY-MM-DD (ISO 8601)
+# 5. Code blocks use language identifier: ```typescript
+# 6. Migration guide pattern: "What Changed" → "Before" → "After" → "Migration Steps"
+
+# CRITICAL: Severity classification
+# Critical = PRD compliance violations, security issues, data loss
+# Major = Core functionality broken, missing features, significant reliability issues
+# Minor = Small issues, easy workarounds, doesn't impact core features
+
+# CRITICAL: Breaking change determination
+# Bug fixes that correct PRD violations are NOT breaking changes
+# Old behavior was "buggy" per specification
+# Only API signature changes are breaking
+
+# GOTCHA: trackTiming default is implicit
+# The default is `true` but determined by `!== false` check, not explicit assignment
+# Document this clearly as it can be confusing
+
+# GOTCHA: Function overloads in TypeScript
+# WorkflowLogger.child() uses overloads for backward compatibility
+# Show ALL overload signatures in documentation, not just implementation
+
+# GOTCHA: Promise.allSettled vs Promise.all
+# Key difference is that allSettled waits for ALL promises to complete
+# This is critical for concurrent task execution reliability
+
+# GOTCHA: Error merge strategy is opt-in
+# Must explicitly enable with `errorMergeStrategy: { enabled: true }`
+# Default behavior (backward compat) throws first error only
+```
+
+---
+
+## Implementation Blueprint
+
+### Data models and structure
+
+No data models needed - this is pure documentation.
+
+The document structure follows a markdown outline:
+
+```markdown
+# Header 1: Document title
+## Header 2: Major sections (Critical Fixes, Major Fixes, etc.)
+### Header 3: Individual bug fixes
+#### Header 4: Subsections (if needed)
+
+Code blocks:
+```typescript
+// For TypeScript code examples
+```
+
+Tables:
+| Severity | Definition | Examples |
+|----------|-----------|----------|
+
+Links:
+[text](url)
+[text](file.ts#L##-L##)
+```
+
+### Implementation Tasks (ordered by dependencies)
+
+```yaml
+Task 1: CREATE plan/bugfix/ directory (if not exists)
+  - EXECUTE: mkdir -p plan/bugfix
+  - PLACEMENT: Under plan/ directory at project root
+  - VERIFY: Directory creation succeeds
+
+Task 2: CREATE plan/bugfix/BUG_FIX_SUMMARY.md
+  - WRITE: Complete bug fix summary document
+  - FOLLOW pattern: /home/dustin/projects/groundswell/CHANGELOG.md
+  - INCLUDE: Executive summary with fix count and severity distribution
+  - INCLUDE: All P1.M1 critical fixes with code examples
+  - INCLUDE: All P1.M2 major fixes with code examples
+  - INCLUDE: All P1.M3 minor fixes summarized
+  - INCLUDE: Breaking changes & migration section
+  - INCLUDE: Testing & validation section
+  - INCLUDE: References with file:line links
+  - PLACEMENT: plan/bugfix/BUG_FIX_SUMMARY.md
+
+Task 3: VERIFY markdown syntax
+  - EXECUTE: markdownlint plan/bugfix/BUG_FIX_SUMMARY.md (if available)
+  - ALTERNATIVE: Manual check of markdown formatting
+  - CHECK: All headers properly formatted
+  - CHECK: All code blocks have language identifiers
+  - CHECK: All links use correct syntax
+
+Task 4: VERIFY all file references
+  - EXECUTE: grep -oE '\[.*\]\([^)]+\.ts#L[0-9]+' plan/bugfix/BUG_FIX_SUMMARY.md
+  - VERIFY: All referenced files exist at specified paths
+  - VERIFY: All line number ranges are accurate
+  - MANUAL: Spot-check critical references
+
+Task 5: VERIFY all external URLs
+  - EXECUTE: Extract URLs and test accessibility (optional)
+  - VERIFY: All URLs include section anchors where applicable
+  - CHECK: keepachangelog.com, semver.org links work
+```
+
+### Implementation Patterns & Key Details
+
+```markdown
+# DOCUMENT STRUCTURE PATTERN (from CHANGELOG.md)
+
+## [VERSION] - YYYY-MM-DD
+
+### Fixed
+
+- **boldMethodName() description**: `methodName()` now [correct behavior]. Previously, [incorrect behavior].
+  - Additional context if needed
+  - Implementation: [path/to/file.ts:##-##](path/to/file.ts#L##-L##)
+
+# MIGRATION GUIDE PATTERN
+
+### Migration Guide for FeatureName Change
+
+**What Changed**:
+[One paragraph explanation]
+
+**Before (Buggy Pattern)**:
+```typescript
+// Show old problematic code
+```
+
+**After (Correct Pattern)**:
+```typescript
+// Show new correct code
+```
+
+**Migration Steps**:
+1. [Actionable step]
+2. [Actionable step]
+3. [Verification step]
+
+# CODE EXAMPLE PATTERN
+
+# Always include explanatory comments in code
+const childLogger = this.logger.child({ parentLogId: 'parent-123' });
+// ^ New PRD-compliant signature
+
+childLogger.info('Child message');
+// Result: workflow.node.logs[0].parentLogId === 'parent-123'
+
+# FILE REFERENCE PATTERN
+
+Implementation: [src/core/logger.ts:98-111](src/core/logger.ts#L98-L111)
+Tests: [src/__tests__/unit/logger.test.ts](src/__tests__/unit/logger.test.ts)
+
+# SEVERITY CLASSIFICATION PATTERN
+
+| Severity | Definition | This Fix |
+|----------|-----------|----------|
+| Critical | [Definition] | ✅ Fits because [reason] |
+
+# GOTCHA: Always justify severity
+# Don't just state severity - explain WHY it deserves that level
+# Reference PRD violations, reliability impact, user impact
+```
+
+### Integration Points
+
+```yaml
+FILES_TO_CREATE:
+  - path: plan/bugfix/BUG_FIX_SUMMARY.md
+    template: "Follow CHANGELOG.md pattern with sections for Critical, Major, Minor"
+
+FILES_TO_READ:
+  - src/core/logger.ts (lines 98-111) - child() implementation
+  - src/decorators/task.ts (lines 112-142) - Promise.allSettled
+  - src/types/decorators.ts (lines 25-32) - TaskOptions
+  - src/utils/workflow-error-utils.ts (lines 23-56) - mergeWorkflowErrors
+  - src/decorators/step.ts (lines 94-101) - trackTiming
+  - src/core/workflow.ts - Multiple fixes (lines vary)
+  - src/debugger/tree-debugger.ts (lines 65-84, 92-117) - Tree debugger
+
+DOCUMENTATION_TO_FOLLOW:
+  - CHANGELOG.md - Format, migration guide pattern, code example style
+  - PRD.md - Source specifications that bugs violated
+```
+
+---
+
+## Validation Loop
+
+### Level 1: Syntax & Style (Immediate Feedback)
+
+```bash
+# Verify markdown syntax
+npx markdownlint plan/bugfix/BUG_FIX_SUMMARY.md 2>&1 || echo "markdownlint not available, skip"
+
+# Manual checks if markdownlint unavailable
+grep -E '^#+ ' plan/bugfix/BUG_FIX_SUMMARY.md | head -20  # Check header hierarchy
+grep -E '```typescript' plan/bugfix/BUG_FIX_SUMMARY.md     # Check code blocks
+grep -E '\[.*\]\(.*#' plan/bugfix/BUG_FIX_SUMMARY.md       # Check anchored links
+
+# Expected: Zero markdown syntax errors, proper formatting
+```
+
+### Level 2: Content Validation (Completeness)
+
+```bash
+# Verify all critical fixes are documented
+grep -q "WorkflowLogger.child()" plan/bugfix/BUG_FIX_SUMMARY.md && echo "✓ P1.M1 documented"
+grep -q "Promise.allSettled" plan/bugfix/BUG_FIX_SUMMARY.md && echo "✓ P1.M2.T1 documented"
+grep -q "ErrorMergeStrategy" plan/bugfix/BUG_FIX_SUMMARY.md && echo "✓ P1.M2.T2 documented"
+grep -q "trackTiming" plan/bugfix/BUG_FIX_SUMMARY.md && echo "✓ P1.M2.T3 documented"
+
+# Verify all minor fixes are mentioned
+grep -q "console.error" plan/bugfix/BUG_FIX_SUMMARY.md && echo "✓ P1.M3.T1 documented"
+grep -q "Tree Debugger" plan/bugfix/BUG_FIX_SUMMARY.md && echo "✓ P1.M3.T2 documented"
+grep -q "name validation" plan/bugfix/BUG_FIX_SUMMARY.md && echo "✓ P1.M3.T3 documented"
+grep -q "isDescendantOf" plan/bugfix/BUG_FIX_SUMMARY.md && echo "✓ P1.M3.T4 documented"
+
+# Verify code examples exist
+grep -c '```typescript' plan/bugfix/BUG_FIX_SUMMARY.md | grep -qE '[0-9]+' && echo "✓ Code examples present"
+
+# Expected: All fixes present, multiple code examples
+```
+
+### Level 3: Link Validation (References)
+
+```bash
+# Extract and verify all file:line references
+grep -oE 'src/[^)]+\.ts#L[0-9-]+?' plan/bugfix/BUG_FIX_SUMMARY.md | while read link; do
+  file=$(echo "$link" | cut -d'#' -f1)
+  if [ -f "$file" ]; then
+    echo "✓ $file exists"
+  else
+    echo "✗ $file NOT FOUND"
+  fi
+done
+
+# Expected: All referenced files exist
+```
+
+### Level 4: Documentation Quality Review
+
+```bash
+# Manual quality checklist
+cat << 'EOF'
+# Documentation Quality Checklist
+
+## Content Quality
+- [ ] Executive summary provides clear overview
+- [ ] All fixes have severity justification
+- [ ] Code examples are copy-pasteable and correct
+- [ ] Before/after examples clearly show the difference
+- [ ] Migration steps are actionable (start with verbs)
+
+## Formatting
+- [ ] Follows CHANGELOG.md patterns (bold methods, detailed descriptions)
+- [ ] All headers use proper hierarchy (#, ##, ###)
+- [ ] Code blocks use language identifiers
+- [ ] File references include line number ranges
+- [ ] External URLs include section anchors
+
+## Completeness
+- [ ] All P1.M1 critical fixes documented
+- [ ] All P1.M2 major fixes documented
+- [ ] All P1.M3 minor fixes summarized
+- [ ] Breaking changes section exists (even if "None")
+- [ ] Testing/validation section included
+- [ ] References section with all links
+
+## Clarity
+- [ ] Technical terms explained or linked
+- [ ] Code examples have explanatory comments
+- [ ] Severity classification table included
+- [ ] Migration steps are numbered and sequential
+EOF
+
+# Expected: All checkboxes pass
+```
+
+---
+
+## Final Validation Checklist
+
+### Technical Validation
+
+- [ ] Document created at `plan/bugfix/BUG_FIX_SUMMARY.md`
+- [ ] Markdown syntax is valid (all headers, code blocks, links formatted correctly)
+- [ ] All file references point to existing files
+- [ ] All line number ranges are accurate
+- [ ] Code examples are syntactically correct TypeScript
+
+### Content Validation
+
+- [ ] All P1.M1 (Critical) fixes fully documented
+- [ ] All P1.M2 (Major) fixes fully documented
+- [ ] All P1.M3 (Minor) fixes documented
+- [ ] Severity properly justified for each fix
+- [ ] Before/after code examples for behavior changes
+- [ ] Migration steps provided where applicable
+
+### Documentation Quality
+
+- [ ] Follows CHANGELOG.md formatting patterns
+- [ ] Bold method names for quick scanning
+- [ ] File:line references for all implementation details
+- [ ] External URLs include section anchors
+- [ ] Executive summary with fix count and distribution
+- [ ] References section with all links
+
+### Feature Validation
+
+- [ ] Target user (developer) can understand what was fixed
+- [ ] User can assess relevance to their codebase
+- [ ] User has actionable migration steps if needed
+- [ ] User has validation commands to verify upgrade
+- [ ] Document passes "No Prior Knowledge" test
+
+---
+
+## Anti-Patterns to Avoid
+
+- **Don't** omit severity justification - always explain WHY a fix is critical/major/minor
+- **Don't** use vague descriptions like "bug fix" - be specific about what changed
+- **Don't** skip code examples for behavior changes - show before/after
+- **Don't** use relative file paths - always use absolute paths from project root
+- **Don't** forget line numbers in file references - users need exact locations
+- **Don't** mix up severity levels - use the decision tree from research
+- **Don't** ignore backward compatibility - explicitly state if breaking changes exist
+- **Don't** create new formatting patterns - follow existing CHANGELOG.md conventions
+- **Don't** leave out migration steps - even if "none needed", state it explicitly
+- **Don't** forget to explain the "why" - users need context to understand the fix
+
+---
+
+## Appendix: Complete Bug Fix Reference
+
+This section provides the complete reference information for all bug fixes to be documented.
+
+### P1.M1 - Critical Fixes
+
+#### 1. WorkflowLogger.child() Signature Fix
+
+**File**: `src/core/logger.ts:98-111`
+
+**Before**:
+```typescript
+child(parentLogId: string): WorkflowLogger {
+  return new WorkflowLogger(this.node, this.observers, parentLogId);
+}
+```
+
+**After**:
+```typescript
+child(parentLogId: string): WorkflowLogger;
+child(meta: Partial<LogEntry>): WorkflowLogger;
+child(input: string | Partial<LogEntry>): WorkflowLogger {
+  const parentLogId = typeof input === 'string' ? input : input.parentLogId;
+  return new WorkflowLogger(this.node, this.observers, parentLogId);
+}
+```
+
+**Severity**: Critical - PRD specification violation
+**Breaking**: No - backward compatible
+**Tests**: `src/__tests__/unit/logger.test.ts`
+
+### P1.M2 - Major Fixes
+
+#### 1. Promise.allSettled for Concurrent Tasks
+
+**File**: `src/decorators/task.ts:112-142`
+
+**Before**:
+```typescript
+const results = await Promise.all(runnable.map((w) => w.run()));
+```
+
+**After**:
+```typescript
+const results = await Promise.allSettled(runnable.map((w) => w.run()));
+const rejected = results.filter((r): r is PromiseRejectedResult => r.status === 'rejected');
+// Error collection and handling...
+```
+
+**Severity**: Major - Concurrent execution reliability
+**Breaking**: No - maintains existing behavior with improved reliability
+**Tests**: `src/__tests__/adversarial/concurrent-task-failures.test.ts`
+
+#### 2. ErrorMergeStrategy Implementation
+
+**Files**:
+- `src/types/decorators.ts:25-32` (TaskOptions interface)
+- `src/utils/workflow-error-utils.ts:23-56` (mergeWorkflowErrors)
+
+**Severity**: Major - Missing core functionality from PRD
+**Breaking**: No - additive feature, opt-in
+**Tests**: `src/__tests__/adversarial/error-merge-strategy.test.ts`
+
+#### 3. trackTiming Default Documentation
+
+**Files**:
+- `PRD.md:183` (documentation update)
+- `src/decorators/step.ts:94-101` (implementation)
+
+**Severity**: Minor - Documentation inconsistency
+**Breaking**: No
+**Tests**: Existing tests verify behavior
+
+### P1.M3 - Minor Fixes
+
+#### 1. Console.error to Logger Replacement
+
+**File**: `src/core/workflow.ts:426, 444`
+
+**Before**: `console.error(errorMessage)`
+**After**: `this.logger.error('Observer onEvent error', { error: err, eventType: event.type })`
+
+**Severity**: Minor - Logging consistency
+**Breaking**: No - transparent to users
+**Tests**: `src/__tests__/integration/observer-logging.test.ts`
+
+#### 2. Tree Debugger Optimization
+
+**File**: `src/debugger/tree-debugger.ts:65-84, 92-117`
+
+**Severity**: Minor - Performance optimization
+**Breaking**: No - transparent improvement
+**Tests**: `src/__tests__/unit/tree-debugger-incremental.test.ts`
+**Benchmarks**: `src/__tests__/adversarial/node-map-update-benchmarks.test.ts`
+
+#### 3. Workflow Name Validation
+
+**File**: `src/core/workflow.ts:98-107`
+
+**Validation Rules**:
+- Rejects empty string names
+- Rejects whitespace-only names
+- Enforces 100-character maximum length
+
+**Severity**: Minor - Input validation
+**Breaking**: Potentially - existing code with invalid names will throw
+**Tests**: `src/__tests__/unit/workflow.test.ts`
+
+#### 4. isDescendantOf Public API
+
+**File**: `src/core/workflow.ts:201-219`
+
+**Change**: Method visibility changed from private to public
+**Added**: Comprehensive JSDoc documentation
+
+**Severity**: Minor - API enhancement
+**Breaking**: No - additive change
+**Tests**: `src/__tests__/unit/workflow-isDescendantOf.test.ts`
+
+---
+
+**End of PRP**