Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(conversation): maintain 1:1 mapping of message to item #2910

Closed
wants to merge 25 commits into from
Closed

fix(conversation): maintain 1:1 mapping of message to item #2910

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
2cd42f3
refactor(conversation): use file templates for JS resolvers
atierian Sep 18, 2024
88f2dd3
update resolver generator post cherry-pick
atierian Sep 24, 2024
50f959e
update conversation snapshots
atierian Sep 24, 2024
ffe6c0f
cleanup comments
atierian Sep 24, 2024
e36d2d9
lint and add resolvers/*.js to prettierignore
atierian Sep 24, 2024
93de4e2
use [[KEY]] marker for template substitution
atierian Sep 24, 2024
b05fbac
use .js.template file extension for resolver templates
atierian Sep 26, 2024
e49417d
use .template.js for that sweet sweet syntax highlighting in gh
atierian Sep 26, 2024
4bf5ce5
update prettierignore
atierian Sep 26, 2024
0848609
refactor(generation): use file templates for JS resolvers
atierian Sep 18, 2024
fda30fb
use [[KEY]] marker for substitution
atierian Sep 24, 2024
eab9624
add test case with valid inference configuration
atierian Sep 24, 2024
504d430
lint
atierian Sep 24, 2024
86440f1
update snapshots post lint
atierian Sep 24, 2024
9129e5c
use .js.template for resolver function templates
atierian Sep 26, 2024
e047843
use .template.js for that sweet sweet syntax highlighting in gh
atierian Sep 26, 2024
70ce4fd
fix(conversation): missing aiContext in message payload
atierian Sep 25, 2024
f0e578d
fix(conversation): split messages by role and use existing GSI for cr…
atierian Sep 25, 2024
45d1538
fix IndexName -> index in otherwise unused query list resolver
atierian Sep 12, 2024
af6ffc1
use messageHistoryQuery, remove messages from lambda payload
atierian Sep 25, 2024
0269fa0
update conversation snapshots for listqueryname typo
atierian Sep 26, 2024
bef11d4
use messageHistoryQuery, remove messages from lambda payload
atierian Sep 25, 2024
82fd92c
remove assistantContent, add associatedUserMessageId for item level m…
atierian Sep 25, 2024
59b91b0
update assistant response resolver pipeline to reflect 1:1 message:it…
atierian Sep 26, 2024
ef639c7
update snapshots
atierian Sep 26, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .prettierignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ packages/amplify-graphql-api-construct-tests/amplify-e2e-reports
packages/amplify-graphql-api-construct/README.md
packages/amplify-graphql-api-construct/tsconfig.json
packages/amplify-graphql-conversation-transformer/src/__tests__/schemas/*.graphql
packages/amplify-graphql-conversation-transformer/src/resolvers/*.template.js
packages/amplify-data-construct/README.md
packages/amplify-data-construct/tsconfig.json
packages/amplify-graphql-model-transformer/publish-notification-lambda/lib/
Expand Down
4 changes: 2 additions & 2 deletions packages/amplify-data-construct/.jsii
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
]
},
"bundled": {
"@aws-amplify/ai-constructs": "^0.1.4",
"@aws-amplify/ai-constructs": "0.0.0-test-20240925144932",
"@aws-amplify/backend-output-schemas": "^1.0.0",
"@aws-amplify/backend-output-storage": "^1.0.0",
"@aws-amplify/graphql-auth-transformer": "4.1.1",
Expand Down Expand Up @@ -3970,5 +3970,5 @@
},
"types": {},
"version": "1.10.1",
"fingerprint": "18+pcRZF+HEurnEkAAQsQVHj1oGYG+i0hcJuPIeDP7U="
"fingerprint": "XIFtuREMLP4ld3+h28KKIQLwEL1DxCLKiMqJERSh2wY="
}
2 changes: 1 addition & 1 deletion packages/amplify-data-construct/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@
"semver"
],
"dependencies": {
"@aws-amplify/ai-constructs": "^0.1.4",
"@aws-amplify/ai-constructs": "0.0.0-test-20240925144932",
"@aws-amplify/backend-output-schemas": "^1.0.0",
"@aws-amplify/backend-output-storage": "^1.0.0",
"@aws-amplify/graphql-api-construct": "1.13.0",
Expand Down
4 changes: 2 additions & 2 deletions packages/amplify-graphql-api-construct/.jsii
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
]
},
"bundled": {
"@aws-amplify/ai-constructs": "^0.1.4",
"@aws-amplify/ai-constructs": "0.0.0-test-20240925144932",
"@aws-amplify/backend-output-schemas": "^1.0.0",
"@aws-amplify/backend-output-storage": "^1.0.0",
"@aws-amplify/graphql-auth-transformer": "4.1.1",
Expand Down Expand Up @@ -8887,5 +8887,5 @@
}
},
"version": "1.13.0",
"fingerprint": "fBEq7TqsVFMCTBRZ+ohz6O/ygsZR+7evKnwJZkkf+/A="
"fingerprint": "bwGSsIcxucYASo+SO2R0k8vwf8deNlWleoyg6ajeR1s="
}
2 changes: 1 addition & 1 deletion packages/amplify-graphql-api-construct/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@
"semver"
],
"dependencies": {
"@aws-amplify/ai-constructs": "^0.1.4",
"@aws-amplify/ai-constructs": "0.0.0-test-20240925144932",
"@aws-amplify/backend-output-schemas": "^1.0.0",
"@aws-amplify/backend-output-storage": "^1.0.0",
"@aws-amplify/graphql-auth-transformer": "4.1.1",
Expand Down
5 changes: 3 additions & 2 deletions packages/amplify-graphql-conversation-transformer/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,15 @@
"access": "public"
},
"scripts": {
"build": "tsc",
"build": "tsc && yarn copy-js-resolver-templates",
"watch": "tsc -w",
"clean": "rimraf ./lib",
"copy-js-resolver-templates": "cp ./src/resolvers/*.template.js ./lib/resolvers",
"test": "jest",
"extract-api": "ts-node ../../scripts/extract-api.ts"
},
"dependencies": {
"@aws-amplify/ai-constructs": "^0.1.4",
"@aws-amplify/ai-constructs": "0.0.0-test-20240925144932",
"@aws-amplify/graphql-directives": "2.2.0",
"@aws-amplify/graphql-index-transformer": "3.0.3",
"@aws-amplify/graphql-model-transformer": "3.0.3",
Expand Down
Loading
Loading