" ).appendTo( "#qunit-fixture" ); + + div.data( "foo", "bar" ); + + expectNoMessage( assert, "Regular properties", function() { + assert.strictEqual( div.data( "foo" ), "bar", "data access" ); + assert.strictEqual( jQuery.data( div[ 0 ], "foo" ), "bar", "data access (static method)" ); + } ); + + expectMessage( assert, "Properties from Object.prototype", 2, function() { + assert.ok( div.data().hasOwnProperty( "foo" ), + "hasOwnProperty works" ); + assert.ok( jQuery.data( div[ 0 ] ).hasOwnProperty( "foo" ), + "hasOwnProperty works (static method)" ); + } ); +} ); diff --git a/warnings.md b/warnings.md index f371cba9..b41ebe9c 100644 --- a/warnings.md +++ b/warnings.md @@ -144,6 +144,15 @@ This is _not_ a warning, but a console log message the plugin shows when it firs **Solution:** Always pass string values to `.css()`, and explicitly add units where required. For example, use `$.css("line-height", "2")` to specify 200% of the current line height or `$.css("line-height", "2px")` to specify pixels. When the numeric value is in a variable, ensure the value is converted to string, e.g. `$.css("line-height", String(height))` and `$.css("line-height", height+"px")`. +### \[data-null-proto\] Accessing properties from jQuery.data() inherited from Object.prototype is removed +### \[data-null-proto\] Setting properties from jQuery.data() inherited from Object.prototype is removed +### \[data-null-proto\] Accessing properties from jQuery.fn.data() inherited from Object.prototype is removed +### \[data-null-proto\] Setting properties from jQuery.fn.data() inherited from Object.prototype is removed + +**Cause:** As of jQuery 4.0.0, data objects no longer inherit from `Object.prototype`. This includes properties like `__proto__` or `hasOwnProperty`. + +**Solution:** Don't use properties inherited from `Object.prototype` on data objects. Instead of `jQuery.data( node ).hasOwnProperty( "foo" )` use `Object.hasOwn( jQuery.data( node ), "foo" )` or, if you need to support older browsers like IE 11, use `Object.prototype.hasOwnProperty.call( jQuery.data( node ), "foo" )`. + ### \[self-closed-tags\] JQMIGRATE: HTML tags must be properly nested and closed: _(HTML string)_ **Cause:** jQuery 3.5.0 changed the way it processes HTML strings. Previously, jQuery would attempt to fix self-closed tags like `` that the HTML5 specification says are not self-closed, turning it into ``. This processing can create a [security problem](https://nvd.nist.gov/vuln/detail/CVE-2020-11022) with malicious strings, so the functionality had to be removed. From e18ffd01648d3bc8115f9c1fad670e7ce46e51ee Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Tue, 1 Apr 2025 21:12:06 +0200 Subject: [PATCH 04/12] Build: Bump the github-actions group with 3 updates Bumps the github-actions group with 3 updates: [actions/setup-node](https://github.com/actions/setup-node), [actions/cache](https://github.com/actions/cache) and [github/codeql-action](https://github.com/github/codeql-action). Closes gh-567 Updates `actions/setup-node` from 4.2.0 to 4.3.0 - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](https://github.com/actions/setup-node/compare/1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a...cdca7365b2dadb8aad0a33bc7601856ffabcc48e) Updates `actions/cache` from 4.2.2 to 4.2.3 - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](https://github.com/actions/cache/compare/d4323d4df104b026a6aa633fdb11d772146be0bf...5a3ec84eff668545956fd18022155c47e93e2684) Updates `github/codeql-action` from 3.28.10 to 3.28.13 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](https://github.com/github/codeql-action/compare/b56ba49b26e50535fa1e7f7db0f4f7b4bf65d80d...1b549b9259bda1cb5ddde3b41741a82a2d15a841) --- updated-dependencies: - dependency-name: actions/setup-node dependency-version: 4.3.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: actions/cache dependency-version: 4.2.3 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions - dependency-name: github/codeql-action dependency-version: 3.28.13 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- .github/workflows/browser-tests.yml | 12 ++++++------ .github/workflows/browserstack.yml | 4 ++-- .github/workflows/codeql-analysis.yml | 6 +++--- .github/workflows/filestash.yml | 4 ++-- .github/workflows/node.js.yml | 4 ++-- 5 files changed, 15 insertions(+), 15 deletions(-) diff --git a/.github/workflows/browser-tests.yml b/.github/workflows/browser-tests.yml index 97536b40..c0d5174e 100644 --- a/.github/workflows/browser-tests.yml +++ b/.github/workflows/browser-tests.yml @@ -27,12 +27,12 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0 + uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 with: node-version: ${{ env.NODE_VERSION }} - name: Cache - uses: actions/cache@d4323d4df104b026a6aa633fdb11d772146be0bf # v4.2.2 + uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3 with: path: ~/.npm key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-npm-lock-${{ hashFiles('**/package-lock.json') }} @@ -58,12 +58,12 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0 + uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 with: node-version: ${{ env.NODE_VERSION }} - name: Cache - uses: actions/cache@d4323d4df104b026a6aa633fdb11d772146be0bf # v4.2.2 + uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3 with: path: ~/.npm key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-npm-lock-${{ hashFiles('**/package-lock.json') }} @@ -87,12 +87,12 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0 + uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 with: node-version: ${{ env.NODE_VERSION }} - name: Cache - uses: actions/cache@d4323d4df104b026a6aa633fdb11d772146be0bf # v4.2.2 + uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3 with: path: ~/.npm key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-npm-lock-${{ hashFiles('**/package-lock.json') }} diff --git a/.github/workflows/browserstack.yml b/.github/workflows/browserstack.yml index d5a60f08..00de2d41 100644 --- a/.github/workflows/browserstack.yml +++ b/.github/workflows/browserstack.yml @@ -43,12 +43,12 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0 + uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 with: node-version: ${{ env.NODE_VERSION }} - name: Cache - uses: actions/cache@d4323d4df104b026a6aa633fdb11d772146be0bf # v4.2.2 + uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3 with: path: ~/.npm key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-npm-lock-${{ hashFiles('**/package-lock.json') }} diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml index cc842ea8..6cbef952 100644 --- a/.github/workflows/codeql-analysis.yml +++ b/.github/workflows/codeql-analysis.yml @@ -33,7 +33,7 @@ jobs: # Initializes the CodeQL tools for scanning. - name: Initialize CodeQL - uses: github/codeql-action/init@b56ba49b26e50535fa1e7f7db0f4f7b4bf65d80d # v3.28.10 + uses: github/codeql-action/init@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13 # Override language selection by uncommenting this and choosing your languages # with: # languages: go, javascript, csharp, python, cpp, java @@ -41,7 +41,7 @@ jobs: # Autobuild attempts to build any compiled languages (C/C++, C#, or Java). # If this step fails, then you should remove it and run the build manually (see below) - name: Autobuild - uses: github/codeql-action/autobuild@b56ba49b26e50535fa1e7f7db0f4f7b4bf65d80d # v3.28.10 + uses: github/codeql-action/autobuild@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13 # ℹ️ Command-line programs to run using the OS shell. # 📚 https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun @@ -55,4 +55,4 @@ jobs: # make release - name: Perform CodeQL Analysis - uses: github/codeql-action/analyze@b56ba49b26e50535fa1e7f7db0f4f7b4bf65d80d # v3.28.10 + uses: github/codeql-action/analyze@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13 diff --git a/.github/workflows/filestash.yml b/.github/workflows/filestash.yml index 19f1ac6a..f5aef172 100644 --- a/.github/workflows/filestash.yml +++ b/.github/workflows/filestash.yml @@ -20,12 +20,12 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0 + uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 with: node-version: ${{ env.NODE_VERSION }} - name: Cache - uses: actions/cache@d4323d4df104b026a6aa633fdb11d772146be0bf # v4.2.2 + uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3 with: path: ~/.npm key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-npm-lock-${{ hashFiles('**/package-lock.json') }} diff --git a/.github/workflows/node.js.yml b/.github/workflows/node.js.yml index e25c648d..69bd1b51 100644 --- a/.github/workflows/node.js.yml +++ b/.github/workflows/node.js.yml @@ -17,12 +17,12 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4.2.0 + uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 with: node-version: ${{ env.NODE_VERSION }} - name: Cache - uses: actions/cache@d4323d4df104b026a6aa633fdb11d772146be0bf # v4.2.2 + uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3 with: path: ~/.npm key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-npm-lock-${{ hashFiles('**/package-lock.json') }} From 94ab4c5220376b4929d55a01a784e31dd9b630a7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 14 Apr 2025 18:04:11 +0200 Subject: [PATCH 05/12] Core: Provide a package.json exports setup inspired by jQuery Core The Migrate setup is simpler than Core as it doesn't have the slim or factory versions, but the core ideas are similar. Fixes gh-524 Closes gh-566 Ref jquery/jquery#5255 Ref jquery/jquery#5429 --- .github/workflows/browser-tests.yml | 5 +- .github/workflows/browserstack.yml | 3 + .github/workflows/node.js.yml | 27 +- .gitignore | 9 +- build/command.js | 45 + build/tasks/build-default.js | 3 - build/tasks/build-watch.js | 3 - build/tasks/build.js | 73 +- .../jquery-migrate.node-module-wrapper.js | 5 + .../jquery-migrate.bundler-require-wrapper.js | 5 + eslint.config.js | 54 +- package-lock.json | 1165 ++++++++++++++++- package.json | 42 +- src/wrapper-esm.js | 21 + test/bundler_smoke_tests/lib/run-rollup.js | 42 + test/bundler_smoke_tests/lib/run-webpack.js | 21 + test/bundler_smoke_tests/lib/utils.js | 13 + .../rollup-commonjs.config.js | 19 + .../rollup-pure-esm.config.js | 17 + test/bundler_smoke_tests/run-jsdom-tests.js | 73 ++ .../jquery-migrate-require.cjs | 5 + .../src-esm-commonjs/main.js | 10 + test/bundler_smoke_tests/src-pure-esm/main.js | 6 + test/bundler_smoke_tests/test.html | 11 + test/bundler_smoke_tests/webpack.config.cjs | 9 + .../{smoke_tests.cjs => node_smoke_tests.cjs} | 6 +- 26 files changed, 1634 insertions(+), 58 deletions(-) create mode 100755 build/command.js delete mode 100644 build/tasks/build-default.js delete mode 100644 build/tasks/build-watch.js create mode 100644 dist-module/wrappers/jquery-migrate.node-module-wrapper.js create mode 100644 dist/wrappers/jquery-migrate.bundler-require-wrapper.js create mode 100644 src/wrapper-esm.js create mode 100644 test/bundler_smoke_tests/lib/run-rollup.js create mode 100644 test/bundler_smoke_tests/lib/run-webpack.js create mode 100644 test/bundler_smoke_tests/lib/utils.js create mode 100644 test/bundler_smoke_tests/rollup-commonjs.config.js create mode 100644 test/bundler_smoke_tests/rollup-pure-esm.config.js create mode 100644 test/bundler_smoke_tests/run-jsdom-tests.js create mode 100644 test/bundler_smoke_tests/src-esm-commonjs/jquery-migrate-require.cjs create mode 100644 test/bundler_smoke_tests/src-esm-commonjs/main.js create mode 100644 test/bundler_smoke_tests/src-pure-esm/main.js create mode 100644 test/bundler_smoke_tests/test.html create mode 100644 test/bundler_smoke_tests/webpack.config.cjs rename test/node_smoke_tests/{smoke_tests.cjs => node_smoke_tests.cjs} (72%) diff --git a/.github/workflows/browser-tests.yml b/.github/workflows/browser-tests.yml index c0d5174e..32aed686 100644 --- a/.github/workflows/browser-tests.yml +++ b/.github/workflows/browser-tests.yml @@ -43,10 +43,7 @@ jobs: run: npm install - name: Run tests - run: | - npm run pretest - npm run test:unit -- -c jtr-local.yml \ - --headless -b ${{ matrix.BROWSER }} -f plugin=${{ matrix.MIGRATE_VERSION }} + run: npm run test:browser -- -f plugin=${{ matrix.MIGRATE_VERSION }} ie: runs-on: windows-latest diff --git a/.github/workflows/browserstack.yml b/.github/workflows/browserstack.yml index 00de2d41..08c1d72d 100644 --- a/.github/workflows/browserstack.yml +++ b/.github/workflows/browserstack.yml @@ -58,6 +58,9 @@ jobs: - name: Install dependencies run: npm install + - name: Build jQuery + run: npm run build:all + - name: Pretest script run: npm run pretest diff --git a/.github/workflows/node.js.yml b/.github/workflows/node.js.yml index 69bd1b51..cbaaa91a 100644 --- a/.github/workflows/node.js.yml +++ b/.github/workflows/node.js.yml @@ -5,13 +5,20 @@ on: push: branches-ignore: "dependabot/**" -env: - NODE_VERSION: 22.x - jobs: - node-smoke-test: + build-and-test: runs-on: ubuntu-latest - name: Node smoke tests + name: ${{ matrix.NPM_SCRIPT }} - ${{ matrix.NAME }} (${{ matrix.NODE_VERSION }}) + strategy: + fail-fast: false + matrix: + NAME: ["Node"] + NODE_VERSION: [18.x, 20.x, 22.x, 23.x] + NPM_SCRIPT: ["test:browserless"] + include: + - NAME: "Node" + NODE_VERSION: "22.x" + NPM_SCRIPT: "lint" steps: - name: Checkout uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 @@ -19,18 +26,18 @@ jobs: - name: Use Node.js ${{ env.NODE_VERSION }} uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 with: - node-version: ${{ env.NODE_VERSION }} + node-version: ${{ matrix.NODE_VERSION }} - name: Cache uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3 with: path: ~/.npm - key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-npm-lock-${{ hashFiles('**/package-lock.json') }} + key: ${{ runner.os }}-node-${{ matrix.NODE_VERSION }}-npm-lock-${{ hashFiles('**/package-lock.json') }} restore-keys: | - ${{ runner.os }}-node-${{ env.NODE_VERSION }}-npm-lock- + ${{ runner.os }}-node-${{ matrix.NODE_VERSION }}-npm-lock- - name: Install dependencies run: npm install - - name: Run Node smoke tests - run: npm run test:node_smoke_tests + - name: Run tests + run: npm run ${{ matrix.NPM_SCRIPT }} diff --git a/.gitignore b/.gitignore index b9bbdcd6..70266163 100644 --- a/.gitignore +++ b/.gitignore @@ -11,9 +11,16 @@ CDN .eslintcache # Ignore built files in `dist` folder -# Leave package.json +# Leave the package.json and wrappers /dist/* !/dist/package.json +!/dist/wrappers + +# Ignore built files in `dist-module` folder +# Leave the package.json and wrappers +/dist-module/* +!/dist-module/package.json +!/dist-module/wrappers /external /node_modules diff --git a/build/command.js b/build/command.js new file mode 100755 index 00000000..96a9a620 --- /dev/null +++ b/build/command.js @@ -0,0 +1,45 @@ +import yargs from "yargs/yargs"; +import { build } from "./tasks/build.js"; + +const argv = yargs( process.argv.slice( 2 ) ) + .version( false ) + .command( { + command: "[options]", + describe: "Build a jQuery Migrate bundle" + } ) + .option( "filename", { + alias: "f", + type: "string", + description: + "Set the filename of the built file. Defaults to jquery.js." + } ) + .option( "dir", { + alias: "d", + type: "string", + description: + "Set the dir to which to output the built file. Defaults to /dist." + } ) + .option( "version", { + alias: "v", + type: "string", + description: + "Set the version to include in the built file. " + + "Defaults to the version in package.json plus the " + + "short commit SHA and any excluded modules." + } ) + .option( "watch", { + alias: "w", + type: "boolean", + description: + "Watch the source files and rebuild when they change." + } ) + .option( "esm", { + type: "boolean", + description: + "Build an ES module (ESM) bundle. " + + "By default, a UMD bundle is built." + } ) + .help() + .argv; + +build( argv ); diff --git a/build/tasks/build-default.js b/build/tasks/build-default.js deleted file mode 100644 index 2950aaaf..00000000 --- a/build/tasks/build-default.js +++ /dev/null @@ -1,3 +0,0 @@ -import { build } from "./build.js"; - -build(); diff --git a/build/tasks/build-watch.js b/build/tasks/build-watch.js deleted file mode 100644 index 2835a917..00000000 --- a/build/tasks/build-watch.js +++ /dev/null @@ -1,3 +0,0 @@ -import { build } from "./build.js"; - -build( { watch: true } ); diff --git a/build/tasks/build.js b/build/tasks/build.js index 4763dc73..bc168681 100644 --- a/build/tasks/build.js +++ b/build/tasks/build.js @@ -22,6 +22,33 @@ async function readJSON( filename ) { return JSON.parse( await read( filename ) ); } +async function getOutputRollupOptions( { + esm = false +} = {} ) { + const wrapperFilePath = path.join( "src", `wrapper${ + esm ? "-esm" : "" + }.js` ); + + const wrapperSource = await read( wrapperFilePath ); + + // Catch `// @CODE` and subsequent comment lines event if they don't start + // in the first column. + const wrapper = wrapperSource.split( + /[\x20\t]*\/\/ @CODE\n(?:[\x20\t]*\/\/[^\n]+\n)*/ + ); + + return { + + // The ESM format is not actually used as we strip it during the + // build, inserting our own wrappers; it's just that it doesn't + // generate any extra wrappers so there's nothing for us to remove. + format: "esm", + + intro: wrapper[ 0 ].replace( /\n*$/, "" ), + outro: wrapper[ 1 ].replace( /^\n*/, "" ) + }; +} + async function writeCompiled( { code, dir, filename, version } ) { const compiledContents = code @@ -34,12 +61,12 @@ async function writeCompiled( { code, dir, filename, version } ) { await writeFile( path.join( dir, filename ), compiledContents ); console.log( `[${ getTimestamp() }] ${ filename } v${ version } created.` ); - await minify( { dir, filename, version } ); } export async function build( { dir = "dist", filename = "jquery-migrate.js", + esm = false, watch = false, version } = {} ) { @@ -59,24 +86,8 @@ export async function build( { }`; } - // Catch `// @CODE` and subsequent comment lines event if they don't start - // in the first column. - const wrapperSrc = await read( "src/wrapper.js" ); - const wrapper = wrapperSrc.split( - /[\x20\t]*\/\/ @CODE\n(?:[\x20\t]*\/\/[^\n]+\n)*/ - ); - const inputRollupOptions = {}; - const outputRollupOptions = { - - // The ESM format is not actually used as we strip it during - // the build; it's just that it doesn't generate any extra - // wrappers so there's nothing for us to remove. - format: "esm", - - intro: wrapper[ 0 ].replace( /\n*$/, "" ), - outro: wrapper[ 1 ].replace( /^\n*/, "" ) - }; + const outputRollupOptions = await getOutputRollupOptions( { esm } ); const src = "src/migrate.js"; inputRollupOptions.input = path.resolve( src ); @@ -122,9 +133,33 @@ export async function build( { } = await bundle.generate( outputRollupOptions ); await writeCompiled( { code, dir, filename, version } ); + await minify( { dir, filename, version } ); + } +} +export async function buildDefaultFiles( { + version = process.env.VERSION, + watch +} = {} ) { + await Promise.all( [ + build( { version, watch } ), + build( { + dir: "dist-module", + filename: "jquery-migrate.module.js", + esm: true, + version, + watch + } ) + ] ); + + if ( watch ) { + console.log( "Watching files..." ); + } else { return compareSize( { - files: [ "dist/jquery-migrate.min.js" ] + files: [ + "dist/jquery-migrate.min.js", + "dist-module/jquery-migrate.module.min.js" + ] } ); } } diff --git a/dist-module/wrappers/jquery-migrate.node-module-wrapper.js b/dist-module/wrappers/jquery-migrate.node-module-wrapper.js new file mode 100644 index 00000000..5012bb4f --- /dev/null +++ b/dist-module/wrappers/jquery-migrate.node-module-wrapper.js @@ -0,0 +1,5 @@ +// Node.js is able to import from a CommonJS module in an ESM one. +import jQuery from "../../dist/jquery-migrate.js"; + +export { jQuery, jQuery as $ }; +export default jQuery; diff --git a/dist/wrappers/jquery-migrate.bundler-require-wrapper.js b/dist/wrappers/jquery-migrate.bundler-require-wrapper.js new file mode 100644 index 00000000..ad414483 --- /dev/null +++ b/dist/wrappers/jquery-migrate.bundler-require-wrapper.js @@ -0,0 +1,5 @@ +"use strict"; + +// Bundlers are able to synchronously require an ESM module from a CommonJS one. +const { jQuery } = require( "../../dist-module/jquery-migrate.module.js" ); +module.exports = jQuery; diff --git a/eslint.config.js b/eslint.config.js index b22b02c6..31947c0d 100644 --- a/eslint.config.js +++ b/eslint.config.js @@ -12,7 +12,13 @@ export default [ }, { - files: [ "eslint.config.js", "build/**" ], + files: [ + "eslint.config.js", + ".release-it.cjs", + "build/**", + "test/node_smoke_tests/**", + "test/bundler_smoke_tests/**/*" + ], languageOptions: { ecmaVersion: "latest", globals: { @@ -60,18 +66,44 @@ export default [ } }, + { + files: [ + "src/wrapper.js", + "src/wrapper-esm.js", + "src/wrapper-factory.js", + "src/wrapper-factory-esm.js" + ], + languageOptions: { + globals: { + jQuery: false + } + }, + rules: { + "no-unused-vars": "off", + indent: [ + "error", + "tab", + { + + // This makes it so code within the wrapper is not indented. + ignoredNodes: [ + "Program > FunctionDeclaration > *" + ] + } + ] + } + }, + { files: [ "src/wrapper.js" ], languageOptions: { sourceType: "script", globals: { - jQuery: false, define: false, module: false } }, rules: { - "no-unused-vars": "off", indent: [ "error", "tab", @@ -203,5 +235,21 @@ export default [ ecmaVersion: 5, sourceType: "script" } + }, + + { + files: [ "dist-module/**" ], + languageOptions: { + ecmaVersion: 2015, + sourceType: "module" + } + }, + + { + files: [ "dist/wrappers/*.js" ], + languageOptions: { + ecmaVersion: 2015, + sourceType: "commonjs" + } } ]; diff --git a/package-lock.json b/package-lock.json index a2309d04..dee1b057 100644 --- a/package-lock.json +++ b/package-lock.json @@ -9,6 +9,8 @@ "version": "4.0.0-pre", "license": "MIT", "devDependencies": { + "@rollup/plugin-commonjs": "28.0.3", + "@rollup/plugin-node-resolve": "16.0.1", "chalk": "5.4.1", "commitplease": "3.2.0", "enquirer": "2.4.1", @@ -24,7 +26,9 @@ "qunit": "2.24.1", "rollup": "4.34.8", "sinon": "9.2.4", - "uglify-js": "3.19.3" + "uglify-js": "3.19.3", + "webpack": "5.98.0", + "yargs": "^17.7.2" }, "peerDependencies": { "jquery": ">=4 <5" @@ -283,6 +287,70 @@ "dev": true, "license": "BSD-3-Clause" }, + "node_modules/@jridgewell/gen-mapping": { + "version": "0.3.8", + "resolved": "https://registry.npmjs.org/@jridgewell/gen-mapping/-/gen-mapping-0.3.8.tgz", + "integrity": "sha512-imAbBGkb+ebQyxKgzv5Hu2nmROxoDOXHh80evxdoXNOrvAnVx7zimzc1Oo5h9RlfV4vPXaE2iM5pOFbvOCClWA==", + "dev": true, + "license": "MIT", + "dependencies": { + "@jridgewell/set-array": "^1.2.1", + "@jridgewell/sourcemap-codec": "^1.4.10", + "@jridgewell/trace-mapping": "^0.3.24" + }, + "engines": { + "node": ">=6.0.0" + } + }, + "node_modules/@jridgewell/resolve-uri": { + "version": "3.1.2", + "resolved": "https://registry.npmjs.org/@jridgewell/resolve-uri/-/resolve-uri-3.1.2.tgz", + "integrity": "sha512-bRISgCIjP20/tbWSPWMEi54QVPRZExkuD9lJL+UIxUKtwVJA8wW1Trb1jMs1RFXo1CBTNZ/5hpC9QvmKWdopKw==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=6.0.0" + } + }, + "node_modules/@jridgewell/set-array": { + "version": "1.2.1", + "resolved": "https://registry.npmjs.org/@jridgewell/set-array/-/set-array-1.2.1.tgz", + "integrity": "sha512-R8gLRTZeyp03ymzP/6Lil/28tGeGEzhx1q2k703KGWRAI1VdvPIXdG70VJc2pAMw3NA6JKL5hhFu1sJX0Mnn/A==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=6.0.0" + } + }, + "node_modules/@jridgewell/source-map": { + "version": "0.3.6", + "resolved": "https://registry.npmjs.org/@jridgewell/source-map/-/source-map-0.3.6.tgz", + "integrity": "sha512-1ZJTZebgqllO79ue2bm3rIGud/bOe0pP5BjSRCRxxYkEZS8STV7zN84UBbiYu7jy+eCKSnVIUgoWWE/tt+shMQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@jridgewell/gen-mapping": "^0.3.5", + "@jridgewell/trace-mapping": "^0.3.25" + } + }, + "node_modules/@jridgewell/sourcemap-codec": { + "version": "1.5.0", + "resolved": "https://registry.npmjs.org/@jridgewell/sourcemap-codec/-/sourcemap-codec-1.5.0.tgz", + "integrity": "sha512-gv3ZRaISU3fjPAgNsriBRqGWQL6quFx04YMPW/zD8XMLsU32mhCCbfbO6KZFLjvYpCZ8zyDEgqsgf+PwPaM7GQ==", + "dev": true, + "license": "MIT" + }, + "node_modules/@jridgewell/trace-mapping": { + "version": "0.3.25", + "resolved": "https://registry.npmjs.org/@jridgewell/trace-mapping/-/trace-mapping-0.3.25.tgz", + "integrity": "sha512-vNk6aEwybGtawWmy/PzwnGDOjCkLWSD2wqvjGGAgOAwCGWySYXfYoxt00IJkTF+8Lb57DwOb3Aa0o9CApepiYQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@jridgewell/resolve-uri": "^3.1.0", + "@jridgewell/sourcemap-codec": "^1.4.14" + } + }, "node_modules/@nodelib/fs.scandir": { "version": "2.1.5", "resolved": "https://registry.npmjs.org/@nodelib/fs.scandir/-/fs.scandir-2.1.5.tgz", @@ -321,6 +389,81 @@ "node": ">= 8" } }, + "node_modules/@rollup/plugin-commonjs": { + "version": "28.0.3", + "resolved": "https://registry.npmjs.org/@rollup/plugin-commonjs/-/plugin-commonjs-28.0.3.tgz", + "integrity": "sha512-pyltgilam1QPdn+Zd9gaCfOLcnjMEJ9gV+bTw6/r73INdvzf1ah9zLIJBm+kW7R6IUFIQ1YO+VqZtYxZNWFPEQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@rollup/pluginutils": "^5.0.1", + "commondir": "^1.0.1", + "estree-walker": "^2.0.2", + "fdir": "^6.2.0", + "is-reference": "1.2.1", + "magic-string": "^0.30.3", + "picomatch": "^4.0.2" + }, + "engines": { + "node": ">=16.0.0 || 14 >= 14.17" + }, + "peerDependencies": { + "rollup": "^2.68.0||^3.0.0||^4.0.0" + }, + "peerDependenciesMeta": { + "rollup": { + "optional": true + } + } + }, + "node_modules/@rollup/plugin-node-resolve": { + "version": "16.0.1", + "resolved": "https://registry.npmjs.org/@rollup/plugin-node-resolve/-/plugin-node-resolve-16.0.1.tgz", + "integrity": "sha512-tk5YCxJWIG81umIvNkSod2qK5KyQW19qcBF/B78n1bjtOON6gzKoVeSzAE8yHCZEDmqkHKkxplExA8KzdJLJpA==", + "dev": true, + "license": "MIT", + "dependencies": { + "@rollup/pluginutils": "^5.0.1", + "@types/resolve": "1.20.2", + "deepmerge": "^4.2.2", + "is-module": "^1.0.0", + "resolve": "^1.22.1" + }, + "engines": { + "node": ">=14.0.0" + }, + "peerDependencies": { + "rollup": "^2.78.0||^3.0.0||^4.0.0" + }, + "peerDependenciesMeta": { + "rollup": { + "optional": true + } + } + }, + "node_modules/@rollup/pluginutils": { + "version": "5.1.4", + "resolved": "https://registry.npmjs.org/@rollup/pluginutils/-/pluginutils-5.1.4.tgz", + "integrity": "sha512-USm05zrsFxYLPdWWq+K3STlWiT/3ELn3RcV5hJMghpeAIhxfsUIg6mt12CBJBInWMV4VneoV7SfGv8xIwo2qNQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/estree": "^1.0.0", + "estree-walker": "^2.0.2", + "picomatch": "^4.0.2" + }, + "engines": { + "node": ">=14.0.0" + }, + "peerDependencies": { + "rollup": "^1.20.0||^2.0.0||^3.0.0||^4.0.0" + }, + "peerDependenciesMeta": { + "rollup": { + "optional": true + } + } + }, "node_modules/@rollup/rollup-android-arm-eabi": { "version": "4.34.8", "resolved": "https://registry.npmjs.org/@rollup/rollup-android-arm-eabi/-/rollup-android-arm-eabi-4.34.8.tgz", @@ -633,6 +776,28 @@ "dev": true, "license": "(Unlicense OR Apache-2.0)" }, + "node_modules/@types/eslint": { + "version": "9.6.1", + "resolved": "https://registry.npmjs.org/@types/eslint/-/eslint-9.6.1.tgz", + "integrity": "sha512-FXx2pKgId/WyYo2jXw63kk7/+TY7u7AziEJxJAnSFzHlqTAS3Ync6SvgYAN/k4/PQpnnVuzoMuVnByKK2qp0ag==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/estree": "*", + "@types/json-schema": "*" + } + }, + "node_modules/@types/eslint-scope": { + "version": "3.7.7", + "resolved": "https://registry.npmjs.org/@types/eslint-scope/-/eslint-scope-3.7.7.tgz", + "integrity": "sha512-MzMFlSLBqNF2gcHWO0G1vP/YQyfvrxZ0bF+u7mzUdZ1/xK4A4sru+nraZz5i3iEIk1l1uyicaDVTB4QbbEkAYg==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/eslint": "*", + "@types/estree": "*" + } + }, "node_modules/@types/estree": { "version": "1.0.6", "resolved": "https://registry.npmjs.org/@types/estree/-/estree-1.0.6.tgz", @@ -640,6 +805,13 @@ "dev": true, "license": "MIT" }, + "node_modules/@types/json-schema": { + "version": "7.0.15", + "resolved": "https://registry.npmjs.org/@types/json-schema/-/json-schema-7.0.15.tgz", + "integrity": "sha512-5+fP8P8MFNC+AyZCDxrB2pkZFPGzqQWUzpSeuuVLvm8VMcorNYavBqoFcxK8bQz4Qsbn4oUEEem4wDLfcysGHA==", + "dev": true, + "license": "MIT" + }, "node_modules/@types/json5": { "version": "0.0.29", "resolved": "https://registry.npmjs.org/@types/json5/-/json5-0.0.29.tgz", @@ -647,6 +819,23 @@ "dev": true, "license": "MIT" }, + "node_modules/@types/node": { + "version": "22.13.14", + "resolved": "https://registry.npmjs.org/@types/node/-/node-22.13.14.tgz", + "integrity": "sha512-Zs/Ollc1SJ8nKUAgc7ivOEdIBM8JAKgrqqUYi2J997JuKO7/tpQC+WCetQ1sypiKCQWHdvdg9wBNpUPEWZae7w==", + "dev": true, + "license": "MIT", + "dependencies": { + "undici-types": "~6.20.0" + } + }, + "node_modules/@types/resolve": { + "version": "1.20.2", + "resolved": "https://registry.npmjs.org/@types/resolve/-/resolve-1.20.2.tgz", + "integrity": "sha512-60BCwRFOZCQhDncwQdxxeOEEkbc5dIMccYLwbxsS4TUNeVECQ/pBJ0j09mrHOl/JJvpRPGwO9SvE4nR2Nb/a4Q==", + "dev": true, + "license": "MIT" + }, "node_modules/@ungap/structured-clone": { "version": "1.3.0", "resolved": "https://registry.npmjs.org/@ungap/structured-clone/-/structured-clone-1.3.0.tgz", @@ -654,6 +843,181 @@ "dev": true, "license": "ISC" }, + "node_modules/@webassemblyjs/ast": { + "version": "1.14.1", + "resolved": "https://registry.npmjs.org/@webassemblyjs/ast/-/ast-1.14.1.tgz", + "integrity": "sha512-nuBEDgQfm1ccRp/8bCQrx1frohyufl4JlbMMZ4P1wpeOfDhF6FQkxZJ1b/e+PLwr6X1Nhw6OLme5usuBWYBvuQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@webassemblyjs/helper-numbers": "1.13.2", + "@webassemblyjs/helper-wasm-bytecode": "1.13.2" + } + }, + "node_modules/@webassemblyjs/floating-point-hex-parser": { + "version": "1.13.2", + "resolved": "https://registry.npmjs.org/@webassemblyjs/floating-point-hex-parser/-/floating-point-hex-parser-1.13.2.tgz", + "integrity": "sha512-6oXyTOzbKxGH4steLbLNOu71Oj+C8Lg34n6CqRvqfS2O71BxY6ByfMDRhBytzknj9yGUPVJ1qIKhRlAwO1AovA==", + "dev": true, + "license": "MIT" + }, + "node_modules/@webassemblyjs/helper-api-error": { + "version": "1.13.2", + "resolved": "https://registry.npmjs.org/@webassemblyjs/helper-api-error/-/helper-api-error-1.13.2.tgz", + "integrity": "sha512-U56GMYxy4ZQCbDZd6JuvvNV/WFildOjsaWD3Tzzvmw/mas3cXzRJPMjP83JqEsgSbyrmaGjBfDtV7KDXV9UzFQ==", + "dev": true, + "license": "MIT" + }, + "node_modules/@webassemblyjs/helper-buffer": { + "version": "1.14.1", + "resolved": "https://registry.npmjs.org/@webassemblyjs/helper-buffer/-/helper-buffer-1.14.1.tgz", + "integrity": "sha512-jyH7wtcHiKssDtFPRB+iQdxlDf96m0E39yb0k5uJVhFGleZFoNw1c4aeIcVUPPbXUVJ94wwnMOAqUHyzoEPVMA==", + "dev": true, + "license": "MIT" + }, + "node_modules/@webassemblyjs/helper-numbers": { + "version": "1.13.2", + "resolved": "https://registry.npmjs.org/@webassemblyjs/helper-numbers/-/helper-numbers-1.13.2.tgz", + "integrity": "sha512-FE8aCmS5Q6eQYcV3gI35O4J789wlQA+7JrqTTpJqn5emA4U2hvwJmvFRC0HODS+3Ye6WioDklgd6scJ3+PLnEA==", + "dev": true, + "license": "MIT", + "dependencies": { + "@webassemblyjs/floating-point-hex-parser": "1.13.2", + "@webassemblyjs/helper-api-error": "1.13.2", + "@xtuc/long": "4.2.2" + } + }, + "node_modules/@webassemblyjs/helper-wasm-bytecode": { + "version": "1.13.2", + "resolved": "https://registry.npmjs.org/@webassemblyjs/helper-wasm-bytecode/-/helper-wasm-bytecode-1.13.2.tgz", + "integrity": "sha512-3QbLKy93F0EAIXLh0ogEVR6rOubA9AoZ+WRYhNbFyuB70j3dRdwH9g+qXhLAO0kiYGlg3TxDV+I4rQTr/YNXkA==", + "dev": true, + "license": "MIT" + }, + "node_modules/@webassemblyjs/helper-wasm-section": { + "version": "1.14.1", + "resolved": "https://registry.npmjs.org/@webassemblyjs/helper-wasm-section/-/helper-wasm-section-1.14.1.tgz", + "integrity": "sha512-ds5mXEqTJ6oxRoqjhWDU83OgzAYjwsCV8Lo/N+oRsNDmx/ZDpqalmrtgOMkHwxsG0iI//3BwWAErYRHtgn0dZw==", + "dev": true, + "license": "MIT", + "dependencies": { + "@webassemblyjs/ast": "1.14.1", + "@webassemblyjs/helper-buffer": "1.14.1", + "@webassemblyjs/helper-wasm-bytecode": "1.13.2", + "@webassemblyjs/wasm-gen": "1.14.1" + } + }, + "node_modules/@webassemblyjs/ieee754": { + "version": "1.13.2", + "resolved": "https://registry.npmjs.org/@webassemblyjs/ieee754/-/ieee754-1.13.2.tgz", + "integrity": "sha512-4LtOzh58S/5lX4ITKxnAK2USuNEvpdVV9AlgGQb8rJDHaLeHciwG4zlGr0j/SNWlr7x3vO1lDEsuePvtcDNCkw==", + "dev": true, + "license": "MIT", + "dependencies": { + "@xtuc/ieee754": "^1.2.0" + } + }, + "node_modules/@webassemblyjs/leb128": { + "version": "1.13.2", + "resolved": "https://registry.npmjs.org/@webassemblyjs/leb128/-/leb128-1.13.2.tgz", + "integrity": "sha512-Lde1oNoIdzVzdkNEAWZ1dZ5orIbff80YPdHx20mrHwHrVNNTjNr8E3xz9BdpcGqRQbAEa+fkrCb+fRFTl/6sQw==", + "dev": true, + "license": "Apache-2.0", + "dependencies": { + "@xtuc/long": "4.2.2" + } + }, + "node_modules/@webassemblyjs/utf8": { + "version": "1.13.2", + "resolved": "https://registry.npmjs.org/@webassemblyjs/utf8/-/utf8-1.13.2.tgz", + "integrity": "sha512-3NQWGjKTASY1xV5m7Hr0iPeXD9+RDobLll3T9d2AO+g3my8xy5peVyjSag4I50mR1bBSN/Ct12lo+R9tJk0NZQ==", + "dev": true, + "license": "MIT" + }, + "node_modules/@webassemblyjs/wasm-edit": { + "version": "1.14.1", + "resolved": "https://registry.npmjs.org/@webassemblyjs/wasm-edit/-/wasm-edit-1.14.1.tgz", + "integrity": "sha512-RNJUIQH/J8iA/1NzlE4N7KtyZNHi3w7at7hDjvRNm5rcUXa00z1vRz3glZoULfJ5mpvYhLybmVcwcjGrC1pRrQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@webassemblyjs/ast": "1.14.1", + "@webassemblyjs/helper-buffer": "1.14.1", + "@webassemblyjs/helper-wasm-bytecode": "1.13.2", + "@webassemblyjs/helper-wasm-section": "1.14.1", + "@webassemblyjs/wasm-gen": "1.14.1", + "@webassemblyjs/wasm-opt": "1.14.1", + "@webassemblyjs/wasm-parser": "1.14.1", + "@webassemblyjs/wast-printer": "1.14.1" + } + }, + "node_modules/@webassemblyjs/wasm-gen": { + "version": "1.14.1", + "resolved": "https://registry.npmjs.org/@webassemblyjs/wasm-gen/-/wasm-gen-1.14.1.tgz", + "integrity": "sha512-AmomSIjP8ZbfGQhumkNvgC33AY7qtMCXnN6bL2u2Js4gVCg8fp735aEiMSBbDR7UQIj90n4wKAFUSEd0QN2Ukg==", + "dev": true, + "license": "MIT", + "dependencies": { + "@webassemblyjs/ast": "1.14.1", + "@webassemblyjs/helper-wasm-bytecode": "1.13.2", + "@webassemblyjs/ieee754": "1.13.2", + "@webassemblyjs/leb128": "1.13.2", + "@webassemblyjs/utf8": "1.13.2" + } + }, + "node_modules/@webassemblyjs/wasm-opt": { + "version": "1.14.1", + "resolved": "https://registry.npmjs.org/@webassemblyjs/wasm-opt/-/wasm-opt-1.14.1.tgz", + "integrity": "sha512-PTcKLUNvBqnY2U6E5bdOQcSM+oVP/PmrDY9NzowJjislEjwP/C4an2303MCVS2Mg9d3AJpIGdUFIQQWbPds0Sw==", + "dev": true, + "license": "MIT", + "dependencies": { + "@webassemblyjs/ast": "1.14.1", + "@webassemblyjs/helper-buffer": "1.14.1", + "@webassemblyjs/wasm-gen": "1.14.1", + "@webassemblyjs/wasm-parser": "1.14.1" + } + }, + "node_modules/@webassemblyjs/wasm-parser": { + "version": "1.14.1", + "resolved": "https://registry.npmjs.org/@webassemblyjs/wasm-parser/-/wasm-parser-1.14.1.tgz", + "integrity": "sha512-JLBl+KZ0R5qB7mCnud/yyX08jWFw5MsoalJ1pQ4EdFlgj9VdXKGuENGsiCIjegI1W7p91rUlcB/LB5yRJKNTcQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@webassemblyjs/ast": "1.14.1", + "@webassemblyjs/helper-api-error": "1.13.2", + "@webassemblyjs/helper-wasm-bytecode": "1.13.2", + "@webassemblyjs/ieee754": "1.13.2", + "@webassemblyjs/leb128": "1.13.2", + "@webassemblyjs/utf8": "1.13.2" + } + }, + "node_modules/@webassemblyjs/wast-printer": { + "version": "1.14.1", + "resolved": "https://registry.npmjs.org/@webassemblyjs/wast-printer/-/wast-printer-1.14.1.tgz", + "integrity": "sha512-kPSSXE6De1XOR820C90RIo2ogvZG+c3KiHzqUoO/F34Y2shGzesfqv7o57xrxovZJH/MetF5UjroJ/R/3isoiw==", + "dev": true, + "license": "MIT", + "dependencies": { + "@webassemblyjs/ast": "1.14.1", + "@xtuc/long": "4.2.2" + } + }, + "node_modules/@xtuc/ieee754": { + "version": "1.2.0", + "resolved": "https://registry.npmjs.org/@xtuc/ieee754/-/ieee754-1.2.0.tgz", + "integrity": "sha512-DX8nKgqcGwsc0eJSqYt5lwP4DH5FlHnmuWWBRy7X0NcaGR0ZtuyeESgMwTYVEtxmsNGY+qit4QYT/MIYTOTPeA==", + "dev": true, + "license": "BSD-3-Clause" + }, + "node_modules/@xtuc/long": { + "version": "4.2.2", + "resolved": "https://registry.npmjs.org/@xtuc/long/-/long-4.2.2.tgz", + "integrity": "sha512-NuHqBY1PB/D8xU6s/thBgOAiAP7HOYDQ32+BFZILJ8ivkUkAHQnWfn6WhL79Owj1qmUnoN/YPhktdIoucipkAQ==", + "dev": true, + "license": "Apache-2.0" + }, "node_modules/acorn": { "version": "8.14.0", "resolved": "https://registry.npmjs.org/acorn/-/acorn-8.14.0.tgz", @@ -707,6 +1071,48 @@ "url": "https://github.com/sponsors/epoberezkin" } }, + "node_modules/ajv-formats": { + "version": "2.1.1", + "resolved": "https://registry.npmjs.org/ajv-formats/-/ajv-formats-2.1.1.tgz", + "integrity": "sha512-Wx0Kx52hxE7C18hkMEggYlEifqWZtYaRgouJor+WMdPnQyEK13vgEWyVNup7SoeeoLMsr4kf5h6dOW11I15MUA==", + "dev": true, + "license": "MIT", + "dependencies": { + "ajv": "^8.0.0" + }, + "peerDependencies": { + "ajv": "^8.0.0" + }, + "peerDependenciesMeta": { + "ajv": { + "optional": true + } + } + }, + "node_modules/ajv-formats/node_modules/ajv": { + "version": "8.17.1", + "resolved": "https://registry.npmjs.org/ajv/-/ajv-8.17.1.tgz", + "integrity": "sha512-B/gBuNg5SiMTrPkC+A2+cW0RszwxYmn6VYxB/inlBStS5nx6xHIt/ehKRhIMhqusl7a8LjQoZnjCs5vhwxOQ1g==", + "dev": true, + "license": "MIT", + "dependencies": { + "fast-deep-equal": "^3.1.3", + "fast-uri": "^3.0.1", + "json-schema-traverse": "^1.0.0", + "require-from-string": "^2.0.2" + }, + "funding": { + "type": "github", + "url": "https://github.com/sponsors/epoberezkin" + } + }, + "node_modules/ajv-formats/node_modules/json-schema-traverse": { + "version": "1.0.0", + "resolved": "https://registry.npmjs.org/json-schema-traverse/-/json-schema-traverse-1.0.0.tgz", + "integrity": "sha512-NM8/P9n3XjXhIZn1lLhkFaACTOURQXjWhV4BA/RnOv8xvgqtqpAX9IO4mRQxSx1Rlo4tqzeqb0sOlruaOy3dug==", + "dev": true, + "license": "MIT" + }, "node_modules/ansi-colors": { "version": "4.1.3", "resolved": "https://registry.npmjs.org/ansi-colors/-/ansi-colors-4.1.3.tgz", @@ -914,6 +1320,39 @@ "concat-map": "0.0.1" } }, + "node_modules/browserslist": { + "version": "4.24.4", + "resolved": "https://registry.npmjs.org/browserslist/-/browserslist-4.24.4.tgz", + "integrity": "sha512-KDi1Ny1gSePi1vm0q4oxSF8b4DR44GF4BbmS2YdhPLOEqd8pDviZOGH/GsmRwoWJ2+5Lr085X7naowMwKHDG1A==", + "dev": true, + "funding": [ + { + "type": "opencollective", + "url": "https://opencollective.com/browserslist" + }, + { + "type": "tidelift", + "url": "https://tidelift.com/funding/github/npm/browserslist" + }, + { + "type": "github", + "url": "https://github.com/sponsors/ai" + } + ], + "license": "MIT", + "dependencies": { + "caniuse-lite": "^1.0.30001688", + "electron-to-chromium": "^1.5.73", + "node-releases": "^2.0.19", + "update-browserslist-db": "^1.1.1" + }, + "bin": { + "browserslist": "cli.js" + }, + "engines": { + "node": "^6 || ^7 || ^8 || ^9 || ^10 || ^11 || ^12 || >=13.7" + } + }, "node_modules/browserstack-local": { "version": "1.5.6", "resolved": "https://registry.npmjs.org/browserstack-local/-/browserstack-local-1.5.6.tgz", @@ -928,6 +1367,13 @@ "temp-fs": "^0.9.9" } }, + "node_modules/buffer-from": { + "version": "1.1.2", + "resolved": "https://registry.npmjs.org/buffer-from/-/buffer-from-1.1.2.tgz", + "integrity": "sha512-E+XQCRwSbaaiChtv6k6Dwgc+bx+Bs6vuKJHHl5kox/BaKbhiXzqQOwK4cO22yElGp2OCmjwVhT3HmxgyPGnJfQ==", + "dev": true, + "license": "MIT" + }, "node_modules/bytes": { "version": "3.1.2", "resolved": "https://registry.npmjs.org/bytes/-/bytes-3.1.2.tgz", @@ -998,6 +1444,27 @@ "node": ">=6" } }, + "node_modules/caniuse-lite": { + "version": "1.0.30001707", + "resolved": "https://registry.npmjs.org/caniuse-lite/-/caniuse-lite-1.0.30001707.tgz", + "integrity": "sha512-3qtRjw/HQSMlDWf+X79N206fepf4SOOU6SQLMaq/0KkZLmSjPxAkBOQQ+FxbHKfHmYLZFfdWsO3KA90ceHPSnw==", + "dev": true, + "funding": [ + { + "type": "opencollective", + "url": "https://opencollective.com/browserslist" + }, + { + "type": "tidelift", + "url": "https://tidelift.com/funding/github/npm/caniuse-lite" + }, + { + "type": "github", + "url": "https://github.com/sponsors/ai" + } + ], + "license": "CC-BY-4.0" + }, "node_modules/chalk": { "version": "5.4.1", "resolved": "https://registry.npmjs.org/chalk/-/chalk-5.4.1.tgz", @@ -1011,6 +1478,31 @@ "url": "https://github.com/chalk/chalk?sponsor=1" } }, + "node_modules/chrome-trace-event": { + "version": "1.0.4", + "resolved": "https://registry.npmjs.org/chrome-trace-event/-/chrome-trace-event-1.0.4.tgz", + "integrity": "sha512-rNjApaLzuwaOTjCiT8lSDdGN1APCiqkChLMJxJPWLunPAt5fy8xgU9/jNOchV84wfIxrA0lRQB7oCT8jrn/wrQ==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=6.0" + } + }, + "node_modules/cliui": { + "version": "8.0.1", + "resolved": "https://registry.npmjs.org/cliui/-/cliui-8.0.1.tgz", + "integrity": "sha512-BSeNnyus75C4//NQ9gQt1/csTXyo/8Sb+afLAkzAptFuMsod9HFokGNudZpi/oQV73hnVK+sR+5PVRMd+Dr7YQ==", + "dev": true, + "license": "ISC", + "dependencies": { + "string-width": "^4.2.0", + "strip-ansi": "^6.0.1", + "wrap-ansi": "^7.0.0" + }, + "engines": { + "node": ">=12" + } + }, "node_modules/color-convert": { "version": "2.0.1", "resolved": "https://registry.npmjs.org/color-convert/-/color-convert-2.0.1.tgz", @@ -1131,6 +1623,13 @@ "node": ">=0.8.0" } }, + "node_modules/commondir": { + "version": "1.0.1", + "resolved": "https://registry.npmjs.org/commondir/-/commondir-1.0.1.tgz", + "integrity": "sha512-W9pAhw0ja1Edb5GVdIF1mjZw/ASI0AlShXM83UUGe2DVr5TdAPEA1OA8m/g8zWp9x6On7gqufY+FatDbC3MDQg==", + "dev": true, + "license": "MIT" + }, "node_modules/concat-map": { "version": "0.0.1", "resolved": "https://registry.npmjs.org/concat-map/-/concat-map-0.0.1.tgz", @@ -1274,6 +1773,16 @@ "dev": true, "license": "MIT" }, + "node_modules/deepmerge": { + "version": "4.3.1", + "resolved": "https://registry.npmjs.org/deepmerge/-/deepmerge-4.3.1.tgz", + "integrity": "sha512-3sUqbMEc77XqpdNO7FRyRog+eW3ph+GYCbj+rK+uYyRMuwsVy0rMiVtPn+QJlKFvWP/1PYpapqYn0Me2knFn+A==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=0.10.0" + } + }, "node_modules/define-data-property": { "version": "1.1.4", "resolved": "https://registry.npmjs.org/define-data-property/-/define-data-property-1.1.4.tgz", @@ -1375,6 +1884,34 @@ "dev": true, "license": "MIT" }, + "node_modules/electron-to-chromium": { + "version": "1.5.128", + "resolved": "https://registry.npmjs.org/electron-to-chromium/-/electron-to-chromium-1.5.128.tgz", + "integrity": "sha512-bo1A4HH/NS522Ws0QNFIzyPcyUUNV/yyy70Ho1xqfGYzPUme2F/xr4tlEOuM6/A538U1vDA7a4XfCd1CKRegKQ==", + "dev": true, + "license": "ISC" + }, + "node_modules/emoji-regex": { + "version": "8.0.0", + "resolved": "https://registry.npmjs.org/emoji-regex/-/emoji-regex-8.0.0.tgz", + "integrity": "sha512-MSjYzcWNOA0ewAHpz0MxpYFvwg6yjy1NG3xteoqz644VCo/RPgnr1/GGt+ic3iJTzQ8Eu3TdM14SawnVUmGE6A==", + "dev": true, + "license": "MIT" + }, + "node_modules/enhanced-resolve": { + "version": "5.18.1", + "resolved": "https://registry.npmjs.org/enhanced-resolve/-/enhanced-resolve-5.18.1.tgz", + "integrity": "sha512-ZSW3ma5GkcQBIpwZTSRAI8N71Uuwgs93IezB7mf7R60tC8ZbJideoDNKjHn2O9KIlx6rkGTTEk1xUCK2E1Y2Yg==", + "dev": true, + "license": "MIT", + "dependencies": { + "graceful-fs": "^4.2.4", + "tapable": "^2.2.0" + }, + "engines": { + "node": ">=10.13.0" + } + }, "node_modules/enquirer": { "version": "2.4.1", "resolved": "https://registry.npmjs.org/enquirer/-/enquirer-2.4.1.tgz", @@ -1488,6 +2025,13 @@ "node": ">= 0.4" } }, + "node_modules/es-module-lexer": { + "version": "1.6.0", + "resolved": "https://registry.npmjs.org/es-module-lexer/-/es-module-lexer-1.6.0.tgz", + "integrity": "sha512-qqnD1yMU6tk/jnaMosogGySTZP8YtUgAffA9nMN+E/rjxcfRQ6IEk7IiozUjgxKoFHBGjTLnrHB/YC45r/59EQ==", + "dev": true, + "license": "MIT" + }, "node_modules/es-object-atoms": { "version": "1.1.1", "resolved": "https://registry.npmjs.org/es-object-atoms/-/es-object-atoms-1.1.1.tgz", @@ -1548,6 +2092,16 @@ "url": "https://github.com/sponsors/ljharb" } }, + "node_modules/escalade": { + "version": "3.2.0", + "resolved": "https://registry.npmjs.org/escalade/-/escalade-3.2.0.tgz", + "integrity": "sha512-WUj2qlxaQtO4g6Pq5c29GTcWGDyd8itL8zTlipgECz3JesAiiOKotd8JU6otB3PACgG6xkJUyVhboMS+bje/jA==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=6" + } + }, "node_modules/escape-string-regexp": { "version": "4.0.0", "resolved": "https://registry.npmjs.org/escape-string-regexp/-/escape-string-regexp-4.0.0.tgz", @@ -1875,6 +2429,13 @@ "node": ">=4.0" } }, + "node_modules/estree-walker": { + "version": "2.0.2", + "resolved": "https://registry.npmjs.org/estree-walker/-/estree-walker-2.0.2.tgz", + "integrity": "sha512-Rfkk/Mp/DL7JVje3u18FxFujQlTNR2q6QfMSMB7AvCBx91NGj/ba3kCfza0f6dVDbw7YlRf/nDrn7pQrCCyQ/w==", + "dev": true, + "license": "MIT" + }, "node_modules/esutils": { "version": "2.0.3", "resolved": "https://registry.npmjs.org/esutils/-/esutils-2.0.3.tgz", @@ -1901,6 +2462,16 @@ "through": "~2.3.1" } }, + "node_modules/events": { + "version": "3.3.0", + "resolved": "https://registry.npmjs.org/events/-/events-3.3.0.tgz", + "integrity": "sha512-mQw+2fkQbALzQ7V0MY0IqdnXNOeTtP4r0lN9z7AAawCXgqea7bDii20AYrIBrFd/Hx0M2Ocz6S111CaFkUcb0Q==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=0.8.x" + } + }, "node_modules/exit-hook": { "version": "4.0.0", "resolved": "https://registry.npmjs.org/exit-hook/-/exit-hook-4.0.0.tgz", @@ -1935,6 +2506,23 @@ "dev": true, "license": "MIT" }, + "node_modules/fast-uri": { + "version": "3.0.6", + "resolved": "https://registry.npmjs.org/fast-uri/-/fast-uri-3.0.6.tgz", + "integrity": "sha512-Atfo14OibSv5wAp4VWNsFYE1AchQRTv9cBGWET4pZWHzYshFSS9NQI6I57rdKn9croWVMbYFbLhJ+yJvmZIIHw==", + "dev": true, + "funding": [ + { + "type": "github", + "url": "https://github.com/sponsors/fastify" + }, + { + "type": "opencollective", + "url": "https://opencollective.com/fastify" + } + ], + "license": "BSD-3-Clause" + }, "node_modules/fastq": { "version": "1.19.0", "resolved": "https://registry.npmjs.org/fastq/-/fastq-1.19.0.tgz", @@ -1945,6 +2533,21 @@ "reusify": "^1.0.4" } }, + "node_modules/fdir": { + "version": "6.4.3", + "resolved": "https://registry.npmjs.org/fdir/-/fdir-6.4.3.tgz", + "integrity": "sha512-PMXmW2y1hDDfTSRc9gaXIuCCRpuoz3Kaz8cUelp3smouvfT632ozg2vrT6lJsHKKOF59YLbOGfAWGUcKEfRMQw==", + "dev": true, + "license": "MIT", + "peerDependencies": { + "picomatch": "^3 || ^4" + }, + "peerDependenciesMeta": { + "picomatch": { + "optional": true + } + } + }, "node_modules/file-entry-cache": { "version": "6.0.1", "resolved": "https://registry.npmjs.org/file-entry-cache/-/file-entry-cache-6.0.1.tgz", @@ -2099,6 +2702,16 @@ "url": "https://github.com/sponsors/ljharb" } }, + "node_modules/get-caller-file": { + "version": "2.0.5", + "resolved": "https://registry.npmjs.org/get-caller-file/-/get-caller-file-2.0.5.tgz", + "integrity": "sha512-DyFP3BM/3YHTQOCUL/w0OZHR0lpKeGrxotcHWcqNEdnltqFwXVfhEBQ94eIo34AfQpo0rGki4cyIiftY06h2Fg==", + "dev": true, + "license": "ISC", + "engines": { + "node": "6.* || 8.* || >= 10.*" + } + }, "node_modules/get-intrinsic": { "version": "1.2.7", "resolved": "https://registry.npmjs.org/get-intrinsic/-/get-intrinsic-1.2.7.tgz", @@ -2200,6 +2813,13 @@ "node": ">=10.13.0" } }, + "node_modules/glob-to-regexp": { + "version": "0.4.1", + "resolved": "https://registry.npmjs.org/glob-to-regexp/-/glob-to-regexp-0.4.1.tgz", + "integrity": "sha512-lkX1HJXwyMcprw/5YUZc2s7DrpAiHB21/V+E1rHUrVNokkvB6bqMzT0VfV6/86ZNabt1k14YOIaT7nDvOX3Iiw==", + "dev": true, + "license": "BSD-2-Clause" + }, "node_modules/globals": { "version": "15.15.0", "resolved": "https://registry.npmjs.org/globals/-/globals-15.15.0.tgz", @@ -2257,6 +2877,13 @@ "url": "https://github.com/sponsors/ljharb" } }, + "node_modules/graceful-fs": { + "version": "4.2.11", + "resolved": "https://registry.npmjs.org/graceful-fs/-/graceful-fs-4.2.11.tgz", + "integrity": "sha512-RbJ5/jmFcNNCcDV5o9eTnBLJ/HszWV0P73bc+Ff4nS/rJj+YaS6IGyiOL0VoBYX+l1Wrl3k63h/KrH+nhJ0XvQ==", + "dev": true, + "license": "ISC" + }, "node_modules/graphemer": { "version": "1.4.0", "resolved": "https://registry.npmjs.org/graphemer/-/graphemer-1.4.0.tgz", @@ -2724,6 +3351,16 @@ "url": "https://github.com/sponsors/ljharb" } }, + "node_modules/is-fullwidth-code-point": { + "version": "3.0.0", + "resolved": "https://registry.npmjs.org/is-fullwidth-code-point/-/is-fullwidth-code-point-3.0.0.tgz", + "integrity": "sha512-zymm5+u+sCsSWyD9qNaejV3DFvhCKclKdizYaJUuHA83RLjb7nSuGnddCHGv0hk+KY7BMAlsWeK4Ueg6EV6XQg==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=8" + } + }, "node_modules/is-generator-function": { "version": "1.1.0", "resolved": "https://registry.npmjs.org/is-generator-function/-/is-generator-function-1.1.0.tgz", @@ -2769,6 +3406,13 @@ "url": "https://github.com/sponsors/ljharb" } }, + "node_modules/is-module": { + "version": "1.0.0", + "resolved": "https://registry.npmjs.org/is-module/-/is-module-1.0.0.tgz", + "integrity": "sha512-51ypPSPCoTEIN9dy5Oy+h4pShgJmPCygKfyRCISBI+JoWT/2oJvK8QPxmwv7b/p239jXrm9M1mlQbyKJ5A152g==", + "dev": true, + "license": "MIT" + }, "node_modules/is-number-object": { "version": "1.1.1", "resolved": "https://registry.npmjs.org/is-number-object/-/is-number-object-1.1.1.tgz", @@ -2803,6 +3447,16 @@ "dev": true, "license": "MIT" }, + "node_modules/is-reference": { + "version": "1.2.1", + "resolved": "https://registry.npmjs.org/is-reference/-/is-reference-1.2.1.tgz", + "integrity": "sha512-U82MsXXiFIrjCK4otLT+o2NA2Cd2g5MLoOVXUZjIOhLurrRxpEXzI8O0KZHr3IjLvlAH1kTPYSuqer5T9ZVBKQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/estree": "*" + } + }, "node_modules/is-regex": { "version": "1.2.1", "resolved": "https://registry.npmjs.org/is-regex/-/is-regex-1.2.1.tgz", @@ -2969,6 +3623,37 @@ "dev": true, "license": "ISC" }, + "node_modules/jest-worker": { + "version": "27.5.1", + "resolved": "https://registry.npmjs.org/jest-worker/-/jest-worker-27.5.1.tgz", + "integrity": "sha512-7vuh85V5cdDofPyxn58nrPjBktZo0u9x1g8WtjQol+jZDaE+fhN+cIvTj11GndBnMnyfrUOG1sZQxCdjKh+DKg==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/node": "*", + "merge-stream": "^2.0.0", + "supports-color": "^8.0.0" + }, + "engines": { + "node": ">= 10.13.0" + } + }, + "node_modules/jest-worker/node_modules/supports-color": { + "version": "8.1.1", + "resolved": "https://registry.npmjs.org/supports-color/-/supports-color-8.1.1.tgz", + "integrity": "sha512-MpUEN2OodtUzxvKQl72cUF7RQ5EiHsGvSsVG0ia9c5RbWGL2CI4C7EpPS8UTBIplnlzZiNuV56w+FuNxy3ty2Q==", + "dev": true, + "license": "MIT", + "dependencies": { + "has-flag": "^4.0.0" + }, + "engines": { + "node": ">=10" + }, + "funding": { + "url": "https://github.com/chalk/supports-color?sponsor=1" + } + }, "node_modules/jquery": { "version": "4.0.0-beta.2", "resolved": "https://registry.npmjs.org/jquery/-/jquery-4.0.0-beta.2.tgz", @@ -3083,6 +3768,13 @@ "dev": true, "license": "MIT" }, + "node_modules/json-parse-even-better-errors": { + "version": "2.3.1", + "resolved": "https://registry.npmjs.org/json-parse-even-better-errors/-/json-parse-even-better-errors-2.3.1.tgz", + "integrity": "sha512-xyFwyhro/JEof6Ghe2iz2NcXoj2sloNsWr/XsERDK/oiPCfaNhl5ONfp+jQdAZRQQ0IJWNzH9zIZF7li91kh2w==", + "dev": true, + "license": "MIT" + }, "node_modules/json-schema-traverse": { "version": "0.4.1", "resolved": "https://registry.npmjs.org/json-schema-traverse/-/json-schema-traverse-0.4.1.tgz", @@ -3164,6 +3856,16 @@ "immediate": "~3.0.5" } }, + "node_modules/loader-runner": { + "version": "4.3.0", + "resolved": "https://registry.npmjs.org/loader-runner/-/loader-runner-4.3.0.tgz", + "integrity": "sha512-3R/1M+yS3j5ou80Me59j7F9IMs4PXs3VqRrm0TU3AbKPxlmpoY1TNscJV/oGJXo8qCatFGTfDbY6W6ipGOYXfg==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=6.11.5" + } + }, "node_modules/locate-path": { "version": "6.0.0", "resolved": "https://registry.npmjs.org/locate-path/-/locate-path-6.0.0.tgz", @@ -3202,6 +3904,16 @@ "dev": true, "license": "ISC" }, + "node_modules/magic-string": { + "version": "0.30.17", + "resolved": "https://registry.npmjs.org/magic-string/-/magic-string-0.30.17.tgz", + "integrity": "sha512-sNPKHvyjVf7gyjwS4xGTaW/mCnF8wnjtifKBEhxfZ7E/S8tQ0rssrwGNn6q8JH/ohItJfSQp9mBtQYuTlH5QnA==", + "dev": true, + "license": "MIT", + "dependencies": { + "@jridgewell/sourcemap-codec": "^1.5.0" + } + }, "node_modules/map-stream": { "version": "0.1.0", "resolved": "https://registry.npmjs.org/map-stream/-/map-stream-0.1.0.tgz", @@ -3218,6 +3930,13 @@ "node": ">= 0.4" } }, + "node_modules/merge-stream": { + "version": "2.0.0", + "resolved": "https://registry.npmjs.org/merge-stream/-/merge-stream-2.0.0.tgz", + "integrity": "sha512-abv/qOcuPfk3URPfDzmZU1LKmuw8kT+0nIHvKrKgFrwifol/doWcdA4ZqsWQ8ENrFKkd67Mfpo/LovbIUsbt3w==", + "dev": true, + "license": "MIT" + }, "node_modules/mime-db": { "version": "1.52.0", "resolved": "https://registry.npmjs.org/mime-db/-/mime-db-1.52.0.tgz", @@ -3285,6 +4004,13 @@ "dev": true, "license": "MIT" }, + "node_modules/neo-async": { + "version": "2.6.2", + "resolved": "https://registry.npmjs.org/neo-async/-/neo-async-2.6.2.tgz", + "integrity": "sha512-Yd3UES5mWCSqR+qNT93S3UoYUkqAZ9lLg8a7g9rimsWmYGK8cVToA4/sF3RrshdyV3sAGMXVUmpMYOw+dLpOuw==", + "dev": true, + "license": "MIT" + }, "node_modules/nise": { "version": "4.1.0", "resolved": "https://registry.npmjs.org/nise/-/nise-4.1.0.tgz", @@ -3299,6 +4025,13 @@ "path-to-regexp": "^1.7.0" } }, + "node_modules/node-releases": { + "version": "2.0.19", + "resolved": "https://registry.npmjs.org/node-releases/-/node-releases-2.0.19.tgz", + "integrity": "sha512-xxOWJsBKtzAq7DY0J+DTzuz58K8e7sJbdgwkbMWQe8UYB6ekmsQ45q0M/tJDsGaZmbC+l7n57UV8Hl5tHxO9uw==", + "dev": true, + "license": "MIT" + }, "node_modules/node-watch": { "version": "0.7.3", "resolved": "https://registry.npmjs.org/node-watch/-/node-watch-0.7.3.tgz", @@ -3591,6 +4324,26 @@ "through": "~2.3" } }, + "node_modules/picocolors": { + "version": "1.1.1", + "resolved": "https://registry.npmjs.org/picocolors/-/picocolors-1.1.1.tgz", + "integrity": "sha512-xceH2snhtb5M9liqDsmEw56le376mTZkEX/jEb/RxNFyegNul7eNslCXP9FDj/Lcu0X8KEyMceP2ntpaHrDEVA==", + "dev": true, + "license": "ISC" + }, + "node_modules/picomatch": { + "version": "4.0.2", + "resolved": "https://registry.npmjs.org/picomatch/-/picomatch-4.0.2.tgz", + "integrity": "sha512-M7BAV6Rlcy5u+m6oPhAPFgJTzAioX/6B0DxyvDlo9l8+T3nLKbrczg2WLUyzd45L8RqfUMyGPzekbMvX2Ldkwg==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=12" + }, + "funding": { + "url": "https://github.com/sponsors/jonschlinkert" + } + }, "node_modules/possible-typed-array-names": { "version": "1.1.0", "resolved": "https://registry.npmjs.org/possible-typed-array-names/-/possible-typed-array-names-1.1.0.tgz", @@ -3693,6 +4446,16 @@ "node": ">= 10" } }, + "node_modules/randombytes": { + "version": "2.1.0", + "resolved": "https://registry.npmjs.org/randombytes/-/randombytes-2.1.0.tgz", + "integrity": "sha512-vYl3iOX+4CKUWuxGi9Ukhie6fsqXqS9FE2Zaic4tNFD2N2QQaXOMFbuKK4QmDHC0JO6B1Zp41J0LpT0oR68amQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "safe-buffer": "^5.1.0" + } + }, "node_modules/raw-body": { "version": "3.0.0", "resolved": "https://registry.npmjs.org/raw-body/-/raw-body-3.0.0.tgz", @@ -3776,6 +4539,26 @@ "url": "https://github.com/sponsors/ljharb" } }, + "node_modules/require-directory": { + "version": "2.1.1", + "resolved": "https://registry.npmjs.org/require-directory/-/require-directory-2.1.1.tgz", + "integrity": "sha512-fGxEI7+wsG9xrvdjsrlmL22OMTTiHRwAMroiEeMgq8gzoLC/PQr7RsRDSTLUg/bZAZtF+TVIkHc6/4RIKrui+Q==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=0.10.0" + } + }, + "node_modules/require-from-string": { + "version": "2.0.2", + "resolved": "https://registry.npmjs.org/require-from-string/-/require-from-string-2.0.2.tgz", + "integrity": "sha512-Xf0nWe6RseziFMu+Ap9biiUbmplq6S9/p+7w7YXP/JBHhrUDDUhwa+vANyubuqfZWTveU//DYVGsDG7RKL/vEw==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=0.10.0" + } + }, "node_modules/resolve": { "version": "1.22.10", "resolved": "https://registry.npmjs.org/resolve/-/resolve-1.22.10.tgz", @@ -3987,6 +4770,63 @@ "node": ">=v12.22.7" } }, + "node_modules/schema-utils": { + "version": "4.3.0", + "resolved": "https://registry.npmjs.org/schema-utils/-/schema-utils-4.3.0.tgz", + "integrity": "sha512-Gf9qqc58SpCA/xdziiHz35F4GNIWYWZrEshUc/G/r5BnLph6xpKuLeoJoQuj5WfBIx/eQLf+hmVPYHaxJu7V2g==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/json-schema": "^7.0.9", + "ajv": "^8.9.0", + "ajv-formats": "^2.1.1", + "ajv-keywords": "^5.1.0" + }, + "engines": { + "node": ">= 10.13.0" + }, + "funding": { + "type": "opencollective", + "url": "https://opencollective.com/webpack" + } + }, + "node_modules/schema-utils/node_modules/ajv": { + "version": "8.17.1", + "resolved": "https://registry.npmjs.org/ajv/-/ajv-8.17.1.tgz", + "integrity": "sha512-B/gBuNg5SiMTrPkC+A2+cW0RszwxYmn6VYxB/inlBStS5nx6xHIt/ehKRhIMhqusl7a8LjQoZnjCs5vhwxOQ1g==", + "dev": true, + "license": "MIT", + "dependencies": { + "fast-deep-equal": "^3.1.3", + "fast-uri": "^3.0.1", + "json-schema-traverse": "^1.0.0", + "require-from-string": "^2.0.2" + }, + "funding": { + "type": "github", + "url": "https://github.com/sponsors/epoberezkin" + } + }, + "node_modules/schema-utils/node_modules/ajv-keywords": { + "version": "5.1.0", + "resolved": "https://registry.npmjs.org/ajv-keywords/-/ajv-keywords-5.1.0.tgz", + "integrity": "sha512-YCS/JNFAUyr5vAuhk1DWm1CBxRHW9LbJ2ozWeemrIqpbsqKjHVxYPyi5GC0rjZIT5JxJ3virVTS8wk4i/Z+krw==", + "dev": true, + "license": "MIT", + "dependencies": { + "fast-deep-equal": "^3.1.3" + }, + "peerDependencies": { + "ajv": "^8.8.2" + } + }, + "node_modules/schema-utils/node_modules/json-schema-traverse": { + "version": "1.0.0", + "resolved": "https://registry.npmjs.org/json-schema-traverse/-/json-schema-traverse-1.0.0.tgz", + "integrity": "sha512-NM8/P9n3XjXhIZn1lLhkFaACTOURQXjWhV4BA/RnOv8xvgqtqpAX9IO4mRQxSx1Rlo4tqzeqb0sOlruaOy3dug==", + "dev": true, + "license": "MIT" + }, "node_modules/selenium-webdriver": { "version": "4.28.1", "resolved": "https://registry.npmjs.org/selenium-webdriver/-/selenium-webdriver-4.28.1.tgz", @@ -4023,6 +4863,16 @@ "semver": "bin/semver" } }, + "node_modules/serialize-javascript": { + "version": "6.0.2", + "resolved": "https://registry.npmjs.org/serialize-javascript/-/serialize-javascript-6.0.2.tgz", + "integrity": "sha512-Saa1xPByTTq2gdeFZYLLo+RFE35NHZkAbqZeWNd3BpzppeVisAqpDjcp8dyf6uIvEqJRd46jemmyA4iFIeVk8g==", + "dev": true, + "license": "BSD-3-Clause", + "dependencies": { + "randombytes": "^2.1.0" + } + }, "node_modules/set-function-length": { "version": "1.2.2", "resolved": "https://registry.npmjs.org/set-function-length/-/set-function-length-1.2.2.tgz", @@ -4215,6 +5065,27 @@ "node": ">=0.3.1" } }, + "node_modules/source-map": { + "version": "0.6.1", + "resolved": "https://registry.npmjs.org/source-map/-/source-map-0.6.1.tgz", + "integrity": "sha512-UjgapumWlbMhkBgzT7Ykc5YXUT46F0iKu8SGXq0bcwP5dz/h0Plj6enJqjz1Zbq2l5WaqYnrVbwWOWMyF3F47g==", + "dev": true, + "license": "BSD-3-Clause", + "engines": { + "node": ">=0.10.0" + } + }, + "node_modules/source-map-support": { + "version": "0.5.21", + "resolved": "https://registry.npmjs.org/source-map-support/-/source-map-support-0.5.21.tgz", + "integrity": "sha512-uBHU3L3czsIyYXKX88fdrGovxdSCoTGDRZ6SYXtSRxLZUzHg5P/66Ht6uoUlHu9EZod+inXhKo3qQgwXUT/y1w==", + "dev": true, + "license": "MIT", + "dependencies": { + "buffer-from": "^1.0.0", + "source-map": "^0.6.0" + } + }, "node_modules/spawnback": { "version": "1.0.1", "resolved": "https://registry.npmjs.org/spawnback/-/spawnback-1.0.1.tgz", @@ -4264,6 +5135,21 @@ "safe-buffer": "~5.1.0" } }, + "node_modules/string-width": { + "version": "4.2.3", + "resolved": "https://registry.npmjs.org/string-width/-/string-width-4.2.3.tgz", + "integrity": "sha512-wKyQRQpjJ0sIp62ErSZdGsjMJWsap5oRNihHhu6G7JVO/9jIB6UyevL+tXuOqrng8j/cxKTWyWUwvSTriiZz/g==", + "dev": true, + "license": "MIT", + "dependencies": { + "emoji-regex": "^8.0.0", + "is-fullwidth-code-point": "^3.0.0", + "strip-ansi": "^6.0.1" + }, + "engines": { + "node": ">=8" + } + }, "node_modules/string.prototype.trim": { "version": "1.2.10", "resolved": "https://registry.npmjs.org/string.prototype.trim/-/string.prototype.trim-1.2.10.tgz", @@ -4392,6 +5278,16 @@ "dev": true, "license": "MIT" }, + "node_modules/tapable": { + "version": "2.2.1", + "resolved": "https://registry.npmjs.org/tapable/-/tapable-2.2.1.tgz", + "integrity": "sha512-GNzQvQTOIP6RyTfE2Qxb8ZVlNmw0n88vp1szwWRimP02mnTsx3Wtn5qRdqY9w2XduFNUgvOwhNnQsjwCp+kqaQ==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=6" + } + }, "node_modules/temp-fs": { "version": "0.9.9", "resolved": "https://registry.npmjs.org/temp-fs/-/temp-fs-0.9.9.tgz", @@ -4419,6 +5315,67 @@ "rimraf": "bin.js" } }, + "node_modules/terser": { + "version": "5.39.0", + "resolved": "https://registry.npmjs.org/terser/-/terser-5.39.0.tgz", + "integrity": "sha512-LBAhFyLho16harJoWMg/nZsQYgTrg5jXOn2nCYjRUcZZEdE3qa2zb8QEDRUGVZBW4rlazf2fxkg8tztybTaqWw==", + "dev": true, + "license": "BSD-2-Clause", + "dependencies": { + "@jridgewell/source-map": "^0.3.3", + "acorn": "^8.8.2", + "commander": "^2.20.0", + "source-map-support": "~0.5.20" + }, + "bin": { + "terser": "bin/terser" + }, + "engines": { + "node": ">=10" + } + }, + "node_modules/terser-webpack-plugin": { + "version": "5.3.14", + "resolved": "https://registry.npmjs.org/terser-webpack-plugin/-/terser-webpack-plugin-5.3.14.tgz", + "integrity": "sha512-vkZjpUjb6OMS7dhV+tILUW6BhpDR7P2L/aQSAv+Uwk+m8KATX9EccViHTJR2qDtACKPIYndLGCyl3FMo+r2LMw==", + "dev": true, + "license": "MIT", + "dependencies": { + "@jridgewell/trace-mapping": "^0.3.25", + "jest-worker": "^27.4.5", + "schema-utils": "^4.3.0", + "serialize-javascript": "^6.0.2", + "terser": "^5.31.1" + }, + "engines": { + "node": ">= 10.13.0" + }, + "funding": { + "type": "opencollective", + "url": "https://opencollective.com/webpack" + }, + "peerDependencies": { + "webpack": "^5.1.0" + }, + "peerDependenciesMeta": { + "@swc/core": { + "optional": true + }, + "esbuild": { + "optional": true + }, + "uglify-js": { + "optional": true + } + } + }, + "node_modules/terser/node_modules/commander": { + "version": "2.20.3", + "resolved": "https://registry.npmjs.org/commander/-/commander-2.20.3.tgz", + "integrity": "sha512-GpVkmM8vF2vQUkj2LvZmD35JxeJOLCwJ9cUkugyk2nuhbv3+mJvpLYYt+0+USMxE+oj+ey/lJEnhZw75x/OMcQ==", + "dev": true, + "license": "MIT" + }, "node_modules/text-table": { "version": "0.2.0", "resolved": "https://registry.npmjs.org/text-table/-/text-table-0.2.0.tgz", @@ -4669,6 +5626,13 @@ "url": "https://github.com/sponsors/ljharb" } }, + "node_modules/undici-types": { + "version": "6.20.0", + "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-6.20.0.tgz", + "integrity": "sha512-Ny6QZ2Nju20vw1SRHe3d9jVu6gJ+4e3+MMpqu7pqE5HT6WsTSlce++GQmK5UXS8mzV8DSYHrQH+Xrf2jVcuKNg==", + "dev": true, + "license": "MIT" + }, "node_modules/unpipe": { "version": "1.0.0", "resolved": "https://registry.npmjs.org/unpipe/-/unpipe-1.0.0.tgz", @@ -4679,6 +5643,37 @@ "node": ">= 0.8" } }, + "node_modules/update-browserslist-db": { + "version": "1.1.3", + "resolved": "https://registry.npmjs.org/update-browserslist-db/-/update-browserslist-db-1.1.3.tgz", + "integrity": "sha512-UxhIZQ+QInVdunkDAaiazvvT/+fXL5Osr0JZlJulepYu6Jd7qJtDZjlur0emRlT71EN3ScPoE7gvsuIKKNavKw==", + "dev": true, + "funding": [ + { + "type": "opencollective", + "url": "https://opencollective.com/browserslist" + }, + { + "type": "tidelift", + "url": "https://tidelift.com/funding/github/npm/browserslist" + }, + { + "type": "github", + "url": "https://github.com/sponsors/ai" + } + ], + "license": "MIT", + "dependencies": { + "escalade": "^3.2.0", + "picocolors": "^1.1.1" + }, + "bin": { + "update-browserslist-db": "cli.js" + }, + "peerDependencies": { + "browserslist": ">= 4.21.0" + } + }, "node_modules/uri-js": { "version": "4.4.1", "resolved": "https://registry.npmjs.org/uri-js/-/uri-js-4.4.1.tgz", @@ -4709,6 +5704,20 @@ "node": ">=18" } }, + "node_modules/watchpack": { + "version": "2.4.2", + "resolved": "https://registry.npmjs.org/watchpack/-/watchpack-2.4.2.tgz", + "integrity": "sha512-TnbFSbcOCcDgjZ4piURLCbJ3nJhznVh9kw6F6iokjiFPl8ONxe9A6nMDVXDiNbrSfLILs6vB07F7wLBrwPYzJw==", + "dev": true, + "license": "MIT", + "dependencies": { + "glob-to-regexp": "^0.4.1", + "graceful-fs": "^4.1.2" + }, + "engines": { + "node": ">=10.13.0" + } + }, "node_modules/webidl-conversions": { "version": "7.0.0", "resolved": "https://registry.npmjs.org/webidl-conversions/-/webidl-conversions-7.0.0.tgz", @@ -4719,6 +5728,87 @@ "node": ">=12" } }, + "node_modules/webpack": { + "version": "5.98.0", + "resolved": "https://registry.npmjs.org/webpack/-/webpack-5.98.0.tgz", + "integrity": "sha512-UFynvx+gM44Gv9qFgj0acCQK2VE1CtdfwFdimkapco3hlPCJ/zeq73n2yVKimVbtm+TnApIugGhLJnkU6gjYXA==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/eslint-scope": "^3.7.7", + "@types/estree": "^1.0.6", + "@webassemblyjs/ast": "^1.14.1", + "@webassemblyjs/wasm-edit": "^1.14.1", + "@webassemblyjs/wasm-parser": "^1.14.1", + "acorn": "^8.14.0", + "browserslist": "^4.24.0", + "chrome-trace-event": "^1.0.2", + "enhanced-resolve": "^5.17.1", + "es-module-lexer": "^1.2.1", + "eslint-scope": "5.1.1", + "events": "^3.2.0", + "glob-to-regexp": "^0.4.1", + "graceful-fs": "^4.2.11", + "json-parse-even-better-errors": "^2.3.1", + "loader-runner": "^4.2.0", + "mime-types": "^2.1.27", + "neo-async": "^2.6.2", + "schema-utils": "^4.3.0", + "tapable": "^2.1.1", + "terser-webpack-plugin": "^5.3.11", + "watchpack": "^2.4.1", + "webpack-sources": "^3.2.3" + }, + "bin": { + "webpack": "bin/webpack.js" + }, + "engines": { + "node": ">=10.13.0" + }, + "funding": { + "type": "opencollective", + "url": "https://opencollective.com/webpack" + }, + "peerDependenciesMeta": { + "webpack-cli": { + "optional": true + } + } + }, + "node_modules/webpack-sources": { + "version": "3.2.3", + "resolved": "https://registry.npmjs.org/webpack-sources/-/webpack-sources-3.2.3.tgz", + "integrity": "sha512-/DyMEOrDgLKKIG0fmvtz+4dUX/3Ghozwgm6iPp8KRhvn+eQf9+Q7GWxVNMk3+uCPWfdXYC4ExGBckIXdFEfH1w==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=10.13.0" + } + }, + "node_modules/webpack/node_modules/eslint-scope": { + "version": "5.1.1", + "resolved": "https://registry.npmjs.org/eslint-scope/-/eslint-scope-5.1.1.tgz", + "integrity": "sha512-2NxwbF/hZ0KpepYN0cNbo+FN6XoK7GaHlQhgx/hIZl6Va0bF45RQOOwhLIy8lQDbuCiadSLCBnH2CFYquit5bw==", + "dev": true, + "license": "BSD-2-Clause", + "dependencies": { + "esrecurse": "^4.3.0", + "estraverse": "^4.1.1" + }, + "engines": { + "node": ">=8.0.0" + } + }, + "node_modules/webpack/node_modules/estraverse": { + "version": "4.3.0", + "resolved": "https://registry.npmjs.org/estraverse/-/estraverse-4.3.0.tgz", + "integrity": "sha512-39nnKffWz8xN1BU/2c79n9nB9HDzo0niYUqx6xyqUnyoAnQyyWpOTdZEeiCch8BBu515t4wp9ZmgVfVhn9EBpw==", + "dev": true, + "license": "BSD-2-Clause", + "engines": { + "node": ">=4.0" + } + }, "node_modules/whatwg-encoding": { "version": "3.1.1", "resolved": "https://registry.npmjs.org/whatwg-encoding/-/whatwg-encoding-3.1.1.tgz", @@ -4870,6 +5960,40 @@ "node": ">=0.10.0" } }, + "node_modules/wrap-ansi": { + "version": "7.0.0", + "resolved": "https://registry.npmjs.org/wrap-ansi/-/wrap-ansi-7.0.0.tgz", + "integrity": "sha512-YVGIj2kamLSTxw6NsZjoBxfSwsn0ycdesmc4p+Q21c5zPuZ1pl+NfxVdxPtdHvmNVOQ6XSYG4AUtyt/Fi7D16Q==", + "dev": true, + "license": "MIT", + "dependencies": { + "ansi-styles": "^4.0.0", + "string-width": "^4.1.0", + "strip-ansi": "^6.0.0" + }, + "engines": { + "node": ">=10" + }, + "funding": { + "url": "https://github.com/chalk/wrap-ansi?sponsor=1" + } + }, + "node_modules/wrap-ansi/node_modules/ansi-styles": { + "version": "4.3.0", + "resolved": "https://registry.npmjs.org/ansi-styles/-/ansi-styles-4.3.0.tgz", + "integrity": "sha512-zbB9rCJAT1rbjiVDb2hqKFHNYLxgtk8NURxZ3IZwD3F6NtxbXZQCnnSi1Lkx+IDohdPlFp222wVALIheZJQSEg==", + "dev": true, + "license": "MIT", + "dependencies": { + "color-convert": "^2.0.1" + }, + "engines": { + "node": ">=8" + }, + "funding": { + "url": "https://github.com/chalk/ansi-styles?sponsor=1" + } + }, "node_modules/wrappy": { "version": "1.0.2", "resolved": "https://registry.npmjs.org/wrappy/-/wrappy-1.0.2.tgz", @@ -4916,6 +6040,16 @@ "dev": true, "license": "MIT" }, + "node_modules/y18n": { + "version": "5.0.8", + "resolved": "https://registry.npmjs.org/y18n/-/y18n-5.0.8.tgz", + "integrity": "sha512-0pfFzegeDWJHJIAmTLRP2DwHjdF5s7jo9tuztdQxAhINCdvS+3nGINqPd00AphqJR/0LhANUS6/+7SCb98YOfA==", + "dev": true, + "license": "ISC", + "engines": { + "node": ">=10" + } + }, "node_modules/yaml": { "version": "2.7.0", "resolved": "https://registry.npmjs.org/yaml/-/yaml-2.7.0.tgz", @@ -4929,6 +6063,35 @@ "node": ">= 14" } }, + "node_modules/yargs": { + "version": "17.7.2", + "resolved": "https://registry.npmjs.org/yargs/-/yargs-17.7.2.tgz", + "integrity": "sha512-7dSzzRQ++CKnNI/krKnYRV7JKKPUXMEh61soaHKg9mrWEhzFWhFnxPxGl+69cD1Ou63C13NUPCnmIcrvqCuM6w==", + "dev": true, + "license": "MIT", + "dependencies": { + "cliui": "^8.0.1", + "escalade": "^3.1.1", + "get-caller-file": "^2.0.5", + "require-directory": "^2.1.1", + "string-width": "^4.2.3", + "y18n": "^5.0.5", + "yargs-parser": "^21.1.1" + }, + "engines": { + "node": ">=12" + } + }, + "node_modules/yargs-parser": { + "version": "21.1.1", + "resolved": "https://registry.npmjs.org/yargs-parser/-/yargs-parser-21.1.1.tgz", + "integrity": "sha512-tVpsJW7DdjecAiFpbIB1e3qxIQsE6NoPc5/eTdrbbIC4h0LVsWhnoa3g+m2HclBIujHzsxZ4VJVA+GUuc2/LBw==", + "dev": true, + "license": "ISC", + "engines": { + "node": ">=12" + } + }, "node_modules/yocto-queue": { "version": "0.1.0", "resolved": "https://registry.npmjs.org/yocto-queue/-/yocto-queue-0.1.0.tgz", diff --git a/package.json b/package.json index fe02ade2..2ca0b76f 100644 --- a/package.json +++ b/package.json @@ -2,9 +2,21 @@ "name": "jquery-migrate", "title": "jQuery Migrate", "description": "Migrate older jQuery code to jQuery 4.x", - "main": "dist/jquery-migrate.js", "version": "4.0.0-pre", "type": "module", + "exports": { + "node": { + "import": "./dist-module/wrappers/jquery-migrate.node-module-wrapper.js", + "default": "./dist/jquery-migrate.js" + }, + "module": { + "import": "./dist-module/jquery-migrate.module.js", + "default": "./dist/wrappers/jquery-migrate.bundler-require-wrapper.js" + }, + "import": "./dist-module/jquery-migrate.module.js", + "default": "./dist/jquery-migrate.js" + }, + "main": "dist/jquery-migrate.js", "homepage": "https://github.com/jquery/jquery-migrate", "author": { "name": "OpenJS Foundation and other contributors", @@ -19,24 +31,32 @@ }, "license": "MIT", "scripts": { - "build": "node build/tasks/build-default.js", + "build": "node ./build/command.js", + "build:all": "node --input-type=module -e \"import { buildDefaultFiles } from './build/tasks/build.js'; buildDefaultFiles()\"", + "build:clean": "rimraf --glob dist/*.{js,map} --glob dist-module/*.{js,map}", + "build:main": "node --input-type=module -e \"import { build } from './build/tasks/build.js'; build()\"", "lint": "eslint --cache .", "npmcopy": "node build/tasks/npmcopy.js", "prepare": "husky", - "pretest": "npm run npmcopy && npm run build && npm run lint", - "start": "npm run npmcopy && node build/tasks/build-watch.js", - "test:browser": "npm run pretest && npm run test:unit -- -b chrome -b firefox --headless", - "test:ie": "npm run pretest && npm run test:unit -- -v -b ie", - "test:node_smoke_tests": "npm run pretest && node test/node_smoke_tests/smoke_tests.cjs", - "test:safari": "npm run pretest && npm run test:unit -- -v -b safari", + "pretest": "npm run npmcopy", + "start": "node --input-type=module -e \"import { buildDefaultFiles } from './build/tasks/build.js'; buildDefaultFiles({ watch: true })\"", + "test:browser": "npm run pretest && npm run build:all && npm run test:unit -- -b chrome -b firefox --headless", + "test:browserless": "npm run pretest && npm run build:all && node test/bundler_smoke_tests/run-jsdom-tests.js && node test/node_smoke_tests/node_smoke_tests.cjs", + "test:ie": "npm run pretest && npm run build:all && npm run test:unit -- -v -b ie", + "test:bundlers": "npm run pretest && npm run build:all && node test/bundler_smoke_tests/run-jsdom-tests.js", + "test:node_smoke_tests": "npm run pretest && npm run build:all && node test/node_smoke_tests/node_smoke_tests.cjs", + "test:safari": "npm run pretest && npm run build:all && npm run test:unit -- -v -b safari", "test:server": "jtr serve", + "test:esm": "npm run pretest && npm run build:main && npm run test:unit -- -f plugin=esmodules --headless ", "test:unit": "jtr", - "test": "npm run test:node_smoke_tests && npm run test:browser" + "test": "npm run build:all && npm run lint && npm run test:browserless && npm run test:browser && npm run test:esm" }, "peerDependencies": { "jquery": ">=4 <5" }, "devDependencies": { + "@rollup/plugin-commonjs": "28.0.3", + "@rollup/plugin-node-resolve": "16.0.1", "chalk": "5.4.1", "commitplease": "3.2.0", "enquirer": "2.4.1", @@ -52,7 +72,9 @@ "qunit": "2.24.1", "rollup": "4.34.8", "sinon": "9.2.4", - "uglify-js": "3.19.3" + "uglify-js": "3.19.3", + "webpack": "5.98.0", + "yargs": "^17.7.2" }, "keywords": [ "jquery", diff --git a/src/wrapper-esm.js b/src/wrapper-esm.js new file mode 100644 index 00000000..4b3390e7 --- /dev/null +++ b/src/wrapper-esm.js @@ -0,0 +1,21 @@ +/*! + * jQuery Migrate - v@VERSION - @DATE + * Copyright OpenJS Foundation and other contributors + */ +import $ from "jquery"; + +// For ECMAScript module environments where a proper `window` +// is present, execute the factory and get jQuery. +function jQueryFactory( jQuery, window ) { + +// @CODE +// build.js inserts compiled jQuery here + +return jQuery; +} + +var jQuery = jQueryFactory( $, window ); + +export { jQuery, jQuery as $ }; + +export default jQuery; diff --git a/test/bundler_smoke_tests/lib/run-rollup.js b/test/bundler_smoke_tests/lib/run-rollup.js new file mode 100644 index 00000000..d18c450c --- /dev/null +++ b/test/bundler_smoke_tests/lib/run-rollup.js @@ -0,0 +1,42 @@ +import { rollup } from "rollup"; + +import { loadConfigFile } from "rollup/loadConfigFile"; +import path from "node:path"; +import { fileURLToPath } from "node:url"; + +const dirname = path.dirname( fileURLToPath( import.meta.url ) ); +const pureEsmConfigPath = path.resolve( + dirname, "..", "rollup-pure-esm.config.js" ); +const commonJSConfigPath = path.resolve( + dirname, "..", "rollup-commonjs.config.js" ); + +// See https://rollupjs.org/javascript-api/#programmatically-loading-a-config-file +async function runRollup( name, configPath ) { + + console.log( `Running Rollup, version: ${ name }` ); + + // options is an array of "inputOptions" objects with an additional + // "output" property that contains an array of "outputOptions". + // We generate a single output so the array only has one element. + const { + options: [ optionsObj ], + warnings + } = await loadConfigFile( configPath, {} ); + + // "warnings" wraps the default `onwarn` handler passed by the CLI. + // This prints all warnings up to this point: + warnings.flush(); + + const bundle = await rollup( optionsObj ); + await Promise.all( optionsObj.output.map( bundle.write ) ); + + console.log( `Build completed: Rollup, version: ${ name }` ); +} + +export async function runRollupPureEsm() { + await runRollup( "pure ESM", pureEsmConfigPath ); +} + +export async function runRollupEsmAndCommonJs() { + await runRollup( "ESM + CommonJS", commonJSConfigPath ); +} diff --git a/test/bundler_smoke_tests/lib/run-webpack.js b/test/bundler_smoke_tests/lib/run-webpack.js new file mode 100644 index 00000000..3f3b7a86 --- /dev/null +++ b/test/bundler_smoke_tests/lib/run-webpack.js @@ -0,0 +1,21 @@ +import webpack from "webpack"; + +// See https://webpack.js.org/api/node/#webpack +export async function runWebpack() { + return new Promise( async( resolve, reject ) => { + console.log( "Running Webpack" ); + + const { default: config } = await import( "../webpack.config.cjs" ); + + webpack( config, ( err, stats ) => { + if ( err || stats.hasErrors() ) { + console.error( "Errors detected during Webpack compilation" ); + reject( err ); + return; + } + + console.log( "Build completed: Webpack" ); + resolve(); + } ); + } ); +} diff --git a/test/bundler_smoke_tests/lib/utils.js b/test/bundler_smoke_tests/lib/utils.js new file mode 100644 index 00000000..0b960dbc --- /dev/null +++ b/test/bundler_smoke_tests/lib/utils.js @@ -0,0 +1,13 @@ +import fs from "node:fs/promises"; +import path from "node:path"; +import { fileURLToPath } from "node:url"; + +const dirname = path.dirname( fileURLToPath( import.meta.url ) ); +const TMP_BUNDLERS_DIR = path.resolve( dirname, "..", "tmp" ); + +export async function cleanTmpBundlersDir() { + await fs.rm( TMP_BUNDLERS_DIR, { + force: true, + recursive: true + } ); +} diff --git a/test/bundler_smoke_tests/rollup-commonjs.config.js b/test/bundler_smoke_tests/rollup-commonjs.config.js new file mode 100644 index 00000000..3060d239 --- /dev/null +++ b/test/bundler_smoke_tests/rollup-commonjs.config.js @@ -0,0 +1,19 @@ +import path from "node:path"; +import { fileURLToPath } from "node:url"; +import resolve from "@rollup/plugin-node-resolve"; +import commonjs from "@rollup/plugin-commonjs"; + +const dirname = path.dirname( fileURLToPath( import.meta.url ) ); + +export default { + input: `${ dirname }/src-esm-commonjs/main.js`, + output: { + dir: `${ dirname }/tmp/rollup-commonjs`, + format: "iife", + sourcemap: true + }, + plugins: [ + resolve(), + commonjs() + ] +}; diff --git a/test/bundler_smoke_tests/rollup-pure-esm.config.js b/test/bundler_smoke_tests/rollup-pure-esm.config.js new file mode 100644 index 00000000..7476e7be --- /dev/null +++ b/test/bundler_smoke_tests/rollup-pure-esm.config.js @@ -0,0 +1,17 @@ +import path from "node:path"; +import { fileURLToPath } from "node:url"; +import resolve from "@rollup/plugin-node-resolve"; + +const dirname = path.dirname( fileURLToPath( import.meta.url ) ); + +export default { + input: `${ dirname }/src-pure-esm/main.js`, + output: { + dir: `${ dirname }/tmp/rollup-pure-esm`, + format: "iife", + sourcemap: true + }, + plugins: [ + resolve() + ] +}; diff --git a/test/bundler_smoke_tests/run-jsdom-tests.js b/test/bundler_smoke_tests/run-jsdom-tests.js new file mode 100644 index 00000000..f4eb5480 --- /dev/null +++ b/test/bundler_smoke_tests/run-jsdom-tests.js @@ -0,0 +1,73 @@ +import fs from "node:fs/promises"; +import jsdom, { JSDOM } from "jsdom"; +import path from "node:path"; +import { fileURLToPath } from "node:url"; +import { runRollupEsmAndCommonJs, runRollupPureEsm } from "./lib/run-rollup.js"; +import { runWebpack } from "./lib/run-webpack.js"; +import { cleanTmpBundlersDir } from "./lib/utils.js"; + +const dirname = path.dirname( fileURLToPath( import.meta.url ) ); + +async function runJSDOMTest( { title, folder } ) { + console.log( "Running bundlers tests:", title ); + + const template = await fs.readFile( `${ dirname }/test.html`, "utf-8" ); + const scriptSource = await fs.readFile( + `${ dirname }/tmp/${ folder }/main.js`, "utf-8" ); + + const html = template + .replace( /@TITLE\b/, () => title ) + .replace( /@SCRIPT\b/, () => scriptSource ); + + const virtualConsole = new jsdom.VirtualConsole(); + virtualConsole.sendTo( console ); + virtualConsole.on( "assert", ( success ) => { + if ( !success ) { + process.exitCode = 1; + } + } ); + + new JSDOM( html, { + resources: "usable", + runScripts: "dangerously", + virtualConsole + } ); + + if ( process.exitCode === 0 || process.exitCode == null ) { + console.log( "Bundlers tests passed for:", title ); + } else { + console.error( "Bundlers tests failed for:", title ); + } +} + +async function buildAndTest() { + await cleanTmpBundlersDir(); + + await Promise.all( [ + runRollupPureEsm(), + runRollupEsmAndCommonJs(), + runWebpack() + ] ); + + await Promise.all( [ + runJSDOMTest( { + title: "Rollup with pure ESM setup", + folder: "rollup-pure-esm" + } ), + + runJSDOMTest( { + title: "Rollup with ESM + CommonJS", + folder: "rollup-commonjs" + } ), + + runJSDOMTest( { + title: "Webpack", + folder: "webpack" + } ) + ] ); + + // The directory won't be cleaned in case of failures; this may aid debugging. + await cleanTmpBundlersDir(); +} + +await buildAndTest(); diff --git a/test/bundler_smoke_tests/src-esm-commonjs/jquery-migrate-require.cjs b/test/bundler_smoke_tests/src-esm-commonjs/jquery-migrate-require.cjs new file mode 100644 index 00000000..b683d96c --- /dev/null +++ b/test/bundler_smoke_tests/src-esm-commonjs/jquery-migrate-require.cjs @@ -0,0 +1,5 @@ +"use strict"; + +const $ = require( "jquery-migrate" ); + +module.exports.$required = $; diff --git a/test/bundler_smoke_tests/src-esm-commonjs/main.js b/test/bundler_smoke_tests/src-esm-commonjs/main.js new file mode 100644 index 00000000..726ed31d --- /dev/null +++ b/test/bundler_smoke_tests/src-esm-commonjs/main.js @@ -0,0 +1,10 @@ +import { $ as $imported } from "jquery-migrate"; + +import { $required } from "./jquery-migrate-require.cjs"; + +console.assert( $required === $imported, + "Only one copy of full jQuery should exist" ); +console.assert( /^jQuery/.test( $imported.expando ), + "jQuery.expando should be detected on full jQuery" ); +console.assert( typeof $imported.migrateVersion === "string" && $imported.migrateVersion.length > 0, + "jQuery.migrateVersion was not detected, the jQuery Migrate bootstrap process has failed" ); diff --git a/test/bundler_smoke_tests/src-pure-esm/main.js b/test/bundler_smoke_tests/src-pure-esm/main.js new file mode 100644 index 00000000..ab41fddd --- /dev/null +++ b/test/bundler_smoke_tests/src-pure-esm/main.js @@ -0,0 +1,6 @@ +import { $ } from "jquery-migrate"; + +console.assert( /^jQuery/.test( $.expando ), + "jQuery.expando should be detected on full jQuery" ); +console.assert( typeof $.migrateVersion === "string" && $.migrateVersion.length > 0, + "jQuery.migrateVersion was not detected, the jQuery Migrate bootstrap process has failed" ); diff --git a/test/bundler_smoke_tests/test.html b/test/bundler_smoke_tests/test.html new file mode 100644 index 00000000..fa1795bb --- /dev/null +++ b/test/bundler_smoke_tests/test.html @@ -0,0 +1,11 @@ + + + + + + + + + + + diff --git a/test/bundler_smoke_tests/webpack.config.cjs b/test/bundler_smoke_tests/webpack.config.cjs new file mode 100644 index 00000000..7ea427ee --- /dev/null +++ b/test/bundler_smoke_tests/webpack.config.cjs @@ -0,0 +1,9 @@ +"use strict"; + +module.exports = { + entry: `${ __dirname }/src-esm-commonjs/main.js`, + output: { + filename: "main.js", + path: `${ __dirname }/tmp/webpack` + } +}; diff --git a/test/node_smoke_tests/smoke_tests.cjs b/test/node_smoke_tests/node_smoke_tests.cjs similarity index 72% rename from test/node_smoke_tests/smoke_tests.cjs rename to test/node_smoke_tests/node_smoke_tests.cjs index a93ccc23..c7bdc38b 100644 --- a/test/node_smoke_tests/smoke_tests.cjs +++ b/test/node_smoke_tests/node_smoke_tests.cjs @@ -5,14 +5,12 @@ console.log( "Running Node.js smoke tests..." ); const assert = require( "node:assert/strict" ); const { JSDOM } = require( "jsdom" ); -const { window } = new JSDOM( `` ); +const { window } = new JSDOM( "" ); // Set the window global. globalThis.window = window; -// Require jQuery Migrate. Since Migrate doesn't specify exports, -// `require( "jquery-migrate" )` won't work here. -const $ = require( "../.." ); +const $ = require( "jquery-migrate" ); assert( /^jQuery/.test( $.expando ), "jQuery.expando was not detected, the jQuery bootstrap process has failed" ); From 0ea101503fe675109f4135b0c5aacf9154b7faaa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 14 Apr 2025 18:24:47 +0200 Subject: [PATCH 06/12] Core: Make migrateWarnProp/etc. not return props for disabled patches Changes: 1. Make `migrateWarnProp` / `migratePatchAndWarnFunc` and other similar functions not return the APIs when patches are disabled. Previously, `migrateWarnProp` would only stop printing warnings but the API would still be patched, while `migratePatchAndWarnFunc` would return a noop function instead of `undefined`. 2. Don't warn just on access of `jQuery.fn.push`, etc., but when it's called. 3. Rename some internal methods. Closes gh-571 --- src/jquery/core.js | 8 ++--- src/main.js | 73 ++++++++++++++++++++++++++++++-------------- test/unit/migrate.js | 37 +++++++++++++++++++--- 3 files changed, 86 insertions(+), 32 deletions(-) diff --git a/src/jquery/core.js b/src/jquery/core.js index d9cb256a..d11403b7 100644 --- a/src/jquery/core.js +++ b/src/jquery/core.js @@ -1,4 +1,4 @@ -import { migrateWarnProp, migratePatchAndWarnFunc, migratePatchAndInfoFunc } from "../main.js"; +import { migratePatchAndWarnFunc, migratePatchAndInfoFunc } from "../main.js"; import "../disablePatches.js"; var arr = [], @@ -92,9 +92,9 @@ migratePatchAndWarnFunc( jQuery, "isWindow", migratePatchAndInfoFunc( jQuery, "proxy", jQuery.proxy, "proxy", "jQuery.proxy() is deprecated" ); -migrateWarnProp( jQuery.fn, "push", push, "push", +migratePatchAndWarnFunc( jQuery.fn, "push", push, "push", "jQuery.fn.push() is removed; use .add() or convert to an array" ); -migrateWarnProp( jQuery.fn, "sort", sort, "sort", +migratePatchAndWarnFunc( jQuery.fn, "sort", sort, "sort", "jQuery.fn.sort() is removed; convert to an array before sorting" ); -migrateWarnProp( jQuery.fn, "splice", splice, "splice", +migratePatchAndWarnFunc( jQuery.fn, "splice", splice, "splice", "jQuery.fn.splice() is removed; use .slice() or .not() with .eq()" ); diff --git a/src/main.js b/src/main.js index 849bef56..dfad6f67 100644 --- a/src/main.js +++ b/src/main.js @@ -70,38 +70,71 @@ export function migrateInfo( code, msg ) { migrateMessageInternal( code, msg, "info" ); } -function migrateMessagePropInternal( +function migratePatchPropInternal( obj, prop, value, code, msg, migrateMessageFn ) { + var orig = obj[ prop ]; Object.defineProperty( obj, prop, { configurable: true, enumerable: true, + get: function() { - migrateMessageFn( code, msg ); - return value; + if ( jQuery.migrateIsPatchEnabled( code ) ) { + + // If `msg` not provided, do not message; more sophisticated + // messaging logic is most likely embedded in `value`. + if ( msg ) { + migrateMessageFn( code, msg ); + } + + return value; + } + + return orig; }, + set: function( newValue ) { - migrateMessageFn( code, msg ); - value = newValue; + if ( jQuery.migrateIsPatchEnabled( code ) ) { + + // See the comment in the getter. + if ( msg ) { + migrateMessageFn( code, msg ); + } + } + + // If a new value was set, apply it regardless if + // the patch is later disabled or not. + orig = value = newValue; } } ); } export function migrateWarnProp( obj, prop, value, code, msg ) { - migrateMessagePropInternal( obj, prop, value, code, msg, migrateWarn ); + if ( !msg ) { + throw new Error( "No warning message provided" ); + } + migratePatchPropInternal( obj, prop, value, code, msg, migrateWarn ); } export function migrateInfoProp( obj, prop, value, code, msg ) { - migrateMessagePropInternal( obj, prop, value, code, msg, migrateInfo ); + if ( !msg ) { + throw new Error( "No warning message provided" ); + } + migratePatchPropInternal( obj, prop, value, code, msg, migrateInfo ); +} + +export function migratePatchProp( obj, prop, value, code ) { + migratePatchPropInternal( obj, prop, value, code ); } -function migrateMessageFuncInternal( +// The value of the "Func" APIs is that for method we want to allow +// checking for the method existence without triggering a warning. +// For other deprecated properties, we do need to warn on access. +function migratePatchFuncInternal( obj, prop, newFunc, code, msg, migrateMessageFn ) { - var finalFunc, - origFunc = obj[ prop ]; - obj[ prop ] = function() { + function wrappedNewFunc() { // If `msg` not provided, do not warn; more sophisticated warnings // logic is most likely embedded in `newFunc`, in that case here @@ -111,34 +144,28 @@ function migrateMessageFuncInternal( migrateMessageFn( code, msg ); } - // Since patches can be disabled & enabled dynamically, we - // need to decide which implementation to run on each invocation. - finalFunc = jQuery.migrateIsPatchEnabled( code ) ? - newFunc : - - // The function may not have existed originally so we need a fallback. - ( origFunc || jQuery.noop ); + return newFunc.apply( this, arguments ); + } - return finalFunc.apply( this, arguments ); - }; + migratePatchPropInternal( obj, prop, wrappedNewFunc, code ); } export function migratePatchAndWarnFunc( obj, prop, newFunc, code, msg ) { if ( !msg ) { throw new Error( "No warning message provided" ); } - return migrateMessageFuncInternal( obj, prop, newFunc, code, msg, migrateWarn ); + return migratePatchFuncInternal( obj, prop, newFunc, code, msg, migrateWarn ); } export function migratePatchAndInfoFunc( obj, prop, newFunc, code, msg ) { if ( !msg ) { throw new Error( "No info message provided" ); } - return migrateMessageFuncInternal( obj, prop, newFunc, code, msg, migrateInfo ); + return migratePatchFuncInternal( obj, prop, newFunc, code, msg, migrateInfo ); } export function migratePatchFunc( obj, prop, newFunc, code ) { - return migrateMessageFuncInternal( obj, prop, newFunc, code ); + return migratePatchFuncInternal( obj, prop, newFunc, code ); } if ( window.document.compatMode === "BackCompat" ) { diff --git a/test/unit/migrate.js b/test/unit/migrate.js index e3c2ffb7..a70ba9a2 100644 --- a/test/unit/migrate.js +++ b/test/unit/migrate.js @@ -46,7 +46,7 @@ QUnit.test( "jQuery.migrateDeduplicateMessages", function( assert ) { } ); QUnit.test( "disabling/enabling patches", function( assert ) { - assert.expect( 14 ); + assert.expect( 27 ); var elem = jQuery( "

" ); @@ -62,6 +62,10 @@ QUnit.test( "disabling/enabling patches", function( assert ) { true, "patch enabled by default (proxy)" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "shorthand-deprecated-v3" ), true, "patch enabled by default (shorthand-deprecated-v3)" ); + assert.strictEqual( jQuery.migrateIsPatchEnabled( "push" ), + true, "patch enabled by default (push)" ); + assert.strictEqual( jQuery.migrateIsPatchEnabled( "isArray" ), + true, "patch enabled by default (isArray)" ); expectMessage( assert, "pre-on-methods (default)", function() { jQuery().bind(); @@ -72,24 +76,47 @@ QUnit.test( "disabling/enabling patches", function( assert ) { expectMessage( assert, "shorthand-deprecated-v3 (default)", function() { jQuery().click(); } ); + expectMessage( assert, "push (default)", function() { + jQuery().push(); + } ); + expectMessage( assert, "isArray (default)", function() { + jQuery.isArray(); + } ); + + expectNoMessage( assert, "push access without calling (default)", function() { + assert.strictEqual( typeof jQuery().push, "function", + "access check doesn't trigger a message (push)" ); + } ); + expectNoMessage( assert, "isArray access without calling (default)", function() { + assert.strictEqual( typeof jQuery.isArray, "function", + "access check doesn't trigger a message (isArray)" ); + } ); - jQuery.migrateDisablePatches( "pre-on-methods", "proxy" ); + jQuery.migrateDisablePatches( "pre-on-methods", "proxy", "push", "isArray" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "pre-on-methods" ), false, "patch disabled (pre-on-methods)" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "proxy" ), false, "patch disabled (proxy)" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "shorthand-deprecated-v3" ), true, "patch still enabled (shorthand-deprecated-v3)" ); + assert.strictEqual( jQuery.migrateIsPatchEnabled( "push" ), + false, "patch disabled (push)" ); - expectNoMessage( assert, "pre-on-methods (default)", function() { + expectNoMessage( assert, "pre-on-methods (disabled)", function() { jQuery().bind(); } ); - expectNoMessage( assert, "proxy (default)", function() { + expectNoMessage( assert, "proxy (disabled)", function() { jQuery.proxy( jQuery.noop ); } ); - expectMessage( assert, "shorthand-deprecated-v3 (default)", function() { + expectMessage( assert, "shorthand-deprecated-v3 (not disabled)", function() { jQuery().click(); } ); + expectNoMessage( assert, "push (disabled)", function() { + assert.strictEqual( jQuery().push, undefined, "`push` patch no longer defined" ); + } ); + expectNoMessage( assert, "isArray (disabled)", function() { + assert.strictEqual( jQuery.isArray, undefined, "`jQuery.isArray` patch no longer defined" ); + } ); jQuery.migrateDisablePatches( "shorthand-deprecated-v3" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "shorthand-deprecated-v3" ), From 99dbd0e4abeb0679aa8d9cb613713c9b794c94e8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 28 Apr 2025 23:24:32 +0200 Subject: [PATCH 07/12] Build: Make browser tests correctly run on multiple jQuery versions There was a typo in `jtr-local.yml`, making the runs using it load with a huge query string containing all the jQuery versions listed joined. Also, only IE and Safari runs even tried to use this config - the default run was not even reading the BROWSERS env variable. This all meant only one jQuery version was actually tested. Closes gh-575 Ref gh-576 --- .github/workflows/browser-tests.yml | 6 +++++- jtr-local.yml | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/browser-tests.yml b/.github/workflows/browser-tests.yml index 32aed686..4274dda9 100644 --- a/.github/workflows/browser-tests.yml +++ b/.github/workflows/browser-tests.yml @@ -43,7 +43,11 @@ jobs: run: npm install - name: Run tests - run: npm run test:browser -- -f plugin=${{ matrix.MIGRATE_VERSION }} + run: | + npm run pretest + npm run build:all + npm run test:unit -- -c jtr-local.yml \ + --headless -b ${{ matrix.BROWSER }} -f plugin=${{ matrix.MIGRATE_VERSION }} ie: runs-on: windows-latest diff --git a/jtr-local.yml b/jtr-local.yml index 9a7da180..e198c889 100644 --- a/jtr-local.yml +++ b/jtr-local.yml @@ -1,6 +1,6 @@ version: 1 -flags: +runs: jquery: - git - git.min From 94fbdb3265f541cafcf0f71ffa0356aac4afd334 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 28 Apr 2025 23:29:19 +0200 Subject: [PATCH 08/12] Deferred: Don't warn on setting getStackHook to the getErrorHook value Don't warn if `jQuery.Deferred.getStackHook` is set to the value of `jQuery.Deferred.getErrorHook`. This is to facilitate plugins supporting both jQuery <3.7 and older without triggering Migrate warnings and without requiring complex logic parsing `jQuery.fn.jquery`. Closes gh-577 Ref gh-578 --- src/jquery/deferred.js | 10 +++++++--- test/unit/jquery/deferred.js | 7 ++++++- warnings.md | 6 +++--- 3 files changed, 16 insertions(+), 7 deletions(-) diff --git a/src/jquery/deferred.js b/src/jquery/deferred.js index 7a1ea3a2..f1d4cc80 100644 --- a/src/jquery/deferred.js +++ b/src/jquery/deferred.js @@ -48,9 +48,13 @@ Object.defineProperty( jQuery.Deferred, "getStackHook", { }, set: function( newValue ) { if ( jQuery.migrateIsPatchEnabled( "deferred-getStackHook" ) ) { - migrateWarn( "deferred-getStackHook", - "jQuery.Deferred.getStackHook is removed; use jQuery.Deferred.getErrorHook" ); - jQuery.Deferred.getErrorHook = newValue; + + // Only warn if `getErrorHook` wasn't set to the same value first. + if ( jQuery.Deferred.getErrorHook !== newValue ) { + migrateWarn( "deferred-getStackHook", + "jQuery.Deferred.getStackHook is removed; use jQuery.Deferred.getErrorHook" ); + jQuery.Deferred.getErrorHook = newValue; + } } else { unpatchedGetStackHookValue = newValue; } diff --git a/test/unit/jquery/deferred.js b/test/unit/jquery/deferred.js index 07eeb0d3..66e6a93e 100644 --- a/test/unit/jquery/deferred.js +++ b/test/unit/jquery/deferred.js @@ -76,7 +76,7 @@ QUnit.test( "jQuery.Deferred.getStackHook - getter, no getErrorHook", function( } ); QUnit.test( "jQuery.Deferred.getStackHook - setter", function( assert ) { - assert.expect( 5 ); + assert.expect( 6 ); var exceptionHookSpy, done = assert.async(); @@ -90,6 +90,11 @@ QUnit.test( "jQuery.Deferred.getStackHook - setter", function( assert ) { "getStackHook mirrors getErrorHook (setter)" ); } ); + expectNoMessage( assert, "jQuery.Deferred.getStackHook - setter", 1, function() { + var mockFn = function() {}; + jQuery.Deferred.getStackHook = jQuery.Deferred.getErrorHook = mockFn; + } ); + expectMessage( assert, "asyncHook from jQuery.Deferred.getStackHook reported", 1, function() { jQuery.Deferred.getStackHook = function() { diff --git a/warnings.md b/warnings.md index b41ebe9c..2bf15c21 100644 --- a/warnings.md +++ b/warnings.md @@ -161,15 +161,15 @@ This is _not_ a warning, but a console log message the plugin shows when it firs ### \[jsonp-promotion\] JQMIGRATE: JSON-to-JSONP auto-promotion is removed -**Cause:** `jQuery.ajax` calls with `dataType: 'json'` with a provided callback are automatically converted by jQuery to JSONP requests unless the options also specify `jsonp: false`. Auto-promoting JSON requests to JSONP introduces a security risk as the developer may be unaware they're not just downloading data but executing code from a remote domain. This auto-promoting behavior is deprecated and will be removed in jQuery 4.0.0. +**Cause:** `jQuery.ajax` calls with `dataType: 'json'` with a provided callback are automatically converted by jQuery to JSONP requests unless the options also specify `jsonp: false`. Auto-promoting JSON requests to JSONP introduces a security risk as the developer may be unaware they're not just downloading data but executing code from a remote domain. This auto-promoting behavior has been removed in jQuery 4.0.0. **Solution:** To trigger a JSONP request, specify the `dataType: "jsonp"` option. ### \[deferred-getStackHook\] JQMIGRATE: jQuery.Deferred.getStackHook is removed; use jQuery.Deferred.getErrorHook -**Cause:** `jQuery.Deferred.getStackHook` was originally created to pass the stack trace from before an async barrier to report when a user error (like calling a non-existing function) causes a promise to be rejected. However, passing a stack trace doesn't take source maps into account, so we started advising to pass the whole error object. To make it clearer, we also renamed the API to `jQuery.Deferred.getErrorHook`. The legacy alias will be removed in jQuery 4.0.0 +**Cause:** `jQuery.Deferred.getStackHook` was originally created to pass the stack trace from before an async barrier to report when a user error (like calling a non-existing function) causes a promise to be rejected. However, passing a stack trace doesn't take source maps into account, so we started advising to pass the whole error object. To make it clearer, we also renamed the API to `jQuery.Deferred.getErrorHook`. The legacy alias has been removed in jQuery 4.0.0. -**Solution:** Rename all usage of `jQuery.Deferred.getStackHook` to `jQuery.Deferred.getErrorHook`. If you previously assigned a function returning an error stack to `jQuery.Deferred.getStackHook` or `jQuery.Deferred.getErrorHook`, change it to return a full error object. +**Solution:** Rename all usage of `jQuery.Deferred.getStackHook` to `jQuery.Deferred.getErrorHook`. If you previously assigned a function returning an error stack to `jQuery.Deferred.getStackHook` or `jQuery.Deferred.getErrorHook`, change it to return a full error object. If you aim to still support jQuery <3.7, assign the hook to `jQuery.Deferred.getErrorHook` first and only later to `jQuery.Deferred.getStackHook` to avoid a Migrate warning. ## Deprecated APIs From ed1ae5f692062a20bf2983294041d32dde7818f6 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Sun, 4 May 2025 13:41:02 +0200 Subject: [PATCH 09/12] Build: Bump the github-actions group with 2 updates Bumps the github-actions group with 2 updates: [actions/setup-node](https://github.com/actions/setup-node) and [github/codeql-action](https://github.com/github/codeql-action). Closes gh-580 Updates `actions/setup-node` from 4.3.0 to 4.4.0 - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](https://github.com/actions/setup-node/compare/cdca7365b2dadb8aad0a33bc7601856ffabcc48e...49933ea5288caeca8642d1e84afbd3f7d6820020) Updates `github/codeql-action` from 3.28.13 to 3.28.16 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](https://github.com/github/codeql-action/compare/1b549b9259bda1cb5ddde3b41741a82a2d15a841...28deaeda66b76a05916b6923827895f2b14ab387) --- updated-dependencies: - dependency-name: actions/setup-node dependency-version: 4.4.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: github/codeql-action dependency-version: 3.28.16 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- .github/workflows/browser-tests.yml | 6 +++--- .github/workflows/browserstack.yml | 2 +- .github/workflows/codeql-analysis.yml | 6 +++--- .github/workflows/filestash.yml | 2 +- .github/workflows/node.js.yml | 2 +- 5 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/workflows/browser-tests.yml b/.github/workflows/browser-tests.yml index 4274dda9..885d52a8 100644 --- a/.github/workflows/browser-tests.yml +++ b/.github/workflows/browser-tests.yml @@ -27,7 +27,7 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 + uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0 with: node-version: ${{ env.NODE_VERSION }} @@ -59,7 +59,7 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 + uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0 with: node-version: ${{ env.NODE_VERSION }} @@ -88,7 +88,7 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 + uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0 with: node-version: ${{ env.NODE_VERSION }} diff --git a/.github/workflows/browserstack.yml b/.github/workflows/browserstack.yml index 08c1d72d..6da8c830 100644 --- a/.github/workflows/browserstack.yml +++ b/.github/workflows/browserstack.yml @@ -43,7 +43,7 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 + uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0 with: node-version: ${{ env.NODE_VERSION }} diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml index 6cbef952..5344af32 100644 --- a/.github/workflows/codeql-analysis.yml +++ b/.github/workflows/codeql-analysis.yml @@ -33,7 +33,7 @@ jobs: # Initializes the CodeQL tools for scanning. - name: Initialize CodeQL - uses: github/codeql-action/init@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13 + uses: github/codeql-action/init@28deaeda66b76a05916b6923827895f2b14ab387 # v3.28.16 # Override language selection by uncommenting this and choosing your languages # with: # languages: go, javascript, csharp, python, cpp, java @@ -41,7 +41,7 @@ jobs: # Autobuild attempts to build any compiled languages (C/C++, C#, or Java). # If this step fails, then you should remove it and run the build manually (see below) - name: Autobuild - uses: github/codeql-action/autobuild@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13 + uses: github/codeql-action/autobuild@28deaeda66b76a05916b6923827895f2b14ab387 # v3.28.16 # ℹ️ Command-line programs to run using the OS shell. # 📚 https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun @@ -55,4 +55,4 @@ jobs: # make release - name: Perform CodeQL Analysis - uses: github/codeql-action/analyze@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13 + uses: github/codeql-action/analyze@28deaeda66b76a05916b6923827895f2b14ab387 # v3.28.16 diff --git a/.github/workflows/filestash.yml b/.github/workflows/filestash.yml index f5aef172..741dd45d 100644 --- a/.github/workflows/filestash.yml +++ b/.github/workflows/filestash.yml @@ -20,7 +20,7 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 + uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0 with: node-version: ${{ env.NODE_VERSION }} diff --git a/.github/workflows/node.js.yml b/.github/workflows/node.js.yml index cbaaa91a..68b9866a 100644 --- a/.github/workflows/node.js.yml +++ b/.github/workflows/node.js.yml @@ -24,7 +24,7 @@ jobs: uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2 - name: Use Node.js ${{ env.NODE_VERSION }} - uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0 + uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0 with: node-version: ${{ matrix.NODE_VERSION }} From b1c0d149dfc81921d1efdb21c6ee44bc04ee9a3d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 5 May 2025 17:49:42 +0200 Subject: [PATCH 10/12] Data: Patch `$._data` as well Also, extract the util to patch a prototype as we'll need it in the event module as well. Closes gh-569 Ref gh-570 --- src/jquery/data.js | 77 ++++++++++------------------------------ src/utils.js | 59 ++++++++++++++++++++++++++++++ test/unit/jquery/data.js | 8 +++-- 3 files changed, 83 insertions(+), 61 deletions(-) diff --git a/src/jquery/data.js b/src/jquery/data.js index be86342f..612deafd 100644 --- a/src/jquery/data.js +++ b/src/jquery/data.js @@ -1,86 +1,45 @@ -import { migratePatchFunc, migrateWarn } from "../main.js"; +import { migratePatchFunc } from "../main.js"; +import { patchProto } from "../utils.js"; function patchDataProto( original, options ) { - var i, + var warningId = options.warningId, apiName = options.apiName, - isInstanceMethod = options.isInstanceMethod, + isInstanceMethod = options.isInstanceMethod; - // `Object.prototype` keys are not enumerable so list the - // official ones here. An alternative would be wrapping - // data objects with a Proxy but that creates additional issues - // like breaking object identity on subsequent calls. - objProtoKeys = [ - "__proto__", - "__defineGetter__", - "__defineSetter__", - "__lookupGetter__", - "__lookupSetter__", - "hasOwnProperty", - "isPrototypeOf", - "propertyIsEnumerable", - "toLocaleString", - "toString", - "valueOf" - ], - - // Use a null prototype at the beginning so that we can define our - // `__proto__` getter & setter. We'll reset the prototype afterwards. - intermediateDataObj = Object.create( null ); - - for ( i = 0; i < objProtoKeys.length; i++ ) { - ( function( key ) { - Object.defineProperty( intermediateDataObj, key, { - get: function() { - migrateWarn( "data-null-proto", - "Accessing properties from " + apiName + - " inherited from Object.prototype is removed" ); - return ( key + "__cache" ) in intermediateDataObj ? - intermediateDataObj[ key + "__cache" ] : - Object.prototype[ key ]; - }, - set: function( value ) { - migrateWarn( "data-null-proto", - "Setting properties from " + apiName + - " inherited from Object.prototype is removed" ); - intermediateDataObj[ key + "__cache" ] = value; - } - } ); - } )( objProtoKeys[ i ] ); - } - - Object.setPrototypeOf( intermediateDataObj, Object.prototype ); - - return function jQueryDataProtoPatched() { + return function apiWithProtoPatched() { var result = original.apply( this, arguments ); if ( arguments.length !== ( isInstanceMethod ? 0 : 1 ) || result === undefined ) { return result; } - // Insert an additional object in the prototype chain between `result` - // and `Object.prototype`; that intermediate object proxies properties - // to `Object.prototype`, warning about their usage first. - Object.setPrototypeOf( result, intermediateDataObj ); + patchProto( result, { + warningId: warningId, + apiName: apiName + } ); return result; }; } -// Yes, we are patching jQuery.data twice; here & above. This is necessary -// so that each of the two patches can be independently disabled. migratePatchFunc( jQuery, "data", patchDataProto( jQuery.data, { + warningId: "data-null-proto", apiName: "jQuery.data()", - isPrivateData: false, + isInstanceMethod: false + } ), + "data-null-proto" ); +migratePatchFunc( jQuery, "_data", + patchDataProto( jQuery._data, { + warningId: "data-null-proto", + apiName: "jQuery._data()", isInstanceMethod: false } ), "data-null-proto" ); migratePatchFunc( jQuery.fn, "data", patchDataProto( jQuery.fn.data, { + warningId: "data-null-proto", apiName: "jQuery.fn.data()", - isPrivateData: true, isInstanceMethod: true } ), "data-null-proto" ); - -// TODO entry in warnings.md diff --git a/src/utils.js b/src/utils.js index d51891c1..5aa5f0a0 100644 --- a/src/utils.js +++ b/src/utils.js @@ -1,5 +1,64 @@ +import { migrateWarn } from "./main.js"; + export function camelCase( string ) { return string.replace( /-([a-z])/g, function( _, letter ) { return letter.toUpperCase(); } ); } + +// Make `object` inherit from `Object.prototype` via an additional object +// in between; that intermediate object proxies properties +// to `Object.prototype`, warning about their usage first. +export function patchProto( object, options ) { + var i, + warningId = options.warningId, + apiName = options.apiName, + + // `Object.prototype` keys are not enumerable so list the + // official ones here. An alternative would be wrapping + // objects with a Proxy but that creates additional issues + // like breaking object identity on subsequent calls. + objProtoKeys = [ + "__proto__", + "__defineGetter__", + "__defineSetter__", + "__lookupGetter__", + "__lookupSetter__", + "hasOwnProperty", + "isPrototypeOf", + "propertyIsEnumerable", + "toLocaleString", + "toString", + "valueOf" + ], + + // Use a null prototype at the beginning so that we can define our + // `__proto__` getter & setter. We'll reset the prototype afterward. + intermediateObj = Object.create( null ); + + for ( i = 0; i < objProtoKeys.length; i++ ) { + ( function( key ) { + Object.defineProperty( intermediateObj, key, { + get: function() { + migrateWarn( warningId, + "Accessing properties from " + apiName + + " inherited from Object.prototype is removed" ); + return ( key + "__cache" ) in intermediateObj ? + intermediateObj[ key + "__cache" ] : + Object.prototype[ key ]; + }, + set: function( value ) { + migrateWarn( warningId, + "Setting properties from " + apiName + + " inherited from Object.prototype is removed" ); + intermediateObj[ key + "__cache" ] = value; + } + } ); + } )( objProtoKeys[ i ] ); + } + + Object.setPrototypeOf( intermediateObj, Object.prototype ); + Object.setPrototypeOf( object, intermediateObj ); + + return object; +} diff --git a/test/unit/jquery/data.js b/test/unit/jquery/data.js index 97f00f25..895829f3 100644 --- a/test/unit/jquery/data.js +++ b/test/unit/jquery/data.js @@ -1,21 +1,25 @@ QUnit.module( "data" ); QUnit.test( "properties from Object.prototype", function( assert ) { - assert.expect( 6 ); + assert.expect( 8 ); var div = jQuery( "

" ).appendTo( "#qunit-fixture" ); div.data( "foo", "bar" ); + jQuery._data( div[ 0 ], "baz", "qaz" ); expectNoMessage( assert, "Regular properties", function() { assert.strictEqual( div.data( "foo" ), "bar", "data access" ); assert.strictEqual( jQuery.data( div[ 0 ], "foo" ), "bar", "data access (static method)" ); + assert.strictEqual( jQuery._data( div[ 0 ], "baz" ), "qaz", "private data access" ); } ); - expectMessage( assert, "Properties from Object.prototype", 2, function() { + expectMessage( assert, "Properties from Object.prototype", 3, function() { assert.ok( div.data().hasOwnProperty( "foo" ), "hasOwnProperty works" ); assert.ok( jQuery.data( div[ 0 ] ).hasOwnProperty( "foo" ), "hasOwnProperty works (static method)" ); + assert.ok( jQuery._data( div[ 0 ] ).hasOwnProperty( "baz" ), + "hasOwnProperty works (private data)" ); } ); } ); From b1e09907b4619b6c508aaf7f7e32b461dc2884e5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 12 May 2025 18:16:23 +0200 Subject: [PATCH 11/12] Event: Patch jQuery.event.global The API has been write-only since 1.9.0 and is going to be removed in jQuery 4.0.0. Also: * make it more explicit why certain patches are tested in the context of `jQuery.migrateDisablePatches` & related APIs * fix an erroneous future tense in a few warnings Closes gh-574 --- src/jquery/event.js | 4 ++++ test/unit/jquery/event.js | 9 +++++++++ test/unit/migrate.js | 30 ++++++++++++++++++++++++++---- warnings.md | 6 ++++++ 4 files changed, 45 insertions(+), 4 deletions(-) diff --git a/src/jquery/event.js b/src/jquery/event.js index e13a5085..ce56591f 100644 --- a/src/jquery/event.js +++ b/src/jquery/event.js @@ -1,5 +1,6 @@ import { migrateWarn, + migrateWarnProp, migratePatchAndInfoFunc, migratePatchFunc } from "../main.js"; @@ -41,3 +42,6 @@ migratePatchAndInfoFunc( jQuery.fn, "undelegate", jQuery.fn.undelegate, migratePatchAndInfoFunc( jQuery.fn, "hover", jQuery.fn.hover, "hover", "jQuery.fn.hover() is deprecated" ); + +migrateWarnProp( jQuery.event, "global", {}, "event-global", + "jQuery.event.global is removed" ); diff --git a/test/unit/jquery/event.js b/test/unit/jquery/event.js index 519c1e01..e23383df 100644 --- a/test/unit/jquery/event.js +++ b/test/unit/jquery/event.js @@ -84,3 +84,12 @@ TestManager.runIframeTest( "Load within a ready handler", "event-lateload.html", JSON.stringify( jQuery.migrateMessages ) ); assert.ok( /load/.test( jQuery.migrateMessages[ 0 ] ), "message ok" ); } ); + +QUnit.test( "jQuery.event.global", function( assert ) { + assert.expect( 3 ); + + expectMessage( assert, "jQuery.event.global", 2, function() { + assert.ok( jQuery.isPlainObject( jQuery.event.global ), "is a plain object" ); + assert.deepEqual( jQuery.event.global, {}, "is an empty object" ); + } ); +} ); diff --git a/test/unit/migrate.js b/test/unit/migrate.js index a70ba9a2..27a4865d 100644 --- a/test/unit/migrate.js +++ b/test/unit/migrate.js @@ -46,7 +46,7 @@ QUnit.test( "jQuery.migrateDeduplicateMessages", function( assert ) { } ); QUnit.test( "disabling/enabling patches", function( assert ) { - assert.expect( 27 ); + assert.expect( 32 ); var elem = jQuery( "

" ); @@ -56,17 +56,28 @@ QUnit.test( "disabling/enabling patches", function( assert ) { // existing warnings. If the ones we rely on here get removed, // replace them with ones that still exist. + // A few APIs that are not slated for removal to make these tests more stable: assert.strictEqual( jQuery.migrateIsPatchEnabled( "pre-on-methods" ), true, "patch enabled by default (pre-on-methods)" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "proxy" ), true, "patch enabled by default (proxy)" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "shorthand-deprecated-v3" ), true, "patch enabled by default (shorthand-deprecated-v3)" ); + + // APIs patched via `migratePatchAndWarnFunc` or `migratePatchAndInfoFunc`; + // we're testing that: + // * they don't warn on access but only when called + // * they don't exist (access evaluates to `undefined`) if patch is disabled assert.strictEqual( jQuery.migrateIsPatchEnabled( "push" ), true, "patch enabled by default (push)" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "isArray" ), true, "patch enabled by default (isArray)" ); + // APIs patched via `migrateWarnProp` or `migrateInfoProp`; we're testing that: + // * they don't exist (access evaluates to `undefined`) if patch is disabled + assert.strictEqual( jQuery.migrateIsPatchEnabled( "event-global" ), + true, "patch enabled by default (event-global)" ); + expectMessage( assert, "pre-on-methods (default)", function() { jQuery().bind(); } ); @@ -82,6 +93,11 @@ QUnit.test( "disabling/enabling patches", function( assert ) { expectMessage( assert, "isArray (default)", function() { jQuery.isArray(); } ); + expectMessage( assert, "event-global (default)", function() { + + // eslint-disable-next-line no-unused-expressions + jQuery.event.global; + } ); expectNoMessage( assert, "push access without calling (default)", function() { assert.strictEqual( typeof jQuery().push, "function", @@ -92,7 +108,7 @@ QUnit.test( "disabling/enabling patches", function( assert ) { "access check doesn't trigger a message (isArray)" ); } ); - jQuery.migrateDisablePatches( "pre-on-methods", "proxy", "push", "isArray" ); + jQuery.migrateDisablePatches( "pre-on-methods", "proxy", "push", "isArray", "event-global" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "pre-on-methods" ), false, "patch disabled (pre-on-methods)" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "proxy" ), @@ -101,6 +117,8 @@ QUnit.test( "disabling/enabling patches", function( assert ) { true, "patch still enabled (shorthand-deprecated-v3)" ); assert.strictEqual( jQuery.migrateIsPatchEnabled( "push" ), false, "patch disabled (push)" ); + assert.strictEqual( jQuery.migrateIsPatchEnabled( "event-global" ), + false, "patch disabled (event-global)" ); expectNoMessage( assert, "pre-on-methods (disabled)", function() { jQuery().bind(); @@ -112,10 +130,14 @@ QUnit.test( "disabling/enabling patches", function( assert ) { jQuery().click(); } ); expectNoMessage( assert, "push (disabled)", function() { - assert.strictEqual( jQuery().push, undefined, "`push` patch no longer defined" ); + assert.strictEqual( jQuery().push, undefined, "`jQuery.fn.push` no longer defined" ); } ); expectNoMessage( assert, "isArray (disabled)", function() { - assert.strictEqual( jQuery.isArray, undefined, "`jQuery.isArray` patch no longer defined" ); + assert.strictEqual( jQuery.isArray, undefined, "`jQuery.isArray` no longer defined" ); + } ); + expectNoMessage( assert, "event-global (disabled)", function() { + assert.strictEqual( jQuery.event.global, undefined, + "`jQuery.event.global` no longer defined" ); } ); jQuery.migrateDisablePatches( "shorthand-deprecated-v3" ); diff --git a/warnings.md b/warnings.md index 2bf15c21..ffbd9630 100644 --- a/warnings.md +++ b/warnings.md @@ -171,6 +171,12 @@ This is _not_ a warning, but a console log message the plugin shows when it firs **Solution:** Rename all usage of `jQuery.Deferred.getStackHook` to `jQuery.Deferred.getErrorHook`. If you previously assigned a function returning an error stack to `jQuery.Deferred.getStackHook` or `jQuery.Deferred.getErrorHook`, change it to return a full error object. If you aim to still support jQuery <3.7, assign the hook to `jQuery.Deferred.getErrorHook` first and only later to `jQuery.Deferred.getStackHook` to avoid a Migrate warning. +### \[event-global\] JQMIGRATE: jQuery.Deferred.getStackHook is removed; use jQuery.Deferred.getErrorHook + +**Cause:** `jQuery.event.global` was an object with keys being event names for which event listeners have ever been added. Originally, it was needed for performance reasons and to fix memory leaks in old IE, but since jQuery 1.9.0, the library has only been recording the events, but it was not using that information anywhere. jQuery 4.0.0 removes the API. + +**Solution:** Remove all usage of `jQuery.event.global`; it's unlikely any existing usage is needed. + ## Deprecated APIs From 8f5205ef2e0396d711899eb50f5cad5a6bde98f1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 12 May 2025 18:23:59 +0200 Subject: [PATCH 12/12] Event: Patch jQuery.event.special's prototype Allow to use common `Object.prototype` properties on `jQuery.event.special` but warn as well. Closes gh-573 Fixes gh-542 --- src/jquery/event.js | 11 ++++++++++- test/unit/jquery/event.js | 22 ++++++++++++++++++++++ 2 files changed, 32 insertions(+), 1 deletion(-) diff --git a/src/jquery/event.js b/src/jquery/event.js index ce56591f..59229947 100644 --- a/src/jquery/event.js +++ b/src/jquery/event.js @@ -2,9 +2,11 @@ import { migrateWarn, migrateWarnProp, migratePatchAndInfoFunc, - migratePatchFunc + migratePatchFunc, + migratePatchProp } from "../main.js"; import "../disablePatches.js"; +import { patchProto } from "../utils.js"; var oldEventAdd = jQuery.event.add; @@ -45,3 +47,10 @@ migratePatchAndInfoFunc( jQuery.fn, "hover", jQuery.fn.hover, migrateWarnProp( jQuery.event, "global", {}, "event-global", "jQuery.event.global is removed" ); + +migratePatchProp( jQuery.event, "special", + patchProto( jQuery.extend( Object.create( null ), jQuery.event.special ), { + warningId: "event-special-null-proto", + apiName: "jQuery.event.special" + } ), + "event-special-null-proto" ); diff --git a/test/unit/jquery/event.js b/test/unit/jquery/event.js index e23383df..c30eaaa5 100644 --- a/test/unit/jquery/event.js +++ b/test/unit/jquery/event.js @@ -93,3 +93,25 @@ QUnit.test( "jQuery.event.global", function( assert ) { assert.deepEqual( jQuery.event.global, {}, "is an empty object" ); } ); } ); + +QUnit.test( "jQuery.event.special: properties from Object.prototype", function( assert ) { + assert.expect( 4 ); + + try { + expectNoMessage( assert, "Regular properties", function() { + jQuery.event.special.fakeevent = {}; + + // eslint-disable-next-line no-unused-expressions + jQuery.event.special.fakeevent; + } ); + + expectMessage( assert, "Properties from Object.prototype", 2, function() { + assert.ok( jQuery.event.special.hasOwnProperty( "fakeevent" ), + "hasOwnProperty works (property present)" ); + assert.ok( !jQuery.event.special.hasOwnProperty( "fakeevent2" ), + "hasOwnProperty works (property missing)" ); + } ); + } finally { + delete jQuery.event.special.fakeevent; + } +} );