Delete the selection-bus shim: its one cycle is over
Landing 1 left plugin-designs/research-dashboard/src/composables/useSelectionBus.js re-exporting the moved file, so the move and its consumers did not have to land together. Both consumers moved in that same landing — useDashboardShell's import and check-facet-cascade's BUS_SRC — so the shim has had zero importers ever since. Measured before deleting: a search across plugin-designs, library and internal finds two references to useSelectionBus and both name the LIBRARY path. A re-export with no importers is not harmless. It is a second entry point that reads like a supported one, and the next author who finds it has no way to tell that it is scaffolding rather than the plugin-local home of the bus. ★ HOLD THIS COMMIT UNTIL 2bb90852 IS ON MAIN. It is deliberately the tip of the stack and touches nothing else, so it can be dropped or deferred on its own. If landing 1 is not in main, the file this deletes is the only thing making the old import path resolve. Landing 2's own shim (useDashboardLayout) STAYS for one more cycle, on the same rule that earned this one its deletion: it is one landing old, and it gets the same measured check before it goes.
This commit is contained in:
parent
8fc7e2d62a
commit
7f3435cc1b
1 changed files with 0 additions and 18 deletions
|
|
@ -1,18 +0,0 @@
|
|||
/**
|
||||
* useSelectionBus.js — RE-EXPORT SHIM (dashboard-kit landing 1).
|
||||
*
|
||||
* The bus now lives at library/composables/useSelectionBus.js. This file exists so the move and
|
||||
* its consumers do not have to land in the same commit: every importer keeps working unchanged for
|
||||
* one cycle, and the ones that still point here are migrated in the next landing, after which this
|
||||
* file is deleted.
|
||||
*
|
||||
* WHY A SHIM AND NOT A SIMULTANEOUS REWRITE. The extraction's governing constraint is that Big
|
||||
* Picture must not change, and the only way to prove that for nine landings in a row is to make
|
||||
* each one small enough to verify on its own. A landing that moves a file AND rewrites every
|
||||
* import is two changes whose failures look identical from the outside.
|
||||
*
|
||||
* NOT A SECOND COPY. It re-exports; it does not duplicate. A copy would drift silently and both
|
||||
* halves would keep passing their gates — which is the exact failure check:no-cross-plugin-imports
|
||||
* and check:shared-reached exist to stop.
|
||||
*/
|
||||
export { createSelectionBus, deriveHierarchy } from '@map/composables/useSelectionBus.js'
|
||||
Loading…
Reference in a new issue