Sentry middleware example doesn't work

[opsb]

Describe the bug When an error occurs the sentryMiddleware never reports an error. The error should be reported in

  transformOutput({ result, step }) {
    // Capture step output and log errors
    if (step) {
      Sentry.setTags({
        "inngest.step.name": step.displayName,
        "inngest.step.op": step.op,
      });

      if (result.error) {
        Sentry.captureException(result.error);
      }
    }
  },

but step is always undefined.

To Reproduce Add example middleware as per https://www.inngest.com/docs/reference/middleware/examples#adding-sentry-error-reporting-and-tracing.

Throw an error in a handler.

Expected behavior Error should be report to Sentry

Screenshots / Stack trace dump If applicable, add screenshots or paste what is output in your terminal to help explain your problem.

System info

• inngest-cli version: [e.g. 1.0.2]
• OS: [e.g. macOS]
• Version [e.g. 22]

Additional context Workaround is to report the error without the step metadata

  transformOutput({ result, step }) {
    // Capture step output and log errors
    if (step) {
      Sentry.setTags({
        "inngest.step.name": step.displayName,
        "inngest.step.op": step.op,
      });
    }

    if (result.error) {
      Sentry.captureException(result.error);
    }
  },

[linear[bot]]

INN-3159 Sentry middleware example doesn't work

[linear[bot]]

INN-3161 Sentry middleware example doesn't work

[jpwilliams]

Hi, @opsb! 👋

Ah - good find. What version of Sentry are you using?

We have some discussion in #471 which results in the release of a new @inngest/middleware-sentry package in #598.

I'd love it if you could test out the new package and see if this also solves your issue, though we're targeting @sentry/node>=8.0.0 for that.

[jpwilliams]

Closing this - please address any further issues here in #471 or #598.