test_candidate_service.py 43 KB

12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214
  1. import asyncio
  2. import json
  3. import tempfile
  4. import unittest
  5. from unittest.mock import patch
  6. from pydantic import ValidationError
  7. from cyber_agent.application import (
  8. AgentApplication,
  9. AgentRole,
  10. ApplicationRegistry,
  11. ApplicationServices,
  12. ProviderRef,
  13. )
  14. from cyber_agent.application.candidate import (
  15. CandidateAdoptionReceipt,
  16. CandidateLedger,
  17. CandidateOperationRecord,
  18. CandidatePointer,
  19. CandidateReviewAction,
  20. MAX_CANDIDATES_PER_ROOT,
  21. MAX_CANDIDATE_OPERATIONS_PER_ROOT,
  22. )
  23. from cyber_agent.application.quality import CandidateValidationRecord
  24. from cyber_agent.application.candidate_service import (
  25. CandidateService,
  26. CandidateStateError,
  27. )
  28. from cyber_agent.core.artifacts import (
  29. ArtifactRef,
  30. ValidationMaterial,
  31. material_content_hash,
  32. )
  33. from cyber_agent.core.task_protocol import TaskBrief, new_task_protocol
  34. from cyber_agent.core.validation import (
  35. ScopeValidationResult,
  36. ValidationCheck,
  37. ValidationResult,
  38. )
  39. from cyber_agent.tools.registry import ToolRegistry
  40. from cyber_agent.trace.models import Message as TraceMessage, Trace
  41. from cyber_agent.trace.store import (
  42. FileSystemTraceStore,
  43. TraceStoreCorruptionError,
  44. )
  45. class MemoryCandidateRepository:
  46. def __init__(self):
  47. self.contents = {}
  48. self.put_calls = 0
  49. self.merge_calls = 0
  50. self.bad_artifact = False
  51. self.adoptions = {}
  52. self.adoption_calls = 0
  53. async def put_version(self, request):
  54. self.put_calls += 1
  55. return self._save(request)
  56. async def merge(self, request):
  57. self.merge_calls += 1
  58. return self._save(request)
  59. def _save(self, request):
  60. key = request.operation_id
  61. self.contents.setdefault(key, request)
  62. persisted = self.contents[key]
  63. digest = material_content_hash(persisted.content)
  64. if self.bad_artifact:
  65. digest = "0" * 64
  66. return ArtifactRef(
  67. artifact_id=(
  68. f"candidate:{persisted.candidate_id}:{persisted.revision}"
  69. ),
  70. version=str(persisted.revision),
  71. content_hash=digest,
  72. kind="candidate.output",
  73. mime_type="application/json",
  74. )
  75. async def resolve(self, ref, root_trace_id, uid):
  76. request = next(
  77. item
  78. for item in self.contents.values()
  79. if (
  80. ref.artifact_id
  81. == f"candidate:{item.candidate_id}:{item.revision}"
  82. )
  83. )
  84. return ValidationMaterial(
  85. **ref.model_dump(),
  86. root_trace_id=root_trace_id,
  87. uid=uid,
  88. content=request.content,
  89. )
  90. async def load_version(self, candidate_ref):
  91. return next(
  92. item.content
  93. for item in self.contents.values()
  94. if item.candidate_id == candidate_ref.candidate_id
  95. and item.revision == candidate_ref.revision
  96. )
  97. async def commit_adoption(self, request):
  98. self.adoption_calls += 1
  99. self.adoptions.setdefault(request.operation_id, request.candidate_ref)
  100. return CandidateAdoptionReceipt(
  101. operation_id=request.operation_id,
  102. external_id=f"published:{request.candidate_ref.candidate_id}",
  103. )
  104. def application():
  105. return AgentApplication(
  106. application_id="candidate.test",
  107. application_version="1",
  108. root_role="writer",
  109. roles=(AgentRole(
  110. role_id="writer",
  111. model="fake",
  112. system_prompt="write",
  113. ),),
  114. artifact_resolver_ref=ProviderRef(
  115. provider_id="artifacts",
  116. provider_version="1",
  117. ),
  118. candidate_repository_ref=ProviderRef(
  119. provider_id="candidates",
  120. provider_version="1",
  121. ),
  122. )
  123. class CandidateServiceTest(unittest.IsolatedAsyncioTestCase):
  124. async def asyncSetUp(self):
  125. self.temp = tempfile.TemporaryDirectory()
  126. self.store = FileSystemTraceStore(self.temp.name)
  127. self.repository = MemoryCandidateRepository()
  128. self.binding = ApplicationRegistry().register(
  129. application(),
  130. ApplicationServices(
  131. tool_registry=ToolRegistry(),
  132. artifact_resolver=self.repository,
  133. candidate_repository=self.repository,
  134. ),
  135. )
  136. self.service = CandidateService(
  137. store=self.store,
  138. application_binding=self.binding,
  139. repository=self.repository,
  140. artifact_resolver=self.repository,
  141. )
  142. await self._create_root("root-a")
  143. async def asyncTearDown(self):
  144. self.temp.cleanup()
  145. def _context(self, root_trace_id, *, task_brief=None):
  146. return {
  147. "agent_mode": "recursive",
  148. "agent_mode_revision": 3,
  149. "root_trace_id": root_trace_id,
  150. "root_task_anchor_hash": "a" * 64,
  151. "application_ref": self.binding.application_ref.model_dump(mode="json"),
  152. "application_role_id": "writer",
  153. "application_role_hash": self.binding.role("writer").role_hash,
  154. "task_protocol": new_task_protocol(task_brief),
  155. }
  156. async def _create_root(self, trace_id, *, uid="user-1"):
  157. await self.store.create_trace(Trace(
  158. trace_id=trace_id,
  159. mode="agent",
  160. agent_type="writer",
  161. uid=uid,
  162. context=self._context(trace_id),
  163. ))
  164. async def _create_child(self, trace_id, root_trace_id="root-a"):
  165. await self.store.create_trace(Trace(
  166. trace_id=trace_id,
  167. mode="agent",
  168. agent_type="writer",
  169. uid="user-1",
  170. parent_trace_id=root_trace_id,
  171. context=self._context(
  172. root_trace_id,
  173. task_brief=TaskBrief(
  174. objective="write candidate",
  175. reason="the root needs an option",
  176. completion_criteria=["candidate is complete"],
  177. expected_outputs=["one candidate"],
  178. ),
  179. ),
  180. ))
  181. async def _record_validation(self, trace_id, candidate, outcome="passed"):
  182. plan_hash = material_content_hash({
  183. "candidate": candidate.model_dump(mode="json"),
  184. "outcome": outcome,
  185. })
  186. status = "passed" if outcome == "passed" else "failed"
  187. scope = ScopeValidationResult(
  188. validator_trace_id=f"{trace_id}@validator",
  189. scope="output",
  190. outcome=outcome,
  191. checks=[ValidationCheck(
  192. check_id="quality.rule",
  193. status=status,
  194. issue=None if status == "passed" else "needs revision",
  195. )],
  196. reason="checked",
  197. retry_from=None if outcome == "passed" else "output",
  198. plan_hash=plan_hash,
  199. )
  200. result = ValidationResult(
  201. evaluated_trace_id=trace_id,
  202. outcome=outcome,
  203. scope_results=[scope],
  204. issues=[] if outcome == "passed" else ["needs revision"],
  205. retry_from=None if outcome == "passed" else "output",
  206. plan_hash=plan_hash,
  207. )
  208. await self.service.record_validation(
  209. trace_id,
  210. CandidateValidationRecord(
  211. candidate_ref=candidate,
  212. plan_hash=plan_hash,
  213. validation_result=result.model_dump(mode="json"),
  214. validated_at_sequence=candidate.created_at_sequence,
  215. ),
  216. )
  217. async def test_create_fork_merge_and_idempotent_recovery(self):
  218. first, duplicate = await asyncio.gather(
  219. self.service.manage(
  220. "root-a",
  221. operation="create",
  222. content={"text": "A"},
  223. parent_refs=[],
  224. effective_at_sequence=1,
  225. ),
  226. self.service.manage(
  227. "root-a",
  228. operation="create",
  229. content={"text": "A"},
  230. parent_refs=[],
  231. effective_at_sequence=1,
  232. ),
  233. )
  234. self.assertEqual(first, duplicate)
  235. self.assertEqual(1, self.repository.put_calls)
  236. second = await self.service.manage(
  237. "root-a",
  238. operation="create",
  239. content={"text": "B"},
  240. parent_refs=[],
  241. effective_at_sequence=2,
  242. )
  243. fork = await self.service.manage(
  244. "root-a",
  245. operation="fork",
  246. content={"text": "A2"},
  247. parent_refs=[CandidatePointer(
  248. candidate_id=first.candidate_id,
  249. revision=first.revision,
  250. )],
  251. effective_at_sequence=3,
  252. )
  253. self.assertEqual(first.candidate_id, fork.candidate_id)
  254. self.assertEqual(2, fork.revision)
  255. merged = await self.service.manage(
  256. "root-a",
  257. operation="merge",
  258. content={"text": "AB"},
  259. parent_refs=[
  260. CandidatePointer(
  261. candidate_id=fork.candidate_id,
  262. revision=fork.revision,
  263. ),
  264. CandidatePointer(
  265. candidate_id=second.candidate_id,
  266. revision=second.revision,
  267. ),
  268. ],
  269. effective_at_sequence=4,
  270. )
  271. self.assertNotIn(
  272. merged.candidate_id,
  273. {first.candidate_id, second.candidate_id},
  274. )
  275. self.assertEqual(2, len(merged.parent_refs))
  276. ledger = CandidateLedger.model_validate(
  277. await self.store.get_candidate_ledger("root-a")
  278. )
  279. self.assertEqual(4, len(ledger.candidates))
  280. self.assertTrue(all(
  281. ledger.current_state(item) == "proposed"
  282. for item in ledger.candidates
  283. ))
  284. async def test_finalize_crash_retries_same_semantic_command_identity(self):
  285. original_replace = self.store.replace_candidate_ledger
  286. writes = 0
  287. async def fail_completed_publish(root_trace_id, ledger):
  288. nonlocal writes
  289. writes += 1
  290. if writes == 2:
  291. raise OSError("simulated ledger finalize crash")
  292. await original_replace(root_trace_id, ledger)
  293. with patch.object(
  294. self.store,
  295. "replace_candidate_ledger",
  296. side_effect=fail_completed_publish,
  297. ):
  298. with self.assertRaisesRegex(OSError, "finalize crash"):
  299. await self.service.manage(
  300. "root-a",
  301. operation="create",
  302. content={"text": "survives retry"},
  303. parent_refs=[],
  304. effective_at_sequence=2,
  305. command_id="candidate-crash-call",
  306. )
  307. pending = CandidateLedger.model_validate(
  308. await self.store.get_candidate_ledger("root-a")
  309. )
  310. self.assertEqual("pending", pending.operations[0].status)
  311. operation_id = pending.operations[0].operation_id
  312. recovered = await self.service.manage(
  313. "root-a",
  314. operation="create",
  315. content={"text": "survives retry"},
  316. parent_refs=[],
  317. effective_at_sequence=99,
  318. command_id="candidate-crash-call",
  319. )
  320. ledger = CandidateLedger.model_validate(
  321. await self.store.get_candidate_ledger("root-a")
  322. )
  323. self.assertEqual(operation_id, ledger.operations[0].operation_id)
  324. self.assertEqual("completed", ledger.operations[0].status)
  325. self.assertEqual(1, len(ledger.candidates))
  326. self.assertEqual(1, len(self.repository.contents))
  327. self.assertEqual(
  328. recovered,
  329. ledger.candidate(ledger.operations[0].candidate),
  330. )
  331. async def test_distinct_tool_calls_with_identical_payload_create_versions(self):
  332. first = await self.service.manage(
  333. "root-a",
  334. operation="create",
  335. content={"text": "same payload"},
  336. parent_refs=[],
  337. effective_at_sequence=10,
  338. command_id="candidate-call-one",
  339. )
  340. await self.store.add_message(TraceMessage.create(
  341. trace_id="root-a",
  342. role="tool",
  343. sequence=10,
  344. tool_call_id="candidate-call-one",
  345. content={"tool_name": "manage_candidate", "result": "registered"},
  346. ))
  347. await self.store.update_trace("root-a", head_sequence=10)
  348. second = await self.service.manage(
  349. "root-a",
  350. operation="create",
  351. content={"text": "same payload"},
  352. parent_refs=[],
  353. effective_at_sequence=20,
  354. command_id="candidate-call-two",
  355. )
  356. await self.store.add_message(TraceMessage.create(
  357. trace_id="root-a",
  358. role="tool",
  359. sequence=20,
  360. parent_sequence=10,
  361. tool_call_id="candidate-call-two",
  362. content={"tool_name": "manage_candidate", "result": "registered"},
  363. ))
  364. await self.store.update_trace("root-a", head_sequence=20)
  365. self.assertNotEqual(first, second)
  366. self.assertEqual(2, self.repository.put_calls)
  367. ledger = CandidateLedger.model_validate(
  368. await self.store.get_candidate_ledger("root-a")
  369. )
  370. self.assertEqual(2, len(ledger.candidates))
  371. self.assertEqual(2, len(ledger.operations))
  372. async def test_rewind_makes_future_branch_candidate_unreportable(self):
  373. await self.store.add_message(TraceMessage.create(
  374. trace_id="root-a",
  375. role="user",
  376. sequence=1,
  377. content="before candidate",
  378. ))
  379. await self.store.update_trace("root-a", head_sequence=1)
  380. candidate = await self.service.manage(
  381. "root-a",
  382. operation="create",
  383. content={"text": "future branch"},
  384. parent_refs=[],
  385. effective_at_sequence=10,
  386. command_id="future-candidate-call",
  387. )
  388. await self.store.add_message(TraceMessage.create(
  389. trace_id="root-a",
  390. role="tool",
  391. sequence=10,
  392. parent_sequence=1,
  393. tool_call_id="future-candidate-call",
  394. content={"tool_name": "manage_candidate", "result": "registered"},
  395. ))
  396. await self.store.update_trace("root-a", head_sequence=10)
  397. await self.service.validate_report_refs("root-a", [candidate])
  398. await self.store.update_trace("root-a", head_sequence=1)
  399. with self.assertRaisesRegex(
  400. CandidateStateError,
  401. "current owner Trace branch",
  402. ):
  403. await self.service.validate_report_refs("root-a", [candidate])
  404. async def test_checkpoint_update_cannot_mutate_identity_or_sequence(self):
  405. candidate = await self.service.manage(
  406. "root-a",
  407. operation="create",
  408. content={"text": "checkpoint"},
  409. parent_refs=[],
  410. effective_at_sequence=3,
  411. )
  412. plan_hash = "a" * 64
  413. await self.service.begin_validation_checkpoint(
  414. "root-a",
  415. candidate,
  416. plan={"checks": []},
  417. plan_hash=plan_hash,
  418. validated_at_sequence=3,
  419. material_usage_recorded=False,
  420. )
  421. with self.assertRaisesRegex(
  422. CandidateStateError,
  423. "fields are immutable",
  424. ):
  425. await self.service.update_validation_checkpoint(
  426. "root-a",
  427. candidate,
  428. plan_hash,
  429. validated_at_sequence=-1,
  430. )
  431. ledger = CandidateLedger.model_validate(
  432. await self.store.get_candidate_ledger("root-a")
  433. )
  434. self.assertEqual(
  435. 3,
  436. ledger.validation_checkpoints[0]["validated_at_sequence"],
  437. )
  438. async def test_cross_root_and_cross_task_parent_access_is_rejected(self):
  439. first = await self.service.manage(
  440. "root-a",
  441. operation="create",
  442. content={"text": "A"},
  443. parent_refs=[],
  444. effective_at_sequence=1,
  445. )
  446. await self._create_root("root-b")
  447. with self.assertRaisesRegex(CandidateStateError, "not found"):
  448. await self.service.manage(
  449. "root-b",
  450. operation="fork",
  451. content={"text": "stolen"},
  452. parent_refs=[CandidatePointer(
  453. candidate_id=first.candidate_id,
  454. revision=first.revision,
  455. )],
  456. effective_at_sequence=2,
  457. )
  458. await self._create_child("child-a")
  459. with self.assertRaisesRegex(CandidateStateError, "another application, root, or Task"):
  460. await self.service.manage(
  461. "child-a",
  462. operation="fork",
  463. content={"text": "stolen"},
  464. parent_refs=[CandidatePointer(
  465. candidate_id=first.candidate_id,
  466. revision=first.revision,
  467. )],
  468. effective_at_sequence=3,
  469. )
  470. async def test_bad_artifact_fails_without_registering_candidate(self):
  471. self.repository.bad_artifact = True
  472. with self.assertRaisesRegex(CandidateStateError, "invalid ArtifactRef"):
  473. await self.service.manage(
  474. "root-a",
  475. operation="create",
  476. content={"text": "bad"},
  477. parent_refs=[],
  478. effective_at_sequence=1,
  479. )
  480. ledger = CandidateLedger.model_validate(
  481. await self.store.get_candidate_ledger("root-a")
  482. )
  483. self.assertEqual(0, len(ledger.candidates))
  484. self.assertEqual("failed", ledger.operations[0].status)
  485. async def test_report_validation_rejects_tamper_and_wrong_owner(self):
  486. candidate = await self.service.manage(
  487. "root-a",
  488. operation="create",
  489. content={"text": "A"},
  490. parent_refs=[],
  491. effective_at_sequence=1,
  492. )
  493. await self.service.validate_report_refs("root-a", [candidate])
  494. tampered = candidate.model_copy(update={
  495. "artifact_ref": candidate.artifact_ref.model_copy(update={
  496. "version": "tampered",
  497. }),
  498. })
  499. with self.assertRaisesRegex(CandidateStateError, "does not match"):
  500. await self.service.validate_report_refs("root-a", [tampered])
  501. await self._create_child("child-a")
  502. with self.assertRaisesRegex(CandidateStateError, "another application, root, or Task"):
  503. await self.service.validate_report_refs("child-a", [candidate])
  504. async def test_ledger_rejects_future_lineage_and_store_reports_corruption(self):
  505. candidate = await self.service.manage(
  506. "root-a",
  507. operation="create",
  508. content={"text": "A"},
  509. parent_refs=[],
  510. effective_at_sequence=1,
  511. )
  512. raw = (await self.store.get_candidate_ledger("root-a"))
  513. future = candidate.model_copy(update={
  514. "revision": 2,
  515. "parent_refs": (CandidatePointer(
  516. candidate_id=candidate.candidate_id,
  517. revision=3,
  518. ),),
  519. })
  520. raw["candidates"].append(future.model_dump(mode="json"))
  521. with self.assertRaisesRegex(ValidationError, "unknown parent"):
  522. CandidateLedger.model_validate(raw)
  523. ledger_path = self.store._get_candidate_ledger_file("root-a")
  524. ledger_path.write_text("{broken", encoding="utf-8")
  525. with self.assertRaisesRegex(TraceStoreCorruptionError, "candidate ledger"):
  526. await self.store.get_candidate_ledger("root-a")
  527. async def test_cyclic_lineage_is_rejected_before_repository_side_effect(self):
  528. first = await self.service.manage(
  529. "root-a",
  530. operation="create",
  531. content={"text": "A"},
  532. parent_refs=[],
  533. effective_at_sequence=1,
  534. )
  535. raw = await self.store.get_candidate_ledger("root-a")
  536. second = first.model_copy(update={
  537. "candidate_id": "candidate-b",
  538. "parent_refs": (CandidatePointer(
  539. candidate_id=first.candidate_id,
  540. revision=first.revision,
  541. ),),
  542. })
  543. raw["candidates"][0]["parent_refs"] = [{
  544. "candidate_id": second.candidate_id,
  545. "revision": second.revision,
  546. }]
  547. raw["candidates"].append(second.model_dump(mode="json"))
  548. ledger_path = self.store._get_candidate_ledger_file("root-a")
  549. ledger_path.write_text(json.dumps(raw), encoding="utf-8")
  550. put_calls = self.repository.put_calls
  551. with self.assertRaisesRegex(ValidationError, "lineage cycle"):
  552. await self.service.manage(
  553. "root-a",
  554. operation="create",
  555. content={"text": "must not be persisted"},
  556. parent_refs=[],
  557. effective_at_sequence=2,
  558. )
  559. self.assertEqual(put_calls, self.repository.put_calls)
  560. async def test_review_adoption_is_idempotent_and_creates_rewind_barrier(self):
  561. await self._create_child("child-a")
  562. candidate = await self.service.manage(
  563. "child-a",
  564. operation="create",
  565. content={"text": "A"},
  566. parent_refs=[],
  567. effective_at_sequence=4,
  568. )
  569. await self._record_validation("child-a", candidate)
  570. action = CandidateReviewAction(
  571. action="adopt",
  572. candidate_ref=candidate,
  573. reason="selected exact revision",
  574. )
  575. first = await self.service.apply_review_actions(
  576. "root-a",
  577. "child-a",
  578. report_refs=[candidate],
  579. actions=[action],
  580. effective_at_sequence=9,
  581. )
  582. second = await self.service.apply_review_actions(
  583. "root-a",
  584. "child-a",
  585. report_refs=[candidate],
  586. actions=[action],
  587. effective_at_sequence=9,
  588. )
  589. self.assertEqual(first, second)
  590. self.assertEqual(1, self.repository.adoption_calls)
  591. self.assertEqual(1, len(self.repository.adoptions))
  592. with self.assertRaisesRegex(CandidateStateError, "committed"):
  593. await self.service.assert_rewind_allowed("root-a", 8)
  594. await self.service.assert_rewind_allowed("root-a", 9)
  595. with self.assertRaisesRegex(CandidateStateError, "terminal"):
  596. await self.service.apply_review_actions(
  597. "root-a",
  598. "child-a",
  599. report_refs=[candidate],
  600. actions=[CandidateReviewAction(
  601. action="discard",
  602. candidate_ref=candidate,
  603. reason="too late",
  604. )],
  605. effective_at_sequence=10,
  606. )
  607. async def test_adoption_retry_uses_same_command_after_parent_sequence_changes(self):
  608. await self._create_child("child-a")
  609. candidate = await self.service.manage(
  610. "child-a",
  611. operation="create",
  612. content={"text": "publish once"},
  613. parent_refs=[],
  614. effective_at_sequence=4,
  615. )
  616. await self._record_validation("child-a", candidate)
  617. peer = await self.service.manage(
  618. "child-a",
  619. operation="create",
  620. content={"text": "peer"},
  621. parent_refs=[],
  622. effective_at_sequence=5,
  623. )
  624. await self._record_validation("child-a", peer)
  625. action = CandidateReviewAction(
  626. action="adopt",
  627. candidate_ref=candidate,
  628. reason="stable parent decision",
  629. )
  630. first = await self.service.apply_review_actions(
  631. "root-a",
  632. "child-a",
  633. report_refs=[candidate, peer],
  634. actions=[action],
  635. effective_at_sequence=10,
  636. command_id="adoption-review-call",
  637. )
  638. recovered = await self.service.apply_review_actions(
  639. "root-a",
  640. "child-a",
  641. report_refs=[candidate, peer],
  642. actions=[action],
  643. effective_at_sequence=999,
  644. command_id="adoption-review-call",
  645. )
  646. self.assertEqual(first, recovered)
  647. self.assertEqual(1, self.repository.adoption_calls)
  648. self.assertEqual(1, len(self.repository.adoptions))
  649. with self.assertRaisesRegex(
  650. CandidateStateError,
  651. "review batch payload changed",
  652. ):
  653. await self.service.apply_review_actions(
  654. "root-a",
  655. "child-a",
  656. report_refs=[candidate, peer],
  657. actions=[action.model_copy(update={
  658. "reason": "tampered decision payload",
  659. })],
  660. effective_at_sequence=1_000,
  661. command_id="adoption-review-call",
  662. )
  663. self.assertEqual(1, self.repository.adoption_calls)
  664. with self.assertRaisesRegex(
  665. CandidateStateError,
  666. "review batch payload changed",
  667. ):
  668. await self.service.apply_review_actions(
  669. "root-a",
  670. "child-a",
  671. report_refs=[candidate, peer],
  672. actions=[
  673. action,
  674. CandidateReviewAction(
  675. action="discard",
  676. candidate_ref=peer,
  677. reason="injected tail action",
  678. ),
  679. ],
  680. effective_at_sequence=1_001,
  681. command_id="adoption-review-call",
  682. )
  683. self.assertEqual("proposed", (
  684. CandidateLedger.model_validate(
  685. await self.store.get_candidate_ledger("root-a")
  686. ).current_state(peer)
  687. ))
  688. with self.assertRaisesRegex(
  689. CandidateStateError,
  690. "review batch payload changed",
  691. ):
  692. await self.service.apply_review_actions(
  693. "root-a",
  694. "child-a",
  695. report_refs=[candidate, peer],
  696. actions=[],
  697. effective_at_sequence=1_002,
  698. command_id="adoption-review-call",
  699. )
  700. async def test_review_batch_replay_rejects_deleted_and_reordered_actions(self):
  701. await self._create_child("child-a")
  702. candidates = []
  703. for sequence, text in ((4, "A"), (5, "B")):
  704. candidate = await self.service.manage(
  705. "child-a",
  706. operation="create",
  707. content={"text": text},
  708. parent_refs=[],
  709. effective_at_sequence=sequence,
  710. )
  711. await self._record_validation("child-a", candidate)
  712. candidates.append(candidate)
  713. actions = [
  714. CandidateReviewAction(
  715. action="discard",
  716. candidate_ref=candidate,
  717. reason=f"discard {candidate.candidate_id}",
  718. )
  719. for candidate in candidates
  720. ]
  721. await self.service.apply_review_actions(
  722. "root-a",
  723. "child-a",
  724. report_refs=candidates,
  725. actions=actions,
  726. effective_at_sequence=10,
  727. command_id="two-action-review",
  728. )
  729. for altered in (actions[:1], list(reversed(actions))):
  730. with self.assertRaisesRegex(
  731. CandidateStateError,
  732. "review batch payload changed",
  733. ):
  734. await self.service.apply_review_actions(
  735. "root-a",
  736. "child-a",
  737. report_refs=candidates,
  738. actions=altered,
  739. effective_at_sequence=999,
  740. command_id="two-action-review",
  741. )
  742. async def test_empty_review_batch_cannot_gain_an_action_during_replay(self):
  743. await self._create_child("child-a")
  744. candidate = await self.service.manage(
  745. "child-a",
  746. operation="create",
  747. content={"text": "A"},
  748. parent_refs=[],
  749. effective_at_sequence=4,
  750. )
  751. await self._record_validation("child-a", candidate)
  752. self.assertEqual([], await self.service.apply_review_actions(
  753. "root-a",
  754. "child-a",
  755. report_refs=[candidate],
  756. actions=[],
  757. effective_at_sequence=10,
  758. command_id="empty-review-batch",
  759. ))
  760. with self.assertRaisesRegex(
  761. CandidateStateError,
  762. "review batch payload changed",
  763. ):
  764. await self.service.apply_review_actions(
  765. "root-a",
  766. "child-a",
  767. report_refs=[candidate],
  768. actions=[CandidateReviewAction(
  769. action="discard",
  770. candidate_ref=candidate,
  771. reason="injected after empty batch",
  772. )],
  773. effective_at_sequence=999,
  774. command_id="empty-review-batch",
  775. )
  776. self.assertEqual(
  777. "proposed",
  778. CandidateLedger.model_validate(
  779. await self.store.get_candidate_ledger("root-a")
  780. ).current_state(candidate),
  781. )
  782. async def test_review_capacity_rejects_before_envelope_or_repository_write(self):
  783. await self._create_child("child-a")
  784. candidate = await self.service.manage(
  785. "child-a",
  786. operation="create",
  787. content={"text": "capacity"},
  788. parent_refs=[],
  789. effective_at_sequence=4,
  790. )
  791. await self._record_validation("child-a", candidate)
  792. ledger = CandidateLedger.model_validate(
  793. await self.store.get_candidate_ledger("root-a")
  794. )
  795. fillers = [
  796. CandidateOperationRecord(
  797. operation_id=f"capacity-filler-{index}",
  798. operation="review_batch",
  799. input_hash="0" * 64,
  800. status="completed",
  801. )
  802. for index in range(
  803. MAX_CANDIDATE_OPERATIONS_PER_ROOT - len(ledger.operations)
  804. )
  805. ]
  806. saturated = CandidateLedger.model_validate({
  807. **ledger.model_dump(mode="json"),
  808. "operations": [
  809. *[item.model_dump(mode="json") for item in ledger.operations],
  810. *[item.model_dump(mode="json") for item in fillers],
  811. ],
  812. })
  813. await self.store.replace_candidate_ledger(
  814. "root-a",
  815. saturated.model_dump(mode="json"),
  816. )
  817. before = await self.store.get_candidate_ledger("root-a")
  818. with self.assertRaisesRegex(
  819. CandidateStateError,
  820. "operation limit exceeded before execution",
  821. ):
  822. await self.service.apply_review_actions(
  823. "root-a",
  824. "child-a",
  825. report_refs=[candidate],
  826. actions=[CandidateReviewAction(
  827. action="adopt",
  828. candidate_ref=candidate,
  829. reason="must not reach repository",
  830. )],
  831. effective_at_sequence=10,
  832. command_id="capacity-overflow-review",
  833. )
  834. self.assertEqual(0, self.repository.adoption_calls)
  835. self.assertEqual(before, await self.store.get_candidate_ledger("root-a"))
  836. async def test_review_capacity_allows_exact_envelope_and_action_boundary(self):
  837. await self._create_child("child-a")
  838. candidate = await self.service.manage(
  839. "child-a",
  840. operation="create",
  841. content={"text": "exact capacity"},
  842. parent_refs=[],
  843. effective_at_sequence=4,
  844. )
  845. await self._record_validation("child-a", candidate)
  846. ledger = CandidateLedger.model_validate(
  847. await self.store.get_candidate_ledger("root-a")
  848. )
  849. target = MAX_CANDIDATE_OPERATIONS_PER_ROOT - 2
  850. fillers = [
  851. CandidateOperationRecord(
  852. operation_id=f"exact-filler-{index}",
  853. operation="review_batch",
  854. input_hash="1" * 64,
  855. status="completed",
  856. )
  857. for index in range(target - len(ledger.operations))
  858. ]
  859. near_limit = CandidateLedger.model_validate({
  860. **ledger.model_dump(mode="json"),
  861. "operations": [
  862. *[item.model_dump(mode="json") for item in ledger.operations],
  863. *[item.model_dump(mode="json") for item in fillers],
  864. ],
  865. })
  866. await self.store.replace_candidate_ledger(
  867. "root-a",
  868. near_limit.model_dump(mode="json"),
  869. )
  870. await self.service.apply_review_actions(
  871. "root-a",
  872. "child-a",
  873. report_refs=[candidate],
  874. actions=[CandidateReviewAction(
  875. action="discard",
  876. candidate_ref=candidate,
  877. reason="fits exact operation boundary",
  878. )],
  879. effective_at_sequence=10,
  880. command_id="exact-capacity-review",
  881. )
  882. completed = CandidateLedger.model_validate(
  883. await self.store.get_candidate_ledger("root-a")
  884. )
  885. self.assertEqual(MAX_CANDIDATE_OPERATIONS_PER_ROOT, len(completed.operations))
  886. self.assertEqual("discarded", completed.current_state(candidate))
  887. async def test_candidate_capacity_rejects_before_repository_or_intent_write(self):
  888. await self._create_child("child-a")
  889. candidate = await self.service.manage(
  890. "child-a",
  891. operation="create",
  892. content={"text": "seed"},
  893. parent_refs=[],
  894. effective_at_sequence=4,
  895. )
  896. ledger = CandidateLedger.model_validate(
  897. await self.store.get_candidate_ledger("root-a")
  898. )
  899. saturated_candidates = [candidate]
  900. saturated_candidates.extend(
  901. candidate.model_copy(update={
  902. "candidate_id": f"capacity-candidate-{index}",
  903. })
  904. for index in range(1, MAX_CANDIDATES_PER_ROOT)
  905. )
  906. saturated = CandidateLedger.model_validate({
  907. **ledger.model_dump(mode="json"),
  908. "candidates": [
  909. item.model_dump(mode="json") for item in saturated_candidates
  910. ],
  911. })
  912. await self.store.replace_candidate_ledger(
  913. "root-a",
  914. saturated.model_dump(mode="json"),
  915. )
  916. before = await self.store.get_candidate_ledger("root-a")
  917. repository_calls = self.repository.put_calls
  918. with self.assertRaisesRegex(
  919. CandidateStateError,
  920. "candidate limit exceeded before execution",
  921. ):
  922. await self.service.manage(
  923. "child-a",
  924. operation="create",
  925. content={"text": "must remain external-orphan free"},
  926. parent_refs=[],
  927. effective_at_sequence=5,
  928. command_id="candidate-capacity-overflow",
  929. )
  930. self.assertEqual(repository_calls, self.repository.put_calls)
  931. self.assertEqual(before, await self.store.get_candidate_ledger("root-a"))
  932. async def test_pending_candidate_reserves_capacity_until_recovery(self):
  933. await self._create_child("child-a")
  934. seed = await self.service.manage(
  935. "child-a",
  936. operation="create",
  937. content={"text": "seed"},
  938. parent_refs=[],
  939. effective_at_sequence=4,
  940. )
  941. ledger = CandidateLedger.model_validate(
  942. await self.store.get_candidate_ledger("root-a")
  943. )
  944. near_limit_candidates = [seed]
  945. near_limit_candidates.extend(
  946. seed.model_copy(update={
  947. "candidate_id": f"reserved-candidate-{index}",
  948. })
  949. for index in range(1, MAX_CANDIDATES_PER_ROOT - 1)
  950. )
  951. near_limit = CandidateLedger.model_validate({
  952. **ledger.model_dump(mode="json"),
  953. "candidates": [
  954. item.model_dump(mode="json") for item in near_limit_candidates
  955. ],
  956. })
  957. await self.store.replace_candidate_ledger(
  958. "root-a",
  959. near_limit.model_dump(mode="json"),
  960. )
  961. original_replace = self.store.replace_candidate_ledger
  962. writes = 0
  963. async def crash_after_repository(root_trace_id, raw_ledger):
  964. nonlocal writes
  965. writes += 1
  966. if writes == 2:
  967. raise OSError("crash before candidate publish")
  968. await original_replace(root_trace_id, raw_ledger)
  969. with patch.object(
  970. self.store,
  971. "replace_candidate_ledger",
  972. side_effect=crash_after_repository,
  973. ):
  974. with self.assertRaisesRegex(OSError, "candidate publish"):
  975. await self.service.manage(
  976. "child-a",
  977. operation="create",
  978. content={"text": "reserved A"},
  979. parent_refs=[],
  980. effective_at_sequence=5,
  981. command_id="reserved-candidate-A",
  982. )
  983. pending = CandidateLedger.model_validate(
  984. await self.store.get_candidate_ledger("root-a")
  985. )
  986. self.assertEqual(MAX_CANDIDATES_PER_ROOT - 1, len(pending.candidates))
  987. self.assertEqual(
  988. 1,
  989. sum(
  990. item.status == "pending" and item.operation == "create"
  991. for item in pending.operations
  992. ),
  993. )
  994. calls_after_a = self.repository.put_calls
  995. with self.assertRaisesRegex(
  996. CandidateStateError,
  997. "candidate limit exceeded before execution",
  998. ):
  999. await self.service.manage(
  1000. "child-a",
  1001. operation="create",
  1002. content={"text": "must not steal A's slot"},
  1003. parent_refs=[],
  1004. effective_at_sequence=6,
  1005. command_id="reserved-candidate-B",
  1006. )
  1007. self.assertEqual(calls_after_a, self.repository.put_calls)
  1008. recovered = await self.service.manage(
  1009. "child-a",
  1010. operation="create",
  1011. content={"text": "reserved A"},
  1012. parent_refs=[],
  1013. effective_at_sequence=99,
  1014. command_id="reserved-candidate-A",
  1015. )
  1016. completed = CandidateLedger.model_validate(
  1017. await self.store.get_candidate_ledger("root-a")
  1018. )
  1019. self.assertEqual(MAX_CANDIDATES_PER_ROOT, len(completed.candidates))
  1020. self.assertEqual(recovered, completed.candidate(recovered))
  1021. async def test_descendant_adoption_blocks_ancestor_rewind_without_mapping(self):
  1022. await self._create_child("child-a")
  1023. await self.store.create_trace(Trace(
  1024. trace_id="grandchild-a",
  1025. mode="agent",
  1026. agent_type="writer",
  1027. uid="user-1",
  1028. parent_trace_id="child-a",
  1029. context=self._context(
  1030. "root-a",
  1031. task_brief=TaskBrief(
  1032. objective="write descendant candidate",
  1033. reason="the child needs an option",
  1034. completion_criteria=["candidate is complete"],
  1035. expected_outputs=["one candidate"],
  1036. ),
  1037. ),
  1038. ))
  1039. candidate = await self.service.manage(
  1040. "grandchild-a",
  1041. operation="create",
  1042. content={"text": "nested candidate"},
  1043. parent_refs=[],
  1044. effective_at_sequence=4,
  1045. )
  1046. await self._record_validation("grandchild-a", candidate)
  1047. await self.service.apply_review_actions(
  1048. "child-a",
  1049. "grandchild-a",
  1050. report_refs=[candidate],
  1051. actions=[CandidateReviewAction(
  1052. action="adopt",
  1053. candidate_ref=candidate,
  1054. reason="commit descendant output",
  1055. )],
  1056. effective_at_sequence=9,
  1057. )
  1058. with self.assertRaisesRegex(CandidateStateError, "committed"):
  1059. await self.service.assert_rewind_allowed("root-a", 100)
  1060. async def test_failed_candidate_can_only_be_revised_or_discarded(self):
  1061. await self._create_child("child-a")
  1062. candidate = await self.service.manage(
  1063. "child-a",
  1064. operation="create",
  1065. content={"text": "placeholder"},
  1066. parent_refs=[],
  1067. effective_at_sequence=4,
  1068. )
  1069. await self._record_validation("child-a", candidate, outcome="failed")
  1070. with self.assertRaisesRegex(CandidateStateError, "passed"):
  1071. await self.service.apply_review_actions(
  1072. "root-a",
  1073. "child-a",
  1074. report_refs=[candidate],
  1075. actions=[CandidateReviewAction(
  1076. action="adopt",
  1077. candidate_ref=candidate,
  1078. reason="invalid",
  1079. )],
  1080. effective_at_sequence=9,
  1081. )
  1082. records = await self.service.apply_review_actions(
  1083. "root-a",
  1084. "child-a",
  1085. report_refs=[candidate],
  1086. actions=[CandidateReviewAction(
  1087. action="revise",
  1088. candidate_ref=candidate,
  1089. reason="remove placeholder",
  1090. )],
  1091. effective_at_sequence=10,
  1092. )
  1093. self.assertEqual("discarded", records[0].state)
  1094. self.assertIn("retry_from=output", records[0].reason)
  1095. async def test_adoption_replays_same_operation_after_finalize_crash(self):
  1096. await self._create_child("child-a")
  1097. candidate = await self.service.manage(
  1098. "child-a",
  1099. operation="create",
  1100. content={"text": "A"},
  1101. parent_refs=[],
  1102. effective_at_sequence=4,
  1103. )
  1104. await self._record_validation("child-a", candidate)
  1105. action = CandidateReviewAction(
  1106. action="adopt",
  1107. candidate_ref=candidate,
  1108. reason="publish once",
  1109. )
  1110. original_replace = self.store.replace_candidate_ledger
  1111. calls = 0
  1112. async def fail_finalize(root_trace_id, ledger):
  1113. nonlocal calls
  1114. calls += 1
  1115. if calls == 3:
  1116. raise OSError("crash before framework finalize")
  1117. await original_replace(root_trace_id, ledger)
  1118. with patch.object(
  1119. self.store,
  1120. "replace_candidate_ledger",
  1121. side_effect=fail_finalize,
  1122. ):
  1123. with self.assertRaisesRegex(OSError, "finalize"):
  1124. await self.service.apply_review_actions(
  1125. "root-a",
  1126. "child-a",
  1127. report_refs=[candidate],
  1128. actions=[action],
  1129. effective_at_sequence=9,
  1130. )
  1131. pending = CandidateLedger.model_validate(
  1132. await self.store.get_candidate_ledger("root-a")
  1133. )
  1134. self.assertEqual("adoption_pending", pending.current_state(candidate))
  1135. await self.service.apply_review_actions(
  1136. "root-a",
  1137. "child-a",
  1138. report_refs=[candidate],
  1139. actions=[action],
  1140. effective_at_sequence=9,
  1141. )
  1142. self.assertEqual(2, self.repository.adoption_calls)
  1143. self.assertEqual(1, len(self.repository.adoptions))
  1144. completed = CandidateLedger.model_validate(
  1145. await self.store.get_candidate_ledger("root-a")
  1146. )
  1147. self.assertEqual("adopted", completed.current_state(candidate))
  1148. if __name__ == "__main__":
  1149. unittest.main()