Skip to content

Commit fdaf4ee

Browse files
committed
fix(sucrase): prepare for Rollup 3
BREAKING CHANGES: Requires Node 14
1 parent 32aa6d2 commit fdaf4ee

File tree

9 files changed

+55
-40
lines changed

9 files changed

+55
-40
lines changed

packages/sucrase/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313

1414
## Requirements
1515

16-
This plugin requires an [LTS](https://github.com/nodejs/Release) Node version (v8.0.0+) and Rollup v1.20.0+.
16+
This plugin requires an [LTS](https://github.com/nodejs/Release) Node version (v14.0.0+) and Rollup v2.53.1+.
1717

1818
## Install
1919

packages/sucrase/package.json

Lines changed: 19 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,15 @@
1313
"author": "Rich Harris",
1414
"homepage": "https://github.com/rollup/plugins/tree/master/packages/sucrase/#readme",
1515
"bugs": "https://github.com/rollup/plugins/issues",
16-
"main": "dist/index.js",
17-
"module": "dist/index.es.js",
16+
"main": "./dist/cjs/index.js",
17+
"module": "./dist/es/index.js",
18+
"type": "commonjs",
19+
"exports": {
20+
"require": "./dist/cjs/index.js",
21+
"import": "./dist/es/index.js"
22+
},
1823
"engines": {
19-
"node": ">=12.0.0"
24+
"node": ">=14.0.0"
2025
},
2126
"scripts": {
2227
"build": "rollup -c",
@@ -33,7 +38,7 @@
3338
"test:ts": "tsc types/index.d.ts test/types.ts --noEmit"
3439
},
3540
"files": [
36-
"dist",
41+
"dist/**/*.{js,json}",
3742
"types",
3843
"README.md",
3944
"LICENSE"
@@ -47,21 +52,24 @@
4752
"jsx"
4853
],
4954
"peerDependencies": {
50-
"rollup": "^2.53.1"
55+
"rollup": "^2.53.1||^3.0.0"
56+
},
57+
"peerDependenciesMeta": {
58+
"rollup": {
59+
"optional": true
60+
}
5161
},
5262
"dependencies": {
53-
"@rollup/pluginutils": "^4.1.1",
54-
"sucrase": "^3.20.0"
63+
"@rollup/pluginutils": "^4.2.1",
64+
"sucrase": "^3.27.0"
5565
},
5666
"devDependencies": {
5767
"@rollup/plugin-alias": "^3.1.4",
58-
"rollup": "^2.67.3"
68+
"rollup": "^3.0.0-7"
5969
},
6070
"types": "./types/index.d.ts",
6171
"ava": {
62-
"babel": {
63-
"compileEnhancements": false
64-
},
72+
"workerThreads": false,
6573
"files": [
6674
"!**/fixtures/**",
6775
"!**/output/**",

packages/sucrase/rollup.config.js

Lines changed: 0 additions & 12 deletions
This file was deleted.

packages/sucrase/rollup.config.mjs

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
import { readFileSync } from 'fs';
2+
3+
import { createConfig } from '../../shared/rollup.config.mjs';
4+
5+
export default {
6+
...createConfig({
7+
pkg: JSON.parse(readFileSync(new URL('./package.json', import.meta.url), 'utf8'))
8+
}),
9+
input: 'src/index.js',
10+
plugins: []
11+
};

packages/sucrase/src/index.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
1-
const fs = require('fs');
2-
const path = require('path');
1+
import fs from 'fs';
2+
import path from 'path';
33

4-
const { transform } = require('sucrase');
5-
const { createFilter } = require('@rollup/pluginutils');
4+
import { transform } from 'sucrase';
5+
import { createFilter } from '@rollup/pluginutils';
66

7-
module.exports = function sucrase(opts = {}) {
7+
export default function sucrase(opts = {}) {
88
const filter = createFilter(opts.include, opts.exclude);
99

1010
return {
@@ -59,4 +59,4 @@ module.exports = function sucrase(opts = {}) {
5959
};
6060
}
6161
};
62-
};
62+
}
-10 Bytes
Binary file not shown.

pnpm-lock.yaml

Lines changed: 15 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

shared/rollup.config.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,14 @@ export function createConfig({ pkg, external = [] }) {
2323
output: [
2424
{
2525
format: 'cjs',
26-
file: pkg.main,
26+
file: pkg.exports.require,
2727
exports: 'named',
2828
footer: 'module.exports = Object.assign(exports.default, exports);',
2929
sourcemap: true
3030
},
3131
{
3232
format: 'es',
33-
file: pkg.module,
33+
file: pkg.exports.import,
3434
plugins: [emitModulePackageFile()],
3535
sourcemap: true
3636
}

tsconfig.base.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
"pretty": true,
1313
"sourceMap": true,
1414
"strict": true,
15-
"target": "es2017"
15+
"target": "es2019"
1616
},
1717
"exclude": ["dist", "node_modules", "test/types"]
1818
}

0 commit comments

Comments
 (0)