I just happened to look at the documentation on this which reads:
That kind of goes against what we have discussed. It is the reason I appended time() to the end of my ID to make it unique because my product is somewhat complex as to the number of factors that make it a unique cart item.
You might want to update that in the docs.
Steven