Safe Load - [merged]

[benloh]

Merges dev-bl/next-gui-safeload -> dev-next-gui

Background

When editing scripts, especially if using the code editor, it is easy to accidentally save a script that will not compile. This would cause Main to stop running, and ScriptEditor would either show "Waiting for Main..." or could not submit updated scripts to the server.

Once this happens, the student would have to drop out into VSCode and fix the project file by hand, and restart the server to recover.

The Fix

This introduces a number of fixes that should keep Main running, and therefore ScriptEditor running, and allow ScriptEditor to submit script changes to fix any issues.

We also now try to catch and display errors so that they are not invisibly being dumped to the dev console.

Specific fixes:

• Blueprints are no longer compiled immediately upon project loading to prevent Main from crashing on a bad script
• Compile blueprints during Main Initialize but catch and display errors
• Catch compile errors when student submits a new script via ScriptEditor
• Catch project load errors if there is an error in the gemprj JSON file
• Default script now uses only INIT and UPDATE only
• addFeature, addProp now write to correct bundle
• Features are now reset between blueprint updates. Should fix undefined agent errors when switching features in and out.
• dbgOut messages can now be input via the GUI Wizard
• charControllers now work again. Change in blueprint compilation broke other derived state updates needed by charController.
• Deselecting script page line now also deselects slot editor
• Restore slot editor is now selected after adding a line -- state changes caused async issues
• Script names are now properly updated when editing the # BLUEPRINT pragma. Prevents ScriptEditor from switching to bp selection view.
• Touches type parameter (e.g. b2b) can now be entered using the GUI

To Test

1. Add a line prop xxx to a script and submit it. You should see an error on Main, but both Main and ScriptEditor remain running.
2. Reload Main. In spite of the script error, Main should load. But you should see "?" in place of any sprites. There is no other obvious error, but we'll work on that next.
3. Change prop xxx to prop and submit it. This should no longer trigger an error and the sprites should load.
4. Open the gemprj file you're playing with and mess up the JSON (e.g. remove a bracket).
5. Reload Main. You should get an alert that says something like "The project "starter_034" could not be loaded! Are you sure it exists?", followed by a second error.
6. Fix the JSON and reload Main. It should load again.

[benloh]

added 1 commit

• 28e2950f - next-gui-safeload: Need to symbolize blueprints on load.

Compare with previous version

[benloh]

added 8 commits

• e3bdae7d - 1 commit from branch dev-next-gui
• 0b61819b - dev-next-safeload-keyword-updates: changes from dev-next-gui-errhelp for keywords:
• f9568c30 - next-gui-safeload-keyword-updates: required scripts are now just INIT and UPDATE
• 802e6767 - next-gui-safeload-keyword-updates: squelch unnecessary error messages
• ae7f19af - Merge branch 'dev-bl/next-gui-safeload' into dev-bl/next-gui-safeload-keyword-updates
• 6f2b582f - safeload-keyword-updates: fix statement underflow in Transpiler.EnforceBlueprintPragmas()
• 658ead7e - next-gui-safeload: Add ResetAllFeatures to clear agent feature subscriptions...
• abd3b156 - next-gui-safeload: Reset ALL features

Compare with previous version

[benloh]

added 1 commit

• 9280041a - next-gui-safeload: Always recompile ALL blueprints whenever a blueprint is...

Compare with previous version

[benloh]

added 2 commits

• 780e88f4 - next-gui-misc: Add dbgOut message wiz UI
• 767a1aee - Merge branch 'dev-bl/next-gui-misc' into dev-bl/next-gui-safeload

Compare with previous version

[benloh]

added 1 commit

• 34dd47d0 - next-gui-safeload: Add 'reset' to IFeature

Compare with previous version

[benloh]

added 1 commit

• 89e48675 - next-gui-safeload: Update derived states after blueprint compile. Fixes charController load.

Compare with previous version

[benloh]

added 1 commit

• 7182938d - next-gui-safeload: Deselect slot editor when deselecting line.

Compare with previous version

[benloh]

added 1 commit

• d367f303 - next-gui-safeload: Remove debug logs

Compare with previous version

[benloh]

added 3 commits

• bfcef06c - next-gui-safeload: When ScriptEditor updates blueprints, just symbolize them,...
• 7c7ab63e - next-gui-safeload: Restore slot selection after adding a line.
• 5cc34e0b - next-gui-safeload: Catch phantom InstnaceRequestEdit on undefined agent. Displays warning.

Compare with previous version

[benloh]

added 1 commit

• 0a7c0d4d - next-gui-safeload: Update ScriptEditor bpName when '# BLUEPRINT' pragma is...

Compare with previous version

[benloh]

added 2 commits

• 178ab05b - next-gui-safeload-sri: identifier support so feat-touches monitor b2b works
• f40e8e48 - Merge next-gui-safeload-sri into dev-bl/next-gui-safeload

Compare with previous version

[benloh]

mentioned in commit 5acb47428bf373d7436c517509cbba3ed7f80717