Conversation
|
Claude finished @jordanl17's task in 49s —— View job ReviewOne-line telemetry event rename. Aligns with the Findings
No other issues. |
📦 Bundle Stats —
|
| Metric | Value | vs main (6165c80) |
|---|---|---|
| Internal (raw) | 2.1 KB | - |
| Internal (gzip) | 799 B | - |
| Bundled (raw) | 10.97 MB | - |
| Bundled (gzip) | 2.06 MB | - |
| Import time | 758ms | +9ms, +1.2% |
bin:sanity
| Metric | Value | vs main (6165c80) |
|---|---|---|
| Internal (raw) | 1023 B | - |
| Internal (gzip) | 486 B | - |
| Bundled (raw) | 9.87 MB | - |
| Bundled (gzip) | 1.77 MB | - |
| Import time | 849ms | +15ms, +1.8% |
🗺️ View treemap · Artifacts
Details
- Import time regressions over 10% are flagged with
⚠️ - Sizes shown as raw / gzip 🗜️. Internal bytes = own code only. Total bytes = with all dependencies. Import time = Node.js cold-start median.
📦 Bundle Stats — @sanity/cli-core
Compared against main (6165c80c)
| Metric | Value | vs main (6165c80) |
|---|---|---|
| Internal (raw) | 96.3 KB | - |
| Internal (gzip) | 22.7 KB | - |
| Bundled (raw) | 21.64 MB | - |
| Bundled (gzip) | 3.43 MB | - |
| Import time | 718ms | -1ms, -0.1% |
🗺️ View treemap · Artifacts
Details
- Import time regressions over 10% are flagged with
⚠️ - Sizes shown as raw / gzip 🗜️. Internal bytes = own code only. Total bytes = with all dependencies. Import time = Node.js cold-start median.
📦 Bundle Stats — create-sanity
Compared against main (6165c80c)
| Metric | Value | vs main (6165c80) |
|---|---|---|
| Internal (raw) | 908 B | - |
| Internal (gzip) | 483 B | - |
| Bundled (raw) | 931 B | - |
| Bundled (gzip) | 491 B | - |
| Import time | ❌ ChildProcess denied: node | - |
Details
- Import time regressions over 10% are flagged with
⚠️ - Sizes shown as raw / gzip 🗜️. Internal bytes = own code only. Total bytes = with all dependencies. Import time = Node.js cold-start median.
Coverage Delta
Comparing 1 changed file against main @ Overall Coverage
|
|
@copilot resolve the merge conflicts in this pull request |
Resolved. I merged |
|
@copilot fix the lint checks now in CI |
Fixed in |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit b664eb6. Configure here.
| "lint": {}, | ||
| "publint": { | ||
| "dependsOn": ["^build:types", "^publint"], | ||
| "dependsOn": ["build:types", "^build:types", "^publint"], |
There was a problem hiding this comment.
Unrelated turbo.json build config change in rename PR
Low Severity
The publint task's dependsOn now includes same-package "build:types" (without ^ prefix), in addition to the existing "^build:types". This build pipeline change is unrelated to the stated PR goal of renaming a telemetry event. It was likely introduced during Copilot's automated merge conflict resolution and may not have been intentionally reviewed.
Reviewed by Cursor Bugbot for commit b664eb6. Configure here.


Description
A March 2026 audit of Studio telemetry by data and analytics flagged SAPP-3865 - the
Schema deploy action executedevent uses the wrong tense and pattern relative to the rest of our event vocabulary. This is the follow-up to SAPP-3818, which covered the Canvas rename in the studio repo.This PR renames the event to
Schema Deployed. No payload or trigger changes, just the event name.Note
Low Risk
Single string change on a telemetry trace name; no runtime or security impact beyond analytics possibly treating it as a new event label.
Overview
Renames the
SchemaDeploytelemetry trace’s displaynamefromSchema deploy action executedtoSchema DeployedinextractSchema.telemetry.ts, aligning Studio telemetry vocabulary with past-tense event naming (e.g.Schema Extracted). TheSchemaDeployexport, payload shape, and when the trace fires are unchanged.Reviewed by Cursor Bugbot for commit 2dced76. Bugbot is set up for automated code reviews on this repo. Configure here.