remove nbttablescriptobject and just use tables bro

Author: diamonddevv

autodocs/autodoc.lua

@@ -2,7 +2,7 @@
 
 --#region Information
 -- GENERATED AUTODOC
--- Generated: 2026-01-20T18:25:29.060359500
+-- Generated: 2026-01-21T10:19:56.649403100
 -- Luafy Version: 2.0.0
 -- Format: Lua LS library file
 --#endregion
@@ -102,11 +102,6 @@ function ScriptResult.await_error() end
 function ScriptResult.release() end
 
 
---- NBT Compound Tag
----@class NbtTable
-local NbtTable = {}
-
-
 --- An entity.
 ---@class Entity
 local Entity = {}
@@ -322,10 +317,10 @@ function luafy.provide_hello_world() end
 ---@return string
 function luafy.get_luaj_version() end
 
---- temp; convert table to nbt
----@param table table table
----@return NbtTable
-function luafy.nbt(table) end
+--- Dump a table to a string.
+---@param table table Table to dump.
+---@return string
+function luafy.dump(table) end
 
 math = {}
 

src/main/java/dev/diamond/luafy/lua/LuaTableBuilder.java

@@ -152,6 +152,7 @@ public static CompoundTag toNbtCompound(LuaTable table) {
     }
 
     private static Optional<Tag> toNbt(LuaValue value) {
+
         return switch (value.type()) {
             case LuaValue.TINT ->       Optional.of(IntTag.valueOf(value.toint()));
             case LuaValue.TNUMBER ->    Optional.of(FloatTag.valueOf(value.tofloat()));
@@ -160,7 +161,7 @@ private static Optional<Tag> toNbt(LuaValue value) {
             case LuaValue.TTABLE ->     {
                 LuaTable table = value.checktable();
 
-                if (Arrays.stream(table.keys()).allMatch(k -> k.type() == LuaValue.TINT)) {
+                if (Arrays.stream(table.keys()).allMatch(LuaValue::isint)) {
                     ListTag list = new ListTag();
                     for (LuaValue key : table.keys()) {
                         toNbt(table.get(key)).ifPresent(list::add);

src/main/java/dev/diamond/luafy/registry/ScriptObjects.java

@@ -16,7 +16,6 @@ public class ScriptObjects {
     public static Vec3dScriptObject VEC3D = new Vec3dScriptObject();
     public static ModScriptObject MOD = new ModScriptObject();
     public static ScriptResultScriptObject SCRIPT_RESULT = new ScriptResultScriptObject();
-    public static NbtTableScriptObject NBT_TABLE = new NbtTableScriptObject();
 
     // entities
     public static EntityScriptObject ENTITY = new EntityScriptObject();
@@ -32,7 +31,6 @@ public static void registerAll() {
         Registry.register(LuafyRegistries.SCRIPT_OBJECTS, Luafy.id("vec3d"), VEC3D);
         Registry.register(LuafyRegistries.SCRIPT_OBJECTS, Luafy.id("mod"), MOD);
         Registry.register(LuafyRegistries.SCRIPT_OBJECTS, Luafy.id("script_result"), SCRIPT_RESULT);
-        Registry.register(LuafyRegistries.SCRIPT_OBJECTS, Luafy.id("nbt_table"), NBT_TABLE);
 
         Registry.register(LuafyRegistries.SCRIPT_OBJECTS, Luafy.id("entity"), ENTITY);
         Registry.register(LuafyRegistries.SCRIPT_OBJECTS, Luafy.id("living_entity"), LIVING_ENTITY);

src/main/java/dev/diamond/luafy/script/api/LuafyApi.java

@@ -13,6 +13,8 @@
 import org.luaj.vm2.LuaTable;
 import org.luaj.vm2.LuaValue;
 
+import java.util.function.Function;
+
 public class LuafyApi extends AbstractScriptApi {
     public LuafyApi(LuaScript script) {
         super("luafy", script);
@@ -47,14 +49,35 @@ public void addFunctions(ScriptApiBuilder apiBuilder) {
                 return LuaString.valueOf(Luafy.LUAJ_VER);
             }, "Returns the version of LuaJ used by the mod.", args -> {}, Argtypes.STRING);
 
+            builder.add("dump", args -> {
+               LuaTable table = args.arg1().checktable();
 
-
-            builder.add("nbt", args -> {
-                return LuaTableBuilder.provide(ScriptObjects.NBT_TABLE, LuaTableBuilder.toNbtCompound(args.arg1().checktable()), script);
-            }, "temp; convert table to nbt", args -> {
-                args.add("table", Argtypes.TABLE, "table");
-            }, ScriptObjects.NBT_TABLE);
+               return LuaValue.valueOf(dumpTable(table));
+            }, "Dump a table to a string.", args -> {
+                args.add("table", Argtypes.TABLE, "Table to dump.");
+            }, Argtypes.STRING);
         });
 
     }
+
+
+    private static String dumpTable(LuaTable t) {
+        StringBuilder b = new StringBuilder();
+
+        b.append("{");
+        for (LuaValue key : t.keys()) {
+            LuaValue v = t.get(key);
+            b.append(MetamethodImpl.tostring(key));
+            b.append(":");
+            if (v.istable()) {
+                b.append(dumpTable(v.checktable()));
+            } else {
+                b.append(MetamethodImpl.tostring(v));
+            }
+            b.append(",");
+        }
+        b.append("}");
+
+        return b.toString();
+    }
 }

src/main/java/dev/diamond/luafy/script/api/MinecraftApi.java

@@ -31,6 +31,7 @@
 import net.minecraft.world.phys.Vec3;
 import org.luaj.vm2.LuaBoolean;
 import org.luaj.vm2.LuaString;
+import org.luaj.vm2.LuaTable;
 import org.luaj.vm2.LuaValue;
 
 import java.util.List;
@@ -204,7 +205,6 @@ public void addFunctions(ScriptApiBuilder apiBuilder) {
                 args.add("count", Argtypes.INTEGER, "Count.");
             }, ScriptObjects.ITEM_STACK);
 
-
         });
     }
 

src/main/java/dev/diamond/luafy/script/object/NbtTableScriptObject.java

@@ -86,8 +86,9 @@ public void toTable(ItemStack obj, LuaTableBuilder builder, LuaScript script) {
 
             // get the component
             DataComponentType<Object> type = (DataComponentType<Object>) BuiltInRegistries.DATA_COMPONENT_TYPE.get(Identifier.parse(key)).orElseThrow().value();
-
             CompoundTag nbt = LuaTableBuilder.toNbtCompound(data);
+            Object component = type.codec().decode(NbtOps.INSTANCE, nbt).getOrThrow().getFirst();
+            obj.set(type, component);
 
             return LuaValue.NIL;
         });