This section is inconsistent with the styling of the rest of the project. It's my bad for not having documentation for this, or at least modelines settings, so I created an issue https://github.com/SpiderOak/crypton/issues/155

Because this section basically replicates code above it, you should adhere to the original styling here, or change both. If I were to write that code again today I would probably keep the one-line style to keep it digestible for readers. If that's unacceptable, my personal style and the way most of the rest of the project is written is to 1) treat () {} and [] as "blocks" and 2) indent with 2 spaces each block. So for example

````js
var signKeySecret =
    JSON.parse(sjcl.decrypt(keypairKey,
                            JSON.stringify(this.signKeyPrivateCiphertext),
                            crypton.cipherOptions));
````

would become 

````js
var signKeySecret = JSON.parse(
  sjcl.decrypt(
    keypairKey,
    JSON.stringify(
      this.signKeyPrivateCiphertext
    ),
    crypton.cipherOptions
  )
);
````

or something similar. But I think in this situation the following is cleanest in the context of the amount of the number of variables we have floating around in this scope:

````js
var signKeySecret = JSON.parse(sjcl.decrypt(keypairKey, JSON.stringify(this.signKeyPrivateCiphertext), crypton.cipherOptions));
````

Obviously up to you :) but either way we should take out the extraneous indentation.

Secondly, I don't think we have to stringify signKeyPrivateCiphertext here. If we do have do, where are we parsing it into JSON and can we optimize by not?