This is not really related to scan base clean up. Due to #404 (closed) these scans stopped working more than a year ago. If there are no unit tests and nobody uses them this cannot be avoided.
well it was working as is 1.2 right before the cleanup merge. also, the return values were never used in the first place. you did edit (one of) the files even and added comments. instead you could have just changed the api and everything would have been fine.
changing the scan api is all that is needed. already implemented in !382 (merged)