feat(skills): add tracing skill family - #391
Merged
Merged
Conversation
Adds a docs-driven tracing skill group (nodejs, nextjs, python, go, java, dotnet, php, ruby, other/OTLP) mirroring the logs and metrics families, and a posthog-tracing marketplace plugin. posthog-node 5.52.0 exports spans natively, so agents get a PostHog-specific route on Node and the OpenTelemetry route everywhere else. Claude-Session: https://claude.ai/code/session_01QPLdjqTgK93WtVxnhx6aVr
1 task
🧙 Wizard CIRun the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands: Test all apps:
Test all apps in a directory:
Test an individual app:
Show more apps
Test against a wizard branch:
Add Results will be posted here when complete. |
This was referenced Sep 11, 2026
Merged
turnipdabeets
marked this pull request as ready for review
September 11, 2026 17:59
marandaneto
approved these changes
Sep 17, 2026
marandaneto
left a comment
marandaneto
left a comment
Member
There was a problem hiding this comment.
The code looks good; no qualifying findings. Verdict: correct.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Agents working through the PostHog MCP server or the wizard have skills for logs and metrics but nothing for distributed tracing, so an agent asked to "add tracing" has no PostHog guidance at all. That gap got wider with
posthog-node5.52.0 (PostHog/posthog-js#4579), which exports spans itself with no OpenTelemetry dependency – a route an agent can't discover from generic OTel knowledge.Changes
context/skills/tracing/– a docs-only skill group mirroringlogsandmetrics: one variant per page under/docs/distributed-tracing/installation(nodejs,nextjs,python,go,java,dotnet,php,ruby), plusotherpointing at the install index for generic OTLP. Shared docs:start-hereandbasics.description.md– how to pick the capture path (posthog-node≥ 5.52.0 → nativetraces+withSpan; existing OpenTelemetry → point the exporter at/i/v1/traces; otherwise the language's OTel SDK), where to instrument, and the rules that break traces when skipped: low-cardinality span names,traceparentpropagation, theposthogDistinctId/sessionIdjoin, flushing in serverless.marketplace.yaml– aposthog-tracingplugin, same shape asposthog-metrics.tracing-skills.test.js– asserts the variant set and each variant's install doc, likemetrics-skills.test.js.Every
docs_urlsentry returns 200 as.mdtoday, and the Node page already carries theposthog-noderoute (merged in PostHog/posthog.com#19837).Deliberately not included:
cli:block – reachable throughwizard skill tracing-<variant>and the MCP skill menu. Awizard tracingcommand likewizard metricsis a separate call.omnibus/instrument-tracingaggregate yet – easy follow-up if wanted for parity with logs/metrics.metricsis owned by@PostHog/apm; tracing probably belongs there too, but that team couldn't be resolved from here – happy to add it if confirmed.How did you test this code?
pnpm test– passes, including the newtracing-skills.test.js.pnpm build– emits ninetracing-*.zipskills, atracinggroup inskill-menu.json, and aposthog-tracingmarketplace plugin. Read the generatedtracing-nodejsSKILL.mdand its references.description.md(traces.serviceName,withSpan/startSpan,beforeSpanSend, theposthogDistinctId/sessionIdattributes, browsertracing_headers) checked againstposthog-jsmain.Related drafts from the same audit: PostHog/posthog#99441 (in-app empty state), PostHog/posthog.com#20109 (tracing docs landing page).
🤖 Generated with Claude Code
https://claude.ai/code/session_01QPLdjqTgK93WtVxnhx6aVr