Warn when there's a typo in proprietary files

[theetrain]

Describe the problem

This is a follow up to #10528

Sometimes, users will encounter unexplainable behaviours (or lack thereof) when seemingly adding the correct files, but those files were misspelled.

Common mistakes include:

• hook.server.js, hook.server.ts

Describe the proposed solution

During dev mode or project build, log a warning if any of the above misspellings are found in the codebase.

Alternatives considered

No response

Importance

nice to have

Additional Information

Some users reported spending a day figuring out their files were misspelled. #10558 helped a lot with +-prefixed files, and I think adding typo detection for hooks.server.ts would be another big win.

[Conduitry]

I'm not sure how to handle this generally. There might be something possible in a major breaking version if we start erroring on unknown files outside of, say, lib and routes. That might be too strict, though.

[sacrosanctic]

Why isn't hooks prefixes with +, that would make it easier to check.

Can hooks be part of the template in npx sv create?

[stephane-vanraes]

I think this could be offloaded to the VSCode Extension instead

const SvelteFileNameChecker = vscode.languages.createDiagnosticCollection('svelte file names');

function checkFilename(
  file: vscode.Uri,
  diagnostic: vscode.DiagnosticCollection
) {

  // Somebody smarter than me can probably make some checks ?
  diagnostic.set(file, [
     new vscode.Diagnostic(
       new vscode.Range(0, 0, 0, 0),
       'File has wrong name, did you mean xxx',
       vscode.DiagnosticSeverity.Warning
    ),
  ]);
}

vscode.workspace.onDidCreateFiles((ev) => ev.files.forEach((file) => checkFilename(file, SvelteFileNameChecker )));
vscode.workspace.onDidRenameFiles((ev) => {
  ev.files.forEach((file) => {
    filenameChecker.delete(file.oldUri);
    checkFilename(file.newUri, SvelteFileNameChecker );
  });
});

context.subscriptions.push(SvelteFileNameChecker);

[theetrain]

Since not everyone uses VSCode, I'd still advocate for this to be part of the build system, svelte-check, or both; of course the IDE extension is good to have as well. I see this as a direct follow up to #10528.

[teemingc]

Should be easy enough to add one for mispelling hooks as "hook". There's existing code for the unnecessary "+" prefix already

• Server:

  kit/packages/kit/src/core/sync/write_server.js

  Line 96 in a4c64f9

  const typo = resolve_entry('src/+hooks.server');

• Client:

  kit/packages/kit/src/core/sync/write_client_manifest.js

  Line 126 in a4c64f9

  const typo = resolve_entry('src/+hooks.client');