@stof aw yes, I didn't break the BC, but it should be done in a new version of knpmenu (I added a new arg on createItem). But as you say, this implementation couple the factory to the item and i'm not happy with that either, the fact is that I couldn't make something better without breaking things. Do you have something to refactore without breaking things ? (I was thinking about using a factory extension to manage the weight)
About the break in @folliked 's PR, I didn't notice it, so we should merge #207 . IMO it's not a good idea to fragment knpmenu in many repo (and as it's needed for complete integration, it adds a new dependency for KnpMenuBundle).