Remove unnecessary log suppression

Author: colinrotherham

.storybook/main.ts

@@ -2,7 +2,6 @@ import { type StorybookConfig } from '@storybook/react-vite';
 import preserveDirectives from 'rollup-preserve-directives';
 import { mergeConfig, type InlineConfig } from 'vite';
 import tsConfigPaths from 'vite-tsconfig-paths';
-import { isLogIgnored } from '../rollup.config.js';
 
 const config: StorybookConfig = {
   stories: ['../stories/**/*.stories.@(ts|tsx)', '../stories/**/*.mdx'],
@@ -20,13 +19,6 @@ const config: StorybookConfig = {
     return mergeConfig(config, {
       build: {
         rollupOptions: {
-          onwarn(warning, handler) {
-            if (isLogIgnored(warning)) {
-              return;
-            }
-
-            handler(warning);
-          },
           plugins: [preserveDirectives()],
         },
       },

rollup.config.js

@@ -65,32 +65,12 @@ export default defineConfig(
 
       // Handle warnings as errors
       onwarn(warning) {
-        if (isLogIgnored(warning)) {
-          return;
-        }
-
         throw new Error(warning.message, { cause: warning });
       },
     }),
   ),
 );
 
 /**
- * Whether to ignore Rollup log messages
- *
- * @param {RollupLog} warning
- */
-export function isLogIgnored(warning) {
-  const { code, message } = warning;
-
-  // Skip warnings related to "use client" directives including
-  // source map issues when directives are bundled by Storybook
-  return (
-    code === 'SOURCEMAP_ERROR' ||
-    (code === 'MODULE_LEVEL_DIRECTIVE' && message.includes('"use client"'))
-  );
-}
-
-/**
- * @import { OutputOptions, RollupLog, RollupOptions } from 'rollup'
+ * @import { OutputOptions, RollupOptions } from 'rollup'
  */