Merge branch 'master' of https://github.com/rollup/rollup into sync-9a7abe97

Author: docschina-bot

.github/actions/install-and-cache-node-deps/action.yml

@@ -14,7 +14,7 @@ runs:
         restore-keys: node-modules-${{ runner.os }}-${{ runner.arch }}
     - name: Install Node dependencies
       if: steps.cache-node-modules.outputs.cache-hit != 'true'
-      run: npm ${{ github.ref == 'refs/heads/master' && 'ci' || 'install' }} --ignore-scripts
+      run: npm ${{ github.ref == 'refs/heads/master' && 'ci' || 'install' }} --ignore-scripts && npm run postinstall --if-present
       shell: bash
     - name: Save Node dependencies cache
       if: steps.cache-node-modules.outputs.cache-hit != 'true' && github.ref == 'refs/heads/master'

.github/copilot-instructions.md

@@ -0,0 +1,31 @@
+# GitHub Copilot Agent Instructions for Rollup
+
+This is the Rollup project, a module bundler for JavaScript.
+
+## Technology Stack & Frameworks
+
+- The application is written in TypeScript but contains a growing part of Rust code located in the `rust` folder that is called from TypeScript.
+- The website is a Vitepress project located in the `docs` folder.
+- The build system uses Rollup for bundling the TypeScript code.
+
+## Repository Structure
+
+- TypeScript code is in the `src` directory. The entry points are `src/node-entry.ts` and `src/browser-entry.ts`.
+- Tests are in the `test` directory, with subdirectories for different test types. Most test types have a `samples` subdirectory which in itself contains other folders, one for each test case. The test case is configured via a `_config.js` file in its folder. Additionally, `samples` can also include folders that contain no `_config.js` file but contain multiple sub-folders in tests to group related test cases. All tests run against the full Rollup artifact, there are no fine-grained unit tests to allow easy refactoring of internal APIs.
+- Rust code is in the `rust` directory. It is grouped into the `bindings_napi` and `bindings_wasm` crates to interface with TypeScript via napi-rs or wasmpack, and the `parse_ast` crate that contains a parser for JavaScript that generates a binary AST format.
+- Rust code is called via `native.js` and `native.wasm.js` in the `src` directory.
+- Development scripts are in the `scripts` directory.
+- Documentation is in the `docs` directory.
+
+## Development Workflow
+
+- In order to create a production build and run all tests, use `npm test`.
+- For quicker development cycles
+  - first run either `npm run update:js` if TypeScript code was updated, or `npm run update:napi` if Rust code was updated.
+  - then run `npm run test:only` to only run tests against the built artifact without rebuilding.
+  - you can focus specific tests by adding `solo: true´ in their `\_config.js` file.
+
+## Copilot-Specific Behaviors
+
+- Only make changes that are fully justified by project documentation or explicit user requests. When in doubt, prefer conservative actions and reference relevant documentation.
+- Always consider and test for edge cases, especially when modifying core logic or build/test infrastructure.

.github/workflows/build-and-tests.yml

@@ -263,7 +263,7 @@ jobs:
         with:
           version: 0.14.1
       - name: Install cargo-zigbuild
-        uses: taiki-e/install-action@5b5de1b4da26ad411330c0454bdd72929bfcbeb2 # v2.62.29
+        uses: taiki-e/install-action@80466ef8efa80486cdfbddf929453a4f3565c791 # v2.62.34
         if: ${{ matrix.settings.cross == 'zig' }}
         env:
           GITHUB_TOKEN: ${{ github.token }}

LICENSE.md

@@ -395,7 +395,7 @@ Repository: git+https://gitlab.com/Rich-Harris/locate-character.git
 ## magic-string
 License: MIT
 By: Rich Harris
-Repository: https://github.com/rich-harris/magic-string.git
+Repository: git+https://github.com/Rich-Harris/magic-string.git
 
 > Copyright 2018 Rich Harris
 > 

browser/LICENSE.md

@@ -123,7 +123,7 @@ Repository: git+https://gitlab.com/Rich-Harris/locate-character.git
 ## magic-string
 License: MIT
 By: Rich Harris
-Repository: https://github.com/rich-harris/magic-string
+Repository: https://github.com/rich-harris/magic-string.git
 
 > Copyright 2018 Rich Harris
 > 

browser/src/fs.ts

@@ -3,7 +3,7 @@ import { throwNoFileSystem } from './error';
 import type * as FsType from './fs.ts';
 
 // Ensure this satisfies the RollupFsModule API, will be removed by tree-shaking
-const _typeTest = null as unknown as typeof FsType satisfies RollupFsModule;
+void (null as unknown as typeof FsType satisfies RollupFsModule);
 
 export const appendFile = throwNoFileSystem('fs.appendFile');
 export const copyFile = throwNoFileSystem('fs.copyFile');

docs/.vitepress/config.ts

@@ -1,9 +1,9 @@
 import alias from '@rollup/plugin-alias';
 import { transformerTwoslash } from '@shikijs/vitepress-twoslash';
-import type { Plugin } from 'vite';
 import { defineConfig } from 'vitepress';
 import { moduleAliases } from '../../build-plugins/aliases';
 import replaceBrowserModules from '../../build-plugins/replace-browser-modules';
+import type { ShikiTransformer } from '../../node_modules/vitepress/node_modules/@shikijs/types/dist/index';
 import '../declarations.d';
 import { examplesPlugin } from './create-examples';
 import { renderMermaidGraphsPlugin } from './mermaid';
@@ -57,7 +57,7 @@ export default defineConfig({
 						types: ['node']
 					}
 				}
-			})
+			}) as ShikiTransformer
 		],
 		config(md) {
 			transposeTables(md);
@@ -179,7 +179,7 @@ export default defineConfig({
 				}
 			},
 			examplesPlugin(),
-			alias(moduleAliases) as unknown as Plugin
+			alias(moduleAliases)
 		]
 	}
 });

docs/declarations.d.ts

@@ -1,5 +1,5 @@
 declare module 'examples.json' {
-	import type { Example } from './types';
+	import type { Example } from 'types';
 	const version: Record<string, Example>;
 	export default version;
 }

docs/repl/components/BundleOptions.vue

@@ -23,7 +23,7 @@
 					v-else-if="option.type === 'string'"
 					:value="option.value"
 					:placeholder="option.placeholder"
-					@input="optionsStore.set(option.name, $event.target.value)"
+					@input="optionsStore.set(option.name, ($event.target as HTMLInputElement).value)"
 				/>
 				<input
 					v-else-if="option.type === 'number'"
@@ -32,7 +32,7 @@
 					type="number"
 					min="0"
 					step="1"
-					@input="optionsStore.set(option.name, Number($event.target.value))"
+					@input="optionsStore.set(option.name, Number(($event.target as HTMLInputElement).value))"
 				/>
 				<div
 					v-for="imported in option.keys"
@@ -41,9 +41,12 @@
 					class="input-with-label"
 				>
 					<input
-						:value="option.value[imported]"
+						:value="option.value?.[imported]"
 						@input="
-							optionsStore.set(option.name, { ...option.value, [imported]: $event.target.value })
+							optionsStore.set(option.name, {
+								...option.value,
+								[imported]: ($event.target as HTMLInputElement).value
+							})
 						"
 					/>
 					<code>'{{ imported }}'</code>
@@ -53,8 +56,8 @@
 		<div v-if="optionsStore.additionalAvailableOptions.length > 0" class="add-option">
 			<select
 				@input="
-					optionsStore.addOption($event.target.value);
-					$event.target.value = '_';
+					optionsStore.addOption(($event.target as HTMLSelectElement).value);
+					($event.target as HTMLSelectElement).value = '_';
 				"
 			>
 				<option disabled selected value="_">添加设置</option>

docs/repl/stores/options.ts

@@ -20,7 +20,7 @@ interface OptionTypeSelect<T> extends BaseOptionType<T> {
 }
 
 interface OptionTypeString extends BaseOptionType<string> {
-	placeholder: string | null;
+	placeholder: string | undefined;
 	type: 'string';
 }
 
@@ -30,7 +30,7 @@ interface OptionTypeStringMapping extends BaseOptionType<Record<string, string>>
 }
 
 interface OptionTypeNumber extends BaseOptionType<number> {
-	placeholder: string | null;
+	placeholder: string | undefined;
 	type: 'number';
 }
 
@@ -617,7 +617,7 @@ function getString({
 		available: typeof available === 'function' ? computed(available) : available || alwaysTrue,
 		defaultValue: defaultValue ?? '',
 		name,
-		placeholder: placeholder || null,
+		placeholder,
 		required: required ? computed(required) : alwaysFalse,
 		type: 'string',
 		value: ref(undefined)
@@ -664,7 +664,7 @@ function getNumber({
 		available: typeof available === 'function' ? computed(available) : available || alwaysTrue,
 		defaultValue: defaultValue ?? 0,
 		name,
-		placeholder: placeholder || null,
+		placeholder,
 		required: required ? computed(required) : alwaysFalse,
 		type: 'number',
 		value: ref(undefined)