Fix structure identifiers to use namespace:identifier format

[Copilot]

Structure identifiers were generated as slash-separated paths (e.g. "puff/coin1") instead of the Minecraft Bedrock namespace:identifier format (e.g. puff:coin1). This caused correct usage to error and completions to suggest invalid identifiers.

ID generation (process.ts)

The first folder segment after structures/ is now the namespace; the remainder is the identifier. Files directly in structures/ fall back to the mystructure namespace.

File path	Old ID	New ID
structures/house.mcstructure	house	mystructure:house
structures/puff/coin1.mcstructure	"puff/coin1"	puff:coin1
structures/stuff/towers/diamond.mcstructure	"stuff/towers/diamond"	stuff:towers/diamond

Diagnosis (diagnose.ts)

Removed the slash↔colon translation logic that was converting namespace:identifier input into the wrong lookup key, and the spurious "needs quotes" error for slash-containing IDs. Lookup is now a direct match against the stored namespace:identifier.

[tryashtar]

@DaanV2 This change seems to have broken quoted structure identifiers. Due to MCPE-99757, structure identifiers with slashes must be quoted. Previously the language server checked for this specifically, and accepted quoted identifiers. Now it rejects them, and autocomplete suggests unquoted identifiers.

[DaanV2]

@tryashtar Is there any documentation on this? I hate this back and forwards that keeps happening 😅

[tryashtar]

The only thing I'm aware of is this:

https://github.com/Mojang/bedrock-samples/blob/main/metadata/command_modules/mojang-commands.json

The parsers for each argument are listed. /function's is called PATHCOMMAND and /structure's is called ID. That still doesn't tell you exactly how they're implemented (how they handle quoting), but that does reduce the in-game testing work that's required to merely testing every argument type, since the command tree and types are available.

[DaanV2]

@tryashtar I have an update here:

#456