Skip to content

refactor: replace duplicated concurrency schedulers - #520

Merged
Medformatik merged 1 commit into
mainfrom
refactor/shared-concurrency
Oct 18, 2026
Merged

Medformatik merged 1 commit into
mainfrom
refactor/shared-concurrency

Conversation

@Medformatik

Copy link
Copy Markdown
Collaborator

Replace six duplicated batch schedulers with pinned p-map and the shared place-enrichment limiter with p-limit. Ordered results, settled provider failures, independent journey/stop-area capture limits, and process-wide enrichment limits remain explicit at their call sites. Remove the generic helper exports and tests that only duplicated library coverage.

Fatal batch errors and caller cancellation now stop further dispatch. Provider deadlines and I/O cancellation remain with their owners; active tasks may finish. Transit preparation keeps its explicit aborted result. Reject an Infinity enrichment setting before constructing the limiter so it cannot silently disable the existing finite cap.

Verification

  • pnpm verify:full passed: lint, 30 workspace type tasks, policy checks, generated Entur/SDK/OpenAPI freshness, and 20,334 tests across 1,759 files. 191 tests / 39 files skipped.
  • API and web production builds passed, including service-worker and standalone assets.
  • Consumer regressions cover embedding failure dispatch, capture cancellation/order/separate limits, cross-request enrichment limits/recovery and finite configuration.
  • Independent review: Spec and Quality passed after fixing the Infinity configuration edge.
  • Production dependency audits passed with existing acknowledged advisories. Knip retains three existing reports; duplicate scan reports zero new clone regions.
  • Database, actual browser/native-device and production acceptance were not run. Bridge preparation is covered through its real web-side consumer tests.

Release and upgrade impact

No package release: concurrency implementations change private application/framework workspaces; portable extension contracts and published CLI behavior are unchanged.
Product impact: none

No migration or operator action is required. Existing finite enrichment settings and defaults remain valid.

@Medformatik Medformatik added dependencies Dependency updates and dependency maintenance integration New or changed provider, source or backend integration labels Oct 18, 2026
@Medformatik Medformatik self-assigned this Oct 18, 2026
@Medformatik
Medformatik enabled auto-merge (squash) October 18, 2026 21:36
@Medformatik
Medformatik merged commit 10cb142 into main Oct 18, 2026
25 checks passed
@Medformatik
Medformatik deleted the refactor/shared-concurrency branch October 18, 2026 21:45
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dependencies Dependency updates and dependency maintenance integration New or changed provider, source or backend integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant