fix and improve comments

Author: bashmish

packages/storybook-builder/src/generate-app-script.ts

@@ -1,4 +1,4 @@
-// based on https://github.com/storybookjs/storybook/blob/v7.0.9/code/lib/builder-vite/src/codegen-modern-iframe-script.ts
+// based on https://github.com/storybookjs/storybook/blob/v8.5.0/code/builders/builder-vite/src/codegen-modern-iframe-script.ts
 
 import { normalizePath } from '@rollup/pluginutils';
 import { loadPreviewOrConfigFile } from '@storybook/core-common';

packages/storybook-builder/src/generate-iframe-html.ts

@@ -1,3 +1,5 @@
+// based on https://github.com/storybookjs/storybook/blob/v8.5.0/code/builders/builder-vite/src/transform-iframe-html.ts
+
 import { normalizeStories } from '@storybook/core-common';
 import type { DocsOptions, Options, TagsOptions } from '@storybook/types';
 import { readFile } from 'node:fs/promises';

packages/storybook-builder/src/generate-setup-addons-script.ts

@@ -1,4 +1,4 @@
-// based on https://github.com/storybookjs/storybook/blob/v7.0.9/code/lib/builder-vite/src/codegen-set-addon-channel.ts
+// based on https://github.com/storybookjs/storybook/blob/v8.5.0/code/builders/builder-vite/src/codegen-set-addon-channel.ts
 
 export async function generateSetupAddonsScript() {
   return `

packages/storybook-builder/src/generate-stories-script.ts

@@ -1,4 +1,4 @@
-// based on https://github.com/storybookjs/storybook/blob/v7.0.9/code/lib/builder-vite/src/codegen-importfn-script.ts
+// based on https://github.com/storybookjs/storybook/blob/v8.5.0/code/builders/builder-vite/src/codegen-importfn-script.ts
 
 import { normalizePath } from '@rollup/pluginutils';
 import { logger } from '@storybook/node-logger';

packages/storybook-builder/src/list-stories.ts

@@ -1,4 +1,4 @@
-// based on https://github.com/storybookjs/storybook/blob/v7.0.9/code/lib/builder-vite/src/list-stories.ts
+// based on https://github.com/storybookjs/storybook/blob/v8.5.0/code/builders/builder-vite/src/list-stories.ts
 
 import { normalizeStories } from '@storybook/core-common';
 import type { Options } from '@storybook/types';

packages/storybook-builder/src/rollup-plugin-mdx.ts

@@ -35,11 +35,11 @@ export function rollupPluginMdx(options: Options): Plugin {
       const mdxLoaderOptions = await options.presets.apply('mdxLoaderOptions', {
         ...mdxPluginOptions,
         mdxCompileOptions: {
-          // TODO(storybook): this is done by Storybook in 3 different places:
-          // 1. addon-essential preset (so not always working for all users who install plugins individually)
-          // 2. addon-docs vite plugin (so not applicable to our builder)
-          // 3. addon-docs webpack loader (also no applicable for us)
-          // so we need to do this here too, for people who individually include addon-docs
+          // this is done by Storybook in 3 different places:
+          // 1. addon-essentials preset (not working for users who install addon-docs directly without addon-essentials)
+          // 2. addon-docs vite plugin (not run for our builder, because it's in the builder-vite hook)
+          // 3. addon-docs webpack loader (not run for our builder, because it's in the builder-webpack hook)
+          // so we need to duplicate same logic here
           providerImportSource: join(
             dirname(require.resolve('@storybook/addon-docs/package.json')),
             '/dist/shims/mdx-react-shim.mjs',

packages/storybook-builder/src/rollup-plugin-prebundle-modules.ts

@@ -65,9 +65,7 @@ export function rollupPluginPrebundleModules(
   };
 }
 
-// TODO(storybook): update the CANDIDATES list
-
-// this is different to https://github.com/storybookjs/storybook/blob/v7.0.0/code/lib/builder-vite/src/optimizeDeps.ts#L7
+// this is different to https://github.com/storybookjs/storybook/blob/v8.5.0/code/builders/builder-vite/src/optimizeDeps.ts
 // builder-vite bundles different dependencies for performance reasons
 // we aim only at browserifying NodeJS dependencies (CommonJS/process.env/...)
 export const CANDIDATES = [