Skip to content

refactor: remove obsolete scheduler and private scaffolding - #817

Merged
kvlonge merged 1 commit into
mainfrom
refactor/chi-367-bloat-pass-two
Sep 25, 2026
Merged

kvlonge merged 1 commit into
mainfrom
refactor/chi-367-bloat-pass-two

Conversation

@kvlonge

@kvlonge kvlonge commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Why

The first dead-code pass left a serial Python-node scheduler reachable only from tests, redundant cursor diagnostic fields and private constants hidden from name-based detection by live names in other modules.

Changes

  • Remove the test-only scheduler, dependency helpers and result wrapper; preserve behavioral tests against the production ready-node entry point.
  • Remove 52 unused private compiler/CLI constants and orphaned imports.
  • Use the existing cursor-input key directly instead of carrying two identical field-name strings.
  • Keep distinct cursor-role mappings, public interfaces and compatibility behavior intact.

Verification

  • 324 focused executor, planner, compiler/discovery and CLI-output tests passed.
  • Tracked-file reference checks, Ruff, ty and fensu checks.
  • Exact static CI gate and independent read-only review before delivery.

@kvlonge
kvlonge enabled auto-merge (squash) September 25, 2026 05:57
@kvlonge
kvlonge merged commit 3515fb5 into main Sep 25, 2026
28 checks passed
@kvlonge
kvlonge deleted the refactor/chi-367-bloat-pass-two branch September 25, 2026 06:10
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