fix(clone): honour skip strategy in _clone_verified_data at runtime

_plan_children already forecasts verified-data as skipped under
file_strategy=skip, but the runtime path lacked the matching guard: on a
re-run where documents reached the target by other means, it would create
verified rows the plan said it would skip. Add the early-return guard,
mirroring _clone_documents.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011ja9H1rnSXmPUgQtHm8TNS

Author: chandrasekharan-zipstack

src/unstract/clone/phases/agentic_studio.py

@@ -559,6 +559,17 @@ def _clone_verified_data(
         if not src_rows:
             return
 
+        # Verified data FKs a document; under 'skip' no documents are copied,
+        # so skip the rows too — matching the plan and _clone_documents.
+        if self.ctx.options.file_strategy == "skip":
+            with lock:
+                result.skipped += len(src_rows)
+                result.warnings.append(
+                    f"agentic {name}: {len(src_rows)} verified-data row(s) not "
+                    "copied (file_strategy=skip)"
+                )
+            return
+
         # Verified data FKs a document; map source rows to target docs by
         # filename, the only identity stable across orgs.
         try:

tests/clone/test_agentic_studio_phase.py

@@ -676,6 +676,29 @@ def test_documents_not_copied_under_file_strategy_skip():
     assert result.skipped == 1
 
 
+def test_verified_data_not_copied_under_skip_even_if_doc_on_target():
+    # Docs reached the target by other means (prior non-skip run / manual
+    # upload), so doc-by-name maps. Under skip, verified data must still be
+    # skipped at runtime — matching the plan, not created off the stray doc.
+    src = FakeClient(
+        projects=[_src_project("src-p", "Receipts")],
+        documents={"src-p": [{"id": "d1", "original_filename": "report.pdf"}]},
+        document_blobs={"d1": b"%PDF-r"},
+        verified_data={
+            "src-p": [{"document_name": "report.pdf", "document": "d1", "data": {}}]
+        },
+    )
+    tgt = FakeClient()
+    tgt.projects.append({"id": "tgt-proj-0001", "name": "Receipts"})
+    tgt.documents["tgt-proj-0001"] = [{"id": "t-d1", "original_filename": "report.pdf"}]
+    ctx = _ctx(src, tgt, file_strategy="skip")
+
+    result = AgenticStudioPhase(ctx).run(CloneReport())
+
+    assert tgt.created_verified_data == []
+    assert any("verified-data" in w for w in result.warnings)
+
+
 def test_dry_run_skip_strategy_counts_docs_and_verified_as_skipped():
     # Verified data FKs a document; under skip no docs land on target, so the
     # plan must forecast verified as skipped too — not as un-creatable creates.