Skip to content

Commit

Permalink
- Check if manifest matches semver
Browse files Browse the repository at this point in the history
  • Loading branch information
Xterionix committed Feb 10, 2025
1 parent e5cbf4f commit c4a157a
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion src/Lib/Diagnostics/Minecraft/Manifest.ts
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,18 @@ export function minecraft_manifest_required_module(
return false;
}

export function minecraft_manifest_version(version: number[], diagnoser: DiagnosticsBuilder, path: string): void {
export function minecraft_manifest_version(version: number[] | string, diagnoser: DiagnosticsBuilder, path: string): void {

if (typeof version == 'string') {
if (!(/^(0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*)(?:-((?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*)(?:\.(?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*))*))?(?:\+([0-9a-zA-Z-]+(?:\.[0-9a-zA-Z-]+)*))?$/).test(version)) diagnoser.add(
path,
"Version string needs to match semver",
DiagnosticSeverity.error,
"minecraft.manifest.version.invalid"
);
return;
}

if (version.length != 3) {
diagnoser.add(
path,
Expand Down

0 comments on commit c4a157a

Please sign in to comment.