I'm using the code your provided above:
// Move all Yes fields to the right
var strPrevVal = "", strCurVal, oCurFld;
for(var i=1;i<=4;i++)
{
oCurFld = this.getField("Yes" + i);
strCurVal = oCurFld.valueAsString;
oCurFld.value = strPrevVal;
strPrevVal = strCurVal;
}
// Move all No fields to the right
var strPrevVal = "", strCurVal, oCurFld;
for(var i=1;i<=4;i++)
{
oCurFld = this.getField("No" + i);
strCurVal = oCurFld.valueAsString;
oCurFld.value = strPrevVal;
strPrevVal = strCurVal;
}
// Move all NA fields to the right
var strPrevVal = "", strCurVal, oCurFld;
for(var i=1;i<=4;i++)
{
oCurFld = this.getField("NA" + i);
strCurVal = oCurFld.valueAsString;
oCurFld.value = strPrevVal;
strPrevVal = strCurVal;
}
// Move all Year fields to the right
var strPrevVal = "", strCurVal, oCurFld;
for(var i=1;i<=4;i++)
{
oCurFld = this.getField("Year" + i);
strCurVal = oCurFld.valueAsString;
oCurFld.value = strPrevVal;
strPrevVal = strCurVal;
}
// Move all AuditPeriod fields to the right
var strPrevVal = "", strCurVal, oCurFld;
for(var i=1;i<=4;i++)
{
oCurFld = this.getField("AuditPeriod" + i);
strCurVal = oCurFld.valueAsString;
oCurFld.value = strPrevVal;
strPrevVal = strCurVal;
}

The names of the column fields in Step 2 start with the "5" index. The scripts work by looping over the fields from 1 to 4.
The Step 2 fields is where the consisntent naming convention breaks down.
One solution is to rename the fields so they fit the model. Something like "Step2.Yes1", "Step2.Yes2", etc.
Another solution, and one that is much easier to do, is to write new loops to take the different index into account.
I'd suggest going with the second solution and add new loops to the script:
like this:
for(var i=5;i<=8;i++)
{
oCurFld = this.getField("Yes" + i);
strCurVal = oCurFld.valueAsString;
oCurFld.value = strPrevVal;
strPrevVal = strCurVal;
}