-
Notifications
You must be signed in to change notification settings - Fork 105
Merge ssdk into main #315
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Merge ssdk into main #315
Changes from all commits
Commits
Show all changes
93 commits
Select commit
Hold shift + click to select a range
0f7a7d1
Support disabling client generation
JordonPhillips bceec98
generate response serializers
JordonPhillips eaba822
generate request deserializers
JordonPhillips 98deb6e
Add ssdk generation methods to ProtocolGenerator
JordonPhillips dffffdf
Fix reference
JordonPhillips a7ef2dc
Add error serialization
JordonPhillips eab8bf8
Fix response header serialization
JordonPhillips 1cab0ca
Fix request deser
JordonPhillips b652b73
refactor request deser / response ser
JordonPhillips f996fca
don't generate ssdk by default
JordonPhillips 8080aee
Don't export clients when not generating clients
JordonPhillips 2e1fc68
Generate IO stubs during ssdk generation
JordonPhillips db9ca79
Remove client config when not generating clients
JordonPhillips 9b5d93c
Support query string deser
JordonPhillips 8a38200
Support path deser
JordonPhillips 825a75b
Support endpoint deser
JordonPhillips 8a8a311
Fix path and host parsing.
adamthom-amzn 1c38bcf
Omit the endpoint from response serialization's SerdeContext.
adamthom-amzn 41ff1a9
Make checkstyle happy
JordonPhillips 5621fd0
remove duplicate type generation
JordonPhillips b45eb6d
WIP: Add binding receivers for query and label
JordonPhillips dde0eec
properly reference the query
JordonPhillips cda3ae4
fix reference to hostRegex
JordonPhillips 5802885
Convert query timestamps properly
JordonPhillips 5c3b89e
bypass type check for query value reading
JordonPhillips 24a9bfa
also convert path strings
JordonPhillips 9dd69b4
Fix reading collections bound to the uri
JordonPhillips 5628ea5
Fix ignored trailing slashes in path parsing.
adamthom-amzn cced771
Make consumption of serde functions simpler
adamthom-amzn abbf3e1
WIP: add empty endpoint to response ser
JordonPhillips 1b3ccdf
fix writing out custom status codes
JordonPhillips 9205e10
Add generateX properties to settings warn list
JordonPhillips 3df7c28
Update path parsing regex
JordonPhillips 289385f
Merge pull request #270 from JordonPhillips/server-serde
JordonPhillips 2c1fc47
Add generation of server interfaces.
adamthom-amzn b8f2143
Add router generation for REST-JSON services.
adamthom-amzn b0369f2
Add handler support for server generation.
adamthom-amzn d520610
Merge pull request #278 from adamthom-amzn/server-serde
JordonPhillips 523cf7e
Generate ssdk request protocol tests
JordonPhillips 4bf58c3
Merge pull request #279 from JordonPhillips/server-serde-protocol-tests
JordonPhillips 6e8f090
Introduce a server-specific symbol provider.
adamthom-amzn c389258
Merge pull request #281 from adamthom-amzn/symbolprovider
adamthom-amzn 862f19f
Get rid of needing to cast OperationSerializers.
adamthom-amzn 1a6a906
Merge pull request #282 from adamthom-amzn/norecord
adamthom-amzn 59edca7
Use server symbol provider in protocol test gen
JordonPhillips 976711c
Split up clint/server command generation
JordonPhillips cbcc084
Merge pull request #284 from JordonPhillips/update-ssdk-request-tests
JordonPhillips 128fefa
Use specific type for encoder in comparators
JordonPhillips fb5b097
Add ssdk response tests
JordonPhillips 7f1d368
update TypeScript dependencies version (#276)
AllanZhengYP 250ad49
Check for only structure shapes for circular dependency (#277)
trivikr 0068b7f
Pin fast-xml-parser to v3.17.4 (#280)
trivikr 505b142
Remove tests for shape conflicts (#286)
8e9225f
Move getHandler function to service
JordonPhillips 8299933
generate error handlers
JordonPhillips 45d8486
Use shared operation serializer
JordonPhillips 2a661c6
Catch modeled errors in server serde
JordonPhillips 206de49
Use errors type
JordonPhillips 6f3dda4
Merge pull request #285 from JordonPhillips/ssdk-response-tests
JordonPhillips 1735105
Merge pull request #289 from JordonPhillips/ssdk-exception-handling
JordonPhillips 4c93f11
Dont generate switch for "never" error serializers
JordonPhillips 3c34bc5
Generate ssdk error protocol tests
JordonPhillips 67b2a0d
Look for proper tag on server error tests
JordonPhillips 2e2664f
Merge pull request #293 from JordonPhillips/exception-handling-updates
JordonPhillips 2496c28
Support serializing framework exceptions
adamthom-amzn 3eb717b
Support asynchronous service operations
adamthom-amzn 257d395
Merge pull request #295 from adamthom-amzn/ssdk
JordonPhillips 3af194d
Adjust ssdk protocol tests to use promises
JordonPhillips 60f9b8b
Merge pull request #297 from JordonPhillips/fix-protocol-tests
JordonPhillips 8f4f83d
Add ts-jest as a devDependency (#287)
trivikr ebac15f
Pin fast-xml-parser dependency to 3.19.0 (#290)
trivikr 3acdf7c
Move types in dist folder (#294)
trivikr dc5b386
fix: create jest.config.js for clients (#291)
trivikr 21c0bca
Use service renames (#292)
b021701
Support deserializing query maps
JordonPhillips 044a7e6
Add generated error types for convenience (#298)
adamthom-amzn b49b293
Generate per-operation ServiceHandlers (#304)
adamthom-amzn d8cc74e
Add ssdk typescript libraries
JordonPhillips 63ee958
Add explicit node types dependency
JordonPhillips 5354c7c
Update ssdk node deps to node 14
JordonPhillips 571f11f
Remove esm builds from ssdk libs
JordonPhillips 6660ad6
Refresh ssdk yarn locks
JordonPhillips 2984250
Add todo comments to update ssdk compiler targets
JordonPhillips b0a0cce
Parameterize ServiceHandler
JordonPhillips caa953a
Move ssdk lib types into dist/types
JordonPhillips 00f8856
Merge branch 'main' into merge-ssdk
JordonPhillips 6326801
Rework some of the project setup (#317)
adamthom-amzn fe6edd5
Update generated io types
JordonPhillips efa3f1a
Rework ssdk package setup
JordonPhillips 02935fa
Run prettier on ssdk libs
JordonPhillips a1027b4
remove unused dependency
JordonPhillips 113bf3e
Create and use ServerSerdeContext interface
JordonPhillips 066d60f
Merge branch 'main' into merge-ssdk
JordonPhillips File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since client and server SDK's plugins are divergent, why not adding a flag like
isServerSdkIntegration
into theTypeScriptIntegration
interface or extend to a new interface likeTypeScriptServerIntegration
? Then 2 different codegen plugins can be applied separately.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something like that would be a great idea. I've got a PR that splits client/server up further where we could add something like that: #314