Pass dependency name around when automodeling (#2681)

This commit is contained in:
Charis Kyriakou
2023-08-09 12:39:05 +01:00
committed by GitHub
parent 6dc0b14be7
commit 6791ddb445
6 changed files with 19 additions and 3 deletions

View File

@@ -549,6 +549,7 @@ interface GenerateExternalApiMessage {
interface GenerateExternalApiFromLlmMessage {
t: "generateExternalApiFromLlm";
dependencyName: string;
externalApiUsages: ExternalApiUsage[];
modeledMethods: Record<string, ModeledMethod>;
}

View File

@@ -29,18 +29,26 @@ export class AutoModeler {
) {}
public async startModeling(
dependency: string,
externalApiUsages: ExternalApiUsage[],
modeledMethods: Record<string, ModeledMethod>,
mode: Mode,
): Promise<void> {
await this.modelDependency(externalApiUsages, modeledMethods, mode);
await this.modelDependency(
dependency,
externalApiUsages,
modeledMethods,
mode,
);
}
private async modelDependency(
dependency: string,
externalApiUsages: ExternalApiUsage[],
modeledMethods: Record<string, ModeledMethod>,
mode: Mode,
): Promise<void> {
void extLogger.log(`Modeling dependency ${dependency}`);
await withProgress(async (progress) => {
const maxStep = 3000;

View File

@@ -170,6 +170,7 @@ export class DataExtensionsEditorView extends AbstractWebview<
case "generateExternalApiFromLlm":
if (useLlmGenerationV2()) {
await this.generateModeledMethodsFromLlmV2(
msg.dependencyName,
msg.externalApiUsages,
msg.modeledMethods,
);
@@ -459,10 +460,12 @@ export class DataExtensionsEditorView extends AbstractWebview<
}
private async generateModeledMethodsFromLlmV2(
dependency: string,
externalApiUsages: ExternalApiUsage[],
modeledMethods: Record<string, ModeledMethod>,
): Promise<void> {
await this.autoModeler.startModeling(
dependency,
externalApiUsages,
modeledMethods,
this.mode,

View File

@@ -227,11 +227,13 @@ export function DataExtensionsEditor({
const onGenerateFromLlmClick = useCallback(
(
dependencyName: string,
externalApiUsages: ExternalApiUsage[],
modeledMethods: Record<string, ModeledMethod>,
) => {
vscode.postMessage({
t: "generateExternalApiFromLlm",
dependencyName,
externalApiUsages,
modeledMethods,
});

View File

@@ -85,6 +85,7 @@ type Props = {
modeledMethods: Record<string, ModeledMethod>,
) => void;
onGenerateFromLlmClick: (
dependencyName: string,
externalApiUsages: ExternalApiUsage[],
modeledMethods: Record<string, ModeledMethod>,
) => void;
@@ -119,11 +120,11 @@ export const LibraryRow = ({
const handleModelWithAI = useCallback(
async (e: React.MouseEvent) => {
onGenerateFromLlmClick(externalApiUsages, modeledMethods);
onGenerateFromLlmClick(title, externalApiUsages, modeledMethods);
e.stopPropagation();
e.preventDefault();
},
[externalApiUsages, modeledMethods, onGenerateFromLlmClick],
[title, externalApiUsages, modeledMethods, onGenerateFromLlmClick],
);
const handleModelFromSource = useCallback(

View File

@@ -27,6 +27,7 @@ type Props = {
modeledMethods: Record<string, ModeledMethod>,
) => void;
onGenerateFromLlmClick: (
dependencyName: string,
externalApiUsages: ExternalApiUsage[],
modeledMethods: Record<string, ModeledMethod>,
) => void;