Add npm workspaces

.eslintignore

@@ -0,0 +1,5 @@
+messages/javascript/src/messages.js
+messages/javascript/src/messages.d.ts
+messages/javascript/dist/src/messages.js
+messages/javascript/dist/src/messages.d.ts
+gherkin/javascript/src/Parser.ts

.gitignore

@@ -1,4 +1,5 @@
 .DS_Store
 .rsynced
 .idea/
-executables
+executables
+node_modules

.npmrc

@@ -0,0 +1 @@
+legacy-peer-deps=true

.templates/javascript/.mocharc.json

@@ -1,6 +1,6 @@
 {
-  "require": ["ts-node/register", "source-map-support/register"],
+  "require": ["tsconfig-paths/register", "ts-node/register", "source-map-support/register"],
   "extension": ["ts", "tsx"],
   "recursive": true,
-  "timeout": 10000  
+  "timeout": 10000
 }

.templates/javascript/default.mk

@@ -6,53 +6,19 @@ PRIVATE = $(shell node -e "console.log(require('./package.json').private)")
 NPM ?= npm
 IS_TESTDATA = $(findstring -testdata,${CURDIR})
 
-ifeq (yarn,$(NPM))
-LOCKFILE = yarn.lock
-else
-LOCKFILE = package-lock.json
-endif
-
-default: .tested .built .linted
+default: .tested
 .PHONY: default
 
-.deps: $(LOCKFILE)
-	touch $@
-
-.codegen: .deps
-	touch $@
-
-.built: .codegen $(TYPESCRIPT_SOURCE_FILES)
-	$(NPM) run build
+.codegen:
 	touch $@
 
 .tested: .tested-npm
 
-.tested-npm: .built $(TYPESCRIPT_SOURCE_FILES)
-	TS_NODE_TRANSPILE_ONLY=1 $(NPM) run test
-	touch $@
-
-.linted: $(TYPESCRIPT_SOURCE_FILES) .built
-	$(NPM) run lint-fix
-	touch $@
-
-$(LOCKFILE): package.json
-	$(NPM) install
+.tested-npm: $(TYPESCRIPT_SOURCE_FILES)
+	$(NPM) run test
 	touch $@
 
-update-dependencies:
-	npx npm-check-updates --upgrade
-.PHONY: update-dependencies
-
-remove-local-dependencies:
-ifeq ($(IS_TESTDATA),-testdata)
-	# no-op
-else
-	cat package.json | sed 's/"@cucumber\/\(.*\)": "file:..\/..\/.*"/"@cucumber\/\1": "^0.0.0"/' > package.json.tmp
-	mv package.json.tmp package.json
-endif
-.PHONY: remove-local-dependencies
-
-pre-release: remove-local-dependencies update-version update-dependencies clean default
+pre-release: update-version update-dependencies clean default
 .PHONY: pre-release
 
 update-version:
@@ -81,17 +47,11 @@ endif
 .PHONY: publish
 
 post-release:
-ifeq ($(IS_TESTDATA),-testdata)
-	# no-op
-else
-	cat package.json | sed 's/"@cucumber\/\(.*\)": .*"/"@cucumber\/\1": "file:..\/..\/\1\/javascript"/' > package.json.tmp
-	mv package.json.tmp package.json
-endif
 .PHONY: post-release
 
 clean: clean-javascript
 .PHONY: clean
 
 clean-javascript:
-	rm -rf .deps .codegen .built* .tested* .linted package-lock.json yarn.lock node_modules coverage dist acceptance
+	rm -rf .deps .codegen .tested* node_modules coverage dist acceptance
 .PHONY: clean-javascript

.templates/javascript/tsconfig.json

@@ -1,26 +1,3 @@
 {
-  "compilerOptions": {
-    "declaration": true,
-    "target": "es5",
-    "lib": [
-      "es2019",
-      "dom"
-    ],
-    "sourceMap": true,
-    "allowJs": false,
-    "jsx": "react",
-    "resolveJsonModule": true,
-    "module": "commonjs",
-    "esModuleInterop": true,
-    "noImplicitAny": true,
-    "moduleResolution": "node",
-    "outDir": "dist",
-    "downlevelIteration": true,
-    "skipLibCheck": true
-  },
-  "include": [
-    "src/**/*",
-    "test/**/*",
-    "features/**/*"
-  ]
+  "extends": "../../tsconfig.json"
 }

Makefile

@@ -1,6 +1,5 @@
 SHELL := /usr/bin/env bash
-PACKAGES ?= c21e \
-	messages \
+PACKAGES ?= messages \
 	gherkin \
 	gherkin-utils \
 	cucumber-expressions \
@@ -17,9 +16,14 @@ PACKAGES ?= c21e \
 	demo-formatter \
 	json-to-messages
 
-default: .rsynced $(patsubst %,default-%,$(PACKAGES))
+default: .rsynced .typescript-built $(patsubst %,default-%,$(PACKAGES))
 .PHONY: default
 
+.typescript-built:
+	npm ci
+	npm run build
+.PHONY: .typescript-built
+
 default-%: %
 	cd $< && make default
 

RELEASE_PROCESS.md

@@ -70,14 +70,6 @@ Make sure the package still builds, and that the tests are still passing:
 
     make clean && make
 
-** Special note for html-formatter **
-
-We need to verify that the webpacked JavaScript runs in the browser.
-Run the following on your host OS (it won't work in Docker):
-
-    make clean
-    CHECK_INTEGRITY=1 make
-
 If all is good, proceed to the next step. Otherwise, make the necessary edits
 until the build passes.