Skip to content

feat: add observation review candidates#45

Merged
cafitac merged 1 commit intomainfrom
feat/observation-review-candidates
May 1, 2026
Merged

feat: add observation review candidates#45
cafitac merged 1 commit intomainfrom
feat/observation-review-candidates

Conversation

@cafitac
Copy link
Copy Markdown
Owner

@cafitac cafitac commented May 1, 2026

Summary

  • add read-only observations review-candidates for top retrieval observation refs
  • include fact review explanations, replacement-chain signals, graph summaries, and follow-up commands
  • document the dogfood review workflow and refresh the current handoff

Test Plan

  • TMPDIR=$PWD/.tmp-test uv run pytest tests/test_cli.py::test_python_module_cli_observations_review_candidates_explains_top_refs_without_mutation_or_raw_queries -q
  • TMPDIR=$PWD/.tmp-test uv run pytest tests/test_cli.py::test_python_module_cli_observations_audit_reports_frequent_and_stale_refs_without_raw_queries tests/test_cli.py::test_python_module_cli_observations_review_candidates_explains_top_refs_without_mutation_or_raw_queries tests/test_cli.py::test_python_module_cli_review_explain_fact_shows_decision_context tests/test_cli.py::test_python_module_cli_graph_inspect_returns_read_only_relation_neighborhood -q
  • PYTHONPATH=src uv run python -m agent_memory.api.cli observations review-candidates --help
  • TMPDIR=$PWD/.tmp-test 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
  • temp-DB observations review-candidates smoke with secret-like query strings; verified no secret-like text in output
  • static diff secret scan: finding_count: 0

@cafitac cafitac merged commit 727c560 into main May 1, 2026
2 checks passed
@cafitac cafitac deleted the feat/observation-review-candidates branch May 1, 2026 01:25
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