Skip to content

fix: accept legacy install telemetry snapshots#2656

Merged
Patrick-Erichsen merged 1 commit into
mainfrom
pe/accept-legacy-install-telemetry
Jun 15, 2026
Merged

fix: accept legacy install telemetry snapshots#2656
Patrick-Erichsen merged 1 commit into
mainfrom
pe/accept-legacy-install-telemetry

Conversation

@Patrick-Erichsen

Copy link
Copy Markdown
Collaborator

Summary

  • accept legacy pre-v0.20.2 install telemetry snapshots alongside current per-skill events
  • process legacy snapshots additively in bounded batches so stale or partial reports cannot deactivate installs
  • preserve current deduplicated install/stat event behavior and add compatibility coverage

Validation

  • bunx vitest run convex/httpApi.handlers.test.ts convex/telemetry.test.ts packages/schema/src/schemas.test.ts
  • bun run --cwd packages/clawhub test:src -- src/cli/commands/skills.test.ts src/cli/commands/sync.test.ts src/schema/schemas.test.ts
  • bunx tsc -p packages/schema/tsconfig.json --noEmit
  • bunx tsc -p packages/clawhub/tsconfig.json --noEmit
  • bunx convex dev --once --typecheck=disable
  • bun run ci:static
  • bun run ci:unit
  • bun run ci:types-build
  • bun run ci:packages
  • bun run ci:e2e-http

Review

  • Codex autoreview's concrete focused tests/type checks passed, but its helper recursively launched nested reviews and was stopped before a final clean line.
  • Fallback reviewers were unavailable: Claude had insufficient credits, Pi was absent, and Droid was unauthenticated.
  • Manual review found no actionable issues.

@Patrick-Erichsen Patrick-Erichsen requested a review from a team as a code owner June 15, 2026 19:25
@vercel

vercel Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
clawhub Ready Ready Preview, Comment Jun 15, 2026 7:26pm

@clawsweeper

clawsweeper Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper status: review started.

I am starting a fresh review of this pull request: fix: accept legacy install telemetry snapshots This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@Patrick-Erichsen Patrick-Erichsen merged commit ab3c708 into main Jun 15, 2026
20 checks passed
@Patrick-Erichsen Patrick-Erichsen deleted the pe/accept-legacy-install-telemetry branch June 15, 2026 19:32
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