Skip to content

Commit 6bcff83

Browse files
committed
feat(amazonq): Handle disclaimer acknowledgement
1 parent 72bca8c commit 6bcff83

File tree

7 files changed

+29
-24
lines changed

7 files changed

+29
-24
lines changed

package-lock.json

Lines changed: 25 additions & 8 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/amazonq/package.json

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -127,10 +127,6 @@
127127
"amazonQWorkspaceLspManifestMessage": {
128128
"type": "boolean",
129129
"default": false
130-
},
131-
"amazonQChatDisclaimerAcknowledged": {
132-
"type": "boolean",
133-
"default": false
134130
}
135131
},
136132
"additionalProperties": false

packages/amazonq/src/app/inline/sessionManager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* SPDX-License-Identifier: Apache-2.0
44
*/
55

6-
import { InlineCompletionItemWithReferences } from '@aws/language-server-runtimes-types'
6+
import { InlineCompletionItemWithReferences } from '@aws/language-server-runtimes-types/inlineCompletionWithReferences'
77

88
// TODO: add more needed data to the session interface
99
interface CodeWhispererSession {

packages/amazonq/src/lsp/chat/messages.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import {
1010
CHAT_OPTIONS,
1111
COPY_TO_CLIPBOARD,
1212
AuthFollowUpType,
13-
DISCLAIMER_ACKNOWLEDGED,
1413
} from '@aws/chat-client-ui-types'
1514
import {
1615
ChatResult,
@@ -108,7 +107,8 @@ export function registerMessageListeners(
108107
}
109108
break
110109
}
111-
case DISCLAIMER_ACKNOWLEDGED: {
110+
// const DISCLAIMER_ACKNOWLEDGED: "disclaimerAcknowledged"
111+
case 'disclaimerAcknowledged': {
112112
void AmazonQPromptSettings.instance.update('amazonQChatDisclaimerAcknowledged', true)
113113
break
114114
}

packages/core/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -440,7 +440,7 @@
440440
},
441441
"devDependencies": {
442442
"@aws-sdk/types": "^3.13.1",
443-
"@aws/chat-client-ui-types": "^0.1.12",
443+
"@aws/chat-client-ui-types": "^0.0.8",
444444
"@aws/language-server-runtimes": "^0.2.49",
445445
"@aws/language-server-runtimes-types": "^0.1.10",
446446
"@cspotcode/source-map-support": "^0.8.1",

packages/core/src/amazonq/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,6 @@ export * from './lsp/config'
4848
export * as WorkspaceLspInstaller from './lsp/workspaceInstaller'
4949
export * as secondaryAuth from '../auth/secondaryAuth'
5050
export * as authConnection from '../auth/connection'
51-
export * as disclaimer from './util/disclaimer'
5251
import { FeatureContext } from '../shared/featureConfig'
5352

5453
/**

packages/webpack.web.config.js

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -50,13 +50,6 @@ module.exports = (env, argv) => {
5050
new webpack.IgnorePlugin({
5151
resourceRegExp: /ps-list/, // matches the path in the require() statement
5252
}),
53-
/**
54-
* HACK: the glob module breaks Web mode if imported, BUT we still dynamically import this module for non web mode
55-
* environments. The following allows compilation to pass by never bundling the module in the final output for web mode.
56-
*/
57-
new webpack.IgnorePlugin({
58-
resourceRegExp: /glob/, // matches the path in the require() statement
59-
}),
6053
],
6154
resolve: {
6255
extensions: ['.ts', '.js'],

0 commit comments

Comments
 (0)