glm-5.1
466b121f77
Decompose the reviewed architecture specs into taskgraph-managed tasks: - 2 setup tasks (project init, test infrastructure) - 4 schema tasks (enums, node attrs, edge attrs, graph schemas) - 1 error hierarchy task - 6 graph tasks (FlowGraph class, 3 construction paths, queries, validation) - 5 analysis tasks (type-compat, build-type-edges, ordering, template-validation, defaults) - 5 component tasks (Operation, Sequential, Parallel, Conditional, Map) - 2 host config tasks (GraphologyHostConfig, ReactiveHostConfig) - 4 reactive tasks (WorkflowRoot, node-status, max-concurrency, retry-semantics) - 3 review tasks (foundation, reactive-and-hosts, complete-library) - 5 meta cluster tasks (schema, graph, component, reactive, analysis layers) - 1 API exports task Validated with taskgraph: zero cycles, 38 tasks, 12 parallel generations. Critical path: 12 tasks through reactive execution layer.
51 lines
1.8 KiB
Markdown
51 lines
1.8 KiB
Markdown
---
|
|
id: review/foundation
|
|
name: Review foundation layer — schemas, errors, FlowGraph class, construction
|
|
status: pending
|
|
depends_on:
|
|
- schema/enums
|
|
- schema/node-attrs
|
|
- schema/edge-attrs
|
|
- schema/graph-schemas
|
|
- error/hierarchy
|
|
- graph/flowgraph-class
|
|
- graph/construction-operation
|
|
- graph/construction-call
|
|
- graph/construction-json
|
|
- graph/queries
|
|
- graph/validation
|
|
scope: broad
|
|
risk: low
|
|
impact: phase
|
|
level: review
|
|
---
|
|
|
|
## Description
|
|
|
|
Review the foundation layer before proceeding to analysis components, host configs, and reactive execution. Verify schemas match architecture docs, errors are correctly structured, and FlowGraph construction preserves DAG invariants.
|
|
|
|
## Acceptance Criteria
|
|
|
|
- [ ] All TypeBox schemas match [schema.md](../docs/architecture/schema.md) — no missing fields, no extra fields
|
|
- [ ] Error hierarchy matches [error-handling.md](../docs/architecture/error-handling.md) — all classes have correct properties
|
|
- [ ] FlowGraph class wraps graphology correctly per [flowgraph-api.md](../docs/architecture/flowgraph-api.md)
|
|
- [ ] Operation graph construction: fromSpecs builds correct nodes and edges
|
|
- [ ] Call graph construction: fromCallEvents and updateFromEvent process all event types correctly
|
|
- [ ] JSON round-trip is lossless for both operation and call graphs
|
|
- [ ] All query methods delegate correctly to graphology-dag
|
|
- [ ] Validation functions return structured errors, never throw
|
|
- [ ] All tests pass: `npm test`
|
|
- [ ] TypeScript strict mode compilation succeeds: `npx tsc --noEmit`
|
|
|
|
## References
|
|
|
|
- docs/architecture/schema.md, docs/architecture/error-handling.md, docs/architecture/flowgraph-api.md
|
|
- docs/architecture/operation-graph.md, docs/architecture/call-graph.md
|
|
|
|
## Notes
|
|
|
|
> To be filled by implementation agent
|
|
|
|
## Summary
|
|
|
|
> To be filled on completion |