🏄

Summary

• remove ejected interceptor records instead of leaving null slots behind
• keep stable interceptor ids so stale ids cannot remove later handlers after compaction
• add focused InterceptorManager regression tests for compaction, stale ids, and order preservation

Verification

• npm run test:vitest:unit -- tests/unit/core/InterceptorManager.test.js
• npx eslint lib/core/InterceptorManager.js tests/unit/core/InterceptorManager.test.js

Summary by cubic

Removes ejected interceptors instead of leaving null slots and introduces stable numeric IDs to prevent stale removals. Adds focused tests to verify compaction, order, and stale ID safety.

Description

• Summary of changes
  • Remove ejected handlers with splice; no more null slots.
  • Add stable numeric IDs via a private symbol and nextId.
  • Keep forEach simple since there are no nulls to skip.
• Reasoning
  • Avoid sparse arrays and reduce memory growth.
  • Prevent stale IDs from removing later handlers.
  • Preserve insertion order after ejections.
• Additional context
  • Public API is unchanged: use returns a number; eject(id) still works.
  • Behavior is backward compatible and internal-only.

Docs

• Update /docs/ interceptor docs to clarify:
  • IDs are stable and not tied to array indices.
  • Ejected handlers are removed, not set to null.
  • forEach no longer needs to account for null entries.

Testing

• Added unit tests in tests/unit/core/InterceptorManager.test.js:
  • Ensures no null slots remain after many add/eject cycles.
  • Verifies stale IDs cannot remove newer handlers.
  • Confirms insertion order is preserved after middle ejections.

Semantic version impact

• Patch: internal behavior change with no public API breakage.

^{Written for commit 15424b9. Summary will update on new commits.}