@devmoods/sentry 0.5.0 → 0.5.1

package/coverage/clover.xml

@@ -1,10 +1,10 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<coverage generated="1782839143347" clover="3.2.0">
-  <project timestamp="1782839143347" name="All files">
+<coverage generated="1783497115890" clover="3.2.0">
+  <project timestamp="1783497115890" name="All files">
     <metrics statements="1" coveredstatements="1" conditionals="0" coveredconditionals="0" methods="0" coveredmethods="0" elements="1" coveredelements="1" complexity="0" loc="1" ncloc="1" packages="1" files="1" classes="1"/>
     <file name="index.ts" path="/Users/hanse/dev/github.com/devmoods/express-extras/packages/@devmoods/sentry/src/index.ts">
       <metrics statements="1" coveredstatements="1" conditionals="0" coveredconditionals="0" methods="0" coveredmethods="0"/>
-      <line num="14" count="10" type="stmt"/>
+      <line num="14" count="13" type="stmt"/>
     </file>
   </project>
 </coverage>

package/coverage/coverage-final.json

@@ -1,2 +1,2 @@
-{"/Users/hanse/dev/github.com/devmoods/express-extras/packages/@devmoods/sentry/src/index.ts": {"path":"/Users/hanse/dev/github.com/devmoods/express-extras/packages/@devmoods/sentry/src/index.ts","statementMap":{"0":{"start":{"line":14,"column":13},"end":{"line":19,"column":null}}},"fnMap":{},"branchMap":{},"s":{"0":10},"f":{},"b":{},"meta":{"lastBranch":0,"lastFunction":0,"lastStatement":1,"seen":{"s:14:13:19:Infinity":0},"fnNames":{}}}
+{"/Users/hanse/dev/github.com/devmoods/express-extras/packages/@devmoods/sentry/src/index.ts": {"path":"/Users/hanse/dev/github.com/devmoods/express-extras/packages/@devmoods/sentry/src/index.ts","statementMap":{"0":{"start":{"line":14,"column":13},"end":{"line":19,"column":null}}},"fnMap":{},"branchMap":{},"s":{"0":13},"f":{},"b":{},"meta":{"lastBranch":0,"lastFunction":0,"lastStatement":1,"seen":{"s:14:13:19:Infinity":0},"fnNames":{}}}
 }

package/coverage/index.html

@@ -101,7 +101,7 @@
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-06-30T17:05:43.338Z
+                at 2026-07-08T07:51:55.887Z
             </div>
         <script src="prettify.js"></script>
         <script>

package/coverage/index.ts.html

@@ -241,7 +241,66 @@
 <a name='L176'></a><a href='#L176'>176</a>
 <a name='L177'></a><a href='#L177'>177</a>
 <a name='L178'></a><a href='#L178'>178</a>
-<a name='L179'></a><a href='#L179'>179</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
+<a name='L179'></a><a href='#L179'>179</a>
+<a name='L180'></a><a href='#L180'>180</a>
+<a name='L181'></a><a href='#L181'>181</a>
+<a name='L182'></a><a href='#L182'>182</a>
+<a name='L183'></a><a href='#L183'>183</a>
+<a name='L184'></a><a href='#L184'>184</a>
+<a name='L185'></a><a href='#L185'>185</a>
+<a name='L186'></a><a href='#L186'>186</a>
+<a name='L187'></a><a href='#L187'>187</a>
+<a name='L188'></a><a href='#L188'>188</a>
+<a name='L189'></a><a href='#L189'>189</a>
+<a name='L190'></a><a href='#L190'>190</a>
+<a name='L191'></a><a href='#L191'>191</a>
+<a name='L192'></a><a href='#L192'>192</a>
+<a name='L193'></a><a href='#L193'>193</a>
+<a name='L194'></a><a href='#L194'>194</a>
+<a name='L195'></a><a href='#L195'>195</a>
+<a name='L196'></a><a href='#L196'>196</a>
+<a name='L197'></a><a href='#L197'>197</a>
+<a name='L198'></a><a href='#L198'>198</a>
+<a name='L199'></a><a href='#L199'>199</a>
+<a name='L200'></a><a href='#L200'>200</a>
+<a name='L201'></a><a href='#L201'>201</a>
+<a name='L202'></a><a href='#L202'>202</a>
+<a name='L203'></a><a href='#L203'>203</a>
+<a name='L204'></a><a href='#L204'>204</a>
+<a name='L205'></a><a href='#L205'>205</a>
+<a name='L206'></a><a href='#L206'>206</a>
+<a name='L207'></a><a href='#L207'>207</a>
+<a name='L208'></a><a href='#L208'>208</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-yes">13x</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -254,7 +313,6 @@
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
-<span class="cline-any cline-yes">10x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -498,12 +556,41 @@ export async function sentryOptions(
   };
 }
 &nbsp;
+function getHttpStatusCode(error: unknown) {
+  if (error == null || typeof error !== 'object') {
+    return undefined;
+  }
+&nbsp;
+  const {
+    status,
+    statusCode,
+    status_code: statusCodeSnake,
+    output,
+  } = error as {
+    status?: number | string;
+    statusCode?: number | string;
+    status_code?: number | string;
+    output?: { statusCode?: number | string };
+  };
+  const value = status ?? statusCode ?? statusCodeSnake ?? output?.statusCode;
+  const parsed = typeof value === 'string' ? Number.parseInt(value, 10) : value;
+&nbsp;
+  return typeof parsed === 'number' &amp;&amp; Number.isFinite(parsed)
+    ? parsed
+    : undefined;
+}
+&nbsp;
 export function sentryErrorHandler(
   options: Sentry.NodeOptions = {},
 ): ErrorRequestHandler {
   return Sentry.expressErrorHandler({
-    shouldHandleError() {
-      return process.env.NODE_ENV === 'production';
+    shouldHandleError(error) {
+      const status = getHttpStatusCode(error);
+&nbsp;
+      return (
+        process.env.NODE_ENV === 'production' &amp;&amp;
+        (status == null || status &lt; 400 || status &gt;= 500)
+      );
     },
     ...options,
   }) as unknown as ErrorRequestHandler;
@@ -604,7 +691,7 @@ export class SentryLoggerAdapter implements LoggerAdapter {
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at 2026-06-30T17:05:43.338Z
+                at 2026-07-08T07:51:55.887Z
             </div>
         <script src="prettify.js"></script>
         <script>

package/dist/index.js

@@ -53,10 +53,23 @@ export async function sentryOptions(options = {}) {
         ...options,
     };
 }
+function getHttpStatusCode(error) {
+    if (error == null || typeof error !== 'object') {
+        return undefined;
+    }
+    const { status, statusCode, status_code: statusCodeSnake, output, } = error;
+    const value = status ?? statusCode ?? statusCodeSnake ?? output?.statusCode;
+    const parsed = typeof value === 'string' ? Number.parseInt(value, 10) : value;
+    return typeof parsed === 'number' && Number.isFinite(parsed)
+        ? parsed
+        : undefined;
+}
 export function sentryErrorHandler(options = {}) {
     return Sentry.expressErrorHandler({
-        shouldHandleError() {
-            return process.env.NODE_ENV === 'production';
+        shouldHandleError(error) {
+            const status = getHttpStatusCode(error);
+            return (process.env.NODE_ENV === 'production' &&
+                (status == null || status < 400 || status >= 500));
         },
         ...options,
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devmoods/sentry",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -24,13 +24,13 @@
   "dependencies": {
     "@devmoods/config": "0.3.0",
     "@devmoods/observability": "0.4.0",
-    "@sentry/node": "^10.63.0",
-    "@sentry/node-native": "^10.63.0",
-    "@sentry/profiling-node": "^10.63.0",
+    "@sentry/node": "^10.64.0",
+    "@sentry/node-native": "^10.64.0",
+    "@sentry/profiling-node": "^10.64.0",
     "express": "^4.21.2"
   },
   "peerDependencies": {
     "express": ">=4.21.2 <5.0.0"
   },
-  "gitHead": "52519bed307a1eedf13ab334f11c1ad9c7c8db91"
+  "gitHead": "2981e0cfae881922af5b04af07c3d3e641dece37"
 }

package/src/__tests__/index.spec.ts

@@ -59,6 +59,51 @@ test('sentryOptions', async () => {
   reset();
 });
 
+describe('sentryErrorHandler', () => {
+  beforeEach(() => {
+    vi.unstubAllEnvs();
+  });
+
+  test('ignores HTTP 40x errors in production', async () => {
+    vi.stubEnv('NODE_ENV', 'production');
+
+    const { Sentry, sentryErrorHandler } = await loadModuleWithSentryMock();
+
+    sentryErrorHandler();
+
+    const options = Sentry.expressErrorHandler.mock.calls[0]![0];
+    expect(options.shouldHandleError({ status: 400 })).toEqual(false);
+    expect(options.shouldHandleError({ statusCode: 401 })).toEqual(false);
+    expect(options.shouldHandleError({ status_code: '403' })).toEqual(false);
+    expect(options.shouldHandleError({ output: { statusCode: 429 } })).toEqual(
+      false,
+    );
+  });
+
+  test('handles server and unknown errors in production', async () => {
+    vi.stubEnv('NODE_ENV', 'production');
+
+    const { Sentry, sentryErrorHandler } = await loadModuleWithSentryMock();
+
+    sentryErrorHandler();
+
+    const options = Sentry.expressErrorHandler.mock.calls[0]![0];
+    expect(options.shouldHandleError(new Error('boom'))).toEqual(true);
+    expect(options.shouldHandleError({ status: 500 })).toEqual(true);
+  });
+
+  test('does not handle errors outside production', async () => {
+    vi.stubEnv('NODE_ENV', 'test');
+
+    const { Sentry, sentryErrorHandler } = await loadModuleWithSentryMock();
+
+    sentryErrorHandler();
+
+    const options = Sentry.expressErrorHandler.mock.calls[0]![0];
+    expect(options.shouldHandleError({ status: 500 })).toEqual(false);
+  });
+});
+
 describe('SentryMetricsAdapter', () => {
   beforeEach(() => {
     vi.restoreAllMocks();

package/src/index.ts

@@ -77,12 +77,41 @@ export async function sentryOptions(
   };
 }
 
+function getHttpStatusCode(error: unknown) {
+  if (error == null || typeof error !== 'object') {
+    return undefined;
+  }
+
+  const {
+    status,
+    statusCode,
+    status_code: statusCodeSnake,
+    output,
+  } = error as {
+    status?: number | string;
+    statusCode?: number | string;
+    status_code?: number | string;
+    output?: { statusCode?: number | string };
+  };
+  const value = status ?? statusCode ?? statusCodeSnake ?? output?.statusCode;
+  const parsed = typeof value === 'string' ? Number.parseInt(value, 10) : value;
+
+  return typeof parsed === 'number' && Number.isFinite(parsed)
+    ? parsed
+    : undefined;
+}
+
 export function sentryErrorHandler(
   options: Sentry.NodeOptions = {},
 ): ErrorRequestHandler {
   return Sentry.expressErrorHandler({
-    shouldHandleError() {
-      return process.env.NODE_ENV === 'production';
+    shouldHandleError(error) {
+      const status = getHttpStatusCode(error);
+
+      return (
+        process.env.NODE_ENV === 'production' &&
+        (status == null || status < 400 || status >= 500)
+      );
     },
     ...options,
   }) as unknown as ErrorRequestHandler;