First Release

[ericclemmons]

No description provided.

[ericclemmons]

This already needs refactoring to work from the inside-out:

• Is it a function?
• Is the function named like a component?
• Is it a const?
• Does it contain JSX?
• Is it wrapped in a decorator?

[ericclemmons]

Well crap. This was coded in isolation based off of source, so only running it first works:

  module: {
    loaders: [
      {
        test: /\.js$/,
        loader: "babel",
        query: {
          babelrc: false,
          cacheDirectory: true,
          passPerPreset: true,
          presets: [
            {
              plugins: [
                "syntax-jsx",
                "../src/plugin.js",
              ],
            },
            "es2015",
            "stage-0",
            "react",
          ],
        },
        exclude: /node_modules/,
      },
    ],

It needs to be able to be ran after es2015, react, and stage-0 presets.

This...will significantly make it more difficult to tell what's a functional component and what's not...

[ericclemmons]

It's working!!

Next steps:

• Rename repo to babel-plugin-react-pure-to-class
• Add README.md, specific to the new Babel version & how to do multi-pass presets.
• Video of demo. Make it prettier, too.
• Rationale.
• Note the auto-inclusion of React into scope.
• Release!

[ericclemmons]

F*.

Major blocker in an internal project:

ERROR in ./src/routes/index.js
Module build failed: ReferenceError: [BABEL] /Users/Eric/Projects/highereducation/platform/src/routes/index.js: Unknown option: base.passPerPreset
    at Logger.error (/Users/Eric/Projects/highereducation/platform/src/node_modules/babel-core/lib/transformation/file/logger.js:41:11)
    at OptionManager.mergeOptions (/Users/Eric/Projects/highereducation/platform/src/node_modules/babel-core/lib/transformation/file/options/option-manager.js:262:18)
    at OptionManager.init (/Users/Eric/Projects/highereducation/platform/src/node_modules/babel-core/lib/transformation/file/options/option-manager.js:416:10)
    at File.initOptions (/Users/Eric/Projects/highereducation/platform/src/node_modules/babel-core/lib/transformation/file/index.js:191:75)
    at new File (/Users/Eric/Projects/highereducation/platform/src/node_modules/babel-core/lib/transformation/file/index.js:122:22)
    at Pipeline.transform (/Users/Eric/Projects/highereducation/platform/src/node_modules/babel-core/lib/transformation/pipeline.js:42:16)
    at transpile (/Users/Eric/Projects/highereducation/platform/src/node_modules/babel-loader/index.js:14:22)
    at /Users/Eric/Projects/highereducation/platform/src/node_modules/babel-loader/lib/fs-cache.js:145:18
    at ReadFileContext.callback (/Users/Eric/Projects/highereducation/platform/src/node_modules/babel-loader/lib/fs-cache.js:28:23)
    at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:324:13)
 @ ./src/client.js 21:14-33

I researched this for an hour and the best I could find out is that somewhere somehow there are runtime dependency issues (of course) and old versions are still being used.

No idea how to fix this.

Maybe instead of relying on passPerPreset this plugin will have to wrap babel-plugin-react-transform like https://github.com/danmartinez101/babel-preset-react-hmre does.

That way this can run first & control the output rather than competing.

[ericclemmons]

Ok, I couldn't get this working in a local project for the life of me.

So, instead, I'm going to base functionality off of the example. If that works & can be a guide then I consider this a success.

[thangngoc89]

👍

[imranismail]

👍

[ericclemmons]

Because gaearon/babel-plugin-react-transform#85 has hit a snag (largely self-imposed, IMO, and tools of this nature require understanding the caveats of improved functionality), if this feature gets delayed further, some next steps would be:

1. Close this project in favor of my fork.
2. Rename the fork to babel-preset-react-functional-hmre.
3. Replace options with defaults.
4. Add other presets to simplify development.

Improve new & improved workflow!

[ericclemmons]

Merging just to close.