---
id: review-sync-completeness
name: Review Sync Completeness and Cross-Reference Consistency
status: completed
depends_on: [delete-replaced-core-code, relocate-core-remaining-modules, update-agents-and-overview, update-packages-and-dependencies, update-call-graph-and-operations-docs, update-pubsub-doc, update-secondary-docs, update-tasks-doc-for-taskgraph, archive-migration-research]
scope: broad
risk: high
impact: phase
level: review
---

## Description

Final review gate after all sync tasks are completed. Verify that the codebase and documentation are consistent with the core library extractions. This is the stabilization gate before resuming feature development.

### Checks

1. **No stale references**: Search entire `docs/` directory for:
   - `packages/core/operations/` → should be `@alkdev/operations`
   - `packages/core/pubsub/` → should be `@alkdev/pubsub`
   - `packages/core/mcp/` → should be `@alkdev/operations/from-mcp`
   - `core/operations/` → should be `@alkdev/operations`
   - `core/pubsub/` → should be `@alkdev/pubsub`
   - `PubSubPublishArgsByKey` → should be `PubSubEventMap`
   - `@repeaterjs/repeater` as a direct dependency → should note it's inlined in `@alkdev/pubsub`
   - `"Not started"` for WebSocket EventTarget → should note it's implemented
   - `"Not started"` for Call Protocol → should note it's implemented

2. **Package structure**: Verify `packages/core/` is deleted, workspace config is correct, hub package has proper deno.json

3. **Type check**: `deno check` passes across the workspace

4. **AGENTS.md accuracy**: Read through AGENTS.md and verify every claim is current

5. **overview.md accuracy**: Read through overview.md and verify every status is current

6. **Cross-reference integrity**: Every doc that mentions a module has the correct package name

7. **Breaking changes documented**: Verify that all breaking API changes from the review doc (Section 1.5) are reflected in relevant architecture docs

## Acceptance Criteria

- [ ] No stale `packages/core/operations/` or `packages/core/pubsub/` references in docs
- [ ] No stale `Not started` status for WebSocket EventTarget or Call Protocol
- [ ] `packages/core/` is deleted from workspace
- [ ] `deno check` passes on workspace
- [ ] AGENTS.md is accurate and current
- [ ] overview.md status table is accurate
- [ ] All breaking API changes (ResponseEnvelope, buildEnv, ScannerFS, etc.) are documented in relevant specs

## References

- docs/reviews/core-library-extraction-sync-2026-05-18.md (full document)