I wrote this code that once the opportunity is marked as "Closed Won" it is suppoed to clone the opp and primary contact role.  For some FRUSTERATING reason it almost always creates TWO clones instead of one and I can't figure out why.

trigger CreateGBRenewal on Opportunity (before update) {
try {

// List of opportunity IDs
Set<Id> oppIds = new Set<Id>();
//lists used for bulk
List<Opportunity> clonesToInsert = new List<Opportunity>();
List<OpportunityContactRole> RoleToInsert = new list<OpportunityContactRole>();

for( opportunity opp: Trigger.New){
if ((opp.StageName == 'Closed Won') && (opp.Subscription_Type__c == 'GradeBeam'||opp.Subscription_Type__c == 'GradeBeam for Subs' )&& (trigger.oldMap.get(opp.id).StageName != 'Closed Won' )){
oppIds.add(opp.Id);
}
}

for( opportunity oldOpp: [SELECT ID, accountId, account.name, Estimated_Renewal_Date__c FROM opportunity WHERE Id IN :OppIds]){

opportunity renewOpp = New opportunity();
renewOpp.closedate = oldOpp.Estimated_Renewal_Date__c;
renewOpp.AccountId = oldOpp.accountId;
renewOpp.name = oldOpp.account.name + ' Renewal';
renewOpp.stagename = 'Renewal Scheduled';
renewOpp.Type = 'Repeat Client';
renewOpp.RecordTypeId = '012S00000008r99';
clonesToInsert.add(renewOpp);
insert clonesToInsert;

for( opportunityContactRole OppCon: [SELECT ContactID, IsPrimary, Role FROM OpportunityContactRole WHERE OpportunityId IN :OppIds AND IsPrimary = TRUE]) {

OpportunityContactRole ConRole = New OpportunityContactRole();
ConRole.ContactID = OppCon.ContactID;
ConRole.OpportunityId = renewOpp.ID;
ConRole.IsPrimary = OppCon.IsPrimary;
ConRole.Role = oppCon.Role;
RoleToInsert.add(ConRole);
insert RoleToInsert;
}
break;
}
}catch (Exception e) {Trigger.new[0].addError(e.getMessage());}
}