Rotorsoft / act-root / 32648709197

100%

push

github

refactor(act): give ambient context its own module and clear it out of internal (#1549)

* refactor(act): give ambient context its own module and clear it out of internal

internal/ modules are stateless implementations — event-sourcing.ts says so
in its own header ("no module-level mutable state"). #1541 broke that by
adding internal/reacting.ts, an AsyncLocalStorage, and having action() read
it. It was the only module-level state and the only node:async_hooks import
in the whole folder.

Ambient context now lives in src/scoped.ts: both the ports bag (previously
declared in ports.ts) and the reaction context. ports.ts goes back to being
about adapters, internal/ back to being stateless.

Public surface is unchanged. `scoped` and `Scoped` are exported through
index.ts's star-export of ports.ts and are in the stability snapshot, so
ports.ts re-exports both by name from their new home. `reacting` stays
unreachable from the entry point — only its source text appears in the
snapshot, via the transitive-import walk.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K5eB6CMELZJfayCYpQta8N

* refactor(act): move the disposer registry out of internal too

internal/disposers.ts held `const entries: Entry[] = []`, a process-wide
mutable registry — the last module-level state in the folder. It is lifecycle
state, not an implementation detail, and its only consumer is the `dispose()`
port.

Moved to src/disposers.ts alongside scoped.ts. internal/ now holds no
module-level state and no node:async_hooks at all.

Not public surface: internal/index.ts is not star-exported (index.ts takes
only DeferSignal from it), so the snapshot hits for register_disposer are
captured text from the transitive-import walk, not API.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K5eB6CMELZJfayCYpQta8N

* refactor(act): invert the ambient dependency s... (continued)

4237 of 4237 branches covered (100.0%)

Branch coverage included in aggregate %.

16 of 16 new or added lines in 5 files covered. (100.0%)

9195 of 9195 relevant lines covered (100.0%)

1336.54 hits per line