🟡 P1: devplan_parse_brief misses Success Criteria and Performance Requirements

[mmorris35]

Problem

devplan_parse_brief returns empty arrays/objects for sections that exist in the brief:

• successCriteria: [] when brief has explicit success criteria
• performanceRequirements: {} when brief has performance specs

Reproduction

Brief contains:

## Success Criteria

1. All AMP protocol operations pass compliance tests
2. Semantic search finds conceptually similar content
3. Runs on Pi 4 (2GB) with <1.5GB RAM
4. Search returns results in <100ms
5. Can store and search 10,000 lessons
6. Embedding generation <50ms per record
7. Clean shutdown preserves all data

And:

## Performance Requirements

- Binary size: <20MB (excluding model)
- Model size: ~80MB
- RAM usage: <1.5GB working set with 10K records
- Startup time: <5 seconds (model loading)
- Embedding latency: <50ms per text on Pi 4

Parsed result:

{
  "successCriteria": [],
  "performanceRequirements": {}
}

Expected Behavior

Parser should extract:

{
  "successCriteria": [
    "All AMP protocol operations pass compliance tests",
    "Semantic search finds conceptually similar content",
    ...
  ],
  "performanceRequirements": {
    "binarySize": "<20MB",
    "ramUsage": "<1.5GB",
    ...
  }
}

Impact

Moderate - Success criteria and perf requirements are key inputs for generating testable subtasks. Without them, generated plans lack specific acceptance criteria.