fix: Loaded endpoint request fix

Author: shreyashkgupta

extensions/src/SidebarProvider.ts

@@ -16,15 +16,13 @@ export class SidebarProvider implements vscode.WebviewViewProvider {
   private apiHistory: ApiEndpoint[] = [];
   private showHistory: boolean = false;
   private context: vscode.ExtensionContext;
-  private apiRequestView: ApiRequestView;
   private httpClient: HttpClient;
 
   constructor(private readonly _extensionUri: vscode.Uri, context: vscode.ExtensionContext) {
     this.apiRequestProvider = new ApiRequestProvider(context);
     this.context = context;
     this.loadApiHistory();
     this.httpClient = new HttpClient();
-    this.apiRequestView = new ApiRequestView(context, this.httpClient.sendRequest.bind(this.httpClient));
 
     // Add these lines to the constructor
     this.inspectApiHistory();
@@ -108,36 +106,31 @@ export class SidebarProvider implements vscode.WebviewViewProvider {
           break;
         }
         case "openApiManagement": {
-          console.log("API MANAGEMENT CASE");
+
           if (!data.value) {
             return;
           }
           this.openWebview(data.value);
           break;
         }
         case "backButton": {
-          console.log("Going Back");
           this.showHistory = false;
           this.refresh();
           break;
         }
         case "newRequest": {
-          console.log("New request log");
           this.openApiRequestView();
           break;
         }
         case "deleteEndpoint": {
-          console.log("Deleting endpoint");
           this.deleteEndpoint(data.name, data.method);
           break;
         }
         case "selectEndpoint": {
-          console.log("Selection endpoint");
           this.loadEndpoint(data.value);
           break;
         }
         case "exportHistory": {
-          console.log("Export history");
           this.exportHistory();
           break;
         }
@@ -226,7 +219,6 @@ export class SidebarProvider implements vscode.WebviewViewProvider {
   private async openWebview(webviewType: string) {
     if (webviewType === 'apiManagement') {
       if (this.apiRequestProvider) {
-        console.log("Opening API Request View");
         this.apiRequestProvider.openApiRequestView();
         this.showHistory = true;
         this.refresh();
@@ -251,7 +243,6 @@ export class SidebarProvider implements vscode.WebviewViewProvider {
 
   private openApiRequestView() {
     if (this.apiRequestProvider) {
-      console.log("Opening API Request View");
       this.apiRequestProvider.openApiRequestView();
     } else {
       console.error("apiRequestProvider is not initialized");
@@ -348,8 +339,8 @@ export class SidebarProvider implements vscode.WebviewViewProvider {
   private loadEndpoint(endpoint: { method: string, name: string }) {
     const savedEndpoint = this.apiHistory.find(e => e.method === endpoint.method && e.name === endpoint.name);
     if (savedEndpoint) {
-      this.apiRequestView.show();
-      this.apiRequestView.loadEndpoint(savedEndpoint);
+      this.apiRequestProvider.openApiRequestView();
+      this.apiRequestProvider.loadEndpoint(savedEndpoint);
     }
   }
 }

extensions/src/apiRequest/apiRequestProvider.ts

@@ -1,6 +1,7 @@
 import * as vscode from "vscode";
 import { ApiRequestView } from "./apiRequestView";
 import { HttpClient } from "../utils/httpClient";
+import { ApiEndpoint } from "../types";
 
 interface Collection {
   name: string;
@@ -31,7 +32,6 @@ export class ApiRequestProvider {
   }
 
   public openApiRequestView() {
-    console.log("API Request View : This is the request view");
     this.view = new ApiRequestView(this.context, this.handleApiRequest);
     this.view.show();
     this.updateCollectionsView();
@@ -46,18 +46,6 @@ export class ApiRequestProvider {
     body: string,
     bodyType: string
   ): Promise<void> {
-    console.log("Handle API Request called");
-
-    // Log the incoming parameters
-    console.log("Parameters:", {
-      method,
-      url,
-      headers,
-      queryParams,
-      formData,
-      body,
-      bodyType,
-    });
 
     let urlObj: URL;
 
@@ -97,13 +85,6 @@ export class ApiRequestProvider {
       requestBody = body;
     }
 
-    console.log("Final Request:", {
-      url: urlObj.toString(),
-      method,
-      headers,
-      requestBody,
-    });
-
     const startTime = Date.now();
 
     try {
@@ -181,4 +162,10 @@ export class ApiRequestProvider {
   private saveEnvironment() {
     this.context.globalState.update("apiEnvironment", this.environment);
   }
+
+  public loadEndpoint(endpoint: ApiEndpoint) {
+    if (this.view) {
+      this.view.loadEndpoint(endpoint);
+    }
+  }
 }

extensions/src/apiRequest/apiRequestView.ts

@@ -25,7 +25,6 @@ export class ApiRequestView {
     }
 
     public async show() {
-        console.log("Show from View");
             this.panel = vscode.window.createWebviewPanel(
                 'apiRequest',
                 'API Request',
@@ -36,7 +35,6 @@ export class ApiRequestView {
                 }
             );
             const content = await this.getWebviewContent();
-            console.log("Webview Content:", content);
             this.panel.webview.html = content;
 
             this.panel.webview.onDidReceiveMessage(
@@ -95,13 +93,11 @@ export class ApiRequestView {
     }
 
     public loadEndpoint(endpoint: ApiEndpoint) {
-        console.log("Loading Endpoint");
         if (this.panel) {
             this.panel.webview.postMessage({
                 command: 'loadEndpoint',
                 endpoint: endpoint
             });
-            console.log(this.panel.webview.postMessage);
         }
     }
 

extensions/src/utils/httpClient.ts

@@ -1,8 +1,6 @@
 export class HttpClient {
     public async sendRequest(url: string, method: string, headers: Record<string, string>, body?: string | FormData) {
         console.log("Request sent from https client");
-console.log('Sending Request:',{url, method}); // Exclude headers and body from logs
-
         try {
             const options: RequestInit = {
                 method,
@@ -19,18 +17,14 @@ console.log('Sending Request:',{url, method}); // Exclude headers and body from
                 }
             }
 
-            console.log("Request Options:", options);
-
             const response = await fetch(url, options);
-            console.log("Response received:", response);
 
             const responseHeaders: Record<string, string> = {};
             response.headers.forEach((value, key) => {
                 responseHeaders[key] = value;
             });
 
             const responseBody = await response.text(); // Get the response body as text
-            console.log("Response Body:", responseBody);
 
             return {
                 status: response.status,