Skip to content

cleanup: remove dead code - #59672

Open
rekram1-node wants to merge 1 commit into
v1from
v1-core-remove-dead-code
Open

rekram1-node wants to merge 1 commit into
v1from
v1-core-remove-dead-code

Conversation

@rekram1-node

Copy link
Copy Markdown
Contributor

Problem

packages/core had a few things nothing uses anymore, plus a stray one-line editor prompt (src/effect/dfdf) that got committed by mistake.

Change

Deletion only. I grepped the whole repo (all packages, tests, scripts, the ./* export pattern) before removing each one.

 packages/core/src/
-├── effect/dfdf                  # stray editor prompt
-├── public-event-manifest.ts     # never imported
 ├── shell.ts
-│   └── killTree()               # plus its now-unused imports and timeout const
 └── util/
     ├── binary.ts
-    │   └── Binary.insert()
     ├── hash.ts
-    │   └── Hash.sha256()
     └── path.ts
-        └── getFileExtension()

I also dropped the unused imports oxlint flags in packages/core (src and test). Bun strips unused imports at runtime and in bun build, even with verbatimModuleSyntax, so none of these were loading anything. I checked that with a tiny side-effect repro.

Out of scope

Unused variables oxlint reports in core (like aisdk and plugin). I only touched imports.

Verification

  • bun typecheck in packages/core, packages/opencode, packages/app and packages/session-ui: pass
  • bun test test/shell.test.ts test/npm.test.ts test/project.test.ts in packages/core: 21 pass

@rekram1-node rekram1-node changed the title chore(core): remove dead code cleanup: remove dead code Oct 27, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant