Skip to content

feat: explain fact review decisions#27

Merged
cafitac merged 1 commit intomainfrom
feat/conflict-decision-explanations
Apr 30, 2026
Merged

feat: explain fact review decisions#27
cafitac merged 1 commit intomainfrom
feat/conflict-decision-explanations

Conversation

@cafitac
Copy link
Copy Markdown
Owner

@cafitac cafitac commented Apr 30, 2026

Summary

  • add agent-memory review explain fact to combine fact status, default retrieval visibility, same claim-slot alternatives, transition history, and replacement chain into one forensic decision context
  • reuse existing transition history and supersedes/replaces relation data without changing default approved-only retrieval behavior
  • document the new review surface in README, install-smoke docs, and current handoff

Test Plan

  • uv run pytest tests/test_cli.py::test_python_module_cli_review_explain_fact_shows_decision_context -q
  • uv run pytest tests/test_review_and_scope_ranking.py tests/test_cli.py -q
  • uv run pytest tests/ -q
  • uv run python scripts/check_release_metadata.py
  • uv run python scripts/smoke_release_readiness.py
  • npm pack --dry-run
  • git diff --check
  • node --check bin/agent-memory.js
  • static diff scan found 0 credential findings

@cafitac cafitac merged commit 75e9586 into main Apr 30, 2026
2 checks passed
@cafitac cafitac deleted the feat/conflict-decision-explanations branch April 30, 2026 07:40
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.

1 participant