Skip to content

feat: Go native symbol upload guidance + contract lock - #299

Merged
cat-ph merged 2 commits into
mainfrom
cat/go-symbol-upload-variant
Jul 30, 2026
Merged

feat: Go native symbol upload guidance + contract lock#299
cat-ph merged 2 commits into
mainfrom
cat/go-symbol-upload-variant

Conversation

@cat-ph

@cat-ph cat-ph commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Was stacked on #262 (rust) — that merged, so this now targets main directly.

Related PRs:

Go half of the native symbol-set pair:

  • error-tracking-upload-source-maps: go variant (docs page is live, build fetches it green) + go guidance in the shared description — platform-split linker flags (-B gobuildid on Linux, -compressdwarf=false on macOS, both load-bearing at runtime and upload time), the strip/-trimpath warnings, standalone symbol-sets upload folded into the resolved build command, posthog-go 1.22.0+ requirement with the in-step upgrade command, Windows fallback note, and a NewDefaultException test affordance.
  • Contract-lock test for the go variant, mirroring the rust/android/iOS ones.

Facts verified against posthog-go v1.22.0 source (build-id-gated automatic emission, NewDefaultException signature) and the live docs page.

Labels needed before merge (matching #253/#281/#262): mcp-publish + minor.

🤖 Generated with Claude Code

@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 mcp-analytics
  • /wizard-ci revenue
  • /wizard-ci self-driving

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 mcp-analytics/custom-dispatcher
  • /wizard-ci mcp-analytics/typescript-sdk
  • /wizard-ci revenue/stripe
  • /wizard-ci self-driving/astro
  • /wizard-ci self-driving/fastapi
  • /wizard-ci self-driving/nuxt
  • /wizard-ci self-driving/react-router
  • /wizard-ci self-driving/sveltekit

Results will be posted here when complete.

@ablaszkiewicz ablaszkiewicz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤌

Base automatically changed from cat/rust-symbol-upload-variant to main July 30, 2026 20:33
@cat-ph
cat-ph force-pushed the cat/go-symbol-upload-variant branch from 116d17c to 721b228 Compare July 30, 2026 20:38
@cat-ph
cat-ph merged commit 22b35c2 into main Jul 30, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants