@spyglassmc/core 0.4.0 → 0.4.1

package/lib/common/Dev.js

@@ -26,7 +26,8 @@ export const Dev = Object.freeze({
             switch (typeof current) {
                 case 'bigint': {
                     const bits = Math.ceil(Math.log2(Number(current)));
-                    ans += (2 + Math.ceil(bits / (ByteToBits * PointerSize))) * PointerSize; // https://stackoverflow.com/a/54298760
+                    ans +=
+                        (2 + Math.ceil(bits / (ByteToBits * PointerSize))) * PointerSize; // https://stackoverflow.com/a/54298760
                     break;
                 }
                 case 'boolean':
@@ -75,7 +76,9 @@ export const Dev = Object.freeze({
             catch (ignored) {
                 // Most likely "Maximum callstack size exceeded".
                 // Fall back to a shallow string representation.
-                return `{ ${Object.entries(value).map(([k, v]) => `'${k}': '${String(v)}'`).join(', ')} }`;
+                return `{ ${Object.entries(value)
+                    .map(([k, v]) => `'${k}': '${String(v)}'`)
+                    .join(', ')} }`;
             }
         }
         else if (typeof value === 'symbol') {

package/lib/common/Operations.js

@@ -15,8 +15,12 @@ export class Operations {
     }
     set(obj, key, value, receiver = obj) {
         const oldValue = Reflect.get(obj, key, receiver);
-        const op = () => { Reflect.set(obj, key, value, receiver); };
-        const undoOp = () => { Reflect.set(obj, key, oldValue, receiver); };
+        const op = () => {
+            Reflect.set(obj, key, value, receiver);
+        };
+        const undoOp = () => {
+            Reflect.set(obj, key, oldValue, receiver);
+        };
         this.addRedoOp(op);
         this.addUndoOp(undoOp);
         op();
@@ -27,7 +31,7 @@ export class Operations {
         }
     }
     redo() {
-        this.redoOps.forEach(op => op());
+        this.redoOps.forEach((op) => op());
     }
 }
 //# sourceMappingURL=Operations.js.map

package/lib/common/ReadonlyProxy.js

@@ -9,7 +9,9 @@ class ReadonlyProxyHandler {
     get(target, p, receiver) {
         const value = Reflect.get(target, p, receiver);
         if (p !== 'prototype' && isObject(value)) {
-            return emplaceMap(this.map, p, { insert: () => ReadonlyProxy.create(value) });
+            return emplaceMap(this.map, p, {
+                insert: () => ReadonlyProxy.create(value),
+            });
         }
         return value;
     }

package/lib/common/StateProxy.js

@@ -43,20 +43,31 @@ class StateProxyHandler {
     }
     get(target, p, receiver) {
         switch (p) {
-            case BranchOff: return () => this.#branchOff(target);
-            case Is: return true;
-            case Origin: return target;
-            case Redo: return () => this.rootOps.redo();
-            case Undo: return () => this.rootOps.undo();
+            case BranchOff:
+                return () => this.#branchOff(target);
+            case Is:
+                return true;
+            case Origin:
+                return target;
+            case Redo:
+                return () => this.rootOps.redo();
+            case Undo:
+                return () => this.rootOps.undo();
         }
         const value = Reflect.get(target, p, receiver);
         if (p !== 'prototype' && isObject(value)) {
-            return emplaceMap(this.map, p, { insert: () => _createStateProxy(value, this.rootOps) });
+            return emplaceMap(this.map, p, {
+                insert: () => _createStateProxy(value, this.rootOps),
+            });
         }
         return value;
     }
     set(target, p, value, receiver) {
-        if (p === BranchOff || p === Is || p === Origin || p === Redo || p === Undo) {
+        if (p === BranchOff ||
+            p === Is ||
+            p === Origin ||
+            p === Redo ||
+            p === Undo) {
             throw new TypeError(`Cannot set ${String(p)}`);
         }
         this.rootOps.set(target, p, StateProxy.dereference(value), receiver);

package/lib/common/externals/BrowserExternals.js

@@ -1,7 +1,6 @@
-import { decode as arrayBufferFromBase64, encode as arrayBufferToBase64 } from 'base64-arraybuffer';
+import { decode as arrayBufferFromBase64, encode as arrayBufferToBase64, } from 'base64-arraybuffer';
 import pako from 'pako';
 import { fileUtil } from '../../service/fileUtil.js';
-import { Uri } from '../index.js';
 class BrowserEventEmitter {
     #listeners = new Map();
     emit(eventName, ...args) {
@@ -70,8 +69,7 @@ class BrowserFsWatcher {
         }
         return this;
     }
-    async close() {
-    }
+    async close() { }
 }
 class BrowserFileSystem {
     static LocalStorageKey = 'spyglassmc-browser-fs';
@@ -174,11 +172,6 @@ export const BrowserExternals = {
         EventEmitter: BrowserEventEmitter,
     },
     fs: new BrowserFileSystem(),
-    uri: {
-        normalize(uri) {
-            return new Uri(uri).toString();
-        },
-    },
 };
 function uint8ArrayToHex(array) {
     let ans = '';

package/lib/common/externals/NodeJsExternals.js

@@ -26,7 +26,7 @@ class NodeJsExternalDownloader {
                     reject(new Error(`Status code ${res.statusCode}: ${res.statusMessage}`));
                 }
                 else {
-                    resolve(promisifyAsyncIterable(res, chunks => Buffer.concat(chunks)));
+                    resolve(promisifyAsyncIterable(res, (chunks) => Buffer.concat(chunks)));
                 }
             });
         });
@@ -71,7 +71,7 @@ export const NodeJsExternals = {
             return (await globby(toPath(location) + '**/*', { absolute: true, dot: true })).map(uriFromPath);
         },
         async mkdir(location, options) {
-            return void await fsp.mkdir(toFsPathLike(location), options);
+            return void (await fsp.mkdir(toFsPathLike(location), options));
         },
         readFile(location) {
             return fsp.readFile(toFsPathLike(location));
@@ -90,7 +90,7 @@ export const NodeJsExternals = {
                     command = 'xdg-open';
                     break;
             }
-            return void await execFile(command, [toPath(location)]);
+            return void (await execFile(command, [toPath(location)]));
         },
         stat(location) {
             return fsp.stat(toFsPathLike(location));
@@ -105,16 +105,6 @@ export const NodeJsExternals = {
             return fsp.writeFile(toFsPathLike(location), data, options);
         },
     },
-    uri: {
-        normalize(uri) {
-            if (uri.startsWith('file:')) {
-                return url.pathToFileURL(url.fileURLToPath(uri)).toString();
-            }
-            else {
-                return new Uri(uri).toString();
-            }
-        },
-    },
 };
 Object.freeze(NodeJsExternals);
 /**
@@ -140,10 +130,10 @@ class ChokidarWatcherWrapper extends EventEmitter {
         super();
         this.#watcher = watcher
             .on('ready', () => this.emit('ready'))
-            .on('add', path => this.emit('add', uriFromPath(path)))
-            .on('change', path => this.emit('change', uriFromPath(path)))
-            .on('unlink', path => this.emit('unlink', uriFromPath(path)))
-            .on('error', e => this.emit('error', e));
+            .on('add', (path) => this.emit('add', uriFromPath(path)))
+            .on('change', (path) => this.emit('change', uriFromPath(path)))
+            .on('unlink', (path) => this.emit('unlink', uriFromPath(path)))
+            .on('error', (e) => this.emit('error', e));
     }
     close() {
         return this.#watcher.close();

package/lib/common/externals/index.d.ts

@@ -23,9 +23,6 @@ export interface Externals {
         EventEmitter: new () => ExternalEventEmitter;
     };
     fs: ExternalFileSystem;
-    uri: {
-        normalize: (uri: string) => string;
-    };
 }
 export interface DecompressedFile {
     data: Uint8Array;

package/lib/common/util.d.ts

@@ -114,6 +114,7 @@ export declare function emplaceMap<K, V>(map: Map<K, V>, key: K, handler: {
  * @returns If `val` is an non-null object or a callable object (i.e. function).
  */
 export declare function isObject(val: unknown): val is object;
+export declare function normalizeUri(uri: string): string;
 /**
  * @example
  * ```ts

package/lib/common/util.js

@@ -9,7 +9,7 @@ export const Uri = URL;
  * This is a decorator for async methods. Decorated methods will return the same `Promise` for
  * the same key, provided that the previously returned `Promise` is still pending.
  */
-export function SingletonPromise(getKey = args => args[0]) {
+export function SingletonPromise(getKey = (args) => args[0]) {
     return (_target, _key, descripter) => {
         const promises = new Map();
         const decoratedMethod = descripter.value;
@@ -19,8 +19,7 @@ export function SingletonPromise(getKey = args => args[0]) {
             if (promises.has(key)) {
                 return promises.get(key);
             }
-            const ans = decoratedMethod.apply(this, args)
-                .then(ans => (promises.delete(key), ans), e => (promises.delete(key), Promise.reject(e)));
+            const ans = decoratedMethod.apply(this, args).then((ans) => (promises.delete(key), ans), (e) => (promises.delete(key), Promise.reject(e)));
             promises.set(key, ans);
             return ans;
         };
@@ -35,7 +34,7 @@ export const Singleton = (_target, _key, descripter) => {
     const decoratedMethod = descripter.value;
     // The `function` syntax is used to preserve `this` context from the decorated method.
     descripter.value = function (...args) {
-        return value ??= decoratedMethod.apply(this, args);
+        return (value ??= decoratedMethod.apply(this, args));
     };
     return descripter;
 };
@@ -46,7 +45,7 @@ export const Singleton = (_target, _key, descripter) => {
  *
  * Decorated methods will be scheduled to run after `ms` milliseconds. The timer will reset when the method is called again.
  */
-export function Delay(ms, getKey = args => args[0]) {
+export function Delay(ms, getKey = (args) => args[0]) {
     return (_target, _key, descripter) => {
         const timeouts = new Map();
         const decoratedMethod = descripter.value;
@@ -109,9 +108,7 @@ export function bufferToString(buffer) {
 export var Arrayable;
 (function (Arrayable) {
     function is(value, isT) {
-        return Array.isArray(value)
-            ? value.every(e => isT(e))
-            : isT(value);
+        return Array.isArray(value) ? value.every((e) => isT(e)) : isT(value);
     }
     Arrayable.is = is;
     function toArray(value) {
@@ -178,7 +175,9 @@ export var Lazy;
     }
     Lazy.isUnresolved = isUnresolved;
     function resolve(lazy) {
-        return isUnresolved(lazy) ? lazy.value = lazy.getter() : lazy;
+        return isUnresolved(lazy)
+            ? (lazy.value = lazy.getter())
+            : lazy;
     }
     Lazy.resolve = resolve;
 })(Lazy || (Lazy = {}));
@@ -195,7 +194,7 @@ export function getStates(category, ids, ctx) {
             .query(ctx.doc, category, id)
             .forEachMember((state, stateQuery) => {
             const values = Object.keys(stateQuery.visibleMembers);
-            const set = ans[state] ??= new Set();
+            const set = (ans[state] ??= new Set());
             const defaultValue = stateQuery.symbol?.relations?.default;
             if (defaultValue) {
                 set.add(defaultValue.path[defaultValue.path.length - 1]);
@@ -240,4 +239,9 @@ export function emplaceMap(map, key, handler) {
 export function isObject(val) {
     return typeof val === 'function' || (!!val && typeof val === 'object');
 }
+export function normalizeUri(uri) {
+    const obj = new Uri(uri);
+    obj.pathname = obj.pathname.replace(/%3A/gi, ':');
+    return obj.toString();
+}
 //# sourceMappingURL=util.js.map

package/lib/node/AstNode.js

@@ -4,8 +4,10 @@ export var AstNode;
 (function (AstNode) {
     /* istanbul ignore next */
     function is(obj) {
-        return !!obj && typeof obj === 'object' && typeof obj.type === 'string' &&
-            Range.is(obj.range);
+        return (!!obj &&
+            typeof obj === 'object' &&
+            typeof obj.type === 'string' &&
+            Range.is(obj.range));
     }
     AstNode.is = is;
     function setParents(node) {
@@ -61,7 +63,7 @@ export var AstNode;
         return node.children?.[findLastChildIndex(node, needle, endInclusive)];
     }
     AstNode.findLastChild = findLastChild;
-    function findDeepestChild({ node, needle, endInclusive = false, predicate = () => true }) {
+    function findDeepestChild({ node, needle, endInclusive = false, predicate = () => true, }) {
         let last;
         let head = Range.contains(node, needle, endInclusive) ? node : undefined;
         while (head && predicate(head)) {
@@ -71,7 +73,7 @@ export var AstNode;
         return last;
     }
     AstNode.findDeepestChild = findDeepestChild;
-    function findShallowestChild({ node, needle, endInclusive = false, predicate = () => true }) {
+    function findShallowestChild({ node, needle, endInclusive = false, predicate = () => true, }) {
         let head = Range.contains(node, needle, endInclusive) ? node : undefined;
         while (head && !predicate(head)) {
             head = findChild(head, needle, endInclusive);

package/lib/node/FileNode.js

@@ -1,7 +1,12 @@
 export var FileNode;
 (function (FileNode) {
     function getErrors(node) {
-        return [...node.parserErrors, ...node.binderErrors ?? [], ...node.checkerErrors ?? [], ...node.linterErrors ?? []];
+        return [
+            ...node.parserErrors,
+            ...(node.binderErrors ?? []),
+            ...(node.checkerErrors ?? []),
+            ...(node.linterErrors ?? []),
+        ];
     }
     FileNode.getErrors = getErrors;
 })(FileNode || (FileNode = {}));

package/lib/node/ResourceLocationNode.d.ts

@@ -33,7 +33,7 @@ export interface ResourceLocationNode extends ResourceLocationBaseNode {
 export declare namespace ResourceLocationNode {
     function is(obj: AstNode | undefined): obj is ResourceLocationNode;
     function mock(range: RangeLike, options: ResourceLocationOptions): ResourceLocationNode;
-    function toString(node: DeepReadonly<ResourceLocationBaseNode>, type?: 'full'): FullResourceLocation;
-    function toString(node: DeepReadonly<ResourceLocationBaseNode>, type?: 'origin' | 'full' | 'short'): string;
+    function toString(node: DeepReadonly<ResourceLocationBaseNode>, type?: 'full', includesTagPrefix?: false): FullResourceLocation;
+    function toString(node: DeepReadonly<ResourceLocationBaseNode>, type?: 'origin' | 'full' | 'short', includesTagPrefix?: boolean): string;
 }
 //# sourceMappingURL=ResourceLocationNode.d.ts.map

package/lib/node/ResourceLocationNode.js

@@ -17,7 +17,7 @@ export var ResourceLocationNode;
     const DefaultNamespace = ResourceLocation.DefaultNamespace;
     /* istanbul ignore next */
     function is(obj) {
-        return obj?.type === 'resource_location';
+        return (obj?.type === 'resource_location');
     }
     ResourceLocationNode.is = is;
     function mock(range, options) {
@@ -28,7 +28,7 @@ export var ResourceLocationNode;
         };
     }
     ResourceLocationNode.mock = mock;
-    function toString(node, type = 'origin') {
+    function toString(node, type = 'origin', includesTagPrefix = false) {
         const path = node.path ? node.path.join(PathSep) : '';
         let id;
         /*
@@ -43,7 +43,10 @@ export var ResourceLocationNode;
         switch (type) {
             case 'origin':
                 // Use `node.namespace !== undefined`, so that empty namespaces can be correctly restored to string.
-                id = node.namespace !== undefined ? `${node.namespace}${NamespacePathSep}${path}` : path;
+                id =
+                    node.namespace !== undefined
+                        ? `${node.namespace}${NamespacePathSep}${path}`
+                        : path;
                 break;
             case 'full':
                 // Use `node.namespace` before `||`, so that both undefined and empty value can result in the default namespace.
@@ -52,10 +55,13 @@ export var ResourceLocationNode;
                 break;
             case 'short':
                 // Use `node.namespace` before `&&` for the same reason stated above.
-                id = node.namespace && node.namespace !== DefaultNamespace ? `${node.namespace}${NamespacePathSep}${path}` : path;
+                id =
+                    node.namespace && node.namespace !== DefaultNamespace
+                        ? `${node.namespace}${NamespacePathSep}${path}`
+                        : path;
                 break;
         }
-        return node.isTag ? `${TagPrefix}${id}` : id;
+        return includesTagPrefix && node.isTag ? `${TagPrefix}${id}` : id;
     }
     ResourceLocationNode.toString = toString;
 })(ResourceLocationNode || (ResourceLocationNode = {}));

package/lib/parser/boolean.js

@@ -1,6 +1,6 @@
 import { literal } from './literal.js';
 import { map } from './util.js';
-export const boolean = map(literal('false', 'true'), res => ({
+export const boolean = map(literal('false', 'true'), (res) => ({
     type: 'boolean',
     range: res.range,
     value: res.value === '' ? undefined : res.value === 'true',

package/lib/parser/comment.d.ts

@@ -7,6 +7,6 @@ interface Options {
 /**
  * `Failure` when three isn't a comment.
  */
-export declare function comment({ singleLinePrefixes, includesEol }: Options): Parser<CommentNode>;
+export declare function comment({ singleLinePrefixes, includesEol, }: Options): Parser<CommentNode>;
 export {};
 //# sourceMappingURL=comment.d.ts.map

package/lib/parser/comment.js

@@ -3,7 +3,7 @@ import { Failure } from './Parser.js';
 /**
  * `Failure` when three isn't a comment.
  */
-export function comment({ singleLinePrefixes, includesEol }) {
+export function comment({ singleLinePrefixes, includesEol, }) {
     return (src, _ctx) => {
         const start = src.cursor;
         const ans = {

package/lib/parser/float.js

@@ -43,7 +43,8 @@ export function float(options) {
         else if (!options.pattern.test(raw)) {
             ctx.err.report(localize('parser.float.illegal', options.pattern), ans);
         }
-        else if ((options.min && ans.value < options.min) || (options.max && ans.value > options.max)) {
+        else if ((options.min && ans.value < options.min) ||
+            (options.max && ans.value > options.max)) {
             const onOutOfRange = options.onOutOfRange ?? fallbackOnOutOfRange;
             onOutOfRange(ans, src, ctx, options);
         }

package/lib/parser/integer.js

@@ -32,7 +32,8 @@ export function integer(options) {
         else if (!options.pattern.test(raw) || isOnlySign) {
             ctx.err.report(localize('parser.integer.illegal', options.pattern), ans);
         }
-        else if ((options.min !== undefined && ans.value < options.min) || (options.max !== undefined && ans.value > options.max)) {
+        else if ((options.min !== undefined && ans.value < options.min) ||
+            (options.max !== undefined && ans.value > options.max)) {
             const onOutOfRange = options.onOutOfRange ?? fallbackOnOutOfRange;
             onOutOfRange(ans, src, ctx, options);
         }

package/lib/parser/list.d.ts

@@ -8,5 +8,5 @@ export interface Options<V extends AstNode> {
     trailingSep: boolean;
     end: string;
 }
-export declare function list<V extends AstNode>({ start, value, sep, trailingSep, end }: Options<V>): InfallibleParser<ListNode<V>>;
+export declare function list<V extends AstNode>({ start, value, sep, trailingSep, end, }: Options<V>): InfallibleParser<ListNode<V>>;
 //# sourceMappingURL=list.d.ts.map

package/lib/parser/list.js

@@ -2,7 +2,7 @@ import { localeQuote, localize } from '@spyglassmc/locales';
 import { Range } from '../source/index.js';
 import { Failure } from './Parser.js';
 import { attempt } from './util.js';
-export function list({ start, value, sep, trailingSep, end }) {
+export function list({ start, value, sep, trailingSep, end, }) {
     return (src, ctx) => {
         const ans = {
             type: 'list',
@@ -34,14 +34,14 @@ export function list({ start, value, sep, trailingSep, end }) {
                 let sepRange = undefined;
                 src.skipWhitespace();
                 requiresValueSep = true;
-                if (hasValueSep = src.peek(sep.length) === sep) {
+                if ((hasValueSep = src.peek(sep.length) === sep)) {
                     sepRange = Range.create(src, () => src.skip(sep.length));
                 }
                 // Create item.
                 ans.children.push({
                     type: 'item',
                     range: Range.create(itemStart, src),
-                    ...valueNode ? { children: [valueNode] } : {},
+                    ...(valueNode ? { children: [valueNode] } : {}),
                     value: valueNode,
                     sep: sepRange,
                 });

package/lib/parser/long.js

@@ -36,7 +36,8 @@ export function long(options) {
         else if (!options.pattern.test(raw) || isOnlySign) {
             ctx.err.report(localize('parser.long.illegal', options.pattern), ans);
         }
-        else if ((options.min && ans.value < options.min) || (options.max && ans.value > options.max)) {
+        else if ((options.min && ans.value < options.min) ||
+            (options.max && ans.value > options.max)) {
             const onOutOfRange = options.onOutOfRange ?? fallbackOnOutOfRange;
             onOutOfRange(ans, src, ctx, options);
         }

package/lib/parser/record.d.ts

@@ -18,5 +18,5 @@ export interface Options<K extends AstNode, V extends AstNode> {
 /**
  * @returns A parser that parses something coming in a key-value pair form. e.g. SNBT objects, entity selector arguments.
  */
-export declare function record<K extends AstNode, V extends AstNode>({ start, pair, end }: Options<K, V>): InfallibleParser<RecordNode<K, V>>;
+export declare function record<K extends AstNode, V extends AstNode>({ start, pair, end, }: Options<K, V>): InfallibleParser<RecordNode<K, V>>;
 //# sourceMappingURL=record.d.ts.map

package/lib/parser/record.js

@@ -5,7 +5,7 @@ import { attempt } from './util.js';
 /**
  * @returns A parser that parses something coming in a key-value pair form. e.g. SNBT objects, entity selector arguments.
  */
-export function record({ start, pair, end }) {
+export function record({ start, pair, end, }) {
     return (src, ctx) => {
         const ans = {
             type: 'record',
@@ -26,8 +26,10 @@ export function record({ start, pair, end }) {
                 }
                 // Key.
                 const keyStart = src.cursor;
-                const { result: keyResult, updateSrcAndCtx: updateForKey, endCursor: keyEnd } = attempt(pair.key, src, ctx);
-                if (keyResult === Failure || (keyEnd - keyStart === 0 && ![pair.sep, pair.end, end, '\r', '\n', '\t', ' '].includes(src.peek()))) {
+                const { result: keyResult, updateSrcAndCtx: updateForKey, endCursor: keyEnd, } = attempt(pair.key, src, ctx);
+                if (keyResult === Failure ||
+                    (keyEnd - keyStart === 0 &&
+                        ![pair.sep, pair.end, end, '\r', '\n', '\t', ' '].includes(src.peek()))) {
                     ctx.err.report(localize('expected', localize('parser.record.key')), Range.create(src, () => src.skipUntilOrEnd(pair.sep, pair.end, end, '\r', '\n')));
                 }
                 else {
@@ -45,10 +47,14 @@ export function record({ start, pair, end }) {
                 }
                 // Value.
                 src.skipWhitespace();
-                const valueParser = typeof pair.value === 'function' ? pair.value : pair.value.get(ans, key);
+                const valueParser = typeof pair.value === 'function'
+                    ? pair.value
+                    : pair.value.get(ans, key);
                 const valueStart = src.cursor;
-                const { result: valueResult, updateSrcAndCtx: updateForValue, endCursor: valueEnd } = attempt(valueParser, src, ctx);
-                if (valueResult === Failure || (valueEnd - valueStart === 0 && ![pair.sep, pair.end, end, '\r', '\n', '\t', ' '].includes(src.peek()))) {
+                const { result: valueResult, updateSrcAndCtx: updateForValue, endCursor: valueEnd, } = attempt(valueParser, src, ctx);
+                if (valueResult === Failure ||
+                    (valueEnd - valueStart === 0 &&
+                        ![pair.sep, pair.end, end, '\r', '\n', '\t', ' '].includes(src.peek()))) {
                     ctx.err.report(localize('expected', localize('parser.record.value')), Range.create(src, () => src.skipUntilOrEnd(pair.sep, pair.end, end, '\r', '\n')));
                 }
                 else {
@@ -59,14 +65,18 @@ export function record({ start, pair, end }) {
                 let endCharRange = undefined;
                 src.skipWhitespace();
                 requiresPairEnd = true;
-                if (hasPairEnd = src.peek(pair.end.length) === pair.end) {
+                if ((hasPairEnd = src.peek(pair.end.length) === pair.end)) {
                     endCharRange = Range.create(src, () => src.skip(pair.end.length));
                 }
                 // Create pair.
                 ans.children.push({
                     type: 'pair',
                     range: Range.create(pairStart, src),
-                    ...(key || value) ? { children: [key, value].filter(v => !!v) } : {},
+                    ...(key || value
+                        ? {
+                            children: [key, value].filter((v) => !!v),
+                        }
+                        : {}),
                     key,
                     sep: sepCharRange,
                     value,

package/lib/parser/resourceLocation.js

@@ -1,11 +1,62 @@
 import { arrayToMessage, localize } from '@spyglassmc/locales';
 import { ResourceLocation } from '../common/index.js';
 import { Range } from '../source/index.js';
-const Terminators = new Set([' ', '\r', '\n', '=', ',', '"', "'", '{', '}', '[', ']', '(', ')', ';']);
+const Terminators = new Set([
+    ' ',
+    '\r',
+    '\n',
+    '=',
+    ',',
+    '"',
+    "'",
+    '{',
+    '}',
+    '[',
+    ']',
+    '(',
+    ')',
+    ';',
+]);
 const LegalCharacters = new Set([
-    'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm',
-    'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z',
-    '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '_', '-', '.',
+    'a',
+    'b',
+    'c',
+    'd',
+    'e',
+    'f',
+    'g',
+    'h',
+    'i',
+    'j',
+    'k',
+    'l',
+    'm',
+    'n',
+    'o',
+    'p',
+    'q',
+    'r',
+    's',
+    't',
+    'u',
+    'v',
+    'w',
+    'x',
+    'y',
+    'z',
+    '0',
+    '1',
+    '2',
+    '3',
+    '4',
+    '5',
+    '6',
+    '7',
+    '8',
+    '9',
+    '_',
+    '-',
+    '.',
 ]);
 export function resourceLocation(options) {
     return (src, ctx) => {
@@ -35,10 +86,12 @@ export function resourceLocation(options) {
             ans.path = rawPath.split(ResourceLocation.PathSep);
             // Check characters.
             /* istanbul ignore next */
-            const illegalChars = [...new Set([
-                    ...[...ans.namespace ?? []].filter(c => !LegalCharacters.has(c)),
-                    ...[...rawPath].filter(c => c !== '/' && !LegalCharacters.has(c)),
-                ])];
+            const illegalChars = [
+                ...new Set([
+                    ...[...(ans.namespace ?? [])].filter((c) => !LegalCharacters.has(c)),
+                    ...[...rawPath].filter((c) => c !== '/' && !LegalCharacters.has(c)),
+                ]),
+            ];
             if (illegalChars.length) {
                 ctx.err.report(localize('parser.resource-location.illegal', arrayToMessage(illegalChars, true, 'and')), ans);
             }