Skip to content

Commit

Permalink
Merge pull request #375 from preactjs/react-babel-plugin
Browse files Browse the repository at this point in the history
Add package `@preact/signals-react-transform`
  • Loading branch information
andrewiggins committed Aug 2, 2023
2 parents 6e4dab4 + 255208a commit fe44f6e
Show file tree
Hide file tree
Showing 21 changed files with 3,393 additions and 777 deletions.
5 changes: 5 additions & 0 deletions .changeset/five-kangaroos-joke.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@preact/signals-react": patch
---

Clean up effect store reference after finishing it
5 changes: 5 additions & 0 deletions .changeset/swift-walls-rhyme.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@preact/signals-react": patch
---

Update internal useSignals API
5 changes: 5 additions & 0 deletions .changeset/tender-scissors-perform.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@preact/signals-react-transform": patch
---

Initial alpha release
2 changes: 1 addition & 1 deletion docs/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
"react-dom": "^18.2.0"
},
"devDependencies": {
"@babel/core": "^7.18.10",
"@babel/core": "^7.22.8",
"@preact/preset-vite": "^2.3.0",
"@types/react": "^18.0.18",
"@types/react-dom": "^18.0.6",
Expand Down
1 change: 1 addition & 0 deletions global.d.ts
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
declare const expect: Chai.ExpectStatic;
declare const sinon: import("sinon").SinonStatic;
declare const transformSignalCode: (code: string, options?: import("@preact/signals-react-transform").PluginOptions) => string;
29 changes: 26 additions & 3 deletions karma.conf.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ const subPkgPath = pkgName => {
// Resolve from package.exports field
const pkgJson = path.join(__dirname, pkgName, "package.json");
const pkgExports = require(pkgJson).exports;
const file = pkgExports["."].browser;
const file = pkgExports["."].browser ?? pkgExports["."].import;
return path.join(__dirname, pkgName, file);
};

Expand All @@ -62,6 +62,7 @@ function createEsbuildPlugin() {
"@preact/signals-core": subPkgPath("./packages/core"),
"@preact/signals": subPkgPath("./packages/preact"),
"@preact/signals-react": subPkgPath("./packages/react"),
"@preact/signals-react-transform": subPkgPath("./packages/react-transform"),
};

return {
Expand Down Expand Up @@ -90,6 +91,11 @@ function createEsbuildPlugin() {
};
});

// Mock fs module to run babel in a browser environment
build.onResolve({ filter: /^fs$/ }, () => {
return { path: path.join(__dirname, "test/browser/mockFs.js") };
});

// Apply babel pass whenever we load a .js file
build.onLoad({ filter: /\.[mc]?[jt]sx?$/ }, async args => {
const contents = await fs.readFile(args.path, "utf-8");
Expand Down Expand Up @@ -190,6 +196,8 @@ function createEsbuildPlugin() {
};
}

const pkgList = ["core", "preact", "react", "react/runtime", "react-transform"];

module.exports = function (config) {
config.set({
browsers: Object.keys(localLaunchers),
Expand Down Expand Up @@ -254,7 +262,20 @@ module.exports = function (config) {

files: [
{
pattern: process.env.TESTS || "packages/*/test/**/*.test.tsx",
// Provide some NodeJS globals to run babel in a browser environment
pattern: "test/browser/nodeGlobals.js",
watched: false,
type: "js",
},
{
pattern: "test/browser/babel.js",
watched: false,
type: "js",
},
{
pattern:
process.env.TESTS ||
`packages/{${pkgList.join(",")}}/test/{,browser,shared}/*.test.tsx`,
watched: false,
type: "js",
},
Expand All @@ -265,7 +286,9 @@ module.exports = function (config) {
},

preprocessors: {
"packages/*/test/**/*": ["esbuild"],
[`packages/{${pkgList.join(",")}}/test/**/*`]: ["esbuild"],
[`test/browser/babel.js`]: ["esbuild"],
[`test/browser/nodeGlobals.js`]: ["esbuild"],
},

plugins: [
Expand Down
15 changes: 11 additions & 4 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,12 @@
"private": true,
"scripts": {
"prebuild": "rimraf packages/core/dist/ packages/preact/dist",
"build": "pnpm build:core && pnpm build:preact && pnpm build:react-runtime && pnpm build:react",
"build": "pnpm build:core && pnpm build:preact && pnpm build:react-runtime && pnpm build:react && pnpm build:react-transform",
"build:core": "microbundle --raw --cwd packages/core && pnpm postbuild:core",
"build:preact": "microbundle --raw --cwd packages/preact && pnpm postbuild:preact",
"build:react": "microbundle --raw --cwd packages/react && pnpm postbuild:react",
"build:react-runtime": "microbundle --raw --cwd packages/react/runtime && pnpm postbuild:react-runtime",
"build:react-transform": "microbundle --no-compress --raw --cwd packages/react-transform",
"postbuild:core": "cd packages/core/dist && mv -f index.d.ts signals-core.d.ts",
"postbuild:preact": "cd packages/preact/dist && mv -f preact/src/index.d.ts signals.d.ts && rm -dr preact",
"postbuild:react": "cd packages/react/dist && mv -f react/src/index.d.ts signals.d.ts && rm -dr react",
Expand All @@ -24,8 +25,8 @@
"test:karma:watch": "karma start karma.conf.js --no-single-run",
"test:karma:prod": "cross-env MINIFY=true NODE_ENV=production karma start karma.conf.js --single-run",
"test:karma:prod:watch": "cross-env NODE_ENV=production karma start karma.conf.js --no-single-run",
"test:mocha": "cross-env COVERAGE=true mocha --require test/node/setup.js --recursive packages/react/test/node/**.test.tsx",
"test:mocha:prod": "cross-env COVERAGE=true NODE_ENV=production mocha --require test/node/setup.js --recursive packages/react/test/node/**.test.tsx",
"test:mocha": "cross-env COVERAGE=true mocha --require test/node/setup.js --recursive packages/*/test/node/**.test.tsx",
"test:mocha:prod": "cross-env COVERAGE=true NODE_ENV=production mocha --require test/node/setup.js --recursive packages/*/test/node/**.test.tsx",
"docs:start": "cd docs && pnpm start",
"docs:build": "cd docs && pnpm build",
"docs:preview": "cd docs && pnpm preview",
Expand All @@ -39,12 +40,17 @@
],
"license": "MIT",
"devDependencies": {
"@babel/core": "^7.19.1",
"@babel/core": "^7.22.8",
"@babel/plugin-proposal-explicit-resource-management": "^7.22.6",
"@babel/plugin-syntax-jsx": "^7.21.4",
"@babel/plugin-transform-modules-commonjs": "^7.22.5",
"@babel/plugin-transform-react-jsx": "^7.21.4",
"@babel/plugin-transform-typescript": "^7.19.1",
"@babel/preset-env": "^7.19.1",
"@babel/preset-react": "^7.18.6",
"@babel/preset-typescript": "^7.18.6",
"@babel/register": "^7.21.0",
"@babel/standalone": "^7.22.6",
"@changesets/changelog-github": "^0.4.6",
"@changesets/cli": "^2.24.2",
"@types/chai": "^4.3.3",
Expand All @@ -56,6 +62,7 @@
"@typescript-eslint/parser": "^5.33.0",
"babel-plugin-istanbul": "^6.1.1",
"babel-plugin-transform-rename-properties": "^0.1.0",
"buffer": "^6.0.3",
"chai": "^4.3.6",
"cross-env": "^7.0.3",
"errorstacks": "^2.4.0",
Expand Down
121 changes: 121 additions & 0 deletions packages/react-transform/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,121 @@
# Signals React Transform

> A Babel plugin to transform React components to automatically subscribe to Preact Signals.
Signals is a performant state management library with two primary goals:

1. Make it as easy as possible to write business logic for small up to complex apps. No matter how complex your logic is, your app updates should stay fast without you needing to think about it. Signals automatically optimize state updates behind the scenes to trigger the fewest updates necessary. They are lazy by default and automatically skip signals that no one listens to.
2. Integrate into frameworks as if they were native built-in primitives. You don't need any selectors, wrapper functions, or anything else. Signals can be accessed directly and your component will automatically re-render when the signal's value changes.

Read the [announcement post](https://preactjs.com/blog/introducing-signals/) to learn more about which problems signals solves and how it came to be.

## Installation:

```sh
npm install @preact/signals-react-transform
```

## Usage

This package works with the `@preact/signals-react` package to integrate signals into React. You use the `@preact/signals-react` package to setup and access signals inside your components and this package is one way to automatically subscribe your components to rerender when the signals you use change.

To understand how to use signals in your components, check out the [Signals React documentation](../react/README.md). This babel transform is one of a couple different ways to use signals in React. To see other ways, including integrations that don't require a build step, see the [Signals React documentation](../react/README.md).

Then, setup the transform plugin in your Babel config:

```js
// babel.config.js
module.exports = {
plugins: [["@preact/signals-react-transform"]],
};
```

Here is an example of a component using signals:

```js
import { signal } from "@preact/signals-react";

const count = signal(0);

function CounterValue() {
// Whenever the `count` signal is updated, we'll
// re-render this component automatically for you
return <p>Value: {count.value}</p>;
}
```

After the babel transform runs, it'll look something like:

```js
import { signal, useSignals } from "@preact/signals-react";

const count = signal(0);

function CounterValue() {
const effect = useSignals();
try {
// Whenever the `count` signal is updated, we'll
// re-render this component automatically for you
return <p>Value: {count.value}</p>;
} finally {
effect.endTracking();
}
}
```

The `useSignals` hook setups the machinery to observe what signals are used inside the component and then automatically re-render the component when those signals change. The `endTracking` function notifies the tracking mechanism that this component has finished rendering. When your component unmounts, it also unsubscribes from all signals it was using.

Fundamentally, this Babel transform needs to answer two questions in order to know whether to transform a function:

1. Is a function a component?
2. If so, does this component use signals?

Currently we use the following heuristics to answer these questions:

1. A function is a component if it has a capitalized name (e.g. `function MyComponent() {}`), contains JSX, and is declared at module scope.
2. If a function's body includes a member expression referencing `.value` (i.e. `something.value`), we assume it's a signal.

If your function/component meets these criteria, this plugin will transform it. If not, it will be left alone. If you have a function that uses signals but does not meet these criteria (e.g. a function that manually calls `createElement` instead of using JSX), you can add a comment with the string `@trackSignals` to instruct this plugin to transform this function. You can also manually opt-out of transforming a function by adding a comment with the string `@noTrackSignals`.

```js
// This function will be transformed
/** @trackSignals */
function MyComponent() {
return createElement("h1", null, signal.value);
}

// This function will not be transformed
/** @noTrackSignals */
function MyComponent() {
return <p>{signal.value}</p>;
}
```

Note, this plugin will not transform higher-order components (HOCs) that wrap other components. If you have an HOC that uses signals, you can use the `@trackSignals` comment to transform the body of the higher-order component.

## Plugin Options

### `mode`

The `mode` option enables you to control how the plugin transforms your code. There are two modes:

- `mode: "auto"` (default): This mode will automatically transform any function that meets the criteria described above. This is the easiest way to get started with signals.
- `mode: "manual"`: This mode will only transform functions that have a comment with the string `@trackSignals`. This is useful if you want to manually control which functions are transformed.

```js
// babel.config.js
module.exports = {
plugins: [
[
"@preact/signals-react-transform",
{
mode: "manual",
},
],
],
};
```

## License

`MIT`, see the [LICENSE](../../LICENSE) file.
65 changes: 65 additions & 0 deletions packages/react-transform/package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,65 @@
{
"name": "@preact/signals-react-transform",
"version": "0.0.1-alpha.0",
"license": "MIT",
"description": "Manage state with style in React",
"keywords": [
"babel",
"signals",
"react"
],
"authors": [
"The Preact Authors (https://github.com/preactjs/signals/contributors)"
],
"repository": {
"type": "git",
"url": "https://github.com/preactjs/signals",
"directory": "packages/react"
},
"bugs": "https://github.com/preactjs/signals/issues",
"homepage": "https://preactjs.com",
"funding": {
"type": "opencollective",
"url": "https://opencollective.com/preact"
},
"main": "dist/signals-transform.js",
"module": "dist/signals-transform.module.js",
"types": "dist/signals-transform.d.ts",
"source": "src/index.ts",
"exports": {
".": {
"types": "./dist/signals-transform.d.ts",
"import": "./dist/signals-transform.mjs",
"require": "./dist/signals-transform.js"
}
},
"mangle": "../../mangle.json",
"scripts": {
"prepublishOnly": "cd ../.. && pnpm build:react-transform"
},
"dependencies": {
"@babel/helper-module-imports": "^7.22.5",
"@babel/helper-plugin-utils": "^7.22.5",
"@preact/signals-react": "workspace:^1.3.2",
"use-sync-external-store": "^1.2.0"
},
"peerDependencies": {
"@babel/core": "^7.22.8",
"react": "^16.14.0 || 17.x || 18.x"
},
"devDependencies": {
"@babel/core": "^7.22.8",
"@types/babel__core": "^7.20.1",
"@types/babel__helper-module-imports": "^7.18.0",
"@types/babel__helper-plugin-utils": "^7.10.0",
"@types/react": "^18.0.18",
"@types/react-dom": "^18.0.6",
"@types/use-sync-external-store": "^0.0.3",
"assert": "^2.0.0",
"buffer": "^6.0.3",
"path": "^0.12.7",
"react": "^18.2.0",
"react-dom": "^18.2.0",
"react-router-dom": "^6.9.0"
}
}
Loading

0 comments on commit fe44f6e

Please sign in to comment.