feat(skills): route Go projects to the OpenTelemetry AIO variant - #373
Open
marco-g-pm wants to merge 2 commits into
Open
feat(skills): route Go projects to the OpenTelemetry AIO variant#373marco-g-pm wants to merge 2 commits into
marco-g-pm wants to merge 2 commits into
Conversation
Go has no PostHog wrapper SDK; PostHog/posthog-go#306 adds an OTel bridge instead. Add the opentelemetry-go variant and make a go.mod project take it first, before the framework and provider rules. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
🧙 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. |
carlos-marchal-ph
approved these changes
Sep 1, 2026
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
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
A Go app asking the wizard for AI observability has nowhere to land: the
ai-observabilityskill ships Python and Node variants only, and its decision rules never mention Go. PostHog/posthog-go#306 adds an OTel bridge, making OpenTelemetry the one supported Go path.for PostHog/posthog-go#306
Changes
go.modproject now routes to the newopentelemetry-govariant first, before the framework and provider rules (new rule 1 in1-begin.md).opentelemetry-govariant toconfig.yaml, pointing at the OpenTelemetry install doc and the Go library doc.go.mod, hand-author thegen_ai.*spans, same cardinality rules.Testing
pnpm buildgeneratesai-observability-opentelemetry-go; all 151 skill tests pass.Release notes
Merge wants
mcp-publishplusminorso the variant reaches users. The install doc it points at gains its Go tabs in a posthog monorepo PR that should land first.🤖 Agent context
Written by Claude Code from the Slack thread on posthog-go#306 (https://posthog.slack.com/archives/C087XQ7K9K7/p1788170266901319), with the user directing scope. Skills invoked: writing-pr-descriptions, writing-user-facing-copy.