you're missing a couple of things that must be done in the ZMI and/or Types configlet (I don't remember right now).
an example from one of our projects using GenericSetup:
diff_tool.xml
<?xml version="1.0" encoding="UTF-8"?>
<object>
<difftypes purge="false">
<type portal_type="collective.nitf.content">
<field name="any" difftype="Compound Diff for Dexterity types"/>
</type>
</difftypes>
</object>
repositorytool.xml
<?xml version="1.0"?>
<repositorytool>
<policymap purge="false">
<type name="collective.nitf.content">
<policy name="at_edit_autoversion" />
<policy name="version_on_revert" />
</type>
</policymap>
</repositorytool>
this needs to be documented.