Skip to content

Land cache memo, lock and tags in feature/caches and consolidate the cache ADR - #312

Merged
tmgbedu merged 17 commits into
feature/cachesfrom
task/cache-consolidate
Oct 21, 2026
Merged

tmgbedu merged 17 commits into
feature/cachesfrom
task/cache-consolidate

Conversation

@claude-tmgbedu

Copy link
Copy Markdown

Why

#307, #309 and #310 were merged into their stacked parent branches (task/cache-memo and task/cache-lock), so Cache.lock, Cache.tags and the Redis lock never reached feature/caches. Only #306 (Cache.memo) did.

What

Content verification (against this branch)

Branch Non-merge commits missing Notes
task/cache-config, -core, -remember-decorator, -provider-facade 0 Only the stacked-PR merge commits are not ancestors
task/cache-file-store, -database-store, -redis 0 Ancestors
task/cache-memo, -lock, -tags, -redis-lock 0 Ancestors after this merge

I also checked each PR's added lines (#305, #306, #309 and #310) against this tree: every line is present. The misses in #296–#303 and #307 are all lines that later commits deliberately rewrote, such as the unittest test rewrite, the DB encode() helper, the Redis config and ADR text updates.

Validation

  • Full suite: 3287 passed, 10 skipped.
  • Cache: 241 passed.
  • ruff and basedpyright clean.

Supersedes #311.

# Conflicts:
#	docs/adr/AGENTS.md
# Conflicts:
#	docs/adr/AGENTS.md
Cache.lock(): atomic locks for array, file and database stores
Cache.tags(): namespaced tagged cache with tagged flush
@codecov

codecov Bot commented Oct 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@tmgbedu
tmgbedu merged commit 606df9d into feature/caches Oct 21, 2026
10 checks passed
@tmgbedu
tmgbedu deleted the task/cache-consolidate branch October 21, 2026 20:34
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