Merge pull request 'docs: Improve KNOWN_VIOLATIONS justifications for SOUL.md alignment' (#267) from feature/improve-sovereignty-justification into main
Some checks failed
Forge CI / smoke-and-build (push) Failing after 41s

Merge PR #267: docs: Improve KNOWN_VIOLATIONS justifications for SOUL.md alignment
This commit was merged in pull request #267.
This commit is contained in:
2026-04-10 09:35:51 +00:00

View File

@@ -91,13 +91,17 @@ NETWORK_FUNC_NAMES = {
# Each entry maps to a reason string.
KNOWN_VIOLATIONS = {
"tools/graph_store.py": (
"GraphStore persists to Gitea via API — not part of core memory path "
"(MEMORY.md/USER.md/SQLite). Excluded from sovereignty gate."
"GraphStore persists to Gitea via API. This is a known architectural trade-off "
"for knowledge graph persistence, which is not part of the core memory path "
"(MEMORY.md/USER.md/SQLite). Future work will explore local-first alternatives "
"to align more closely with SOUL.md principles."
),
"tools/session_search_tool.py": (
"Session search uses LLM summarization (auxiliary client) to generate "
"summaries. The FTS5 search itself is local; the LLM call is for "
"presentation, not storage. Known architectural trade-off."
"Session search uses LLM summarization via an auxiliary client. While the FTS5 "
"search is local, the LLM call for summarization is an external dependency. "
"This is a temporary architectural trade-off for enhanced presentation. "
"Research is ongoing to implement local LLM options for full sovereignty, "
"in line with SOUL.md."
),
}