given that most of the completion string generation happens in the ```completionString``` method, don't you think it might be a good idea to work this logic into that too? So that it will all be in one location.