Skip to content

feat(recall): add state_mode=current|history recall alias - #173

Merged
jack-arturo merged 2 commits into
mainfrom
pr/recall-state-mode
Jun 6, 2026
Merged

feat(recall): add state_mode=current|history recall alias#173
jack-arturo merged 2 commits into
mainfrom
pr/recall-state-mode

Conversation

@jack-arturo

@jack-arturo jack-arturo commented Jun 6, 2026

Copy link
Copy Markdown
Member

Summary

Adds a public state_mode=current|history recall query param as a semantic alias over the boolean current_only (#170).

  • Resolution order: explicit current_only wins → else state_mode → else default current.
  • Unknown state_mode values return 400, validated before precedence is applied.
  • The resolved mode is echoed in the response ("state_mode": ...), always present in both modes.
  • state_mode is a pure presentation alias — the downstream current-state filter still reads only the boolean, so there is no behavior change to filtering and the param is fully backward compatible.

Tooling

  • New Tier-0 no-LLM smoke gate: make bench-current-state (pytest -k 'recall_state_mode or recall_current_only').
  • tests/test_makefile_targets.py locks that target as a deterministic, no-LLM pytest gate.

Tests

  • pytest tests/test_api_endpoints.py tests/test_makefile_targets.py → 104 passed / 1 skipped
  • make bench-current-state → 19 passed

Follow-up

🤖 Generated with Claude Code

@jack-arturo @claude
feat(recall): add state_mode=current|history recall alias
25b6e98 
Introduce a public state_mode=current|history query param as a semantic
alias over the boolean current_only (#170). Resolution order: an explicit
current_only wins; otherwise state_mode drives it; otherwise default to
current. Unknown state_mode values are rejected with 400, validated before
precedence is applied. The resolved mode is echoed in the response.

state_mode is a pure presentation alias — the downstream current-state
filter still reads only the boolean, so there is no behavior change to
filtering and the param is fully backward compatible.

Adds a Tier-0 no-LLM smoke gate (make bench-current-state) plus a meta-test
locking that target as a deterministic, no-LLM pytest gate.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 6, 2026 11:30
Copilot AI reviewed Jun 6, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a public state_mode=current|history query parameter to /recall as a semantic alias over current_only, validates unknown values with 400 (before precedence), and always echoes the resolved state_mode in successful responses.

Changes:

  • Implement state_mode parsing/validation + precedence with current_only, and echo state_mode in the recall response.
  • Add API endpoint tests covering defaulting, history behavior, precedence, and invalid state_mode rejection.
  • Add a deterministic Tier-0 smoke target (make bench-current-state) and document it in testing/benchmark docs (plus a test that locks the Makefile target’s no-LLM intent).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
automem/api/recall.py Parse/validate state_mode, resolve precedence with current_only, and include state_mode in the JSON response.
tests/test_api_endpoints.py Add coverage for state_mode defaulting, history mode, precedence, and invalid values.
Makefile Add bench-current-state target running a focused, no-LLM pytest subset.
tests/test_makefile_targets.py Assert the bench-current-state Makefile target remains a no-LLM pytest gate.
docs/API.md Document state_mode contract, precedence, and response echo.
docs/TESTING.md Document make bench-current-state as the deterministic regression gate for recall state semantics.
benchmarks/EXPERIMENT_LOG.md Record bench-current-state as a Tier-0 benchmark/gate for state contract changes.

@jack-arturo
jack-arturo merged commit b1df86c into main Jun 6, 2026
7 checks passed
@jack-arturo
jack-arturo deleted the pr/recall-state-mode branch June 6, 2026 13:09
@jack-arturo jack-arturo mentioned this pull request Jun 6, 2026
Merged
jack-arturo added a commit that referenced this pull request Jun 26, 2026
@jack-arturo
chore(main): release 0.16.0 (#154)
8c67c16 
🤖 I have created a release *beep* *boop*
---


##
[0.16.0](v0.15.2...v0.16.0)
(2026-06-26)


### Features

* **api:** add admin backup endpoint
([#162](#162))
([8b1f264](8b1f264))
* **api:** support bulk memory associations
([1221e36](1221e36))
* **api:** support bulk memory associations
([#198](#198))
([28eb916](28eb916))
* **benchmarks:** LongMemEval failure-mode diagnosis harness + judge
quota preflight
([#183](#183))
([f99bece](f99bece))
* **consolidation:** expose cluster threshold and min size as env vars
([#163](#163))
([7e731f3](7e731f3))
* **enrichment:** expose classification fallback-rate metrics in
/enrichment/status
([#188](#188))
([0b522a9](0b522a9))
* **entity:** harden identity cleanup and repair tooling
([#176](#176))
([827dfbc](827dfbc))
* **eval:** recall-quality optimization harness — lab foundation +
design ([#197](#197))
([431433e](431433e))
* **graph:** support unbounded visualizer snapshots
([#141](#141))
([c730128](c730128))
* **lab:** add aged labelled distractor injection
([cc5d546](cc5d546))
* **lab:** add config_complexity simplicity metric
([dfb10d9](dfb10d9))
* **lab:** add distractor_rate_at_k precision guardrail metric
([872eab2](872eab2))
* **lab:** add lab_corpus with parameterized recall
([5e1e071](5e1e071))
* **lab:** add pick_winner scorecard decision rule
([3187eac](3187eac))
* **lab:** add real consolidation pass helper
([48a7d4a](48a7d4a))
* **lab:** isolate production clone restores
([#171](#171))
([aef90c0](aef90c0))
* **lab:** wire scorecard, distractors, recall params, consolidation
into runner
([589ec30](589ec30))
* **recall:** add metadata sidecar search
([#177](#177))
([4e7956e](4e7956e))
* **recall:** add state_mode=current|history recall alias
([#173](#173))
([b1df86c](b1df86c))
* **recall:** cap tag-score denominator to fix query-length bias
([#193](#193))
([cefa516](cefa516))
* **recall:** date-aware ranking + latest-fact selection
([#158](#158),
[#159](#159))
([#187](#187))
([a6ed945](a6ed945))
* **recall:** make recency decay window and curve configurable
([#182](#182))
([dbb933f](dbb933f))
* **recall:** ranking release — recency config, tag-score cap, relevance
gate, date-aware ranking
([#182](#182),
[#193](#193),
[#186](#186),
[#187](#187),
[#183](#183),
[#184](#184),
[#188](#188))
([#194](#194))
([337fe98](337fe98))
* **scripts:** safer reclassify_with_llm.py with provider flags +
tighter prompt
([#164](#164))
([a742602](a742602))


### Bug Fixes

* **api:** address copilot review on PR
[#198](#198)
([0466a1e](0466a1e))
* **api:** handle grouped association write failures
([cd93df9](cd93df9))
* **backup:** make backup_automem.py runnable as `python
scripts/backup_automem.py`
([#175](#175))
([edd9742](edd9742))
* **benchmarks:** add publication verification bundle
([#166](#166))
([420d721](420d721))
* **consolidation:** skip eager first tick at startup to avoid FalkorDB
load race
([#165](#165))
([1b812cf](1b812cf))
* **docs:** keep dispatch payload arrays stable
([df6e9e8](df6e9e8))
* **embedding:** fall back to per-item real embeddings before
placeholders in batch path
([#189](#189))
([6e9c62c](6e9c62c))
* **entity:** restore person-shape exemption on the slug validation path
([#179](#179))
([5e29960](5e29960))
* **entity:** stop validator over-rejecting real people, code tools, and
event categories
([#178](#178))
([193b730](193b730))
* **lab:** address copilot review on PR
[#197](#197)
([45f80d6](45f80d6))
* **lab:** align scorecard key contract (build_scorecard -&gt;
pick_winner)
([7d91530](7d91530))
* **mcp-sse:** decouple /health liveness from upstream readiness
([#151](#151))
([5bcfb8b](5bcfb8b))
* **mcp:** cap association failure summary
([ea4e08f](ea4e08f))
* **mcp:** surface stored metadata and updated_at in detailed recall
format ([#184](#184))
([230416e](230416e))
* **recall:** address copilot review on PR
[#194](#194)
([50b1647](50b1647))
* **recall:** canonicalize / and : separators in context_tag matching
([3afd9d3](3afd9d3))
* **recall:** canonicalize / and : separators in context_tag matching
([#203](#203))
([ba5e9ff](ba5e9ff))
* **recall:** gate query-independent scoring on topical evidence within
tag scope
([#130](#130))
([#186](#186))
([c11b594](c11b594))
* **recall:** hydrate semantic recall summaries
([#192](#192))
([76e845d](76e845d))
* **recall:** normalize graph keyword scores into the 0-1 component
range ([#191](#191))
([3653ddf](3653ddf))
* **recall:** respect current memory state
([#170](#170))
([ed36b98](ed36b98)),
closes [#169](#169)
[#158](#158)
[#159](#159)
* **scripts:** add sys.path guard to reembed_embeddings.py
([d333cf0](d333cf0))


### Documentation

* add scripts catalog, recall-quality-lab guide, and 0.16.0 migrations
([f20c664](f20c664))
* **bench:** log full judged 500q LongMemEval ship-config run with churn
attribution
([41bf8d0](41bf8d0))
* **eval:** Plan A — lab metric foundation (TDD, 9 tasks)
([0087dda](0087dda))
* **eval:** Plan B — parallel matrix harness (TDD, 9 tasks)
([c8ddfb2](c8ddfb2))
* **evals:** mark Memora/FAMA/WRIT lifecycle diagnostics as
diagnostic-only
([#174](#174))
([e8a3285](e8a3285))
* **eval:** spec for recall-quality optimization harness
([b1a1995](b1a1995))
* fix stale claims and document gated flags for 0.16.0
([b152d64](b152d64))
* note develop-branch contribution policy in README
([ccf02dd](ccf02dd))
* **positioning:** add scout reference
([#168](#168))
([922d23b](922d23b))
* refresh benchmark currency for the neutral AMB run and prune stale
archive docs
([3ff95bd](3ff95bd))
* refresh benchmark currency for the neutral AMB run and prune stale
archive docs
([#204](#204))
([89c30e0](89c30e0))
* refresh README and benchmark guidance
([#157](#157))
([bba31cc](bba31cc))
* **runtime:** align Docker viewer paths and setup guidance
([#155](#155))
([bbda79b](bbda79b))
* scripts catalog, recall quality lab guide, and 0.16.0 migration
runbook ([#199](#199))
([f190ae5](f190ae5))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
@jack-arturo jack-arturo mentioned this pull request Aug 28, 2026
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Balanced
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jack-arturo