Elevate factbase bug/feature reporting to equal priority, require domain-agnostic detail
This commit is contained in:
@@ -1,3 +1,4 @@
|
|||||||
0648e6 1771799566
|
0648e6 1771799566
|
||||||
06d51f 1771799636
|
06d51f 1771799636
|
||||||
25cc1e 1771799702
|
25cc1e 1771799702
|
||||||
|
4fb0a1 1771799764
|
||||||
|
|||||||
|
@@ -469,3 +469,11 @@ To https://gitea.home.everyonce.com/daniel/factbase-ancient-history.git
|
|||||||
|
|
||||||
[0m[1G[0m[0m[?25h
|
[0m[1G[0m[0m[?25h
|
||||||
status: UPDATED | Augustus | changes: fixed duplicate # heading, corrected malformed @t[=14] to @t[=0014], removed answered review queue section, added wife/children facts, added Succession section with Julio-Claudian dynasty info, added religious and social reform details (pontifex maximus, Lex Julia, Ara Pacis, Ludi Saeculares), added two new scholarly footnotes from World History Encyclopedia
|
status: UPDATED | Augustus | changes: fixed duplicate # heading, corrected malformed @t[=14] to @t[=0014], removed answered review queue section, added wife/children facts, added Succession section with Julio-Claudian dynasty info, added religious and social reform details (pontifex maximus, Lex Julia, Ara Pacis, Ludi Saeculares), added two new scholarly footnotes from World History Encyclopedia
|
||||||
|
[main 601d485] improve: Augustus
|
||||||
|
2 files changed, 100 insertions(+)
|
||||||
|
[2026-02-22 22:36:04] ✅ Committed: improve: Augustus
|
||||||
|
[2026-02-22 22:36:04] Done (56s) — UPDATED
|
||||||
|
[2026-02-22 22:36:09] [5/66] Next up...
|
||||||
|
[2026-02-22 22:36:09] ━━━ [Ramesses II] (009548) reviews=0 garbage=0 ━━━
|
||||||
|
[2026-02-22 22:36:09] 🧹 Bash cleanup applied
|
||||||
|
[2026-02-22 22:36:09] 🔍 Enrichment + review pass
|
||||||
|
|||||||
@@ -269,13 +269,36 @@ STEPS — work through in order, skip any that do not apply:
|
|||||||
encyclopedias. Add any new facts not already present, with source citations, following
|
encyclopedias. Add any new facts not already present, with source citations, following
|
||||||
factbase authoring conventions. Do NOT add speculative or poorly-sourced claims.
|
factbase authoring conventions. Do NOT add speculative or poorly-sourced claims.
|
||||||
|
|
||||||
4. IMPROVEMENT IDEAS:
|
4. FACTBASE TOOL BUGS & FEATURE REQUESTS:
|
||||||
If you notice friction or gaps in factbase tools, file a Vikunja feature request:
|
A key purpose of this improvement loop is to make factbase itself better. As you work
|
||||||
|
through steps 1-3, pay close attention to how the factbase tools behave. File a Vikunja
|
||||||
|
task for EACH of the following you encounter:
|
||||||
|
|
||||||
|
BUGS — any MCP tool error, unexpected result, silent failure, data corruption, or behavior
|
||||||
|
that contradicts the tool description. Use title prefix "[factbase]".
|
||||||
|
|
||||||
|
FEATURES — any friction, missing capability, awkward multi-step workaround, or gap that
|
||||||
|
slowed you down or limited what you could accomplish. Use title prefix "[factbase][feature]".
|
||||||
|
|
||||||
|
CRITICAL: Factbase is domain-agnostic — it is used for many different knowledge bases, not
|
||||||
|
just this one. Your bug reports and feature requests MUST describe the issue in terms of
|
||||||
|
factbase tool behavior, not in terms of ancient history content. For example:
|
||||||
|
- GOOD: "apply_review_answers corrupts footnote references when document has >5 citations"
|
||||||
|
- BAD: "apply_review_answers broke the Ramesses II document"
|
||||||
|
- GOOD: "[feature] check_repository should detect duplicate entities across different folders"
|
||||||
|
- BAD: "[feature] factbase should know that Babylon appears in both civilizations/ and cities/"
|
||||||
|
|
||||||
|
Include in every task description:
|
||||||
|
- Which MCP tool was called (or which tool is missing/needed)
|
||||||
|
- What you expected to happen vs what actually happened
|
||||||
|
- Specific parameters or conditions that triggered the issue
|
||||||
|
- For features: the general use case and how it would help ANY factbase, not just this one
|
||||||
|
|
||||||
curl -s -X PUT "https://vikunja.home.everyonce.com/api/v1/projects/2/tasks" \
|
curl -s -X PUT "https://vikunja.home.everyonce.com/api/v1/projects/2/tasks" \
|
||||||
-H "Authorization: Bearer tk_ff251f3d3512775c71913bc2f8ec0dabbf5016a8" \
|
-H "Authorization: Bearer tk_ff251f3d3512775c71913bc2f8ec0dabbf5016a8" \
|
||||||
-H "Content-Type: application/json" \
|
-H "Content-Type: application/json" \
|
||||||
-d '"'"'{"title":"[factbase][feature] <summary>","description":"<details>","priority":2}'"'"'
|
-d '"'"'{"title":"[factbase] <summary>","description":"<detailed description>","priority":3}'"'"'
|
||||||
Only file genuinely useful improvements, not duplicates.
|
Priority: 1-2 for features/minor issues, 3 for normal bugs, 4-5 for data loss or blocking failures.
|
||||||
|
|
||||||
RULES:
|
RULES:
|
||||||
- Use update_document to edit — be surgical, change only what needs changing
|
- Use update_document to edit — be surgical, change only what needs changing
|
||||||
@@ -386,7 +409,7 @@ run_deep_scan() {
|
|||||||
log "🔬 Running deep cross-document validation scan..."
|
log "🔬 Running deep cross-document validation scan..."
|
||||||
local output
|
local output
|
||||||
output=$(kiro-cli chat --trust-all-tools --no-interactive --model "$MODEL" \
|
output=$(kiro-cli chat --trust-all-tools --no-interactive --model "$MODEL" \
|
||||||
"Run check_repository with deep_check=true. Review any new issues found — answer what you can, defer what you cannot. Then commit." 2>&1) || {
|
"Run check_repository with deep_check=true. Review any new issues found — answer what you can, defer what you cannot. If any factbase tool behaves unexpectedly during this process, file a bug to Vikunja (project 2, Authorization: Bearer tk_ff251f3d3512775c71913bc2f8ec0dabbf5016a8). Describe issues in domain-agnostic terms — factbase is used for many knowledge bases, not just this one." 2>&1) || {
|
||||||
log "❌ Deep scan agent failed, continuing..."
|
log "❌ Deep scan agent failed, continuing..."
|
||||||
return 1
|
return 1
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -7,3 +7,6 @@
|
|||||||
[2026-02-22T22:35:02+00:00] 25cc1e | Hammurabi
|
[2026-02-22T22:35:02+00:00] 25cc1e | Hammurabi
|
||||||
status: UPDATED | Hammurabi | changes: Fixed duplicate # heading; corrected stele material from "basalt" to "diorite" (per Louvre and scholarly sources); added father Sin-Muballit; expanded Achievements with specific conquered city-states (Larsa, Eshnunna, Mari) and Assyrian tribute; enriched Code of Hammurabi section with Akkadian cuneiform language, prologue/epilogue structure (~4,130 lines), and Shamash relief description; added World History Encyclopedia as [^3] source
|
status: UPDATED | Hammurabi | changes: Fixed duplicate # heading; corrected stele material from "basalt" to "diorite" (per Louvre and scholarly sources); added father Sin-Muballit; expanded Achievements with specific conquered city-states (Larsa, Eshnunna, Mari) and Assyrian tribute; enriched Code of Hammurabi section with Akkadian cuneiform language, prologue/epilogue structure (~4,130 lines), and Shamash relief description; added World History Encyclopedia as [^3] source
|
||||||
duration: 59s
|
duration: 59s
|
||||||
|
[2026-02-22T22:36:04+00:00] 4fb0a1 | Augustus
|
||||||
|
status: UPDATED | Augustus | changes: fixed duplicate # heading, corrected malformed @t[=14] to @t[=0014], removed answered review queue section, added wife/children facts, added Succession section with Julio-Claudian dynasty info, added religious and social reform details (pontifex maximus, Lex Julia, Ara Pacis, Ludi Saeculares), added two new scholarly footnotes from World History Encyclopedia
|
||||||
|
duration: 56s
|
||||||
|
|||||||
@@ -43,3 +43,5 @@
|
|||||||
{"level":"info","message":"Starting MCP server","service":"mcp-puppeteer","timestamp":"2026-02-22 22:35:10.075"}
|
{"level":"info","message":"Starting MCP server","service":"mcp-puppeteer","timestamp":"2026-02-22 22:35:10.075"}
|
||||||
{"level":"info","message":"MCP server started successfully","service":"mcp-puppeteer","timestamp":"2026-02-22 22:35:10.078"}
|
{"level":"info","message":"MCP server started successfully","service":"mcp-puppeteer","timestamp":"2026-02-22 22:35:10.078"}
|
||||||
{"level":"info","message":"Puppeteer MCP Server closing","service":"mcp-puppeteer","timestamp":"2026-02-22 22:36:03.326"}
|
{"level":"info","message":"Puppeteer MCP Server closing","service":"mcp-puppeteer","timestamp":"2026-02-22 22:36:03.326"}
|
||||||
|
{"level":"info","message":"Starting MCP server","service":"mcp-puppeteer","timestamp":"2026-02-22 22:36:12.664"}
|
||||||
|
{"level":"info","message":"MCP server started successfully","service":"mcp-puppeteer","timestamp":"2026-02-22 22:36:12.668"}
|
||||||
|
|||||||
@@ -30,9 +30,7 @@ Ramesses II (~1279–1213 BCE), also known as Ramesses the Great, was the third
|
|||||||
[^2]: Tyldesley, J. *Ramesses: Egypt's Greatest Pharaoh* (2000)
|
[^2]: Tyldesley, J. *Ramesses: Egypt's Greatest Pharaoh* (2000)
|
||||||
---
|
---
|
||||||
|
|
||||||
## Review Queue
|
|
||||||
|
|
||||||
<!-- factbase:review -->
|
|
||||||
- [x] `@q[temporal]` Line 10: "Reign: ~1279–1213 BCE (66 years, one of the longest in Egyptian history)" - when was this true?
|
- [x] `@q[temporal]` Line 10: "Reign: ~1279–1213 BCE (66 years, one of the longest in Egyptian history)" - when was this true?
|
||||||
> Static historical fact. 1279-1213 BCE. No temporal tag needed.
|
> Static historical fact. 1279-1213 BCE. No temporal tag needed.
|
||||||
- [x] `@q[temporal]` Line 11: "Dynasty: 19th Dynasty, New Kingdom" - when was this true?
|
- [x] `@q[temporal]` Line 11: "Dynasty: 19th Dynasty, New Kingdom" - when was this true?
|
||||||
|
|||||||
Reference in New Issue
Block a user