Skip to content

storage: retire legacy execution writer exports - #1744

Merged
likun666661 merged 1 commit into
apache:mainfrom
likun666661:agent/retire-legacy-execution-writers
Aug 1, 2026
Merged

likun666661 merged 1 commit into
apache:mainfrom
likun666661:agent/retire-legacy-execution-writers

Conversation

@likun666661

Copy link
Copy Markdown
Member

Part of #1649.

Summary

  • remove the retired AgentRun, RuntimeEvent, and ShellRun file-writer constructors from the production @maka/storage entrypoint
  • isolate legacy writer construction behind an explicitly named test-support subpath for import and compatibility fixtures
  • expose RuntimeEvent compatibility readers through a frozen read-only facade for both legacy JSONL and SQLite
  • update cross-package compatibility tests to use the explicit legacy fixture boundary

Invariants

  • production imports cannot obtain createAgentRunStore, createRuntimeEventStore, or createShellRunStore from @maka/storage
  • legacy codecs and file implementations remain internal so existing state can still be imported and compatibility-tested
  • RuntimeEvent readers do not expose appendRuntimeEvent, even when backed by a read-only SQLite connection
  • no production source imports the legacy test-support subpath

Verification

  • npm run build:test
  • npm run typecheck
  • npm run lint
  • npm run format:check
  • full @maka/storage test suite
  • 84/84 affected @maka/runtime tests with test concurrency 1
  • 5/5 affected @maka/headless tests with test concurrency 1
  • focused CLI, Desktop, Runtime Host, RuntimeEvent transfer, and public-surface tests

@likun666661
likun666661 marked this pull request as ready for review August 1, 2026 03:53
@likun666661
likun666661 merged commit 6ac8d80 into apache:main Aug 1, 2026
5 of 6 checks passed
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.

1 participant