Add unsafe API; reimplement CP via safe/unsafe API (#46)

* Add unsafe API

* Move Handle into types

* Move Unsafe to new namespace

* Add Safe API; reimplement CP using it

* Relocate send

* Add Js prefix to all Js options

* Rename execSyncOpts to execSync'

* Reimplement ChildProcess bindings via Unsafe/Safe APIs

* Update changelog

* Update usage of exec'

* Update FFI name to match PS one

* Add missing docs on options

* Add more missing options docs

* Tweak docs one more time

Author: JordanMartinez

CHANGELOG.md

@@ -22,7 +22,30 @@ Breaking changes:
 - Update `pid` type signature to return `Maybe Pid` rather than `Pid` (#44 by @JordanMartinez)
 - Update `kill` returned value from `Effect Unit` to `Effect Boolean` (#44 by @JordanMartinez)
 - Migrate `Error` to `node-os`' `SystemError` (#45 by @JordanMartinez)
+- Breaking changes made to the `Exit` type (#46 by @JordanMartinez)
 
+  - Moved from `Node.ChildProces` to `Node.ChildProces.Types`
+  - Changed the `BySignal`'s constructor's arg type from `Signal` to `String`
+- Breaking changes made to the `Handle` type (#46 by @JordanMartinez)
+  
+  - Moved from `Node.ChildProces` to `Node.ChildProces.Types`
+- Converted `defaultOptions { override = Just 1}` pattern to `(_ { override = Just 1})`  (#46 by @JordanMartinez)
+
+  Before:
+  ```purs
+  spawn "foo" [ "bar" ] (defaultSpawnOptions { someOption = Just overrideValue })
+  spawn "foo" [ "bar" ] defaultSpawnOptions
+  ```
+
+  After:
+  ```purs
+  spawn "foo" [ "bar" ] (_ { someOption = Just overrideValue })
+  spawn "foo" [ "bar" ] identity
+  ```
+- Restrict end-user's ability to configure `stdio` to only those appended to `safeStdio` (#46 by @JordanMartinez)
+
+  See the module docs for `Node.ChildProcess`.
+- All `ChildProcess`-creating functions have been updated to support no args and all args variants (#46 by @JordanMartinez)
 
 New features:
 - Added event handler for `spawn` event (#43 by @JordanMartinez)
@@ -36,14 +59,17 @@ New features:
   - signalCode
   - spawnArgs
   - spawnFile
+- Added unsafe, uncurried API of all ChildProcess-creating functions (#46 by @JordanMartinez)
+- Added safe variant of `spawnSync`/`spawnSync'` (#46 by @JordanMartinez)
 
 Bugfixes:
 
 Other improvements:
 - Bumped CI's node version to `lts/*` (#41 by @JordanMartinez)
 - Updated CI `actions/checkout` and `actions/setup-nodee` to `v3` (#41 by @JordanMartinez)
 - Format codebase & enforce formatting in CI via purs-tidy (#42 by @JordanMartinez)
-- Migrate more FFI to uncurried functions (#44 by @JordanMartinez)
+- Migrate FFI to uncurried functions (#44, #46 by @JordanMartinez)
+- Updated recommended module alias in docs (#46 by @JordanMartinez)
 
 ## [v9.0.0](https://github.com/purescript-node/purescript-node-child-process/releases/tag/v9.0.0) - 2022-04-29
 

src/Node/ChildProcess.js

@@ -1,63 +1,2 @@
-/* eslint-env node*/
-
-import { spawn, exec, execFile, execSync, execFileSync, fork as cp_fork } from "child_process";
-
-export function unsafeFromNullable(msg) {
-  return x => {
-    if (x === null) throw new Error(msg);
-    return x;
-  };
-}
-
-export const connectedImpl = (cp) => cp.connected;
-export const disconnectImpl = (cp) => cp.disconnect();
-export const exitCodeImpl = (cp) => cp.exitCode;
-export const pidImpl = (cp) => cp.pid;
-export const killImpl = (cp) => cp.kill();
-export const killStrImpl = (cp, str) => cp.kill(str);
-export const killedImpl = (cp) => cp.killed;
-export const signalCodeImpl = (cp) => cp.signalCode;
-export const spawnArgs = (cp) => cp.spawnArgs;
-export const spawnFile = (cp) => cp.spawnFile;
-
-export function spawnImpl(command) {
-  return args => opts => () => spawn(command, args, opts);
-}
-
-export function execImpl(command) {
-  return opts => callback => () => exec(
-    command,
-    opts,
-    (err, stdout, stderr) => {
-      callback(err)(stdout)(stderr)();
-    }
-  );
-}
-
-export const execFileImpl = function execImpl(command) {
-  return args => opts => callback => () => execFile(
-    command,
-    args,
-    opts,
-    (err, stdout, stderr) => {
-      callback(err)(stdout)(stderr)();
-    }
-  );
-};
-
-export function execSyncImpl(command) {
-  return opts => () => execSync(command, opts);
-}
-
-export function execFileSyncImpl(command) {
-  return args => opts => () => execFileSync(command, args, opts);
-}
-
-export function fork(cmd) {
-  return args => () => cp_fork(cmd, args);
-}
-
 const _undefined = undefined;
 export { _undefined as undefined };
-import process from "process";
-export { process };