Update spin up to spin watch so npm run spin works.

[tpmccallum]

Signed-off-by: tpmccallum [email protected]

Fixes #849

Before:

$ npm run spin

> [email protected] spin /home/tpmccallum/Fermyon/developer
> nodemon --watch content --watch static --watch templates --watch spin-up-hub --ext html,md,rhai,hbs,css,js --verbose --legacy-watch --signal SIGINT --exec 'npm run build-index && npm run build-hub-index && spin up --file spin.toml --quiet --env PREVIEW_MODE=$PREVIEW_MODE'

[nodemon] 2.0.20
[nodemon] to restart at any time, enter `rs`
[nodemon] or send SIGUSR2 to 2615137 to restart
[nodemon] watching path(s): content/**/* static/**/* templates/**/* spin-up-hub/**/*
[nodemon] watching extensions: html,md,rhai,hbs,css,js
[nodemon] starting `npm run build-index && npm run build-hub-index && spin up --file spin.toml --quiet --env PREVIEW_MODE=$PREVIEW_MODE`
[nodemon] spawning
[nodemon] child pid: 2615159
[nodemon] watching 489 files

> [email protected] build-index /home/tpmccallum/Fermyon/developer
> node md_parser.mjs --dir=./content/ --out=./static/data.json --ignore=./content/api/**/*

[nodemon] files triggering change check: static/data.json
[nodemon] matched rule: /home/tpmccallum/Fermyon/developer/static/**/*
[nodemon] changes after filters (before/after): 1/0

> [email protected] build-hub-index /home/tpmccallum/Fermyon/developer
> node hub_index_generator.mjs --dir=./content/api/hub/ --out=./static/hub-index-data.json --ignore=./content/api/hub/get_list.md,

[nodemon] files triggering change check: static/hub-index-data.json
[nodemon] matched rule: /home/tpmccallum/Fermyon/developer/static/**/*
[nodemon] changes after filters (before/after): 1/0
Error: Failed to load manifest from "spin.toml"

Caused by:
   0: Failed to load Spin app from "spin.toml"
   1: Failed to load component `hub-fileserver-static`
   2: File or directory "spin-up-hub/dist/" does not exist
[nodemon] app crashed - waiting for file changes before starting...

[github-actions]

🚀 preview deployed successfully to Fermyon Cloud and available at https://fermyon-developer-pr-1284-qrcubjnh.fermyon.app

[tpmccallum]

Perhaps there is more to this than meets the eye.

Worked the first and second time but failing consistently now:

✓ built in 5.75s
[nodemon] files triggering change check: spin-up-hub/dist/assets/index-a80a27fc.js
[nodemon] matched rule: /home/tpmccallum/Fermyon/developer/spin-up-hub/**/*
[nodemon] changes after filters (before/after): 1/1
[nodemon] restarting due to changes...
[nodemon] spin-up-hub/dist/assets/index-a80a27fc.js

[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...
[nodemon] still waiting for 1 sub-process to finish...

[karthik2804]

You do not need both spin watch and spin up. They effectively serve the same purpose of restartting. We got to choose one or the other,

[tpmccallum]

Hi @karthik2804
Not sure I understand. There is only one spin command in the package.json (spin watch) so as far as I can tell after this update the site will exclusively use spin watch.

[itowlson]

I think this might be a typo for not needing both spin watch and nodemon --watch. If nodemon is already watching, it shouldn't be telling Spin to watch, just to run. Conversely, if both are watching, spin watch get as far as going "oh there's a change I better restart spin up" and then nodemon will swoop in and kills it.

[karthik2804]

Perhaps an even better idea for this would be to use, spin up --direct-mounts that would make the cycle much faster instead of restarting spin. We could just have nodemon watch for rebuilding the search index.

cc: @itowlson for viz