Skip to content

Maintenance: refactor Tracer types and interfaces to improve consistency and clarity #1730

Closed
@dreamorosi

Description

@dreamorosi

Summary

Review all type and interface definitions and usage throughout the codebase to identify opportunities for refactoring and clean up. This will help improve consistency in naming, structure, and usage of types across the project.

Specific areas to investigate:

  • Inconsistent naming (plural vs singular, capitalization, abbreviations, etc.)
  • Misnamed types that are not actually types but constants or enums
  • Usage of any where more specific types could be defined
  • Opportunities to convert usage of enums to as const types
  • Inheritance hierarchies that could be simplified
  • Re-organization of types into more logical modules/namespaces

Why is this needed?

As the project has grown organically, inconsistencies in how types are defined and used have emerged. By refactoring and cleaning these up, we can improve maintainability, clarity, and reliability of the codebase.

More consistent and intention-revealing names will also improve onboarding and comprehension as well as DX.

Converting usage to proper types where applicable will enable stronger compiler checking and editor tooling support. Overall, this refactoring will leave the code in better shape for continued development and maintenance.

Which area does this relate to?

No response

Solution

Examples of types/interfaces/objects that could be improved:

  • TracerInterface (here) should be moved to types folder
  • MethodDecorator (here) has an outstanding TODO item. It's unclear if this can in fact be improved or the any is necessary, but either way we should spend some time investigating and either fixing it or removing the TODO.
  • ProviderServiceInterface (here) should be moved to types folder and reviewed

Important

Before starting to work on a PR for this issue, the assignee should do an investigation of the code and propose an action plan under this issue.

Once the plan has been discussed and agreed upon with at least one maintainer work on the PR can start.

Acknowledgment

Future readers

Please react with 👍 and your use case to help us understand customer demand.

Metadata

Metadata

Assignees

Labels

completedThis item is complete and has been merged/shippedgood-first-issueSomething that is suitable for those who want to start contributingtracerThis item relates to the Tracer Utility

Type

No type

Projects

Status

Shipped

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions