Skip to content

Audit and eliminate DIFF-SYNC stub files (59 disposition misclassifications) #2192

@alexey-pelykh

Description

@alexey-pelykh

Problem

DIFF-SYNC batches B1-B4 created 59 stub files (<=20 lines each, marked "Gutted in RemoteClaw fork") that didn't exist in the foundation. 183 files total contain gutted markers. Each stub is a band-aid: upstream code was INCLUDE'd that imports from gutted subsystems, then stubs were created to make it compile.

This is too much code for a middleware fork. Every stub represents a file that should have been EXCLUDE'd in disposition.tsv, not shimmed.

Scope

For each of the 59 stub files:

  1. Trace who imports it (grep -rn "from.*{stub-path}" src/)
  2. Determine if the importer should exist in the fork
  3. If NO: reclassify importer as EXCLUDE in disposition.tsv, delete the stub
  4. If YES: the stub may be legitimately needed — document why

Stub files by domain (59 total)

Domain Count Key stubs
src/agents/ 14 model-fallback, model-selection, skills, sandbox, defaults, synthetic/together/venice/huggingface-models
src/commands/ 11 auth-choice.apply.*, doctor-memory/sandbox, model-picker, onboard-skills, auth-token, models
src/infra/ 7 exec-safety, exec-safe-bin-*, skills-remote, system-run-approval-binding
src/config/ 6 sessions, types.acp/memory/sandbox/skills, normalize-exec-safe-bin
src/memory/ 4 index, internal, status-format, types (entire directory is stubs)
src/gateway/ 4 exec-approval-manager, server-methods/exec-approval, test-helpers.openai-mock, test-openai-responses-model
src/acp/ 2 control-plane/manager, policy
Other 11 cli/command-secret-*, browser/form-fields, media-understanding/transcribe-audio, etc.

Expected outcome

  • Net code reduction (delete stubs + their importers where appropriate)
  • Cleaner disposition.tsv (EXCLUDE entries for files that shouldn't be in fork)
  • Future DIFF-SYNC batches won't recreate the same stubs
  • Aligns with Stub Gate (fork-sync Phase D.3) and Middleware Boundary Principle

Priority

Must be done BEFORE the next DIFF-SYNC batch (B5: v2026.3.7). Each sync that touches these domains will recreate stubs that were deleted, unless the importers are reclassified as EXCLUDE.

Context

Discovered during pre-v2026.3.7 regression remediation (PR #2191). The new Stub Gate in fork-sync Phase D.3 will prevent future stub creation, but existing stubs need manual cleanup.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions