Cosmetic refactors related to if/else

[staltz]

Context: in PR #22, arj asked to bring back the else if pattern.

The first commit formats the encoder / decoder object so that:

• Their order is sigilSuffix, string, bool, nil, bytes to reflect the order in bfe.json
• There are spaces in between, as it used to be in version 1.0

The second commit does the else-if refactor

[staltz]

@mixmix Could you check this one?

[mixmix]

I don't see the else if as an improvement, but won't block this.

I mainly like the horizontal alignment of all the clauses and the lack of vertical spread (this adds 20 lines?)

If this started with If (false) If else (x) return... If else (y) return...

Then it would be aligned. Dunno if linter let's you collapse those blocks into single line though

[mixmix]

Basically this PR feels like minor safety improvement vs minor readability improvement (for some). Ie bikeshed!

Let's keep moving

[staltz]

I don't see the else if as an improvement, but won't block this.

I think it's more about keeping the else if pattern that was there in ssb-bfe originally