Skip to content

fix: fix-telemetry-console-span-exporter-import — correct ConsoleSpanExporter import path - #355

Merged
avoidwork merged 2 commits into
mainfrom
fix/telemetry-console-span-exporter-import
Jun 21, 2026
Merged

fix: fix-telemetry-console-span-exporter-import — correct ConsoleSpanExporter import path#355
avoidwork merged 2 commits into
mainfrom
fix/telemetry-console-span-exporter-import

Conversation

@avoidwork

@avoidwork avoidwork commented Jun 21, 2026

Copy link
Copy Markdown
Owner

Description

Fixes a medium-severity bug where ConsoleSpanExporter was imported from the wrong OpenTelemetry package. The import was @opentelemetry/exporter-trace-otlp-http but should be @opentelemetry/sdk-trace-base.

Type of Change

  • Bugfix (non-breaking change which fixes an issue)

Testing

  • All 1161 tests pass (1 skipped)
  • Lint passes — 0 warnings, 0 errors

Coverage

  • 100% line coverage maintained

Checklist

  • npm run lint passes
  • Tests pass with 100% line coverage
  • No forbidden patterns used
  • Conventional Commit style applied

@avoidwork

Copy link
Copy Markdown
Owner Author

Audit Results: fix-telemetry-console-span-exporter-import

Implementation Audit (Step 9)

Goal Fulfillment

  • ✅ Goal 1 (fix ConsoleSpanExporter import) fully satisfied — import corrected on line 4 of src/telemetry/provider.js

Spec Compliance

  • ✅ Code matches requirements in proposal.md, design.md, and tasks.md
  • ✅ Import changed from @opentelemetry/exporter-trace-otlp-http to @opentelemetry/sdk-trace-base

Task Completion

  • ✅ Task 1.1: Import fixed correctly
  • ✅ Task 2.1: All 1161 tests pass (1 skipped)
  • ✅ Task 2.2: Lint passes — 0 warnings, 0 errors

Quality Check

  • ✅ No regressions introduced
  • ✅ No forbidden patterns used
  • ✅ Conventional commit style applied
  • ✅ Application starts correctly (TUI correctly refuses non-interactive terminal — expected behavior)

Result: All goals satisfied, all tasks complete, no issues found.

@avoidwork
avoidwork merged commit 2b95779 into main Jun 21, 2026
2 checks passed
@avoidwork
avoidwork deleted the fix/telemetry-console-span-exporter-import branch June 21, 2026 13:57
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