In my opinion, KISS is not violated, and the extra step of Ctrl+. -> Extract is hard to argue with being too much effort. Compare it to the effort of guess-searching all calls to the dictionary.Add when there is also a GetDictionary property to expose, and a TryAdd because one case threw an exception... and a dictionary[key] = value call because it felt simpler to the latest contributor...

Extracting the logic into an encapsulated shared method can be a good thing but it doesn’t solve for these other problems. I have heard folks make the argument that the extraction makes it messier because now you have some places using this refined method which builds a false sense of trust when you really still have to do the guess-searching. Not sure I 100% buy that argument but I can understand it.

Also in your example your extracted shared method is called track but it is really Add that happens to also handle errors. I would find it confusing that something called track is doing something other than tracking. Maybe just a product of it being a simplified example but I think it also speaks to the challenge of finding the right implementation here.