Some tips:
- Add version cvar, for some reason you totally skipped out on this. (see:
https://forums.alliedmods.net/showthread.php?t=57142)
-Parse the list once as an ADT_Array or even just a keyvalue. That way you are not loading from file ideally twice right after each other when you could have just loaded it and then searched etc. If you need some reference on how to do this I think i could post an example.
-ServerCommand can be replaced with SetConvarString.
Edit:
something like this doesn't really need to use key-value, as it's just a list, although it's probably best to not change it now. but something you could do is readline (ex:
https://forums.alliedmods.net/showthread.php?p=1729023)