В одном из наших проектов много кода, который выглядит следующим образом:

internal static class Extensions
{
    public static string AddFoo(this string s)
    {
        if (!string.IsNullOrEmpty(s)) return s + "Foo";
        return "Foo";
    }
}

Есть ли какая-либо явная причина, чтобы сделать это, кроме того, что «легче сделать тип общедоступным позже?»

Я подозреваю, что это имеет значение только в очень странных крайних случаях (отражение в Silverlight) или вообще не имеет значения.