Three fixes required to run the integration suite after Phase 4:
1. v3_vault_store_test.go makeVaultDoc: AgentID is *string (nullable),
not string — was a pre-existing type error from v3 commit 8f56ddaa
that blocked the test binary build. Pass &agentID.
2. vault_classify_test.go imports: remove unused `store/pg` import —
pre-existing from v3 commit 4b7a2e6e.
3. TestVaultClassify_DeleteDocLinksByTypes_MultipleTypes: fix off-by-one
in target-doc loop. Pre-existing runtime panic from v3 commit — loop
created 7 targets but linkTypes slice has 8 entries, accessing docs[8]
on a len-8 slice. Changed loop to `range 8`.
4. TestVaultClassify_DeleteDocLinksByTypes_NonExistentDoc: swap
"fake-doc-id-12345" placeholder for a well-formed but unused UUID.
Post-Phase-4, parseUUID rejects bare strings at the caller boundary;
the "non-existent = no-op" contract requires a syntactically valid
UUID. Per Q7 documented contract change.
Phase 4 Step 4g integration gate.
viettranx