Skip to content

refactor(hooks): remove dead TheRouter-era exports (markSessionNative, markAlgorithmStarting, setModeToken)#1599

Open
anikinsasha wants to merge 1 commit into
danielmiessler:mainfrom
anikinsasha:refactor/remove-therouter-dead-exports
Open

refactor(hooks): remove dead TheRouter-era exports (markSessionNative, markAlgorithmStarting, setModeToken)#1599
anikinsasha wants to merge 1 commit into
danielmiessler:mainfrom
anikinsasha:refactor/remove-therouter-dead-exports

Conversation

@anikinsasha

Copy link
Copy Markdown

Removes the three exports orphaned by the 7.0.0 retirement of TheRouter.hook.ts (3baba670) — zero callers, imports, tests, or dynamic references on main. Full diagnosis and grep evidence in #1598 (this PR is the mechanical half; the downgrade-guard behavior question stays in the issue).

  • hooks/lib/isa-utils.ts: delete 1252-1400 — the orphaned markAlgorithmStarting docblock (1252-1269, physically detached above the wrong function), markSessionNative doc+fn (1270-1315), markAlgorithmStarting fn (1317-1400). The upsertNativeSession alias (1249-1250) and everything from addRatingPulse (1402+) untouched.
  • hooks/lib/tab-setter.ts: delete 343-393 — setModeToken JSDoc+fn and LIVE_ALGO_PHASES (362), which only it referenced.

Deletion-only; no behavior change. Deliberately does not touch:

Testing: bun build LifeOS/install/hooks/lib/isa-utils.ts LifeOS/install/hooks/lib/tab-setter.ts LifeOS/install/hooks/PromptProcessing.hook.ts --target=bun green on this branch (after bun install in LifeOS/install); post-delete git grep for the three symbols returns only historical doc mentions plus the one guard-comment line noted above.

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