[organizeDeclarations] A property with didSet is treated as computed

[anivaros]

Properties with didSet are placed in the Computed Properties section.

// MARK: - Computed Properties

var foo: Bar = .baz {
  didSet {
    // Do some work
  }
}

But they must be in the section of regular properties.

[calda]

The category grouping here is "instance properties with a body", which includes computed properties and stored properties with didSet / willSet blocks. The problem here is that the mark comment text doesn't accurately describe the section:

case .instancePropertyWithBody:
    return "Computed Properties"

@oiuhr, any thoughts on what we should do here? Some options I can think of:

1. Change the mark comment of instancePropertyWithBody from // MARK: - Computed properties to // MARK: - Properties with body
2. Create a new computedInstanceProperty declaration type and remove instancePropertyWithBody from the default list of --organizationmode type. In that case these properties with didSet / willSet blocks would fall back to being part of the instanceProperty group.

[oiuhr]

either way looks fine to me 👍 i guess adding a new categories (reorganizing existing ones) is a thing tbd anyways

as a workaround op could also just provide a custom mark title for 'computedInstanceProperty'. option is poorly documented but it’s certainly working, maybe we should address this problem here as well

[calda]

Here's a fix: #1952

[nicklockwood]

@anivaros fix landed in 0.55.5