Usability improvements

- Now entity.key() is the same as entity.key.value
- Entity:serialize only serializes component given correctly
- Any other value inside the Entity is ignored
- Disable some diagnostics used in Lua language server by sumneko

.luacheckrc

@@ -1 +1,2 @@
+---@diagnostic disable: lowercase-global
 std="love+luajit"

concord/builtins/key.lua

@@ -22,6 +22,10 @@ function Key:deserialize (data)
   self.value = getKey(self, data)
 end
 
+function Key.__mt:__call()
+  return self.value
+end
+
 function Key:removed (replaced)
   if not replaced then
     local entity = self.__entity

concord/component.lua

@@ -95,6 +95,7 @@ end
 function Component:__initialize(entity, ...)
    local component = self:__new(entity)
 
+   ---@diagnostic disable-next-line: redundant-parameter
    self.__populate(component, ...)
 
    return component

concord/entity.lua

@@ -248,7 +248,8 @@ function Entity:serialize(ignoreKey)
          if not ignoreKey then
             data.key = component.value
          end
-      elseif (name ~= "__world") and (name ~= "__isEntity") and (component.__name == name) then
+      --We only care about components that were properly given to the entity
+      elseif Type.isComponent(component) and (component.__name == name) then
          local componentData = component:serialize()
 
          if componentData ~= nil then

concord/world.lua

@@ -476,6 +476,7 @@ end
 
 return setmetatable(World, {
    __call = function(_, ...)
+      ---@diagnostic disable-next-line: redundant-parameter
       return World.new(...)
    end,
 })