Dave Meehan

NVM Group 1 data CRC never validated on LmHandler NVM Restore

8

The following code is never executed due to the conditon always being false. The effect is that the Group 1 data is not checked for validity before restoration, which could...

LmHandler: inclusion of Commissioning.h seems redundant

1

The remaining parameters would be better handled via LmHandlerParams than using macros that must be defined by the application source.

Used by EventBridge Pipes.

Used for LPWAN devices

EntityManager create() implies either strictPropertyInitialization=false or optional auto-generated properties

1

It seems that TypeDORM assumes/requires Typescripts `strictPropertyInitialization` to be set `false`. It is automatically set to true if the `strict` option is used in tsconfig. The pattern for create() requires...

docs: import syntax requires specific '* as asl' format or exports are not found

5

Current usage documentation shows `import * as asl from '@ts2asl....` when writing the source. I prefer to use `import { deploy } from '@ts2asl...'` and in my initial attempt to...

Here is a typical example of using Powertools Logger, then adding the Lambda context so that its added to the structure log entries sent to CloudWatch: ```ts import { Logger...

docs: where to store runtime state such as service clients across lambda invocations

1

It seems normal practice when implementing a Lambda to put client instantiation outside of the handler, so that they are performed once at cold start, and have their endpoint connections...

e.g. ``` // Update User asl.sdk(DynamoDB).updateItem(...) ``` The AST appears to include the preceeding comment as part of the block, but would need a regex or similar to extract any...

Code in question (which might not be a valid use of execution id, but happens even if execution id is assigned to a variable first): ``` if (lockInfo.currentLockItem?.['$$.Execution.Id'].S === '*Z*')...