Skip to content

refactor(core): inline portable shell scanner - #44026

Merged
kitlangton merged 1 commit into
v2from
shell-scan-core
Aug 22, 2026
Merged

refactor(core): inline portable shell scanner#44026
kitlangton merged 1 commit into
v2from
shell-scan-core

Conversation

@kitlangton

Copy link
Copy Markdown
Contributor

What

Moves the portable shell scanner from the unpublished @opencode-ai/shell-scan workspace package into Core, where it is consumed.

This removes an unpublished package from Core's runtime dependency graph without changing scanner behavior or the experimental feature's opt-in status.

How

  • Moves the scanner implementation to packages/core/src/shell/scan.ts.
  • Moves the scanner regression corpus to packages/core/test/shell-scan/.
  • Updates Core's lazy runtime import and parity test to use the local module.
  • Removes the standalone package, lockfile entry, Core dependency, and root publish step.

Scope

Testing

  • bun install --frozen-lockfile
  • bun run test test/shell-scan test/shell-parse.test.ts test/shell-parse-parity.test.ts test/tool-shell.test.ts from packages/core
  • bun run test from packages/core: 2,220 passed, 16 skipped
  • bun typecheck from packages/core
  • bun run build from packages/core
  • bun run lint from the repository root: 0 errors, existing warnings only
  • Push hook repository typecheck: 32 successful packages
  • bun pm pack from packages/core, verifying dist/shell/{parse,scan}.js and matching declarations are included

@kitlangton
kitlangton enabled auto-merge (squash) August 21, 2026 23:57
@kitlangton
kitlangton merged commit f69f78e into v2 Aug 22, 2026
8 checks passed
@kitlangton
kitlangton deleted the shell-scan-core branch August 22, 2026 00:06
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