App crashing after updating to v4.7.1 with logLevel set to 'info'

[g4rry420]

After upgrading the Smartsheet JavaScript SDK from a v2.126.0 to v4.7.1, our application crashes during initialization when logLevel is set to 'info'

cc @ggoranov-smar

[roomote-v0]

I've analyzed this issue and identified the root cause. The problem is in the buildLoggerFromLevel function in index.ts.

Root Cause

Line 72 attempts to validate the log level using:

if (winston.level[logLevel] == null) {

However, winston.level does not exist in Winston v2.x (which this SDK uses - v2.3.1), causing the application to crash with "Cannot read properties of undefined".

Proposed Fix

Replace the incorrect check with the proper Winston v2.x API:

In index.ts line 72:

// Change from:
if (winston.level[logLevel] == null) {

// To:
if (winston.config.npm.levels[logLevel] == null) {

This uses the correct Winston v2.x API to access the npm logging levels (error, warn, info, verbose, debug, silly).

Optional cleanup in lib/types/CreateClientOptions.ts line 11:
Remove duplicate 'info' and 'http' (not a standard npm level in Winston v2):

// Change from:
logLevel?: 'error' | 'warn' | 'info' | 'http' | 'info' | 'verbose' | 'debug' | 'silly';

// To:
logLevel?: 'error' | 'warn' | 'info' | 'verbose' | 'debug' | 'silly';

This fix will resolve the crash when logLevel is set to any valid value.

Please mention @roomote in a comment saying that you approve this plan, and I'll proceed with the implementation.

[ggoranov-smar]

Fixed by #159 and will be released with v4.7.2