unpkg CDN as a fallback? about servor

[osdevisnot]

Awesome, I'll try to get to this soon. We will need to explore if this can bring any unforeseen hurdles if we make it a default. Alternative would be to have a flag to disable this behavior.

[lukejacksonn]

I think this is a really interesting idea and would like to see it evolve. Currently I use servor with es module only projects (without local dependencies) and currently manage by being explicit about my import urls so doing https://unpkg.com/es-react for example.

But it would be nice not to have to write https://unpkg.com everywhere. You might be interested in a few other threads that I have been having with developers in other bundler camps (pikapkg and webpack) over in lukejacksonn/perflink#15

[osdevisnot]

Digging deeper on this, it seems we have 2 alternative approaches we can go with.

First approach would be to use @babel/core with babel-plugin-bare-import-rewrite and follow what express-transform-bare-module-specifiers does. Something like this:

				const result = babel.transformSync(body, {
					filename: path.resolve(rootDir, request.path.replace(/^\//, '')),
					plugins: [
						['bare-import-rewrite', {
							modulesDir: modulesUrl,
							rootBaseDir: rootDir
						}]
					],
					sourceMaps: 'inline'
				});

Second approach would be to use 😱😱 REGEX 😱😱 to replace the file content before sending a response. Something like this:

https://regex101.com/r/6mZUJu/1

Babel approach would add a dependency to servor, which kind of defeats the no-dependency constraints. However REGEX can be hard to read and add maintenance overhead.

[lukejacksonn]

The regex route sounds much more adoptable at this point. I have written one like this before.. it wasn't too bad but I probably missed some edge cases.

[lukejacksonn]

Hey @osdevisnot if you figure this out then please open a PR and reference this issue. I'm very interested in what it might look like but am doing a tidy up here so am going to close for now!