refactor: remove dotenv usage in apis and update local settings properly

Author: sinedied

packages/agent-api/package.json

@@ -8,11 +8,10 @@
     "build": "tsc",
     "watch": "tsc -w",
     "clean": "rimraf dist",
-    "prestart": "npm run clean && npm run build",
+    "prestart": "npm run clean && npm run update:local-settings && npm run build",
     "start:host": "func start --port 7072",
     "start": "concurrently npm:start:* npm:watch --raw --kill-others",
-    "update:local-settings": "node ./scripts/update-local-settings.mjs",
-    "postinstall": "npm run update:local-settings"
+    "update:local-settings": "node ./scripts/update-local-settings.mjs"
   },
   "author": "Microsoft",
   "license": "MIT",
@@ -31,6 +30,7 @@
   "devDependencies": {
     "@types/node": "^20",
     "azure-functions-core-tools": "^4",
+    "dotenv": "^17.2.1",
     "typescript": "^5"
   }
 }

packages/agent-api/src/functions/chats-delete.ts

@@ -1,7 +1,6 @@
 import process from 'node:process';
 import { HttpRequest, HttpResponseInit, InvocationContext, app } from '@azure/functions';
 import { AzureCosmsosDBNoSQLChatMessageHistory } from '@langchain/azure-cosmosdb';
-import 'dotenv/config';
 import { getCredentials, getUserId } from '../auth.js';
 
 async function deleteChats(request: HttpRequest, context: InvocationContext): Promise<HttpResponseInit> {

packages/agent-api/src/functions/chats-get.ts

@@ -1,7 +1,6 @@
 import process from 'node:process';
 import { HttpRequest, HttpResponseInit, InvocationContext, app } from '@azure/functions';
 import { AzureCosmsosDBNoSQLChatMessageHistory } from '@langchain/azure-cosmosdb';
-import 'dotenv/config';
 import { getCredentials, getUserId } from '../auth.js';
 
 async function getChats(request: HttpRequest, context: InvocationContext): Promise<HttpResponseInit> {

packages/agent-api/src/functions/chats-post.ts

@@ -11,7 +11,6 @@ import { createToolCallingAgent } from 'langchain/agents';
 import { AgentExecutor } from 'langchain/agents';
 import { loadMcpTools } from '@langchain/mcp-adapters';
 import { StreamableHTTPClientTransport } from '@modelcontextprotocol/sdk/client/streamableHttp.js';
-import 'dotenv/config';
 import { getAzureOpenAiTokenProvider, getCredentials, getUserId } from '../auth.js';
 import { Client } from '@modelcontextprotocol/sdk/client/index.js';
 import { ChainValues } from '@langchain/core/utils/types.js';

packages/burger-api/package.json

@@ -8,14 +8,11 @@
     "build": "tsc",
     "watch": "tsc -w",
     "clean": "rimraf dist",
-    "prestart": "npm run clean",
+    "prestart": "npm run clean && npm run update:local-settings",
     "start:storage": "azurite --silent --location ./.azurite --debug ./.azurite/debug.log",
     "start:host": "func start",
     "start": "concurrently npm:start:* npm:watch --raw --kill-others",
-    "update:local-settings": "node ./scripts/update-local-settings.mjs",
-    "postinstall": "npm run update:local-settings",
-    "generate:burgers": "npx -y genaiscript@latest run generate-burgers",
-    "generate:images": "npx -y genaiscript@latest run generate-images"
+    "update:local-settings": "node ./scripts/update-local-settings.mjs"
   },
   "author": "Microsoft",
   "license": "MIT",
@@ -25,14 +22,14 @@
     "@azure/functions": "^4.7.0",
     "@azure/identity": "^4.8.0",
     "@azure/storage-blob": "^12.27.0",
-    "dotenv": "^17.0.0",
     "js-yaml": "^4.1.0"
   },
   "devDependencies": {
     "@types/js-yaml": "^4.0.9",
     "@types/node": "^20",
     "azure-functions-core-tools": "^4",
     "azurite": "^3",
+    "dotenv": "^17.0.0",
     "typescript": "^5"
   }
 }

packages/burger-api/src/blob-service.ts

@@ -3,10 +3,6 @@ import path from 'node:path';
 import { Buffer } from 'node:buffer';
 import { BlobServiceClient, ContainerClient } from '@azure/storage-blob';
 import { DefaultAzureCredential } from '@azure/identity';
-import dotenv from 'dotenv';
-
-// Env file is located in the root of the repository
-dotenv.config({ path: path.join(process.cwd(), '../../.env'), quiet: true });
 
 export class BlobService {
   private static instance: BlobService;

packages/burger-api/src/db-service.ts

@@ -1,16 +1,11 @@
-import path from 'node:path';
 import { Container, CosmosClient, Database } from '@azure/cosmos';
 import { DefaultAzureCredential } from '@azure/identity';
-import dotenv from 'dotenv';
 import burgersData from '../data/burgers.json';
 import toppingsData from '../data/toppings.json';
 import { ToppingCategory, Topping } from './topping.js';
 import { Burger } from './burger.js';
 import { Order, OrderStatus } from './order.js';
 
-// Env file is located in the root of the repository
-dotenv.config({ path: path.join(process.cwd(), '../../.env'), quiet: true });
-
 // Helper to strip properties starting with underscore from an object
 function stripUnderscoreProperties<T extends object>(object: T): T {
   if (!object || typeof object !== 'object') return object;

packages/burger-api/src/functions/openapi-get.ts

@@ -3,10 +3,6 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import { app, type HttpRequest, type InvocationContext } from '@azure/functions';
 import yaml from 'js-yaml';
-import dotenv from 'dotenv';
-
-// Env file is located in the root of the repository
-dotenv.config({ path: path.join(process.cwd(), '../../.env'), quiet: true });
 
 app.http('openapi-get', {
   methods: ['GET'],