shorten internal serde fn names

[kuhe]

Shorten generated serde function names. Source code contains a comment with the original long name, distribution uses shorter names.

The premise is that serde function names do not need to contain the protocol -- it's repetitive when the protocol file containing the functions is named after the protocol already.

JSv3 codegen paired PR aws/aws-sdk-js-v3#4611

Benefits:
reduced code size

Potential Downsides:

• breaking change for anyone is using these internal functions directly (though no one should be).
• no benefit to anyone who was applying mangling already (most browser applications, some server applications).

Example

/**
 * deserializeAws_restJson1PrefetchConsumption
 */
const de_PrefetchConsumption = ...

[kuhe]

Update: no longer shortening the symbol/shape name to prevent thrashing on the function names.

[pose]

Why is this required and cannot be achieved by properly configuring bundling and minifying with esbuild/webpack?

[kuhe]

This is not required, but it benefits users who use the generated clients as they are without minifying. One example is the AWS Lambda provided AWS SDK for JavaScript v3 that users do not bundle into their AWS Lambda applications.

[mtdowling]

Can you update the description to show the current state of what this generates? I'd like these names to be stable cross model updates too, which I think you already added. Is it now [se|de]_ShapeName?

[pose]

Why does the AWS Lambda provided AWS SDK for JavaScript v3 cannot be minified?

[kuhe]

Yes, the PR currently is only a change to [se|de]_ShapeName which is stable over model changes. I've updated the original description.

Why does the AWS Lambda provided AWS SDK for JavaScript v3 cannot be minified?

The AWS Lambda provided AWS SDKs are supposedly for development/testing/convenience purposes, link and should retain readable stack traces without mangling and minification, but customers often prefer to continue to use them for production workloads.

We can provide some benefit to those users without needing to change customer behavior. However, if you think the risk of this change is greater than the potential benefits, I'm fine with closing it.

[mtdowling]

nit - we wrap at 120

[srchase]

Can all the auto-formatting be undone on this? It's hard to read the actual changes and this will be an unnecessarily noisy commit.