Skip to content

docs(memory): pin mcpgrade otel-comment reword as a deferred TODO - #344

Merged
devopam merged 1 commit into
mainfrom
claude/claude-md-mcpgrade-todo
Oct 7, 2026
Merged

devopam merged 1 commit into
mainfrom
claude/claude-md-mcpgrade-todo

Conversation

@devopam

@devopam devopam commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Summary

Pins a deferred TODO in CLAUDE.md's Future-work section (durable project memory — the session task list doesn't survive): reword the src/mcpg/otel_tracing.py comment that lists "Datadog" as an example OTLP backend so mcpgrade.dev's static scan stops raising its false-positive "sends telemetry to Datadog" Transport finding. Comment-only when done; no code/behaviour change here — this PR only records the note + the mcpgrade context for a future session.

Docs-only.

Roadmap linkage

Advances roadmap row: N/A — project-memory note.

Checklist

  • Tests — N/A (docs-only)
  • ruff/mypy — no source change
  • CHANGELOG.md — N/A (internal memory note, not user-facing)
  • Roadmap row cited (N/A)
  • No hand-edits to src/mcpg/_vendor/

🤖 Generated with Claude Code

https://claude.ai/code/session_0122yLZLJ8t4W43sdN6BmTZc


Generated by Claude Code

Summary by Sourcery

Pin a deferred documentation-only task for resolving an mcpgrade.dev static-analysis false positive.

Enhancements:

  • Record a deferred project-memory note to reword the Datadog OTLP-backend example in a tracing comment to avoid a known mcpgrade.dev static-analysis false positive without changing functionality.

Documentation:

  • Document the mcpgrade.dev false-positive context and guidance for addressing it in the future.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

Persist the future TODO in CLAUDE.md's Future-work section so it survives
across sessions (the task list is session-only): reword the
otel_tracing.py comment that names "Datadog" as an example OTLP backend,
to clear mcpgrade.dev's false-positive "sends telemetry to Datadog"
finding. Captures the mcpgrade context (static-analysis "D" with several
false positives + an unscored injection-surface check) so a future
session has the background. No code/behaviour change.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0122yLZLJ8t4W43sdN6BmTZc
@devopam
devopam force-pushed the claude/claude-md-mcpgrade-todo branch from 3e3adc6 to 0ad97a9 Compare October 7, 2026 06:11
@devopam
devopam merged commit e5bcedb into main Oct 7, 2026
29 checks passed
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