Skip to content

[codex] Fix telemetry install packaging and reporting#340

Merged
blove merged 1 commit into
mainfrom
codex/telemetry-postpublish-cleanup
May 15, 2026
Merged

[codex] Fix telemetry install packaging and reporting#340
blove merged 1 commit into
mainfrom
codex/telemetry-postpublish-cleanup

Conversation

@blove

@blove blove commented May 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Preserve publishConfig.access=public when assembling the publishable @ngaf/telemetry dist manifest.
  • Remove the temporary publish workflow projects escape hatch so releases go through the full publishable group again.
  • Add richer ngaf:postinstall metadata for runtime and npm installer context without collecting project paths or raw environment variables.
  • Add local PostHog package telemetry dashboard definitions and document the install-event property taxonomy.

Validation

  • npx vitest run libs/telemetry/scripts/assemble-dist.spec.mjs libs/telemetry/src/node/client.spec.ts --config vite.config.mts
  • NX_DAEMON=false npx nx test telemetry --skip-nx-cache
  • NX_DAEMON=false npx nx test posthog-tools --skip-nx-cache
  • NX_DAEMON=false npx nx run-many -t lint --projects=telemetry,posthog-tools --skip-nx-cache (passes with existing warnings)
  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/publish.yml"); puts "publish workflow yaml ok"'
  • git diff --check
  • NX_DAEMON=false npx nx build telemetry --configuration=production --skip-nx-cache
  • Checked dist/libs/telemetry/package.json includes publishConfig.access=public.
  • Live npm install smoke without NGAF_TELEMETRY_INGEST_URL sent install telemetry for all seven published @ngaf/*@0.0.30 packages.

@vercel

vercel Bot commented May 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cacheplane Ready Ready Preview, Comment May 15, 2026 8:40pm

Request Review

@blove blove merged commit 874be76 into main May 15, 2026
16 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.

1 participant