Earlier quoted context omitted.
As per the other person's comment, yeah basically I could have broken it up but it would've been an arbitrary demarcation. I just deleted our functions and fixed everything that yelled. Admittedly that could've been one and then leveraging the libraries better could've been another, but they would've been 2 PRs that changed almost every line. So done as one to mitigate review time.
Why not leave your functions but have them invoke the libraries instead?
> setup_terminal(); enable_input(); while(...) inp = read_character(); .....
vs
> readline()
So yes I could've stubbed out the other stuff and replaced just one, but that's just adding tech debt