Disambiguation error about citeproc HOT 6 CLOSED

OK, I see what is going on.
In order to avoid expensive re-renders in disambiguation, I operate with a DisambData structure that just contains names and dates. This works fine for author-date styles, but not so well when titles are rendered as part of the citation, as in this case. We may need to do an additional render at the last step to avoid this problem.

from citeproc.

What should the result be (and why)? (If you can explain this you can save me the trouble of combing through the linked thread.)

from citeproc.

Sure.
The expected result for the notes is:

1. Doe, A First Book by John Doe.
2. Doe, A Second Book by John Doe.
3. Smith, A First Book by Jane Smith.

The additional year in the Doe citations is not correct. I guess the date is inserted by this macro which is called on line 1353:

<macro name="date-disambiguate">
    <choose>
      <if disambiguate="true" type="personal_communication" match="any">
        <text macro="issued"/>
      </if>
    </choose>
  </macro>

Now, the spec says about disambiguate="true":

When set to “true” (the only allowed value), the element content is only rendered if it disambiguates two otherwise identical citations. This attempt at disambiguation is only made when all other disambiguation methods have failed to uniquely identify the target source.

But in this case, there is clearly no need to disambiguate these citations by adding the date.

from citeproc.

Is this rule even getting triggered here? This is not a personal_communication but a book.

from citeproc.

Oh, sorry, I guess the match=any matches if EITHER condition is met.
This is definitely getting triggered, because the output changes if I remove it.

from citeproc.

Or maybe re-render at every step and measure the performance impact.

from citeproc.