Skip to content

Remove install context from package telemetry#431

Merged
blove merged 1 commit into
mainfrom
codex/remove-install-context
May 18, 2026
Merged

Remove install context from package telemetry#431
blove merged 1 commit into
mainfrom
codex/remove-install-context

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 18, 2026

Summary

  • remove the derived install_context property from ngaf:postinstall payloads
  • remove package telemetry dashboard filters that depended on install_context
  • keep source-side CI exclusion as the mechanism for excluding our own CI installs
  • update telemetry contract tests and docs to prevent reintroducing the derived field

Verification

  • PATH=/tmp/codex-node-v24/bin:$PATH npx nx test telemetry
  • PATH=/tmp/codex-node-v24/bin:$PATH npx nx run posthog-tools:test
  • PATH=/tmp/codex-node-v24/bin:$PATH npx nx build telemetry
  • PATH=/tmp/codex-node-v24/bin:$PATH npx nx run-many -t lint --projects=telemetry,posthog-tools
  • PATH=/tmp/codex-node-v24/bin:$PATH npx prettier --check ... && git diff --check
  • set -a; source /Users/blove/repos/angular-agent-framework/.env; set +a; PATH=/tmp/codex-node-v24/bin:$PATH npx nx run posthog-tools:sync:apply
  • set -a; source /Users/blove/repos/angular-agent-framework/.env; set +a; export POSTHOG_PERSONAL_API_KEY="${POSTHOG_PERSONAL_API_KEY_READONLY:-$POSTHOG_PERSONAL_API_KEY}"; PATH=/tmp/codex-node-v24/bin:$PATH npx nx run posthog-tools:quality:live -- --days 7 --limit-per-event 25 --require-critical-coverage

Notes

  • The dashboard updates were applied to PostHog with applied: 16, failed: 0; no orphan deletion was requested.
  • Lint still reports existing warnings only.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

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

Project Deployment Actions Updated (UTC)
cacheplane Ready Ready Preview, Comment May 18, 2026 6:56pm

Request Review

@blove blove enabled auto-merge (squash) May 18, 2026 18:49
@blove blove merged commit 9ab0bcc into main May 18, 2026
18 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