feat(namespaces): remove namespaces with only a log filter

[kuhe]

Model Namespaces

Problem: model namespaces containing only a filterSensitiveLog function are generated alongside most model interfaces. This is distracting to readers of the model file, and worse, creates an interruption when users use VSCode’s go-to-definition functionality on model interfaces that have an identical name with these namespaces.

Solution: remove these namespaces and generate the filter functions with unique names.

Because the diff is excessively large, this is what it looks like in general:

Old

models_{n}.ts

export interface ResultType { ... }
export namespace ResultType {
    export const filterSensitiveLog = (obj: any) => { ... }
}

{Name}Command.ts

import { InputType, ResultType } from 'models_{n}';
...
    const handlerExecutionContext: HandlerExecutionContext = {
      logger,
      clientName,
      commandName,
      inputFilterSensitiveLog: InputType.filterSensitiveLog,
      outputFilterSensitiveLog: ResultType.filterSensitiveLog,
    };
...

New

models_{n}.ts

export interface InputType {}
export interface ResultType {}
export type T {}

// aggregated at bottom of file
export const InputTypeFilterSensitiveLog ...
export const ResultTypeFilterSensitiveLog ...
export const TFilterSensitiveLog ...

{Name}Command.ts

import { InputType, InputTypeFilterSensitiveLog ... } from 'models_{n}';

...
    const handlerExecutionContext: HandlerExecutionContext = {
      logger,
      clientName,
      commandName,
      inputFilterSensitiveLog: InputTypeFilterSensitiveLog,
      outputFilterSensitiveLog: ResultTypeFilterSensitiveLog,
    };
...

Testing

CI only

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.