Skip to content

feat(memory): move Memory into core - #1868

Open
sentry-junior[bot] wants to merge 8 commits into
mainfrom
cramer/1861-memory-core-cutover
Open

sentry-junior[bot] wants to merge 8 commits into
mainfrom
cramer/1861-memory-core-cutover

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Closes #1861.

Move Memory into @sentry/junior as one atomic product cutover.

  • move Memory runtime, schema, and migration ownership into core
  • enable Memory in app and CLI composition without plugin discovery
  • add JuniorAppOptions.memory and the @sentry/junior/memory export
  • preserve existing migration journals and data adoption
  • keep @sentry/junior-memory as a thin compatibility package
  • update the example app, evals, tests, and docs to use core Memory

Deferred cleanup: shared utility consolidation, compatibility package removal, and old route removal.

Verification:

  • full @sentry/junior test suite
  • @sentry/junior-memory tests (95)
  • core, compatibility package, and eval typechecks
  • core and compatibility package lint
  • production example Nitro build
  • file length, migration metadata, test architecture, release alignment, and diff checks

via David Cramer.

--

View Junior Session [Sentry]

Co-Authored-By: David Cramer <david@sentry.io>
@vercel

vercel Bot commented Sep 16, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
junior-docs Ready Ready Preview Sep 16, 2026 7:46pm UTC

Request Review

@blacksmith-sh

This comment has been minimized.

Co-Authored-By: David Cramer <david@sentry.io>
@dcramer
dcramer marked this pull request as ready for review September 16, 2026 18:38
@github-actions github-actions Bot added the risk: high PR risk score: high label Sep 16, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread packages/junior-evals/src/behavior-harness.ts
Comment thread packages/junior/src/db/schema.ts
Co-Authored-By: David Cramer <david@sentry.io>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7d65a00. Configure here.

Comment thread packages/junior/migrations/0043_ordinary_young_avengers.sql Outdated
Comment thread packages/junior-memory/tests/storage.test.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move Memory fully into Junior core

0 participants