You need to follow the normal item NBT structure and it should work fine. I believe that your command should be:
/summon Villager ~ ~1 ~ {Offers:{Recipes:[{maxUses:10,buy:{id:388,Count:5},sell:{id:349,Damage:3,Count:3}}]}}
Each recipe (containing a buy item and a sell item) should be in the Recipe
list, looking something like:
Currently, you have the second recipe inside the data of the first recipe's sell item:
The following data should work:
{
Profession: 3,
CustomName: "PvP Villager",
CustomNameVisible: 1,
Career: 1,
CareerLevel: 42,
CanPickUpLoot: 0,
PersistenceRequired: 1,
Invulnerable: 1,
Offers: {
Recipes: [
{
rewardExp: false,
uses: -2147483648,
maxUses: 0,
buy: {
id: gold_block,
Count: 8
},
sell: {
id: potion,
Count: 1,
tag: {
HideFlags: 32,
CustomPotionEffects: [
{
Id: 10,
Amplifier: 1,
Duration: 410
},
{
Id: 22,
Amplifier: 3,
Duration: 2400
},
{
Id: 11,
Amplifier: 0,
Duration: 6000
},
{
Id: 12,
Amplifier: 0,
Duration: 6000
}
],
display: {
Name: "Gods water",
Lore: [
"Who knew water was so good!"
]
}
}
}
},
{
buy: {
id: "diamond_block",
Count: 8
},
maxUses: 9999999,
sell: {
id: "golden_apple",
Count: 1,
Damage: 1
},
rewardExp: false
}
]
}
}
I'd recommend building up commands and testing them along the way, rather than starting with a long command and adding in brackets to try to fix the errors.
Best Answer
You cannot set empty NBT tag nor set something to zero count.
Make it another way - Sell two items and let him give you back the one of it - "renewable token".
So you will sell "anything" + "item you want to sell" , and get "anything" back. Yup, you just lost that thing you wanted to get rid of.
PS: buyb is the second buy tag, as I remember, cannot check now from work :)