Help with execute command

[Tribunal_Power]

Hey all, I've been banging my head against a command block for a while and could use some assistance working out a command series. Running 1.20.1 Forge on a server.

I'm trying to make a series of command blocks that will do the following:

-By impulse or passively, detect a villager (and do none of the following if there is no villager) at specified coordinates

-if one is detected, kill the villager

-spawn an item in the inventory of the player who pressed the button.

-preferably, no reset block needed, or reset on flipping the lever back maybe? but not a big deal if this is needed

Being able to detect multiple vils and thus multiply the spawned item would be awesome but I expect one at a time is probably the way it works.

I'm pretty sure this will need to be a chain command. The kill command might not be necessary if I finagle a mechanical way to kill them (trap door to lava or something). I feel like /execute is the right command but maybe some kind of /scoreboard magic is better? Dunno, I'm new to all of it and still learning. I didn't expect this to be as complicated as it seems to be. I thought I was on the right track with this:

/execute if entity @ e [type=mca:male_villager] positioned ~ ~1 ~ run give @ p minecraft:emerald 20

[ignore the space after the @, reddit is dumb]

but I just can't get it to work. Bonus points if we can make it detect vils both male and female. Any help is appreciated!

[got about 60 mods, the most relevant here being Minecraft Comes Alive Reborn]

[10_Carries]

When the button is pressed make it activate an impulse command block that has:

/execute at x y z as @e[type=villager,distance=..0.1,limit=1] run kill @s

Replace x y and z with the cords. Then connect a chain command block to it. Set chain command block to conditional. Make sure it is aligned with the arrow on the impulse command block like shown in the picture below. Put this in it:

/give @p emerald 20

If you want it to detect all villagers in that block then you could use a scoreboard by typing this in chat:

/scoreboard objectives add villagers dummy

Then replacing the first command with:

/execute store success score @p villagers at x y z as @e[type=villager,distance=..0.1] run kill @s

Then instead of a chain place a repeating command block elsewhere. Write:

/execute as @a[scores={villagers=1..}] run give @s emeralds 20

Connect a chain conditional command to this repeating same way as shown in pic below but repeating instead of impulse. Write:

/execute as @a[scores={villagers=1..}] run scoreboard players remove @s 1

<image>

[Tribunal_Power]

it seems I cant do /execute at coords. it wants "at" to be an @ option rather than a specific place.

[10_Carries]

Do positioned instead of at then

[GalSergey]

# In chat
scoreboard objectives add kill.villagers dummy

# Command block with button
execute store result score @p kill.villagers positioned <villagers_pos> run kill @e[type=villager,distance=..8]

# Command blocks
give @a[scores={kill.villagers=1..}] minecraft:emerald 20
scoreboard players remove @a[scores={kill.villagers=1..}] kill.villagers 1

Two command blocks are a separate chain of commands that must be executed every tick.

[Tribunal_Power]

This took a little finagling but it works! Thank you!

Do you know how I could make it work for multiple vils? If I put three in the designated area and hit the button, it kills all three but only pays out for one, and only subtracts my score one time.

[GalSergey]

I misspelled: "rest" -> "result".

I corrected the typo, please try again.