redundant_type_annotation autocorrect is really buggy

[Jeehut]

I recently turned on redundant_type_annotation which seems to work fine when it comes to showing warnigns. But when running swiftlint autocorrect I get many changes that break my code. Here are a few examples of the changes made by this rule:

BEFORE

 enum EventType {
     case amazingMonitor(isPresenting: Bool)
     case amazingHistory(isPresenting: Bool)
     case startNFCScan
     case scanAmazingQRCode
 }

AFTER (the fist two cases mixed up)

 enum EventType {
     case amazingMonitor(isPresentingcase amazingHistory(isPresenting: Bool)
     case startNFCScan
     case scanAmazingQRCode
 }

BEFORE

 import Foundation
 import MapKit
 
 // swiftlint:disable:next type_body_length
 final class AmazingTracking: NSObject {
     // MARK: - Stored Type Properties
     static let shared = AmazingTracking()

AFTER (the swiftlint disabling broken)

 import Foundation
 import MapKit
 
 // swiftlintlength
 final class AmazingTracking: NSObject {
     // MARK: - Stored Type Properties
     static let shared = AmazingTracking()

BEFORE

 protocol AmazingManagerDelegate: class {
     func manager(_ manager: AmazingManager, didChange region: MKCoordinateRegion)

AFTER (the : class is gone)

 protocol AmazingManagerDelegate {
     func manager(_ manager: AmazingManager, didChange region: MKCoordinateRegion)

There's a lot more if you need more examples, but I thinks this is a start.

[marcelofabri]

@Dschee I wasn't able to reproduce the problem with the examples you've provided. Can you double check that those reduced snippets trigger the issue?

[Jeehut]

You were right, the above examples didn't have this bug. But I could reduce the issue to the following example, where this happens:

class ViewController: UIViewController {
    func someMethod() {
        let myVar: Int = Int(5)
    }
}

When autocorrecting this example, instead of removing the redundant : Int it removes the : UIViewController.

[Jeehut]

@marcelofabri Thank you for fixing this so quickly!