SpyglassMC / Spyglass

Development tools for vanilla Minecraft: Java Edition data pack developers.
https://spyglassmc.com
MIT License
322 stars 34 forks source link

Minecraft mod to integrate Spyglass features #957

Open SPGoding opened 2 years ago

SPGoding commented 2 years ago

We just need autocompletion and semantic coloring, how hard can it be?

Update: I was just informed of a major blocker of this issue: spyglass is already in Minecraft :stuck_out_tongue:

NeunEinser commented 2 years ago

We just need autocompletion and semantic coloring, how hard can it be?

Also registry entries from all loaded data packs. And you need to look for an nbtdoc folder next to the data folder.

MulverineX commented 2 years ago

Actually, by the time this is a thing this should IMO be built with Quilt; an equivalent project that is better practically (tooling, design, management) & ethically (fabric having frequent & repeated abuse of power from top management with no recourse).

osfanbuff63 commented 2 years ago

Actually, by the time this is a thing this should IMO be built with Quilt; an equivalent project that is better practically (tooling, design, management) & ethically (fabric having frequent & repeated abuse of power from top management with no recourse).

I've never heard of this mid loader, and so I suspect it doesn't have a very big player base. If it were possible, I'd go multi-platform, given I pretty much wouldn't use it unless it was Fabric, since I have a number of other mods.

misode commented 2 years ago

Given Quilt has only just moved to beta, I think it's safer to start with Fabric. Presumably switching to Quilt later will be easy.

MulverineX commented 2 years ago

If there is someone active & willing enough to develop this before the stable release of Quilt then yeah.

@osfanbuff63 Quilt when reaching full release will have full support for Fabric mods. The point of [eventually] porting the mod is to support & utilize the more healthy ecosystem. ie. You would be able to use all of your Fabric mods alongside before they are ported.

osfanbuff63 commented 2 years ago

@osfanbuff63 Quilt when reaching full release will have full support for Fabric mods. The point of [eventually] porting the mod is to support & utilize the more healthy ecosystem. ie. You would be able to use all of your Fabric mods alongside before they are ported.

Oh, that makes a lot more sense - thanks!

Gaming32 commented 2 years ago

What would be extremely useful for a mod is to be able to read modded commands and provide them to the language server for autocomplete and such.

SPGoding commented 2 years ago

What would be extremely useful for a mod is to be able to read modded commands and provide them to the language server for autocomplete and such.

Thanks for the suggestion!

Just to be clear this issue is about a mod that provides some of the IntelliSense features in Minecraft's command block UI and chat UI. If you were talking about supporting modded commands in text editors that should already be possible in the current v4 build.

If you're talking about the mod mentioned by this issue then that should also be possible as long as the mod generates the commands.json file in a way similar to the net/minecraft/data/info/CommandsReport#run method (yarn mappings) on the fly.

Gaming32 commented 2 years ago

Oh that's already supported? That's quite useful. How do I load my modded commands?

SPGoding commented 2 years ago

Oh that's already supported? That's quite useful. How do I load my modded commands?

Sorry I wasn't being clear. v4 is still under development and hasn't been released yet. The version available on VS Marketplace doesn't support modded commands.

Gaming32 commented 2 years ago

Oh no, I got that. I'm still curious how to do it in v4 though.