Skip to content

ref(memory): remove viewer collection facade - #1649

Merged
dcramer merged 1 commit into
mainfrom
codex/1600-memory-viewer
Aug 23, 2026
Merged

ref(memory): remove viewer collection facade#1649
dcramer merged 1 commit into
mainfrom
codex/1600-memory-viewer

Conversation

@dcramer

@dcramer dcramer commented Aug 21, 2026

Copy link
Copy Markdown
Member

Replaces the memory viewer object facade with named archive, get, list, stats, and timeline operations used directly by the REST API and User page. Authenticated User authority, public and private visibility, archive rules, cursor behavior, ordering, filters, and response shapes stay unchanged.

This is a hard cutover with no compatibility wrapper. The refactor is a small net deletion and keeps the existing real-SQL integration coverage.

Refs #1600

@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview Aug 23, 2026 3:03am

Request Review

@dcramer
dcramer force-pushed the feat/web-memory-linked-identities branch 2 times, most recently from 20e8bdb to b9db2a6 Compare August 22, 2026 22:57
Base automatically changed from feat/web-memory-linked-identities to main August 23, 2026 02:24
@dcramer
dcramer force-pushed the codex/1600-memory-viewer branch from 5ab8ec9 to 8c8345f Compare August 23, 2026 02:42
@dcramer
dcramer marked this pull request as ready for review August 23, 2026 02:46
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Aug 23, 2026
@dcramer
dcramer force-pushed the codex/1600-memory-viewer branch from 8c8345f to 8d124e1 Compare August 23, 2026 03:02
@dcramer
dcramer merged commit f08f00a into main Aug 23, 2026
39 checks passed
@dcramer
dcramer deleted the codex/1600-memory-viewer branch August 23, 2026 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant