Results 85 comments of SPGoding

Thank you so much for testing! Is this still an issue in 3.4.13?

Those files should already be parsed since `normalizeUri()` in `Project` maps the cache URIs to the logical URIs. One case I can think of where this might not work is...

Oh we need to remove these checks in `Project` or change them to check for all supported URI schemas: https://github.com/SpyglassMC/Spyglass/blob/main/packages%2Fcore%2Fsrc%2Fservice%2FProject.ts#L869 Sorry I forgot I only changed them on a feature...

There is a [`DidChangeWatchedFiles` notification](https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#workspace_didChangeWatchedFiles) in LSP so I believe editor support is not as big of a problem (depending on how many editors implement that notification). I think I...

This is no longer an issue in v4.

![image](https://github.com/SpyglassMC/Spyglass/assets/15277496/3148bfb0-a6c3-4208-9e0f-fbb22caa6f9e) I capped.

I'm leaning towards closing this as Won't Fix, since there is no reason to associate JSON files in a data pack to `jsonc`, as Minecraft won't be able to load...

A string has to be quoted, and an identifier cannot have `:` or `/` in it, so i don't see any potential conflicts this can introduce. Or did i miss...

It's still relevant but not a requirement for v4. I'll rewrite the issue body to make it clearer what this issue is about.

Actually upon second thoughts this just make the resource location parser more complicated for a tiny benefit of being able to write resource locations like `minecraft:foo/bar` without the namespace part....