The promo image is absolutely ridiculous. The lever, button and antenna on the block do not exist in the real mod. The remote control has "on" and "off" buttons, but also an "on/off" button (which also looks garbled). The "L" button makes no sense. The random up arrow in the background means nothing.
The UI is confusing. The two modes are called "lever" and "button" which is a bit strange. It is inconsistent in calling the channels either "Left/Right" or "Left Click/Right Click". The word "click" does not work anymore, because people can rebind those keys now
The actual node texture looks bad, it uses the same texture on all sides and also has an art style mismatch (not mesecons yellow, no 16×16 texture). And it also doesn't look like in the promo image, and looks identical in ON and OFF state
The chat command makes no sense and feels bloated
Still uses the name "Minetest" instead of "Luanti"
When you destroy an active receiver node, it is not removed from the receiver list, leading to garbage in the database. If an LVM destroys a receiver node, same problem
Code garbage:
The receiver ID is overly complex. It also needs to store data in a file, which is risky because that can lead to data inconsistencies. It would be much simpler to just use node positions to identify receivers instead
The receiver ID is broken by design. It just generates a random 4-digit number and when it collides, it tries again. This mod will freeze when there are 10000 receivers
Uses minetest.serialize to store positions instead of minetest.pos_to_string
Wrong use of translation placeholders: %s instead of @1, or outright string concatenation
There is code for sneak+rightclick with remote on a receiver, but it never runs
And I could just go on and on and on …
PLEASE just learn real progamming already. Vibe coding is a dead end.
Thanks for your critique ^^
As for the promo image, indeed it’s not very faithful to the mod, but the second screenshot does show the reality quite well ^^
However, I don’t really understand the confusion between “lever” and “button.”
Regarding Luanti–Minetest, I was rather referring to Minetest Game.
minetest.serialize
to store positions instead ofminetest.pos_to_string
%s
instead of@1
, or outright string concatenationAnd I could just go on and on and on …
PLEASE just learn real progamming already. Vibe coding is a dead end.
I can't agree more.
Thanks for your critique ^^ As for the promo image, indeed it’s not very faithful to the mod, but the second screenshot does show the reality quite well ^^ However, I don’t really understand the confusion between “lever” and “button.” Regarding Luanti–Minetest, I was rather referring to Minetest Game.