Though this predates your patch, my preference is that getContentRule return the complete rule with the trailing ";" included. All consumers of getContentRule simply re-add this. My desire is that things that return attribute settings return "complete" settings to help prevent accidentally creating invalid rules by forgetting to join the semi-colon to the end.