chore: switch to 'npm workspaces' (#156)

Let's try npm workspaces for handling multiple packages.
The primary win is developing local changes to the helpers
package and having the others automatically linked.

Note that npm workspaces requires npm@7 or later, which
typically means node@16 or later. For earler Node.js versions
there are utils/run-* scripts to support install/test/lint.

Author: trentm

.ci/run-install.sh

@@ -21,8 +21,8 @@ jobs:
       - uses: actions/setup-node@v3
         with:
           node-version: 18
-      - run: .ci/run-install.sh
-      - run: .ci/run-lint.sh
+      - run: utils/run-install.sh
+      - run: utils/run-lint.sh
 
   test:
     runs-on: ubuntu-latest
@@ -36,6 +36,6 @@ jobs:
         with:
           node-version: ${{ matrix.node-version }}
       - name: Install dependencies
-        run: .ci/run-install.sh
+        run: utils/run-install.sh
       - name: Test Node.js v${{ matrix.node-version }}
-        run: .ci/run-test.sh
+        run: utils/run-test.sh

.ci/run-lint.sh

@@ -1,49 +1,28 @@
-# All development tasks are typically available via npm scripts, i.e.
-# `npm run <script> ...` or via CI scripts (".ci/*.sh"). This Makefile
-# exists as a convenience for some common tasks.
+# Common development tasks are typically available via npm scripts, e.g.:
+#	  npm --workspaces install
+#	  npm --workspaces test
+#	  npm --workspaces lint
+#	  npm --workspaces lint:fix
+#
+# This Makefile exists as a convenience, to support older Node.js versions
+# (which npm <=7 when workspaces support was added), and to support some less
+# common tasks.
 
 .PHONY: all
 all:
-	./.ci/run-install.sh
-	(cd utils && npm install)
+	./utils/run-install.sh
 
-.PHONY: clean
-clean:
-	(cd helpers && rm -rf node_modules)
-	(cd loggers/winston && rm -rf node_modules)
-	(cd loggers/morgan && rm -rf node_modules)
-	(cd loggers/pino && rm -rf node_modules)
-	(cd utils && rm -rf node_modules)
+.PHONY: test
+test:
+	./utils/run-test.sh
 
 .PHONY: lint
 lint: check-license-headers
-	./.ci/run-lint.sh
+	./utils/run-lint.sh
 
 .PHONY: fmt
 fmt:
-	(cd helpers && npm run lint:fix)
-	(cd loggers/winston && npm run lint:fix)
-	(cd loggers/morgan && npm run lint:fix)
-	(cd loggers/pino && npm run lint:fix)
-	(cd utils && npm run lint:fix)
-
-.PHONY: test
-test:
-	./.ci/run-test.sh
-
-# For local dev, setup each logger to use the local helpers, rather than
-# a version published to npm. Need to be careful to not *push* with that
-# tweak to each package.json.
-.PHONY: install-local-helpers undo-install-local-helpers
-install-local-helpers:
-	(cd loggers/winston && npm install ../../helpers)
-	(cd loggers/morgan && npm install ../../helpers)
-	(cd loggers/pino && npm install ../../helpers)
-undo-install-local-helpers:
-	export HELPERS_VER=$(shell cd helpers && npm info . version) && \
-		(cd loggers/winston && npm install @elastic/ecs-helpers@v$$HELPERS_VER) && \
-		(cd loggers/morgan && npm install @elastic/ecs-helpers@v$$HELPERS_VER) && \
-		(cd loggers/pino && npm install @elastic/ecs-helpers@v$$HELPERS_VER)
+	npm --workspaces lint:fix # requires npm>=7 (aka node>=16)
 
 # Build and open the rendered docs for testing.
 #
@@ -61,7 +40,7 @@ docs-and-open:
 # List licenses of prod deps.
 .PHONY: list-licenses
 list-licenses:
-	@for dir in helpers $(shell ls -d loggers/*); do \
+	@for dir in helpers $(shell ls -d packages/*); do \
 		(cd $$dir && npm ls --prod --parseable | while read subdir; do node -e "console.log(require('$$subdir/package.json').license)"; done) \
 	done | sort | uniq -c | sort -n