Skip to content

Commit 13619e2

Browse files
committed
use the correct / agreed upon X-Request-Origin header
1 parent 5ec82fd commit 13619e2

File tree

6 files changed

+24
-9
lines changed

6 files changed

+24
-9
lines changed

packages/compass-assistant/src/compass-assistant-provider.spec.tsx

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,11 @@ const TestComponent: React.FunctionComponent<{
8585

8686
return (
8787
<DrawerContentProvider>
88-
<MockedProvider appNameForPrompt="MongoDB Compass" chat={chat}>
88+
<MockedProvider
89+
originForPrompt="mongodb-compass"
90+
appNameForPrompt="MongoDB Compass"
91+
chat={chat}
92+
>
8993
<DrawerAnchor>
9094
<div data-testid="provider-children">Provider children</div>
9195
<CompassAssistantDrawer
@@ -106,7 +110,11 @@ describe('useAssistantActions', function () {
106110

107111
return (
108112
<DrawerContentProvider>
109-
<MockedProvider appNameForPrompt="MongoDB Compass" chat={chat}>
113+
<MockedProvider
114+
originForPrompt="mongodb-compass"
115+
appNameForPrompt="MongoDB Compass"
116+
chat={chat}
117+
>
110118
{children}
111119
</MockedProvider>
112120
</DrawerContentProvider>
@@ -659,7 +667,10 @@ describe('CompassAssistantProvider', function () {
659667
render(
660668
<DrawerContentProvider>
661669
<DrawerAnchor />
662-
<MockedProvider appNameForPrompt="MongoDB Compass" />
670+
<MockedProvider
671+
originForPrompt="mongodb-compass"
672+
appNameForPrompt="MongoDB Compass"
673+
/>
663674
</DrawerContentProvider>,
664675
{
665676
preferences: {

packages/compass-assistant/src/compass-assistant-provider.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -256,6 +256,7 @@ export const CompassAssistantProvider = registerCompassPlugin(
256256
children,
257257
}: PropsWithChildren<{
258258
appNameForPrompt: string;
259+
originForPrompt: string;
259260
chat?: Chat<AssistantMessage>;
260261
atlasAiService?: AtlasAiService;
261262
}>) => {
@@ -280,7 +281,7 @@ export const CompassAssistantProvider = registerCompassPlugin(
280281
initialProps.chat ??
281282
new Chat({
282283
transport: new DocsProviderTransport({
283-
appName: initialProps.appNameForPrompt,
284+
origin: initialProps.originForPrompt,
284285
instructions: buildConversationInstructionsPrompt({
285286
target: initialProps.appNameForPrompt,
286287
}),

packages/compass-assistant/src/docs-provider-transport.spec.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ describe('DocsProviderTransport', function () {
6262
});
6363
abortController = new AbortController();
6464
transport = new DocsProviderTransport({
65+
origin: 'mongodb-compass',
6566
instructions: 'Test instructions for MongoDB assistance',
6667
model: mockModel,
6768
});

packages/compass-assistant/src/docs-provider-transport.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,21 +17,21 @@ export function shouldExcludeMessage({ metadata }: AssistantMessage) {
1717

1818
export class DocsProviderTransport implements ChatTransport<AssistantMessage> {
1919
private model: LanguageModel;
20-
private appName: string;
20+
private origin: string;
2121
private instructions: string;
2222

2323
constructor({
2424
instructions,
2525
model,
26-
appName,
26+
origin,
2727
}: {
2828
instructions: string;
2929
model: LanguageModel;
30-
appName: string;
30+
origin: string;
3131
}) {
3232
this.instructions = instructions;
3333
this.model = model;
34-
this.appName = appName;
34+
this.origin = origin;
3535
}
3636

3737
static emptyStream = new ReadableStream<UIMessageChunk>({
@@ -65,7 +65,7 @@ export class DocsProviderTransport implements ChatTransport<AssistantMessage> {
6565
messages: convertToModelMessages(filteredMessages),
6666
abortSignal: abortSignal,
6767
headers: {
68-
'X-Request-Origin': this.appName,
68+
'X-Request-Origin': this.origin,
6969
},
7070
providerOptions: {
7171
openai: {

packages/compass-web/src/entrypoint.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -502,6 +502,7 @@ const CompassWeb = ({
502502
>
503503
<CompassInstanceStorePlugin>
504504
<CompassAssistantProvider
505+
originForPrompt="atlas-data-explorer"
505506
appNameForPrompt={
506507
APP_NAMES_FOR_PROMPT.DataExplorer
507508
}

packages/compass/src/app/components/home.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,7 @@ function HomeWithConnections({
151151
<ConnectionStorageProvider value={connectionStorage}>
152152
<FileInputBackendProvider createFileInputBackend={createFileInputBackend}>
153153
<CompassAssistantProvider
154+
originForPrompt="mongodb-compass"
154155
appNameForPrompt={APP_NAMES_FOR_PROMPT.Compass}
155156
>
156157
<CompassConnections

0 commit comments

Comments
 (0)