Fix builder list (#52)

* Fix builder list

* fix types

* lint

* fix nbt.long type def

* fix types

* update README

Author: extremeheat

README.md

@@ -93,10 +93,10 @@ Provides a way to build complex nbt structures simply:
 const nbt = require('prismarine-nbt')
 writePlayerNbt({
   Air: nbt.short(300),
-  Armor: nbt.list(
-    nbt.comp({ Count: nbt.byte(0), Damage: nbt.short(0), Name: nbt.string("") }),
-    nbt.comp({ Count: nbt.byte(0), Damage: nbt.short(0), Name: nbt.string("") }),
-    nbt.comp({ Count: nbt.byte(0), Damage: nbt.short(0), Name: nbt.string("") }),
-  ),
+  Armor: nbt.list(nbt.comp([
+    { Count: nbt.byte(0), Damage: nbt.short(0), Name: nbt.string("") },
+    { Count: nbt.byte(0), Damage: nbt.short(0), Name: nbt.string("") },
+    { Count: nbt.byte(0), Damage: nbt.short(0), Name: nbt.string("") }
+  ])),
 })
 ```

compiler-tagname.js

@@ -1,7 +1,7 @@
 /* global ctx */
 function readPString (buffer, offset) {
   const { value, size } = ctx.shortString(buffer, offset)
-  for (var c of value) {
+  for (const c of value) {
     if (c === '\0') throw new Error('unexpected tag end')
   }
   return { value, size }

nbt.js

@@ -159,13 +159,13 @@ const builder = {
   short (value) { return { type: 'short', value } },
   byte (value) { return { type: 'byte', value } },
   string (value) { return { type: 'string', value } },
-  comp (value) { return { type: 'compound', value } },
+  comp (value, name = '') { return { type: 'compound', name, value } },
   int (value) { return { type: 'int', value } },
   double (value) { return { type: 'double', value } },
   long (value) { return { type: 'long', value } },
-  list (...value) {
-    const type = value[0]?.type ?? 'end'
-    return { type: 'list', value: { type, value } }
+  list (value) {
+    const type = value?.type ?? 'end'
+    return { type: 'list', value: { type, value: value?.value ?? [] } }
   }
 }
 

typings/index.d.ts

@@ -66,15 +66,15 @@ declare module 'prismarine-nbt'{
   /** @deprecated */
   export function parse(data: Buffer, callback: (err: Error | null, value: NBT) => any): void;
 
-  export function short<T extends number> (val: T): { type: 'short', value: T }
-  export function byte<T extends number> (val: T): { type: 'byte', value: T }
-  export function string<T extends string> (val: T): { type: 'string', value: T }
-  export function comp<T extends object> (val: T): { type: 'compound', value: T }
-  export function int<T extends number> (val: T): { type: 'int', value: T }
-  export function list<T extends string, K extends {type: T}>(...value: K[]): { type: 'list'; value: { type: T | 'end', value: K[] } };
-  export function double<T extends number> (value: T): { type: 'double', value: T}
+  export function short<T extends number | number[]> (val: T): { type: 'short', value: T }
+  export function byte<T extends number | number[]> (val: T): { type: 'byte', value: T }
+  export function string<T extends string | string[]> (val: T): { type: 'string', value: T }
+  export function comp<T extends object | object[]> (val: T, name?: string): { type: 'compound', name, value: T }
+  export function int<T extends number | number[]> (val: T): { type: 'int', value: T }
+  export function list<T extends string, K extends {type: T}>(value: K): { type: 'list'; value: { type: T | 'end', value: K } };
+  export function double<T extends number | number[]> (value: T): { type: 'double', value: T}
   /**
    * @param value Takes a BigInt or an array of two 32-bit integers
    */
-  export function long<T extends number | BigInt> (value: T[] | T): { type: 'long', value: T[] | T}
+  export function long<T extends number | number[] | number[number[]] | BigInt | BigInt[]> (value: T): { type: 'long', value: T}
 }