Проблема в том, что значение аргумента по умолчанию вычисляется один раз — в момент определения функции. В вашем примере lst = создаётся одна и та же список-объект и затем переиспользуется при каждом вызове функции без явной передачи lst. Поэтому последовательные вызовы модифицируют один и тот же список.

Что у вас происходит:

Первый вызов add_item$1$ даёт $$1$$.Второй вызов add_item$2$ добавляет 2 в уже существующий список -> $$1,2$$.

Иллюстрация $можнопосмотретьidсписка$:

def add_item$x,lst=[]$:
print${}^{\prime }id(lst){=}^{\prime },id(lst)$ lst.append$x$ return lst
print$ad{d}_{i}tem(1)$ # id одинаковый
print$ad{d}_{i}tem(2)$

Как исправить
1) Использовать "sentinel" None $рекомендуемыйподход$:

def add_item$x,lst=None$:
if lst is None:
lst = lst.append$x$ return lst
print$ad{d}_{i}tem(1)$ # $$1$$ print$ad{d}_{i}tem(2)$ # $$2$$

2) Всегда передавать новый список при вызове:

print$ad{d}_{i}tem(1,[])$ print$ad{d}_{i}tem(2,[])$

3) Если нужно принять уже существующий список, но не изменять переданный объект — делать копию:

def add_item$x,lst=None$:
if lst is None:
lst = else:
lst = list$lst$ # копия
lst.append$x$ return lst

Это не баг Python, а ожидаемое поведение. Для мутабельных объектов $list,dict,setит.п.$ по умолчанию лучше использовать None и создавать объект внутри функции.