Skip to content

Add reviewed memory correction path - #36

Open
taditya1997 wants to merge 1 commit into
theschoolofai:mainfrom
taditya1997:taditya1997-memory-corrections
Open

taditya1997 wants to merge 1 commit into
theschoolofai:mainfrom
taditya1997:taditya1997-memory-corrections

Conversation

@taditya1997

Copy link
Copy Markdown

Capability

Adds a reviewed durable-memory correction path. Explicit Correction: prompts now supersede the matching same-scope current fact, preserve the old fact as history, expose provenance through memory search, and require review for ambiguous corrections.

Proof

  • uv run ruff check . -> All checks passed!
  • uv run pytest -q -> 47 passed, 1 warning
  • PYTHONPATH=../S13Code uv run pytest -q in S13Proof -> 3 passed
  • PYTHONPATH=../S13Code uv run python run_a2a_proof.py --output a2a-proof.json completed with wait/resume trace.

Boundary notes

  • No provider keys, .env, generated DBs, personal memory, or unrestricted local paths are committed.
  • Proof identities are synthetic: tenant acme, project family, user rohan.
  • S13Code still calls the gateway seam for LLM work; the extension is inside scoped memory/runtime contracts.

@theschoolofai theschoolofai left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved for a reviewed memory-correction path with full floor tests and provenance; duplicate-contribution ordering is considered against earlier contradiction work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants