Skip to content

refactor: remove legacy queue APIs#6

Open
cmilesio wants to merge 4 commits into
mainfrom
refactor/remove-legacy-api
Open

refactor: remove legacy queue APIs#6
cmilesio wants to merge 4 commits into
mainfrom
refactor/remove-legacy-api

Conversation

@cmilesio

Copy link
Copy Markdown
Member

What

Remove the retired public bus and queuefake packages, workflow-observer aliases, root Config.Observer, and Redis ServerLogger. Root queue is now the sole application facade for dispatch, workflows, observation, and fakes.

flowchart LR
    App[Application] --> Q[queue.Queue]
    Q --> W[internal/workflow]
    Q --> D[driver runtime]
    D --> B[backend]
Loading
Removed surface Canonical replacement
bus.* root queue.*
queuefake.New() queue.NewFake()
Config.Observer queue.WithObserver(...)
WorkflowObserver and related aliases queue.Observer and queue.Event
Redis ServerLogger DriverBaseConfig.Logger

Replace facade-only integration coverage with a public root-queue chain and batch contract exercised across sync, workerpool, Redis, NATS, SQS, RabbitMQ, SQLite, MySQL, and Postgres. Regenerate the README API/examples and executed test-count evidence.

Why

The compatibility facade preserved duplicate public models for queueing, workflows, observation, and fakes. Removing it leaves one coherent API while retaining the internal delivery and orchestration separation needed for backend-specific behavior. The migration guidance and generated documentation now describe only supported APIs.

@codecov-commenter

codecov-commenter commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

2 participants