Skip to content

refactor(memory): trim unused Windows spawn exports#101594

Merged
vincentkoc merged 1 commit into
mainfrom
refactor/deadcode-all-surfaces-20260707-v40
Jul 7, 2026
Merged

refactor(memory): trim unused Windows spawn exports#101594
vincentkoc merged 1 commit into
mainfrom
refactor/deadcode-all-surfaces-20260707-v40

Conversation

@vincentkoc

Copy link
Copy Markdown
Member

What Problem This Solves

The private memory host runtime bridge still forwarded seven low-level Windows spawn candidate symbols that had no package or repository consumers.

Why This Change Was Made

Remove the unused candidate-resolution functions and types from the broad runtime and IO facades. The supported memory-host Windows spawn facade keeps its high-level program resolution/materialization API unchanged.

User Impact

No user-visible behavior changes. The private memory host bridge exposes a smaller, more accurate Windows spawn surface.

Evidence

  • ts-unused-exports identified exactly the seven removed symbols in openclaw-runtime-io.ts.
  • Blacksmith Testbox tbx_01kwy3qtz3r2cyh9hcgr4ze7x1:
    • pnpm check:changed -- packages/memory-host-sdk/src/host/openclaw-runtime-io.ts packages/memory-host-sdk/src/host/openclaw-runtime.ts passed.
    • 23 QMD process tests and 8 package-boundary contract tests passed after the final live-main rebase.
    • oxfmt --check passed for both changed files.
    • unused-file and dependency scans passed; all five duplicate scan lanes reported zero clones.
    • plugin boundary summary reported zero eligible compatibility removals and zero unused reserved imports.
  • Fresh Codex autoreview reported no actionable findings at 0.87 confidence.
  • Fresh iOS Periphery dead-code scan passed: https://github.com/openclaw/openclaw/actions/runs/28861053671
  • Android private-declaration and UI exported-singleton scans found no actionable dead code.
  • git diff --check passed.

AI-assisted: yes. The implementation and validation evidence were reviewed before submission; transcript attachment was declined.

@vincentkoc vincentkoc self-assigned this Jul 7, 2026
@openclaw-barnacle openclaw-barnacle Bot added size: XS maintainer Maintainer-authored PR labels Jul 7, 2026
@vincentkoc

Copy link
Copy Markdown
Member Author

Land-ready verification complete.

Change:

  • Removed three unused low-level Windows spawn functions and four unused related types from the private memory-host broad runtime/IO facades.
  • Retained the supported memory-host Windows spawn facade: resolveWindowsSpawnProgram, materializeWindowsSpawnProgram, and its three consumed types.
  • Diff: 2 files, -14 lines. No runtime behavior changes.

Proof:

  • Blacksmith Testbox provider, lease tbx_01kwy3qtz3r2cyh9hcgr4ze7x1.
  • corepack pnpm check:changed -- packages/memory-host-sdk/src/host/openclaw-runtime-io.ts packages/memory-host-sdk/src/host/openclaw-runtime.ts passed.
  • corepack pnpm test:serial packages/memory-host-sdk/src/host/qmd-process.test.ts src/plugins/contracts/extension-package-project-boundaries.test.ts passed: 31 tests.
  • corepack pnpm exec oxfmt --check packages/memory-host-sdk/src/host/openclaw-runtime-io.ts packages/memory-host-sdk/src/host/openclaw-runtime.ts passed.
  • ts-unused-exports identified exactly the seven removed facade symbols.
  • Unused-file and dependency scans passed; all five duplicate lanes reported zero clones.
  • Plugin-boundary summary reported zero eligible compatibility removals and zero unused reserved imports.
  • Fresh iOS Periphery dead-code run passed: https://github.com/openclaw/openclaw/actions/runs/28861053671
  • Fresh Codex autoreview reported no actionable findings at 0.87 confidence.
  • Codebase-memory graph refreshed after the final live-main rebase: 281,014 nodes / 1,138,118 edges.
  • git diff --check passed.

Known proof gaps: none for this behavior-neutral private-surface cleanup.

@vincentkoc
vincentkoc marked this pull request as ready for review July 7, 2026 11:17
@vincentkoc
vincentkoc merged commit 988f30e into main Jul 7, 2026
111 of 113 checks passed
@vincentkoc
vincentkoc deleted the refactor/deadcode-all-surfaces-20260707-v40 branch July 7, 2026 11:17
@vincentkoc

Copy link
Copy Markdown
Member Author

Merged via squash.

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

Labels

maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant