docs(plans): address plan-reviewer recommendations

- Pin @antfly/sdk to exact version (0.0.14, not ^0.0.14)
- Add RepositoryIndexer.initialize() to Part 1.3 surface area
- Add getAll() pagination and embedding completion as spike questions
- Add model mismatch detection test to Part 1.2
- Add concurrent auto-start safety to Part 1.5
- All reviewer findings addressed

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: prosdev

.claude/da-plans/core/phase-1-antfly-migration/1.1-spike-validate-api.md

@@ -145,6 +145,8 @@ If there's a delay, we need to handle this in the index command (wait for embedd
 | 6 | Can we full-scan without a query vector? | |
 | 7 | Does the SDK handle connection errors gracefully? | |
 | 8 | What happens when antfly server is not running? | |
+| 9 | Does `getAll()` paginate beyond 10000 docs? How? | |
+| 10 | Does `dev index` need to wait for embedding completion before returning? | |
 
 ## Exit criteria
 

.claude/da-plans/core/phase-1-antfly-migration/1.2-antfly-vector-store.md

@@ -187,6 +187,7 @@ Use a dedicated test table (`test-antfly-{random}`), clean up after each test.
 | handles empty table search | search on empty table → returns [] |
 | handles missing server gracefully | Connection refused → meaningful error |
 | search(embedding) throws | Direct call to search() with vector → throws with guidance |
+| detects model mismatch | Table has model A, config says model B → warns user to re-index |
 
 ## Exit criteria
 

.claude/da-plans/core/phase-1-antfly-migration/1.3-update-facade.md

@@ -10,6 +10,7 @@ so consumers don't change.
 
 - `packages/core/src/vector/index.ts` — VectorStorage class
 - `packages/core/src/vector/types.ts` — VectorStorageConfig, VectorStats
+- `packages/core/src/indexer/index.ts` — RepositoryIndexer.initialize() passes skipEmbedder
 
 ## Current flow (VectorStorage.add)
 

.claude/da-plans/core/phase-1-antfly-migration/1.4-swap-deps-tests.md

@@ -18,7 +18,7 @@ Remove:
 
 Add:
 ```json
-"@antfly/sdk": "^0.0.14"
+"@antfly/sdk": "0.0.14"
 ```
 
 ### packages/dev-agent/package.json
@@ -31,7 +31,7 @@ Remove:
 
 Add:
 ```json
-"@antfly/sdk": "^0.0.14"
+"@antfly/sdk": "0.0.14"
 ```
 
 ### packages/dev-agent/tsup.config.ts

.claude/da-plans/core/phase-1-antfly-migration/1.5-dev-setup-command.md

@@ -244,6 +244,7 @@ If antfly is installed but not running, it auto-starts silently.
 | ensureAntfly — server not running, binary exists | Mock fetch fail → spawn called → poll succeeds |
 | ensureAntfly — binary missing | Mock execSync throw → error with install instructions |
 | ensureAntfly — server fails to start in time | Mock poll timeout → error message |
+| ensureAntfly — concurrent calls | Two simultaneous calls → only one spawns, second detects running server |
 | dev setup — model missing | Mock termite list → triggers pull |
 | dev setup — everything ready | All checks pass → "Nothing to do" |
 
@@ -255,3 +256,4 @@ If antfly is installed but not running, it auto-starts silently.
 - Antfly binary missing → clear error pointing to install instructions
 - No user-facing command requires typing `antfly` directly
 - `ANTFLY_URL` env var works for custom setups
+- Concurrent auto-start is safe (check port before spawning, brief retry delay)