It wouldn't actually even work, anyway. The corruption isn't in the import process, it's in XF's renderer being broken. All the relevant information is already in the current database.
No, it's not that. Or at least it's not
just that. Those masses of extra added "[/color]" tags are corruption, as well; they weren't present in the original at all. (Never mind the "[color=#A4A77F]", etc, following that, or the mass of irrelevant closing tags at the trailing end of the post.) Additionally, SMF's [code] and [tt] tags appear to have been merged, which is
very wrong: the latter should have been changed to [font="Courier New"] tags or somesuch.
And, as you noted later, the [hr] tags have been lost. It should be almost trivial to add [hr] support with a plugin; at the very least unknown tags should be left in place, untouched, rather than stripped entirely. (As was done for the [nobbc] tag, which XF could sorely use an equivalent to.)
I am currently arguing with the XF devs about the render issue. Apparently, in the specific case of your post, the problem is that it treats [COLOR=something]this[COLOR=somethingelse]that[/COLOR] as two nested tags without a proper closing of one, rather than implicitly closing each tag at the beginning of the next one (as it should, since nesting a [color] tag doesn't even make sense). This means that a big train of this runs into their recursion limit. No info on a resolution as of yet, but working on it.
No, that's not right at all. Nesting a [color] tag
does make sense, if (say) you want a single bit of red text in the middle of some cyan, and I'm pretty sure SMF and XF will handle those cases identically:
basically like this, open open close close. What I remember doing on SMF wasn't just to nest and close color tags, though; there was another tag (explicit default size, maybe? I've forgotten) that I used to surround the whole shebang, implicitly closing all the "color" tags when I closed that outer tag. That's what SMF did that XF doesn't do: in XF, unclosed interior tags
leak out of the enclosing tag.
SMF's code needs to be properly parsed and restructured in order to translate it to legal XF code, and the importer simply did not do this.
It would also be contingent on getting a way to closely manage the import process; it seems that the transforms run over the input data aren't really discretionary in the usual case. I'm classing this as a low-priority fix, since the damage it does is not to the point of illegibility and the fix would entail much effort and some data losses.
It totally
is to the point of illegibility. See
the first link I provided, and scroll down. Highlight. That text shouldn't be transparent.
I don't see why you think the fix would entail loss of data. It should be possible to import only unchanged posts.