Various Windows fixes

build/import-utils.js

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-const path = require('path');
+const path = require('path').posix; // Windows fix: .posix makes paths use '/' instead of '\', and solves issues linked to that.
 
 const alreadyResolvedMatch = /^(\.{0,2}\/|[a-z]\w*\:)/;  // matches start of './' or 'https:' etc
 

build/modern-vfs-middleware.js

@@ -16,7 +16,7 @@
 
 const modernLoader = require('./modern-loader.js');
 const mimeTypes = require('mime-types');
-const path = require('path');
+const path = require('path').posix; // Windows fix: .posix makes paths use '/' instead of '\', and solves issues linked to that.
 const fs = require('fs').promises;
 
 

santa-vfs.js

@@ -15,7 +15,7 @@
  */
 
 const fsp = require('./build/fsp.js');
-const path = require('path');
+const path = require('path').posix; // Windows fix: .posix makes paths use '/' instead of '\', and solves issues linked to that.
 const compileStyles = require('./build/compile-santa-sass.js');
 const compileScene = require('./build/compile-scene.js');
 const JSON5 = require('json5');
@@ -174,7 +174,7 @@ module.exports = (staticScope, options) => {
 
   // TODO(samthor): Closure doesn't have to be tied to scenes. But, it's mostly for historic code,
   // so maybe it's not worth making it generic.
-  const closureSceneMatch = /^static\/scenes\/(\w+)\/:closure(|-\w+)\.js$/;
+  const closureSceneMatch = /(static\/scenes\/(\w+)\/:closure(|-\w+)\.js)/; // Old regex didn't detect closure paths correctly
   const closurePlugin = {
     match(id) {
       const rooted = path.relative(__dirname, id);
@@ -185,9 +185,9 @@ module.exports = (staticScope, options) => {
     },
     async load(id) {
       const m = closureSceneMatch.exec(id);
-      const sceneName = m[1];
+      const sceneName = m[2]; // Was m[1]
 
-      const flags = m[2];
+      const flags = m[3]; // Was m[2]
       if (flags) {
         // nb. This previously allowed e.g., ':closure-typeSafe.js'.
         throw new Error(`unsupported Closure flags: ${flags}`);

serve.js

@@ -45,15 +45,15 @@ const yargs = require('yargs')
       default: false,
       describe: 'Serve static on network address'
     })
-    .option('prefix', {
+    /*.option('prefix', { // Prefix removed to solve the issue with js files going outside of the prefix and not being able to get the files
       type: 'string',
       default: 'st',
       describe: 'Static prefix',
       coerce(v) {
         return v.replace(/[^a-z0-9]/g, '') || 'st';  // ensure prefix is basic ascii only
       },
       requiresArg: true,
-    })
+    })*/
     .option('lang', {
       type: 'string',
       default: 'en',
@@ -92,7 +92,7 @@ log(chalk.red(messages('santatracker')), `[${yargs.lang}]`);
 // nb. matches config in release.js
 const baseurl = `http://127.0.0.1:${yargs.port + 80}/`;
 const config = {
-  staticScope: `${baseurl}${yargs.prefix}/`,
+  staticScope: `${baseurl}`, // Prefix removed to solve the issue with js files going outside of the prefix and not being able to get the files
   version: `dev-${(new Date).toISOString().replace(/[^\d]/g, '')}`,
   baseurl,
 };
@@ -110,7 +110,7 @@ async function serve() {
     serveLink: true,
   });
   const staticServer = polka();
-  staticServer.use(yargs.prefix, vfsMiddleware(vfs, 'static'), staticHost);
+  staticServer.use('/', vfsMiddleware(vfs, 'static'), staticHost); // Prefix removed to solve the issue with js files going outside of the prefix and not being able to get the files
 
   await listen(staticServer, yargs.port + 80, yargs.all);
   log('Static', chalk.green(config.staticScope), yargs.all ? chalk.red('(on all interfaces)') : '');