Ignore popup/index.ts instead of erroring

[aleclarson]

In #276, an error message was added for when both popup/index.html and popup/index.ts exist. As I mentioned in this comment, it would be better to simply ignore the popup/index.ts file, especially since the docs suggest that a popup must be a .html file.

[aklinker1]

For reference, here is where we apply the glob patterns to detect entrypoint files:

wxt/packages/wxt/src/core/utils/building/find-entrypoints.ts

Lines 49 to 51 in ef24358

	const relativePaths = await glob(Object.keys(PATH_GLOB_TO_TYPE_MAP), {
	cwd: wxt.config.entrypointsDir,
	});

There is a test that already exists to cover this. It just needs updated slightly: remove the popup.html input and don't expect any validation errors.

wxt/packages/wxt/src/core/utils/building/__tests__/find-entrypoints.test.ts

Lines 671 to 694 in ef24358

	it('should not allow multiple entrypoints with the same name', async () => {
	globMock.mockResolvedValueOnce([
	'options/index.html',
	'options/index.jsx',
	'popup.html',
	'popup/index.html',
	'popup/index.ts',
	'ui.html',
	]);
	await expect(() => findEntrypoints()).rejects.toThrowError(
	[
	'Multiple entrypoints with the same name detected, only one entrypoint for each name is allowed.',
	'',
	'- options',
	` - ${unnormalizePath('src/entrypoints/options/index.html')}`,
	` - ${unnormalizePath('src/entrypoints/options/index.jsx')}`,
	'- popup',
	` - ${unnormalizePath('src/entrypoints/popup.html')}`,
	` - ${unnormalizePath('src/entrypoints/popup/index.html')}`,
	` - ${unnormalizePath('src/entrypoints/popup/index.ts')}`,
	].join('\n'),
	);
	});

Additionally, we should add another test case, basically a copy of the original, that expects an error for popup.html, popup/index.html, and popup.content.ts files - there can still only be one entrypoint named "popup".

This issue just ignores */index.[jt]sx? when */index.html already exists.