Skip to content

Reverse Dependency: trace-dispatcher no longer depends on trace-forward #6268

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

jutaro
Copy link
Contributor

@jutaro jutaro commented Jul 1, 2025

This PR decouples trace-dispatcher from trace-forward, thereby removing its dependency on network-related libraries and Cardano-specific modules.

Motivation

Previously, trace-dispatcher depended on trace-forward, which in turn pulled in network and a substantial portion of the Cardano ecosystem. This entanglement made it cumbersome to use trace-dispatcher in simpler or non-Cardano settings.

What Changed

  • Removed the direct dependency on trace-forward
  • Refactored or restructured any modules or types that required it
  • Ensured that trace-dispatcher remains focused on its core responsibility: dispatching traces, not networking

Benefits

  • Lighter dependency footprint
  • Easier reuse of trace-dispatcher in new projects or different ecosystems
  • Paves the way for more modular and maintainable tracing infrastructure

@jutaro jutaro requested a review from mgmeier July 1, 2025 17:17
@jutaro jutaro force-pushed the jutaro/trace-dispatcher--forwarding branch 3 times, most recently from f6edd80 to 7602ed5 Compare July 6, 2025 13:05
@jutaro jutaro marked this pull request as ready for review July 6, 2025 18:58
@jutaro jutaro requested review from a team as code owners July 6, 2025 18:58
@jutaro jutaro marked this pull request as draft July 7, 2025 10:23
@jutaro jutaro force-pushed the jutaro/trace-dispatcher--forwarding branch from 83a9f7a to f2fd6f8 Compare July 10, 2025 07:58
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.

1 participant