Skip to content

refactor(core): unify accepted note runtime boundaries - #1102

Merged
phernandez merged 2 commits into
mainfrom
codex/1096-1098-accepted-note-runtime
Jul 18, 2026
Merged

refactor(core): unify accepted note runtime boundaries#1102
phernandez merged 2 commits into
mainfrom
codex/1096-1098-accepted-note-runtime

Conversation

@phernandez

@phernandez phernandez commented Jul 18, 2026

Copy link
Copy Markdown
Member

Why

Accepted-note create, update, and edit could be composed from partial persistence helpers, preparation instantiated the full EntityService graph, and hosted runtime code depended on cloud/local implementation packages instead of neutral core owners.

What

  • add one complete accepted-note snapshot persistence operation for entity content, graph rows, and search state
  • keep move persistence explicitly narrower so path changes do not replace the graph
  • extract immutable preparation values and lightweight preparation functions with explicit dependencies
  • move accepted-note repository/search cleanup values and shared runtime contracts to neutral packages
  • retain thin compatibility exports for downstream transition
  • document and enforce the resulting dependency direction

Verification

  • focused accepted-note, preparation, route, and architecture tests: 84 passed
  • SQLite pre-materialization atomicity regression: passed
  • targeted Postgres create/update/edit/move routes: 9 passed
  • just fast-check
  • just doctor
  • just package-check

Companion

Hosted imports and composition are updated in basicmachines-co/basic-memory-cloud#1548, pinned to this core commit.

Closes #1096
Closes #1097
Closes #1098

Signed-off-by: phernandez <paul@basicmachines.co>
Signed-off-by: phernandez <paul@basicmachines.co>
@phernandez
phernandez force-pushed the codex/1096-1098-accepted-note-runtime branch from 79d3a9f to 613fd02 Compare July 18, 2026 15:22
@phernandez
phernandez marked this pull request as ready for review July 18, 2026 15:25
@phernandez
phernandez merged commit d380b5e into main Jul 18, 2026
22 checks passed
@phernandez
phernandez deleted the codex/1096-1098-accepted-note-runtime branch July 18, 2026 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant