Optional nodejs build?

[jimafisk]

We're finally at a working solution for removing NodeJS as a dependency of the project (#40) 🎸 🤘

That brings up the question of what to do with the old NodeJS dependent build process. My initial thought was to completely remove this, which is what I've already done for the client build. Now I'm thinking there is some advantages to adding it back in 🤦 (and leaving the static html nodejs build as an option as well).

So leaving a NodeJS option would have the following pros / cons:

Pros	Cons
Can test broken things against native NodeJS builds	More code, harder to maintain, changes to things like props have to be made in two places
Allows devs to continue ejecting the core build process for modification	Slightly slower builds since prep for both builds will be run every time because I'm planning on only breaking out the exec.Command()
Could be faster in some scenarios (with defaults it's slower though)	More work to set up the flags for build and serve commands, as well as make these optional when nodejs is run

References:

• NodeJS client prep: https://github.com/plentico/plenti/blob/da8484f8c5b316ffb213239158616af5b32be973/cmd/build/client.go
• NodeJS static prep: https://github.com/plentico/plenti/blob/da8484f8c5b316ffb213239158616af5b32be973/cmd/build/data_source.go
• NodeJS exec script: https://github.com/plentico/plenti/blob/da8484f8c5b316ffb213239158616af5b32be973/cmd/build/exec_node.go
• NodeJS build written in JS (can be ejected from core for modification): https://github.com/plentico/plenti/blob/da8484f8c5b316ffb213239158616af5b32be973/ejected/build.js

If keeping this in as an option for now, we should add a flag to that would look like this when applied in the CLI:

• plenti build --nodejs=true
• plenti serve --nodejs=true

Alternatively you should be able to set this in the site-wide configuration file (plenti.json). We'd have to modify the API to allow something like:

{
  "build": {
    "output": "public",
    "nodejs": true
  }
}

[jimafisk]

This is now incorporated into the 0.2.* versions of plenti: da0ab90

The CLI flag (--nodejs=true) should work with both the build and serve commands.

I did not update the plenti.json config file API to allow declaring nodejs builds there (as proposed above). If this becomes useful to folks down the road, happy to add it.