bug: 'Could not resolve "./headers"'

[FlippingBinary]

Bug

This bug can be reproduced by attempting to build the new SK app in ./demo. It was introduced by 5a6a64f because it adds a new headers module without making adapt(...) copy it to the build directory.

Proposed solution

Copy all the files in ./files to the build directory:

builder.copy(files, tmp, {
	replace: {
		SERVER: `${relativePath}/index.js`,
		MANIFEST: './manifest.js',
		DEBUG: debug.toString()
	}
});

It would replace these two code blocks:

svelte-adapter-azure-swa/index.js

Lines 63 to 69 in 24365cb

	builder.copy(join(files, 'entry.js'), entry, {
	replace: {
	SERVER: `${relativePath}/index.js`,
	MANIFEST: './manifest.js',
	DEBUG: debug.toString()
	}
	});

svelte-adapter-azure-swa/index.js

Line 78 in 24365cb

builder.copy(join(files, 'api'), apiDir);

Benefits

The provided constants SERVER, MANIFEST, and DEBUG could be used in more than just entry.js.

Downside

This would take up more space in the build directory for files that aren't needed there.

[FlippingBinary]

As mentioned here, the api directory does still need to be copied separately.

svelte-adapter-azure-swa/index.js

Line 78 in 24365cb

builder.copy(join(files, 'api'), apiDir);

[geoffrich]

Merging #86 should prevent a bug like this from getting merged in the future, since we run the demo build as part of PR verification.