refactor(schema): remove unused session.compacted event by rekram1-node · Pull Request #57167 · anomalyco/opencode · GitHub
Skip to content

refactor(schema): remove unused session.compacted event - #57167

Merged
rekram1-node merged 1 commit into
v2.1from
remove-compacted-event
Oct 19, 2026
Merged

rekram1-node merged 1 commit into
v2.1from
remove-compacted-event

Conversation

@rekram1-node

Copy link
Copy Markdown
Collaborator

Problem

session.compacted is still defined in @opencode/schema and registered in the event manifest, but nothing publishes it. It came over from the V1 package when event definitions moved into packages/schema (#33579); V1 compaction emitted it, and that package has since been removed. Compaction now reports through session.compaction.started, .delta, .ended, and .failed.

Change

 packages/schema/src/
-├── session-compaction-event.ts    # session.compacted, never published
 └── event-manifest.ts
-      ...SessionCompactionEvent.Definitions,
  • Generated OpenAPI and clients: no change. The event was never part of them, and check:generated passes without regenerating.
  • Subscribers: none in this repo. An external subscriber never received the event, so nothing changes at runtime.
  • Stored data: none. The event was ephemeral.

Out of scope

  • session.compacted in the docs' V1 plugin event list (packages/web/src/content/docs/**/plugins.mdx), which describes V1 plugins.

Verification

Package Command Result
schema bun test (includes event-manifest.test.ts) 85 pass
server bun test 85 pass
core bun test test/worktree.test.ts 35 pass
schema, core, server, client, protocol bun typecheck clean
protocol, client, services/www bun run check:generated ok

@rekram1-node
rekram1-node merged commit 4acb7ff into v2.1 Oct 19, 2026
19 checks passed
@rekram1-node
rekram1-node deleted the remove-compacted-event branch October 19, 2026 19:54
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