54 lines
2.2 KiB
Markdown
54 lines
2.2 KiB
Markdown
---
|
|
id: review/drift-sync
|
|
name: Review drift sync — verify all specs align with source after implementation
|
|
status: pending
|
|
depends_on:
|
|
- drift/any-to-unknown
|
|
- drift/remove-schema-builder
|
|
- drift/metagraph-module
|
|
- drift/replace-types-exports
|
|
- drift/update-sqlite-imports
|
|
- drift/bridge-functions
|
|
- drift/reference-modules
|
|
- drift/crypto-module
|
|
scope: broad
|
|
risk: low
|
|
impact: phase
|
|
level: review
|
|
---
|
|
|
|
## Description
|
|
|
|
Review the entire codebase after all drift tasks are complete. Verify that the source code now matches the architecture specs in all the areas that were previously drifted. Confirm no regressions, no orphaned exports, and no remaining inconsistencies.
|
|
|
|
## Acceptance Criteria
|
|
|
|
- [ ] No `Type.Any()` calls remain in `src/` (should all be `Type.Unknown()`)
|
|
- [ ] `schemaBuilder.ts` is gone, not re-exported anywhere
|
|
- [ ] `types.ts` is either deleted or reduced to re-exports from `modules/metagraph.ts`
|
|
- [ ] No standalone `BaseNodeAttributes`, `BaseEdgeAttributes`, `GraphConfig`, `NodeType`, `EdgeType`, `GraphSchema`, `GRAPH_BASE_TYPE`, `GraphBaseType` exports remain (replaced by Module)
|
|
- [ ] `Metagraph` Module is exported from the main `@alkdev/storage` package
|
|
- [ ] `bridge.ts` exists with `moduleToDbSchema`, `validateNode`, `validateEdge`
|
|
- [ ] `crypto.ts` exists with `encrypt`, `decrypt`, `generateEncryptionKey`, `EncryptedDataSchema`
|
|
- [ ] Reference Modules (`CallGraph`, `SecretGraph`) compose from `Metagraph` via `Import()` and `Composite()`
|
|
- [ ] `deno check mod.ts` passes — no type errors
|
|
- [ ] `deno check src/sqlite/mod.ts` passes — sqlite host still compiles
|
|
- [ ] `deno lint` passes (excluding known slow-types exclusions)
|
|
- [ ] `deno test --allow-all test/` passes (even if tests are empty — no breakage)
|
|
- [ ] Import graph is clean: no circular imports, sqlite host doesn't reach into `graphs/types.ts` anymore
|
|
- [ ] Architecture docs are still accurate — no new drifts introduced by changes
|
|
|
|
## References
|
|
|
|
- docs/architecture/metagraph-module.md
|
|
- docs/architecture/sqlite-host.md
|
|
- docs/architecture/encrypted-data.md
|
|
- docs/architecture/overview.md
|
|
|
|
## Notes
|
|
|
|
> To be filled by implementation agent
|
|
|
|
## Summary
|
|
|
|
> To be filled on completion |