Skip to content

Commit eef29cb

Browse files
committed
circular dep, inferred troubles
1 parent b9b205b commit eef29cb

File tree

4 files changed

+21
-15
lines changed

4 files changed

+21
-15
lines changed

package-lock.json

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

packages/compass-assistant/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@
5656
"@mongodb-js/compass-telemetry": "^1.14.0",
5757
"@mongodb-js/connection-info": "^0.17.1",
5858
"@mongodb-js/compass-logging": "^1.7.12",
59+
"@mongodb-js/compass-generative-ai": "^0.51.0",
5960
"mongodb-connection-string-url": "^3.0.1",
6061
"ai": "^5.0.26",
6162
"compass-preferences-model": "^2.51.0",

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

Lines changed: 17 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -84,23 +84,26 @@ export function useAssistantActions(): AssistantActionsContextType & {
8484
};
8585
}
8686

87-
export const compassAssistantServiceLocator = createServiceLocator(function () {
88-
const { isAssistantEnabled, ...actions } = useAssistantActions();
87+
export type CompassAssistantService = AssistantActionsContextType & {
88+
getIsAssistantEnabled: () => boolean;
89+
};
8990

90-
const assistantEnabledRef = useRef(isAssistantEnabled);
91-
assistantEnabledRef.current = isAssistantEnabled;
91+
export const compassAssistantServiceLocator = createServiceLocator(
92+
function (): CompassAssistantService {
93+
const { isAssistantEnabled, ...actions } = useAssistantActions();
9294

93-
return {
94-
...actions,
95-
getIsAssistantEnabled() {
96-
return assistantEnabledRef.current;
97-
},
98-
};
99-
}, 'compassAssistantLocator');
95+
const assistantEnabledRef = useRef(isAssistantEnabled);
96+
assistantEnabledRef.current = isAssistantEnabled;
10097

101-
export type CompassAssistantService = ReturnType<
102-
typeof compassAssistantServiceLocator
103-
>;
98+
return {
99+
...actions,
100+
getIsAssistantEnabled() {
101+
return assistantEnabledRef.current;
102+
},
103+
};
104+
},
105+
'compassAssistantLocator'
106+
);
104107

105108
export const AssistantProvider: React.FunctionComponent<
106109
PropsWithChildren<{

packages/compass-generative-ai/src/atlas-ai-service.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import {
55
} from 'compass-preferences-model/provider';
66
import type { AtlasService } from '@mongodb-js/atlas-service/provider';
77
import { AtlasServiceError } from '@mongodb-js/atlas-service/renderer';
8-
import type { ConnectionInfo } from '@mongodb-js/compass-connections/provider';
8+
import type { ConnectionInfo } from '@mongodb-js/connection-info';
99
import type { Document } from 'mongodb';
1010
import type { Logger } from '@mongodb-js/compass-logging';
1111
import { EJSON } from 'bson';

0 commit comments

Comments
 (0)