add op's to the spans

Author: nsdeschenes

packages/bundler-plugin-core/src/sentry/telemetry.ts

@@ -152,7 +152,11 @@ export function telemetryPlugin({
           "Sending telemetry data on issues and performance to Codecov. To disable telemetry, set `options.telemetry` to `false`.",
         );
         startSpan(
-          { name: "Codecov Bundler Plugin execution", scope: sentryScope },
+          {
+            name: "Codecov Bundler Plugin execution",
+            op: "bundler-plugin-startup",
+            scope: sentryScope,
+          },
           () => {
             //
           },

packages/bundler-plugin-core/src/utils/Output.ts

@@ -163,7 +163,11 @@ class Output {
     const inputs: ProviderUtilInputs = { envs, args };
 
     const provider = await startSpan(
-      { name: "output.write.detectProvider", scope: this.sentryScope },
+      {
+        name: "Detect Provider",
+        op: "output.write.detectProvider",
+        scope: this.sentryScope,
+      },
       async () => {
         let detectedProvider;
         try {
@@ -206,7 +210,8 @@ class Output {
 
       const slug = provider.slug ?? "";
       const repoIndex = slug.lastIndexOf("/") + 1;
-      const owner = slug.substring(0, repoIndex).trimEnd();
+      // -1 to trim the trailing slash
+      const owner = slug.substring(0, repoIndex - 1).trimEnd();
       if (owner.length > 0) {
         this.sentryScope.setTag("owner", owner);
       }
@@ -219,7 +224,11 @@ class Output {
 
     let url = "";
     await startSpan(
-      { name: "output.write.getPreSignedURL", scope: this.sentryScope },
+      {
+        name: "Get Pre-Signed URL",
+        op: "output.write.getPreSignedURL",
+        scope: this.sentryScope,
+      },
       async () => {
         try {
           url = await getPreSignedURL({
@@ -260,7 +269,11 @@ class Output {
     );
 
     await startSpan(
-      { name: "output.write.uploadStats", scope: this.sentryScope },
+      {
+        name: "Upload Stats",
+        op: "output.write.uploadStats",
+        scope: this.sentryScope,
+      },
       async () => {
         try {
           await uploadStats({

packages/bundler-plugin-core/src/utils/__tests__/Output.test.ts

@@ -431,6 +431,59 @@ describe("Output", () => {
       });
     });
 
+    describe("successful detection of provider", () => {
+      it("sets the owner and repo tags", async () => {
+        setup({});
+        const sentryClient = {
+          captureMessage: vi.fn(),
+        } as unknown as Client;
+
+        const sentryScope = {
+          getClient: vi.fn(),
+          setTag: vi.fn(),
+          addBreadcrumb: vi.fn(),
+        } as unknown as Scope;
+
+        const output = new Output(
+          {
+            apiUrl: "http://localhost",
+            bundleName: "output-test",
+            debug: false,
+            dryRun: false,
+            enableBundleAnalysis: true,
+            retryCount: 1,
+            uploadToken: "token",
+            telemetry: false,
+          },
+          { sentryClient, sentryScope },
+        );
+
+        output.start();
+        output.end();
+
+        await output.write();
+
+        // eslint-disable-next-line @typescript-eslint/unbound-method
+        expect(sentryScope.setTag).toHaveBeenNthCalledWith(
+          1,
+          "service",
+          "github",
+        );
+        // eslint-disable-next-line @typescript-eslint/unbound-method
+        expect(sentryScope.setTag).toHaveBeenNthCalledWith(
+          2,
+          "owner",
+          "codecov",
+        );
+        // eslint-disable-next-line @typescript-eslint/unbound-method
+        expect(sentryScope.setTag).toHaveBeenNthCalledWith(
+          3,
+          "repo",
+          "codecov-javascript-bundler-plugins",
+        );
+      });
+    });
+
     describe("fails to fetch pre-signed URL", () => {
       it("immediately returns", async () => {
         setup({ urlSendError: true });