Skip to content

Add Go docs coverage#153

Merged
marandaneto merged 2 commits into
mainfrom
chore/add-go-doc-coverage
May 27, 2026
Merged

Add Go docs coverage#153
marandaneto merged 2 commits into
mainfrom
chore/add-go-doc-coverage

Conversation

@marandaneto
Copy link
Copy Markdown
Member

Problem

Go had coverage in feature flags, error tracking, and logs, but it was missing the main Go SDK library docs from the core integration and product analytics skill configs. Agents working on Go applications could miss the current posthog-go setup and SDK guidance, especially around client lifecycle, stable distinct IDs, EvaluateFlags, and Go error tracking patterns.

Changes

  • Add a docs-only integration-go skill using the official Go library docs.
  • Add the Go library docs to omnibus integration and product analytics skills.
  • Add a Go-specific integration template with Go app detection and verification guidance.
  • Add Go-specific commandments for SDK installation, one-client-per-process lifecycle, client.Close(), distinct IDs, properties/PII, feature flag snapshots, and error tracking with NewDefaultException / SlogCaptureHandler.
  • Add go.sum to relevant omnibus platform/package detection hints.
  • Keep existing Go feature flag installation docs unchanged per review feedback.
  • Keep this docs-only: no basics sample is added.

Checklist

  • pnpm build
  • pnpm test
  • Changeset not required (chore/docs-only config update)

@marandaneto marandaneto requested a review from a team May 27, 2026 14:10
@github-actions
Copy link
Copy Markdown

🧙 Wizard CI

Run 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:

  • /wizard-ci all

Test all apps in a directory:

  • /wizard-ci basic-integration
  • /wizard-ci misc
  • /wizard-ci revenue

Test an individual app:

  • /wizard-ci basic-integration/android
  • /wizard-ci basic-integration/angular
  • /wizard-ci basic-integration/astro
Show more apps
  • /wizard-ci basic-integration/django
  • /wizard-ci basic-integration/fastapi
  • /wizard-ci basic-integration/flask
  • /wizard-ci basic-integration/javascript-node
  • /wizard-ci basic-integration/javascript-web
  • /wizard-ci basic-integration/laravel
  • /wizard-ci basic-integration/next-js
  • /wizard-ci basic-integration/nuxt
  • /wizard-ci basic-integration/python
  • /wizard-ci basic-integration/rails
  • /wizard-ci basic-integration/react-native
  • /wizard-ci basic-integration/react-router
  • /wizard-ci basic-integration/sveltekit
  • /wizard-ci basic-integration/swift
  • /wizard-ci basic-integration/tanstack-router
  • /wizard-ci basic-integration/tanstack-start
  • /wizard-ci basic-integration/vue
  • /wizard-ci misc/quack-quack
  • /wizard-ci revenue/stripe

Results will be posted here when complete.

Copy link
Copy Markdown
Contributor

@dustinbyrne dustinbyrne left a comment

Choose a reason for hiding this comment

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

Content looks good. Probably worth another pair of eyes on the schema / configuration unless you're feeling confident

@dustinbyrne dustinbyrne requested a review from a team May 27, 2026 15:28
# Conflicts:
#	transformation-config/skills/omnibus/instrument-error-tracking/description.md
#	transformation-config/skills/omnibus/instrument-feature-flags/description.md
#	transformation-config/skills/omnibus/instrument-integration/description.md
#	transformation-config/skills/omnibus/instrument-logs/description.md
#	transformation-config/skills/omnibus/instrument-product-analytics/description.md
@marandaneto marandaneto merged commit eb728e3 into main May 27, 2026
11 of 12 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