Fix SDK build error from console lib

[jmrossy]

No description provided.

[jmrossy]

Rossy
—
Today at 2:18 PM
I'm seeing a build error using the latest SDK coming from this use of an un-listed runtime dep here: https://github.com/abacus-network/abacus-monorepo/pull/859/files#diff-fcbd20c85364887c8a84ce55239f001ab6f6562dbb2106457c5893a99751390bR5
@asaoines Did you mean to import assert from console instead of using the abacus util's existing assert (i.e. utils.assert() )
Rossy
—
Today at 2:27 PM
Actually we use the console lib in two places. RetryProvider above and in utils/logging.ts: https://github.com/abacus-network/abacus-monorepo/blob/main/typescript/utils/src/logging.ts#L1

@Mattie some question as for Asa above. Did you mean to import from console? If so, then we need to explicitly include that as a runtime dep on utils. Though IMO we shouldn't add a dep for this, just logging to the globally available console should be fine 🙂
mattie
—
Today at 2:29 PM
I copied that import from @tkporter's version. I assumed that there was a reason the import was needed.
I am cool with removing if we know it won't change funcunality
Rossy
—
Today at 2:33 PM
@tkporter Is there an env we're using that doesn't have the default global console available? If not I don't think we need that lib
nambrot
—
Today at 2:35 PM
i wanna say this assert comes from the auto-import that VS code does for you and probably wasnt intentional
tkporter
—
Today at 2:35 PM
honestly forgot I added that so feel free to change as you see fit
I think I added it back when we were using console groups so I wanted the groupIndentation: 4, instead of the default of 2, but that's not used anymore so let's get rid of it