Removing Log Statements

Author: Swapnil Nagar

src/addBindingName.ts

@@ -16,7 +16,6 @@ export function addBindingName<T extends { type: string; name?: string }>(
     binding: T,
     suffix: string
 ): T & { name: string } {
-    console.log('addBindingName: binding', JSON.stringify(binding));
     if (!binding.name) {
         let bindingType = binding.type;
         if (!bindingType.toLowerCase().endsWith(suffix.toLowerCase())) {

src/converters/fromRpcTypedData.ts

@@ -10,33 +10,23 @@ export function fromRpcTypedData(data: RpcTypedData | null | undefined): unknown
     if (!data) {
         return undefined;
     } else if (isDefined(data.string)) {
-        const result = tryJsonParse(data.string);
-        return result;
+        return tryJsonParse(data.string);
     } else if (isDefined(data.json)) {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-        const result = JSON.parse(data.json);
-        return result;
+        return JSON.parse(data.json);
     } else if (isDefined(data.bytes)) {
-        const result = Buffer.from(data.bytes);
-        return result;
+        return Buffer.from(data.bytes);
     } else if (isDefined(data.stream)) {
-        const result = Buffer.from(data.stream);
-        return result;
+        return Buffer.from(data.stream);
     } else if (isDefined(data.http)) {
-        const result = new HttpRequest(data.http);
-        return result;
+        return new HttpRequest(data.http);
     } else if (isDefined(data.int)) {
-        console.log('Condition: data.int - returning:', data.int);
         return data.int;
     } else if (isDefined(data.double)) {
-        console.log('Condition: data.double - returning:', data.double);
         return data.double;
     } else if (data.collectionBytes && isDefined(data.collectionBytes.bytes)) {
-        const result = data.collectionBytes.bytes.map((d) => Buffer.from(d));
-        return result;
+        return data.collectionBytes.bytes.map((d) => Buffer.from(d));
     } else if (data.collectionString && isDefined(data.collectionString.string)) {
-        const result = data.collectionString.string.map(tryJsonParse);
-        return result;
+        return data.collectionString.string.map(tryJsonParse);
     } else if (data.collectionDouble && isDefined(data.collectionDouble.double)) {
         return data.collectionDouble.double;
     } else if (data.collectionSint64 && isDefined(data.collectionSint64.sint64)) {
@@ -51,9 +41,7 @@ export function fromRpcTypedData(data: RpcTypedData | null | undefined): unknown
 
 function tryJsonParse(data: string): unknown {
     try {
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-        const parsed = JSON.parse(data);
-        return parsed;
+        return JSON.parse(data);
     } catch {
         return data;
     }

src/converters/toRpcTypedData.ts

@@ -1,9 +1,6 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the MIT License.
 
-// Copyright (c) .NET Foundation. All rights reserved.
-// Licensed under the MIT License.
-
 import { RpcTypedData } from '@azure/functions-core';
 
 export function toRpcTypedData(data: unknown): RpcTypedData | null | undefined {
@@ -26,7 +23,6 @@ export function toRpcTypedData(data: unknown): RpcTypedData | null | undefined {
             return { double: data };
         }
     } else {
-        const jsonString = JSON.stringify(data);
-        return { json: jsonString };
+        return { json: JSON.stringify(data) };
     }
 }

src/deferred-binding/connectionDetails.ts

@@ -42,7 +42,6 @@ export function parseConnectionDetails(jsonBuffer: Buffer | null | undefined): B
         throw new Error('Connection details content is null or undefined');
     }
     const parsedObject: unknown = JSON.parse(jsonBuffer.toString());
-    console.log('Parsed object:', parsedObject);
 
     if (isBlobConnectionDetails(parsedObject)) {
         return parsedObject;

src/deferred-binding/storage-blob/azureStorageBlobClientFactory.ts

@@ -64,18 +64,15 @@ export class AzureStorageBlobClientFactory {
             if (!clientId) {
                 throw new Error(`Environment variable ${connectionName}__clientId is not defined.`);
             }
-            console.log(`Using user-assigned managed identity with client ID: ${clientId.substring(0, 8)}...`);
             return new ManagedIdentityUserStrategy(connectionUrl, clientId);
         }
 
         // Next, check for system-assigned managed identity
         if (isSystemBasedManagedIdentity(connectionName)) {
-            console.log('Using system-assigned managed identity for connection');
             return new ManagedIdentitySystemStrategy(connectionUrl);
         }
 
         // Default to connection string
-        console.log('Using connection string authentication');
         return new ConnectionStringStrategy(connectionUrl);
     }
 }

src/trigger.ts

@@ -50,7 +50,6 @@ export function timer(options: TimerTriggerOptions): TimerTrigger {
 }
 
 export function storageBlob(options: StorageBlobTriggerOptions): StorageBlobTrigger {
-    console.log('storageBlob', options);
     return addTriggerBindingName({
         ...options,
         type: 'blobTrigger',
@@ -132,6 +131,5 @@ export function generic(options: GenericTriggerOptions): FunctionTrigger {
 }
 
 function addTriggerBindingName<T extends { type: string; name?: string }>(binding: T): T & { name: string } {
-    console.log('addTriggerBindingName', binding);
     return addBindingName(binding, 'Trigger');
 }

test/converters/fromRpcTypedData.test.ts

@@ -117,9 +117,6 @@ describe('modelBindingData scenario', () => {
     let mockBlobClient: any;
 
     beforeEach(() => {
-        // Suppress console.log output during tests
-        console.log = () => {};
-
         // Create mock blob client for testing
         mockBlobClient = {
             blobClient: { url: 'https://test.blob.core.windows.net/container/blob' },

test/converters/toCoreFunctionMetadata.test.ts

@@ -240,8 +240,6 @@ describe('toCoreFunctionMetadata deferred binding tests', () => {
             return: output.http({}),
         });
 
-        console.log('Blob trigger result: 3 : ', result);
-
         expect(result).to.deep.equal({
             name: 'blobFunction',
             bindings: {
@@ -276,8 +274,6 @@ describe('toCoreFunctionMetadata deferred binding tests', () => {
             ],
         });
 
-        console.log('Blob trigger result: 4 : ', result);
-
         expect(result).to.deep.equal({
             name: 'funcName',
             bindings: {