Am I undermining the efficiency of StringBuilder?

I see no problem with your extension. If it works for you it’s all good.

I myself prefere:

sb.Append(SettingNode)
  .Append(KeyAttribute)
  .Append(setting.Name);