improve: Treaty of Apamea

.automate/continuous-improve.log

@@ -5304,3 +5304,106 @@ status: UPDATED | Sumer | changes: Applied 32 review answers; fixed duplicate #
 [2026-02-23 00:27:36] ━━━ [Treaty of Apamea] (7207df) reviews=0 garbage=0 ━━━
 [2026-02-23 00:27:36]   🧹 Bash cleanup applied
 [2026-02-23 00:27:36]   🔍 Enrichment + review pass
+All tools are now trusted (!). Kiro will execute tools without asking for confirmation.
+Agents can sometimes do unexpected things so understand the risks.
+
+Learn more at https://kiro.dev/docs/cli/chat/security/#using-tools-trust-all-safely
+
+
+
+[?25l[?25lRunning tool think with the param (from mcp server: sequential-thinking-mcp)
+ ⋮  {
+ ⋮    "thought": "Let me analyze the document. The review questions are all marked as [x] (answered) in the document content. I need to:\n1. Check the review queue to see if there are open questions\n2. Apply any answers if needed\n3. Enrich the document with web search\n\nLooking at the document, all questions appear to be answered (marked with [x]). But I notice the document has a duplicate title (\"# Treaty of Apamea\" appears twice). Also, the review answers note \"BCE temporal tags not yet supported by factbase\" but the steering file says BCE tags ARE supported. The document already has @t[=188 BCE] tags in the content.\n\nLet me check the review queue first, then proceed."
+ ⋮  }
+ - Completed in 0.142s
+
+[?25l[?25lRunning tool get_review_queue with the param (from mcp server: factbase)
+ ⋮  {
+ ⋮    "doc_id": "7207df"
+ ⋮  }
+ - Completed in 0.12s
+
+[?25l> All 20 questions are answered. Now let me apply them and then do web research in parallel.
+[?25l
+[?25lRunning tool apply_review_answers with the param (from mcp server: factbase)
+ ⋮  {
+ ⋮    "doc_id": "7207df"
+ ⋮  }
+Searching the web for: Treaty of Apamea 188 BCE terms significance Seleucid Rome scholarship (using tool: web_search)
+ - Completed in 0.948s
+
+
+ ✓ Found 10 search results
+ - Completed in 0.765s
+
+[?25l> Good — 20 answers applied. Now let me get the current state of the document and do a more targeted search for additional details.
+[?25l
+[?25lRunning tool get_entity with the param (from mcp server: factbase)
+ ⋮  {
+ ⋮    "id": "7207df"
+ ⋮  }
+Searching the web for: Treaty of Apamea 188 BCE hostages Antiochus IV indemnity payment schedule details (using tool: web_search)
+ - Completed in 0.1s
+
+
+ ✓ Found 10 search results
+ - Completed in 1.154s
+
+[?25l> I can see several issues with the document after apply:
+1. Duplicate # Treaty of Apamea title
+2. The "Antiochus III" line got mangled — the answer text was injected into the content
+3. @t[~1984] was incorrectly added to the Gruen citation line
+4. Missing blank line before ## Terms
+5. The indemnity payment schedule details are missing (500 talents immediately, 2,500 on Senate ratification, then 1,000/year for 12 years)
+6. Missing: Antiochus IV was taken as a hostage under the treaty terms
+
+Let me fix the corruption and enrich the document:
+[?25lRunning tool update_document with the param (from mcp server: factbase)
+ ⋮  {
+ ⋮    "id": "7207df",
+ ⋮    "content": "## Overview\nThe Treaty of Apamea (188 BCE) was imposed by Rome on the Seleucid Empire after the Roman-Seleucid War, effectively ending Seleucid power in Anatolia and establishing Roman dominance in the eastern Mediterranean. @t[=188 BCE]\n\n## Key Facts\n- Date: 188 BCE @t[=188 BCE]\n- Parties: Roman Republic and Seleucid Empire (Antiochus III)\n- Context: Followed the Battle of Magnesia (190 BCE) @t[=190 BCE]\n\n## Terms\n- Seleucids withdrew from all territory west of the Taurus Mountains\n- War indemnity of 15,000 Euboic talents of silver: 500 paid immediately, 2,500 on Senate ratification, then 1,000 talents annually for 12 years [^1]\n- Seleucid navy reduced to 10 warships; all war elephants surrendered\n- Territory redistributed to Rome's allies: Pergamon received Lydia, Phrygia, Mysia, Pisidia, and Pamphylia; Rhodes received Caria and Lycia [^1]\n- Antiochus III's son (the future Antiochus IV Epiphanes) taken to Rome as a political hostage [^3]\n- Seleucids prohibited from expanding west of the Taurus Mountains in perpetuity\n\n## Significance\n- Marked the end of Seleucid influence in Anatolia\n- Established Rome as the dominant power in the eastern Mediterranean\n- Established *philia* (amity) between Rome and the Seleucid dynasty, governing relations for the following half-century [^4]\n- Weakened the Seleucid Empire, contributing to its eventual fragmentation [^2]\n\n---\n[^1]: Polybius, *Histories* 21.43\n[^2]: Gruen, E. *The Hellenistic World and the Coming of Rome* (1984)\n[^3]: Wikipedia, \"Antiochus IV Epiphanes\" — hostage clause of the Treaty of Apamea\n[^4]: Ma, J. \"The Treaty of Apamea and the Later Seleucids,\" *Antichthon* (2015), Cambridge University Press\n",
+ ⋮    "title": "Treaty of Apamea"
+ ⋮  }
+ - Completed in 0.2s
+
+[?25l[?25lRunning tool get_entity with the param (from mcp server: factbase)
+ ⋮  {
+ ⋮    "id": "7207df"
+ ⋮  }
+ - Completed in 0.1s
+
+[?25l> Document looks clean. Now let me file a bug for the apply_review_answers corruption, then commit.
+[?25lI will run the following command: curl -s -X PUT "https://vikunja.home.everyonce.com/api/v1/projects/2/tasks" \
+  -H "Authorization: Bearer tk_ff251f3d3512775c71913bc2f8ec0dabbf5016a8" \
+  -H "Content-Type: application/json" \
+  -d '{
+    "title": "[factbase] apply_review_answers injects answer text into document content and adds spurious temporal tags",
+    "description": "Tool: apply_review_answers\nDoc ID: 7207df (Treaty of Apamea)\n\nThree corruption issues observed after calling apply_review_answers(doc_id='\''7207df'\''):\n\n1. Answer text injected into content: The @q[ambiguous] answer for \"what does III mean\" had its answer text (\"the 3rd ruler of that name. Standard historical convention.\") appended directly into the Key Facts bullet point, changing \"Antiochus III\" to \"Antiochus III — the 3rd ruler of that name. Standard historical convention.\"\n\n2. Spurious temporal tag added: The @q[stale] answer about Gruen (1984) caused a @t[~1984] tag to be inserted into the document body next to the Gruen citation, which is incorrect — 1984 is a publication date, not a historical event date.\n\n3. Duplicate title: The document ended up with two consecutive \"# Treaty of Apamea\" headings after apply.\n\nExpected behavior: apply_review_answers should strip @q[] markers and their answers from the review queue section without modifying the main document body content.\n\nActual behavior: Answer text was merged into the document body, and a temporal tag was incorrectly synthesized from a year mentioned in an answer.\n\nWorkaround: Used update_document to manually restore the correct document content.\n\nContext: Running entity improvement loop on treaties/treaty-of-apamea.md.",
+    "priority": 4
+  }' (using tool: shell)
+Purpose: File bug report for apply_review_answers corruption
+
+{"id":100,"title":"[factbase] apply_review_answers injects answer text into document content and adds spurious temporal tags","description":"Tool: apply_review_answers\nDoc ID: 7207df (Treaty of Apamea)\n\nThree corruption issues observed after calling apply_review_answers(doc_id='7207df'):\n\n1. Answer text injected into content: The @q[ambiguous] answer for \"what does III mean\" had its answer text (\"the 3rd ruler of that name. Standard historical convention.\") appended directly into the Key Facts bullet point, changing \"Antiochus III\" to \"Antiochus III — the 3rd ruler of that name. Standard historical convention.\"\n\n2. Spurious temporal tag added: The @q[stale] answer about Gruen (1984) caused a @t[~1984] tag to be inserted into the document body next to the Gruen citation, which is incorrect — 1984 is a publication date, not a historical event date.\n\n3. Duplicate title: The document ended up with two consecutive \"# Treaty of Apamea\" headings after apply.\n\nExpected behavior: apply_review_answers should strip @q[] markers and their answers from the review queue section without modifying the main document body content.\n\nActual behavior: Answer text was merged into the document body, and a temporal tag was incorrectly synthesized from a year mentioned in an answer.\n\nWorkaround: Used update_document to manually restore the correct document content.\n\nContext: Running entity improvement loop on treaties/treaty-of-apamea.md.","done":false,"done_at":"0001-01-01T00:00:00Z","due_date":"0001-01-01T00:00:00Z","reminders":null,"project_id":2,"repeat_after":0,"repeat_mode":0,"priority":4,"start_date":"0001-01-01T00:00:00Z","end_date":"0001-01-01T00:00:00Z","assignees":[],"labels":null,"hex_color":"","percent_done":0,"identifier":"#92","index":92,"related_tasks":null,"attachments":null,"cover_image_attachment_id":0,"is_favorite":false,"created":"2026-02-22T18:28:19.887831836-06:00","updated":"2026-02-22T18:28:19.887835002-06:00","bucket_id":0,"position":2.6469779601696886e-23,"reactions":null,"created_by":{"id":1,"name":"daniel","username":"daniel","created":"2026-02-20T07:27:17-06:00","updated":"2026-02-20T10:17:06-06:00"}}
+ - Completed in 0.183s
+
+[?25l> Bug filed (task #92). Now commit:
+[?25lI will run the following command: cd /home/ubuntu/work/factbase-ancient-history && git add -A && git commit -m "Improve Treaty of Apamea: fix apply corruption, add indemnity schedule, hostage clause, territorial details, philia clause" && git push (using tool: shell)
+Purpose: Commit and push changes
+
+[main bf03959] Improve Treaty of Apamea: fix apply corruption, add indemnity schedule, hostage clause, territorial details, philia clause
+ 7 files changed, 47 insertions(+), 72 deletions(-)
+ create mode 100644 .factbase/factbase.db-shm
+ create mode 100644 .factbase/factbase.db-wal
+ rewrite treaties/treaty-of-apamea.md (82%)
+remote: . Processing 1 references        
+remote: Processed 1 references in total        
+To https://gitea.home.everyonce.com/daniel/factbase-ancient-history.git
+   acb0433..bf03959  main -> main
+ - Completed in 1.428s
+
+[?25l> <action_summary>status: UPDATED | Treaty of Apamea | changes: Applied 20 review answers; fixed apply_review_answers corruption (answer text injected into body, spurious @t[~1984] tag, duplicate title); enriched Terms section with indemnity payment schedule (500 immediately, 2500 on Senate ratification, 1000/year for 12 years), specific territorial redistribution to Pergamon and Rhodes, Antiochus IV hostage clause, perpetual Taurus prohibition; added philia/amity clause to Significance; added footnotes [^3] (Wikipedia/Antiochus IV) and [^4] (Ma 2015, Cambridge); filed bug #92 for apply_review_answers corruption</action_summary>
+
+ ▸ Time: 49s
+
+[?25h
+status: UPDATED | Treaty of Apamea | changes: Applied 20 review answers; fixed apply_review_answers corruption (answer text injected into body, spurious @t[~1984] tag, duplicate title); enriched Terms section with indemnity payment schedule (500 immediately, 2500 on Senate ratification, 1000/year for 12 years), specific territorial redistribution to Pergamon and Rhodes, Antiochus IV hostage clause, perpetual Taurus prohibition; added philia/amity clause to Significance; added footnotes [^3] (Wikipedia/Antiochus IV) and [^4] (Ma 2015, Cambridge); filed bug #92 for apply_review_answers corruption