One thing that's jumping out to me as a bit questionable here is the usage of the name "external-list". All the other group types end with "-group", so having one end with "-list" would be tad bit inconsistent.