@pagopa/dx-mcpserver 0.0.1 → 0.0.3

package/README.md

@@ -4,17 +4,6 @@
 
 This package contains the implementation of a Model Context Protocol (MCP) server.
 
-## Architecture
-
-The architecture allows any Model Context Protocol (MCP) compliant client (such as GitHub Copilot) to query the [PagoPA DX technical documentation](https://dx.pagopa.it/) in natural language, receiving contextualized and up-to-date answers.
-
-1.  **Content Upload**: On each release of the documentation website, Markdown and text files (`.md`, `.txt`) are uploaded to an S3 bucket.
-2.  **Indexing**: From there, the documents are processed by **Amazon Bedrock Knowledge Bases**, which handles the embedding and semantic indexing process.
-3.  **Vector Storage**: The resulting embeddings are saved in a Vector Bucket (an S3-based vector database), enabling efficient and persistent semantic search.
-4.  **Query and Retrieval**: When an MCP client sends a query, an **AWS Lambda** function implementing the MCP Server queries the Knowledge Base to retrieve the most relevant content and returns the response to the client.
-
-This approach allows AI agents like Copilot to access the documentation context in a structured way, keeping the orchestration, storage, and semantic retrieval layers separate.
-
 ## Features
 
 The server currently exposes the following capabilities:
@@ -25,35 +14,50 @@ The server currently exposes the following capabilities:
 - **Prompts**:
   - `GenerateTerraformConfiguration`: Guides the generation of Terraform configurations following PagoPA DX best practices.
 
-## How to use it
+## Authentication
+
+The server requires a fine-grained [GitHub Personal Access Token](https://github.com/settings/personal-access-tokens) for authentication with the following settings:
+
+- **Resource owner:**
+  - Choose the **pagopa** organization
+- **Repository access:**
+  - Public Repositories (read-only)
+- **Organization permissions:**
+  - Members: Read-only (to verify membership in the pagopa organization)
+
+## Usage
 
 This server can be used by any MCP-compliant client.
 
-<details>
-<summary><b>VS Code</b></summary>
+### VS Code
 
 Update your configuration file with the following. See [VS Code MCP docs](https://code.visualstudio.com/docs/copilot/chat/mcp-servers) for more info.
-
-#### VS Code Remote Server Connection
+The GH PAT authentication is done via a prompt, so you will be asked to enter it the first time you use the server.
 
 ```json
 {
   "servers": {
     "dx-docs": {
-      "url": "https://api.dev.dx.pagopa.it/mcp",
+      "url": "https://api.dx.pagopa.it/mcp",
       "type": "http",
       "headers": {
-        "x-gh-pat": "${env:GH_PAT}"
+        "x-gh-pat": "${input:github_mcp_pat}"
       }
-    }
+    },
+    "inputs": [
+      {
+        "type": "promptString",
+        "id": "github_mcp_pat",
+        "description": "GitHub Personal Access Token",
+        "password": true
+      }
+    ]
   }
 }
 ```
 
-</details>
+### GitHub Copilot Coding Agent
 
-<details>
-<summary><b>GitHub Copilot Coding Agent</b></summary>
 You need to configure it in the repository settings. See [GitHub Copilot MCP docs](https://docs.github.com/en/copilot/how-tos/use-copilot-agents/coding-agent/extend-coding-agent-with-mcp) for more info.
 
 1.  **Declare the MCP Server**: In the "Copilot" >> "Coding agent" panel of your repository settings, add an MCP Server declaration as follows:
@@ -62,7 +66,7 @@ You need to configure it in the repository settings. See [GitHub Copilot MCP doc
 {
   "mcpServers": {
     "pagopa-dx": {
-      "url": "https://api.dev.dx.pagopa.it/mcp",
+      "url": "https://api.dx.pagopa.it/mcp",
       "type": "http",
       "tools": ["*"],
       "headers": {
@@ -77,7 +81,19 @@ You need to configure it in the repository settings. See [GitHub Copilot MCP doc
 
 Once configured, Copilot can autonomously invoke the MCP server's tools during task execution, using it to access documentation context and improve the quality of its code generation.
 
-</details>
+### GitHub Copilot CLI
+
+To use the MCP server with [GitHub Copilot CLI](https://github.com/features/copilot/cli/), run the cli with `copilot` and prompt `/mcp add` to start the configuration of the MCP server
+
+Follow the guided wizard to start using the DX MCP server:
+
+1. **Server Name**: `dx-docs`
+2. **Server Type**: `2` (HTTP)
+3. **URL**: `https://api.dx.pagopa.it/mcp`
+4. **HTTP Headers**: `{"x-gh-pat": "<your-gh-PAT>"}`
+5. **Tools**: `*` (leave as is)
+
+Use `Tab` to navigate between fields and `Ctrl+S` to save.
 
 ## Development
 
@@ -102,3 +118,14 @@ To build the Docker container for this application, run the following command fr
 ```bash
 docker build -t dx/mcp-server -f ./apps/mcpserver/Dockerfile .
 ```
+
+## Architecture
+
+The architecture allows any Model Context Protocol (MCP) compliant client (such as GitHub Copilot) to query the [PagoPA DX technical documentation](https://dx.pagopa.it/) in natural language, receiving contextualized and up-to-date answers.
+
+1.  **Content Upload**: On each release of the documentation website, Markdown and text files (`.md`, `.txt`) are uploaded to an S3 bucket.
+2.  **Indexing**: From there, the documents are processed by **Amazon Bedrock Knowledge Bases**, which handles the embedding and semantic indexing process.
+3.  **Vector Storage**: The resulting embeddings are saved in a Vector Bucket (an S3-based vector database), enabling efficient and persistent semantic search.
+4.  **Query and Retrieval**: When an MCP client sends a query, an **AWS Lambda** function implementing the MCP Server queries the Knowledge Base to retrieve the most relevant content and returns the response to the client.
+
+This approach allows AI agents like Copilot to access the documentation context in a structured way, keeping the orchestration, storage, and semantic retrieval layers separate.

package/dist/auth/__tests__/githubAuth.test.js

@@ -1,12 +1,20 @@
+import { getLogger } from "@logtape/logtape";
 import { Octokit } from "@octokit/rest";
 import { beforeEach, describe, expect, it, vi } from "vitest";
-import { logger } from "../../utils/logger.js";
 import * as githubAuth from "../github.js";
 vi.mock("@octokit/rest");
 describe("verifyGithubUser", () => {
+    let loggerSpy;
     beforeEach(() => {
         vi.clearAllMocks();
         process.env.REQUIRED_ORGANIZATIONS = "pagopa";
+        // Get logger and spy on its methods - no need for special configuration
+        const logger = getLogger(["mcpserver", "github-auth"]);
+        loggerSpy = {
+            debug: vi.spyOn(logger, "debug"),
+            error: vi.spyOn(logger, "error"),
+            warn: vi.spyOn(logger, "warn"),
+        };
     });
     it("returns false if no token is provided", async () => {
         const result = await githubAuth.verifyGithubUser("");
@@ -22,10 +30,9 @@ describe("verifyGithubUser", () => {
                 },
             },
         }));
-        const errorLog = vi.spyOn(logger, "error");
         const result = await githubAuth.verifyGithubUser("token");
         expect(result).toBe(false);
-        expect(errorLog).toHaveBeenCalledWith(expect.any(Error), "Error verifying GitHub organization membership:");
+        expect(loggerSpy.error).toHaveBeenCalledWith("Error verifying GitHub organization membership", { error: expect.any(Error) });
     });
     it("returns false if user is not member of required org", async () => {
         vi.mocked(Octokit).mockImplementation(() => ({
@@ -39,6 +46,7 @@ describe("verifyGithubUser", () => {
         }));
         const result = await githubAuth.verifyGithubUser("token");
         expect(result).toBe(false);
+        expect(loggerSpy.warn).toHaveBeenCalledWith("User is not a member of any of the required organizations: pagopa");
     });
     it("returns true if user is member of required org", async () => {
         vi.mocked(Octokit).mockImplementation(() => ({
@@ -52,5 +60,6 @@ describe("verifyGithubUser", () => {
         }));
         const result = await githubAuth.verifyGithubUser("token");
         expect(result).toBe(true);
+        expect(loggerSpy.debug).toHaveBeenCalledWith("User is a member of one of the required organizations: pagopa");
     });
 });

package/dist/auth/github.js

@@ -1,6 +1,6 @@
+import { getLogger } from "@logtape/logtape";
 import { Octokit } from "@octokit/rest";
 import { z } from "zod/v4";
-import { logger } from "../utils/logger.js";
 const organizationsSchema = z
     .array(z.string().nonempty())
     .nonempty()
@@ -14,6 +14,7 @@ const REQUIRED_ORGANIZATIONS = organizationsSchema.parse(process.env.REQUIRED_OR
  * @returns A boolean indicating whether the user is a member of a required organization.
  */
 export async function verifyGithubUser(token) {
+    const logger = getLogger(["mcpserver", "github-auth"]);
     if (!token) {
         return false;
     }
@@ -23,7 +24,7 @@ export async function verifyGithubUser(token) {
         const { data: organizations } = await octokit.rest.orgs.listForAuthenticatedUser();
         const isMember = organizations.some((org) => REQUIRED_ORGANIZATIONS.includes(org.login));
         if (isMember) {
-            logger.info(`User is a member of one of the required organizations: ${REQUIRED_ORGANIZATIONS.join(", ")}`);
+            logger.debug(`User is a member of one of the required organizations: ${REQUIRED_ORGANIZATIONS.join(", ")}`);
         }
         else {
             logger.warn(`User is not a member of any of the required organizations: ${REQUIRED_ORGANIZATIONS.join(", ")}`);
@@ -31,7 +32,7 @@ export async function verifyGithubUser(token) {
         return isMember;
     }
     catch (error) {
-        logger.error(error, "Error verifying GitHub organization membership:");
+        logger.error("Error verifying GitHub organization membership", { error });
         return false;
     }
 }

package/dist/config/__tests__/awsConfig.test.js

@@ -1,11 +1,4 @@
-import { describe, expect, it, vi } from "vitest";
-vi.mock("../../utils/logger", () => ({
-    logger: {
-        error: vi.fn(),
-        info: vi.fn(),
-        warn: vi.fn(),
-    },
-}));
+import { describe, expect, it } from "vitest";
 import * as awsConfig from "../aws.js";
 describe("aws config", () => {
     it("should export kbRerankingEnabled as boolean", () => {

package/dist/config/aws.js

@@ -1,10 +1,11 @@
 import { BedrockAgentRuntimeClient } from "@aws-sdk/client-bedrock-agent-runtime";
-import { logger } from "../utils/logger.js";
+import { getLogger } from "@logtape/logtape";
+const logger = getLogger(["mcpserver", "aws-config"]);
 // When true, enables reranking for the Bedrock knowledge base queries.
 export const kbRerankingEnabled = (process.env.BEDROCK_KB_RERANKING_ENABLED || "true").trim().toLowerCase() ===
     "true";
 export const knowledgeBaseId = process.env.BEDROCK_KNOWLEDGE_BASE_ID || "";
-logger.info(`Default reranking enabled: ${kbRerankingEnabled} (from BEDROCK_KB_RERANKING_ENABLED)`);
+logger.debug(`Default reranking enabled: ${kbRerankingEnabled} (from BEDROCK_KB_RERANKING_ENABLED)`);
 export const region = process.env.AWS_REGION || "eu-central-1";
 // List of AWS regions that support reranking
 export const rerankingSupportedRegions = [
@@ -20,7 +21,7 @@ try {
     kbRuntimeClient = new BedrockAgentRuntimeClient({ region });
 }
 catch (e) {
-    logger.error(e, "Error getting bedrock agent client");
+    logger.error("Error getting bedrock agent client", { error: e });
     process.exit(1);
 }
 export { kbRuntimeClient };

package/dist/config/logging.js

@@ -0,0 +1,56 @@
+/**
+ * LogTape logging configuration for the MCP server.
+ *
+ * Configures logging for both the mcpserver and mcp-prompts packages.
+ * The console sink works perfectly in all environments:
+ * - AWS Lambda: stdout → CloudWatch
+ * - Local: stdout → console
+ *
+ * Log level is controlled via the LOG_LEVEL environment variable.
+ *
+ * Usage:
+ * Instead of importing a global logger, use getLogger directly in each module:
+ *
+ * import { getLogger } from "@logtape/logtape";
+ * const logger = getLogger(["mcpserver", "module-name"]);
+ * logger.info("Message");
+ */
+import { configure, getConsoleSink } from "@logtape/logtape";
+import { z } from "zod";
+/**
+ * Zod schema for validating log levels.
+ * Based on LogTape's LogLevel type.
+ */
+const DEFAULT_LOG_LEVEL = "info";
+const logLevelSchema = z
+    .enum(["debug", "info", "warning", "error"])
+    .catch(DEFAULT_LOG_LEVEL);
+export async function configureLogging() {
+    const logLevel = logLevelSchema.parse(process.env.LOG_LEVEL);
+    if (logLevel !== process.env.LOG_LEVEL) {
+        // Use console.warn for this early logging before LogTape is configured
+        console.warn(`Invalid log level: ${process.env.LOG_LEVEL}. Using ${DEFAULT_LOG_LEVEL}`);
+    }
+    await configure({
+        loggers: [
+            {
+                category: ["mcpserver"],
+                lowestLevel: logLevel,
+                sinks: ["console"],
+            },
+            {
+                category: ["mcp-prompts"],
+                lowestLevel: logLevel,
+                sinks: ["console"],
+            },
+            {
+                category: ["logtape", "meta"],
+                lowestLevel: "warning",
+                sinks: ["console"],
+            },
+        ],
+        sinks: {
+            console: getConsoleSink(),
+        },
+    });
+}

package/dist/config/monitoring.js

@@ -0,0 +1,26 @@
+/**
+ * Azure Application Insights monitoring configuration for the MCP server.
+ *
+ * This configuration works in any environment (AWS Lambda, Azure Functions, local, etc.)
+ * and sends telemetry data to Azure Application Insights.
+ *
+ * Required environment variable:
+ * - APPLICATIONINSIGHTS_CONNECTION_STRING: The connection string for Azure Application Insights
+ *
+ * Optional environment variable:
+ * - APPINSIGHTS_SAMPLING_PERCENTAGE: Sampling percentage (0-100), defaults to 5
+ */
+import { getLogger } from "@logtape/logtape";
+import { initAzureMonitor } from "@pagopa/azure-tracing/azure-monitor";
+const logger = getLogger(["mcpserver", "monitoring"]);
+export function configureAzureMonitoring() {
+    try {
+        // Initialize Azure Monitor using environment variables
+        // This will read APPLICATIONINSIGHTS_CONNECTION_STRING and APPINSIGHTS_SAMPLING_PERCENTAGE
+        initAzureMonitor();
+        logger.info("Azure Application Insights monitoring configured successfully");
+    }
+    catch (error) {
+        logger.warn(`Failed to configure Azure monitoring: ${error instanceof Error ? error.message : String(error)}. Custom events will not be sent to Application Insights.`);
+    }
+}

package/dist/decorators/promptUsageMonitoring.js

@@ -0,0 +1,31 @@
+import { getLogger } from "@logtape/logtape";
+import { emitCustomEvent } from "@pagopa/azure-tracing/logger";
+const logger = getLogger(["mcpserver", "prompt-logging"]);
+/**
+ * Decorator that adds logging to prompt load functions.
+ * Logs when a prompt is requested to both console and Azure Application Insights.
+ */
+export function withPromptLogging(prompt, catalogId) {
+    const originalLoad = prompt.load;
+    return {
+        ...prompt,
+        load: async (args) => {
+            const eventData = {
+                arguments: JSON.stringify(args),
+                promptId: catalogId,
+                promptName: prompt.name,
+                timestamp: new Date().toISOString(),
+            };
+            // Log to console (goes to CloudWatch in Lambda)
+            logger.debug(`Prompt requested: ${prompt.name} - ${JSON.stringify(eventData)}`);
+            // Emit custom event to Azure Application Insights
+            emitCustomEvent("PromptRequested", {
+                arguments: JSON.stringify(args),
+                promptId: catalogId,
+                promptName: prompt.name,
+            })("mcpserver");
+            // Call the original load function and return the result
+            return await originalLoad(args);
+        },
+    };
+}

package/dist/decorators/toolUsageMonitoring.js

@@ -0,0 +1,56 @@
+import { getLogger } from "@logtape/logtape";
+import { emitCustomEvent } from "@pagopa/azure-tracing/logger";
+const logger = getLogger(["mcpserver", "tool-logging"]);
+/**
+ * Decorator that adds logging to tool execute functions.
+ * Logs when a tool is executed to both console and Azure Application Insights.
+ * Preserves the exact original function signature and type.
+ */
+export function withToolLogging(tool) {
+    if (typeof tool !== "object" || !tool.execute || !tool.name) {
+        return tool;
+    }
+    const originalExecute = tool.execute;
+    const toolName = tool.name;
+    return {
+        ...tool,
+        execute: async (...args) => {
+            const startTime = Date.now();
+            const [toolArgs] = args;
+            const eventData = {
+                arguments: JSON.stringify(toolArgs),
+                timestamp: new Date().toISOString(),
+                toolName,
+            };
+            // Log to console (goes to CloudWatch in Lambda)
+            logger.debug(`Tool executed: ${toolName} - ${JSON.stringify(eventData)}`);
+            // Emit custom event to Azure Application Insights
+            emitCustomEvent("ToolExecuted", eventData)("mcpserver");
+            try {
+                // Call the original execute function and return the result
+                const result = await originalExecute(...args);
+                const executionTime = Date.now() - startTime;
+                // Log successful completion
+                logger.debug(`Tool completed: ${toolName} - execution time: ${executionTime}ms`);
+                // Emit completion event to Azure Application Insights
+                emitCustomEvent("ToolCompleted", {
+                    executionTimeMs: executionTime.toString(),
+                    toolName,
+                })("mcpserver");
+                return result;
+            }
+            catch (error) {
+                const executionTime = Date.now() - startTime;
+                // Log error
+                logger.error(`Tool failed: ${toolName} - ${error instanceof Error ? error.message : String(error)} - execution time: ${executionTime}ms`);
+                // Emit error event to Azure Application Insights
+                emitCustomEvent("ToolFailed", {
+                    error: error instanceof Error ? error.message : String(error),
+                    executionTimeMs: executionTime.toString(),
+                    toolName,
+                })("mcpserver");
+                throw error;
+            }
+        },
+    };
+}

package/dist/index.js

@@ -1,10 +1,19 @@
+import { getLogger } from "@logtape/logtape";
+import { getEnabledPrompts } from "@pagopa/dx-mcpprompts";
 import { FastMCP } from "fastmcp";
 import { verifyGithubUser } from "./auth/github.js";
+import { configureLogging } from "./config/logging.js";
+import { configureAzureMonitoring } from "./config/monitoring.js";
 import { serverInstructions } from "./config/server.js";
-import { GenerateTerraformConfigurationPrompt } from "./prompts/GenerateTerraformConfiguration.js";
+import { withPromptLogging } from "./decorators/promptUsageMonitoring.js";
+import { withToolLogging } from "./decorators/toolUsageMonitoring.js";
 import { QueryPagoPADXDocumentationTool } from "./tools/QueryPagoPADXDocumentation.js";
 import { SearchGitHubCodeTool } from "./tools/SearchGitHubCode.js";
-import { logger } from "./utils/logger.js";
+// Configure logging
+await configureLogging();
+await configureAzureMonitoring();
+const logger = getLogger(["mcpserver"]);
+logger.info("MCP Server starting...");
 // Authentication is enabled based on the AUTH_REQUIRED environment variable.
 const server = new FastMCP({
     authenticate: async (request) => {
@@ -31,9 +40,18 @@ const server = new FastMCP({
     version: "0.0.0",
 });
 logger.debug(`Server instructions: \n\n${serverInstructions}`);
-server.addPrompt(GenerateTerraformConfigurationPrompt);
-server.addTool(QueryPagoPADXDocumentationTool);
-server.addTool(SearchGitHubCodeTool);
+logger.debug(`Loading enabled prompts...`);
+getEnabledPrompts().then((prompts) => {
+    prompts.forEach((catalogEntry) => {
+        logger.debug(`Adding prompt: ${catalogEntry.prompt.name}`);
+        // Apply logging decorator to the prompt
+        const decoratedPrompt = withPromptLogging(catalogEntry.prompt, catalogEntry.id);
+        server.addPrompt(decoratedPrompt);
+        logger.debug(`Added prompt: ${catalogEntry.prompt.name}`);
+    });
+});
+server.addTool(withToolLogging(QueryPagoPADXDocumentationTool));
+server.addTool(withToolLogging(SearchGitHubCodeTool));
 // Starts the server in HTTP Stream mode.
 server.start({
     httpStream: {

package/dist/services/__tests__/bedrock.test.js

@@ -1,13 +1,16 @@
-import { describe, expect, it, vi } from "vitest";
-vi.mock("../../utils/logger", () => ({
-    logger: {
-        error: vi.fn(),
-        info: vi.fn(),
-        warn: vi.fn(),
-    },
-}));
+import { getLogger } from "@logtape/logtape";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { queryKnowledgeBase } from "../bedrock.js";
 describe("queryKnowledgeBase", () => {
+    let loggerSpy;
+    beforeEach(() => {
+        vi.clearAllMocks();
+        // Get logger and spy on its methods - no need for special configuration
+        const logger = getLogger(["mcpserver", "bedrock"]);
+        loggerSpy = {
+            warn: vi.spyOn(logger, "warn"),
+        };
+    });
     it("should skip images in results", async () => {
         const mockClient = {
             config: {
@@ -26,6 +29,7 @@ describe("queryKnowledgeBase", () => {
         };
         const result = await queryKnowledgeBase("kbId", "query", mockClient, 2, false);
         expect(result).toContain("doc1");
+        expect(loggerSpy.warn).toHaveBeenCalledWith("Images are not supported at this time. Skipping...");
     });
     it("should warn if reranking is not supported in region", async () => {
         const mockClient = {
@@ -40,5 +44,6 @@ describe("queryKnowledgeBase", () => {
         };
         const result = await queryKnowledgeBase("kbId", "query", mockClient, 2, true);
         expect(typeof result).toBe("string");
+        expect(loggerSpy.warn).toHaveBeenCalledWith("Reranking is not supported in region unsupported-region");
     });
 });

package/dist/services/bedrock.js

@@ -1,6 +1,6 @@
 import { RetrieveCommand, } from "@aws-sdk/client-bedrock-agent-runtime";
+import { getLogger } from "@logtape/logtape";
 import { rerankingSupportedRegions } from "../config/aws.js";
-import { logger } from "../utils/logger.js";
 /**
  * Queries a Bedrock knowledge base with a given query, handling reranking and result serialization.
  * This method interacts with the AWS Bedrock service to retrieve knowledge base results.
@@ -15,6 +15,7 @@ import { logger } from "../utils/logger.js";
  * @returns A serialized string of the query results.
  */
 export async function queryKnowledgeBase(knowledgeBaseId, query, kbAgentClient, numberOfResults = 5, reranking = false, rerankingModelName = "AMAZON") {
+    const logger = getLogger(["mcpserver", "bedrock"]);
     const clientRegion = await kbAgentClient.config.region();
     let rerankingEnabled = reranking;
     // Reranking is only supported in specific AWS regions.

package/dist/tools/QueryPagoPADXDocumentation.js

@@ -2,18 +2,25 @@ import { z } from "zod";
 import { kbRerankingEnabled, kbRuntimeClient, knowledgeBaseId, } from "../config/aws.js";
 import { queryKnowledgeBase } from "../services/bedrock.js";
 /**
- * A tool that provides access to the complete Terraform documentation for PagoPA Dx.
- * It uses a Bedrock knowledge base to answer queries about Terraform modules and best practices.
+ * A tool that provides access to the complete PagoPA DX documentation.
+ * It uses a Bedrock knowledge base to answer queries about DX tools, patterns, and best practices.
  */
 export const QueryPagoPADXDocumentationTool = {
     annotations: {
-        title: "Query PagoPA DX Terraform documentation",
+        title: "Query PagoPA DX documentation",
     },
-    description: `This tool provides access to the complete Terraform documentation for PagoPA Dx.
-Use this knowledge base to generate or review Terraform configurations aligned with the official PagoPA Dx module conventions.
-All prompts and questions should be written in English, so that the tool responds using English resource and variable names.
-The tool should be used to explain, guide, or suggest Terraform usage based on verified module documentation and internal best practices.
-Use only modules from the pagopa-dx namespace. To get terraform modules descriptions, input/output variables and examples, use the \`searchModules\` tool.
+    description: `This tool provides access to the complete PagoPA DX documentation covering:
+- Getting started, monorepo setup, dev containers, and GitHub collaboration
+- Git workflows and pull requests
+- DX pipelines setup and management
+- TypeScript development (npm scripts, ESLint, code review)
+- Terraform (folder structure, DX modules, Azure provider, pre-commit hooks, validation, deployment, drift detection)
+- Azure development (naming conventions, policies, IAM, API Management, monitoring, networking, deployments, static websites, Service Bus, data archiving)
+- Container development (Docker images)
+- Contributing to DX (Azure provider, Terraform modules, documentation)
+
+All prompts and questions should be written in English.
+For Terraform module details (input/output variables, examples), use the \`searchModules\` tool.
 `,
     execute: async (args) => {
         const result = await queryKnowledgeBase(knowledgeBaseId, args.query, kbRuntimeClient, undefined, kbRerankingEnabled);

package/dist/tools/SearchGitHubCode.js

@@ -1,6 +1,6 @@
+import { getLogger } from "@logtape/logtape";
 import { Octokit } from "@octokit/rest";
 import { z } from "zod";
-import { logger } from "../utils/logger.js";
 const defaultOrg = process.env.GITHUB_SEARCH_ORG || "pagopa";
 /**
  * A tool that searches for code in a GitHub organization.
@@ -15,6 +15,7 @@ Use this to find examples of specific code patterns, such as Terraform module us
 For example, search for "pagopa-dx/azure-function-app/azurerm" to find examples of the azure-function-app module usage.
 Returns file contents matching the search query.`,
     execute: async (args, context) => {
+        const logger = getLogger(["mcpserver", "github-search"]);
         const org = defaultOrg;
         const token = context.session?.token;
         if (!token) {
@@ -52,7 +53,7 @@ Returns file contents matching the search query.`,
                     return null;
                 }
                 catch (error) {
-                    logger.error(error, `Error fetching file ${item.path}`);
+                    logger.error(`Error fetching file ${item.path}`, { error });
                     return null;
                 }
             }));
@@ -66,7 +67,7 @@ Returns file contents matching the search query.`,
             });
         }
         catch (error) {
-            logger.error(error, "Error searching GitHub code");
+            logger.error("Error searching GitHub code", { error });
             throw error;
         }
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pagopa/dx-mcpserver",
-  "version": "0.0.1",
+  "version": "0.0.3",
   "type": "module",
   "description": "An MCP server that support developers using DX tools.",
   "repository": {
@@ -20,18 +20,20 @@
   },
   "dependencies": {
     "@aws-sdk/client-bedrock-agent-runtime": "^3.583.0",
+    "@logtape/logtape": "^1.1.1",
     "@octokit/rest": "^22.0.0",
     "axios": "^1.12.2",
     "fastmcp": "^3.19.1",
-    "pino": "^9.1.0",
-    "pino-lambda": "^4.4.1",
-    "pino-pretty": "^13.1.1",
-    "zod": "^3.25.76"
+    "zod": "^3.25.76",
+    "@pagopa/dx-mcpprompts": "^0.0.1",
+    "@pagopa/azure-tracing": "^0.4.8"
   },
   "devDependencies": {
     "@types/node": "^22.16.2",
     "@vitest/coverage-v8": "^3.2.4",
     "eslint": "^9.30.0",
+    "prettier": "3.6.2",
+    "tsx": "^4.20.6",
     "typescript": "~5.8.3",
     "vitest": "^3.2.4",
     "@pagopa/eslint-config": "^5.1.0"
@@ -40,6 +42,7 @@
     "build": "tsc",
     "lint": "eslint --fix src",
     "lint:check": "eslint src",
+    "start": "tsx src/index.ts",
     "format": "prettier --write .",
     "format:check": "prettier --check .",
     "typecheck": "tsc --noEmit",

package/dist/prompts/GenerateTerraformConfiguration.js

@@ -1,29 +0,0 @@
-/**
- * A prompt that generates a Terraform configuration following PagoPA DX best practices.
- * It guides the user to query the knowledge base for information on folder structure,
- * modules, and conventions before generating the code.
- */
-export const GenerateTerraformConfigurationPrompt = {
-    // The arguments for the prompt.
-    arguments: [
-        {
-            description: "Requirements for the Terraform configuration to generate.",
-            name: "requirements",
-            required: false,
-        },
-    ],
-    // The description of the prompt.
-    description: "Generates a Terraform configuration following PagoPA DX best practices.",
-    // The function that loads the prompt.
-    load: async (args) => `To generate a Terraform configuration for ${args.requirements || "any requirement"}, you must follow the PagoPA DX best practices.
-
-You can find information about the "infrastructure folder structure", "Using DX terraform modules", "Using DX terraform provider" and other conventions by using the \`QueryPagoPADXDocumentation\` tool. For example, you can ask "what is the infrastructure folder structure?", but you must ask for the other information as well.
-It's suggested to call the \`QueryPagoPADXDocumentation\` tool multiple times to gather all the necessary information using simple and scoped questions.
-
-When generating the configuration, remember to:
-1.  Use existing Terraform modules from the \`pagopa-dx\` namespace whenever possible. You can search for them using the \`searchModules\` tool.
-2.  Strictly follow the correct folder structure for infrastructure resources.
-3.  Generate HCL code that is clean, readable, and well-documented.`,
-    // The name of the prompt.
-    name: "generate-terraform-configuration",
-};

package/dist/utils/__tests__/logger.test.js

@@ -1,21 +0,0 @@
-import { describe, expect, it, vi } from "vitest";
-vi.mock("../logger", () => ({
-    logger: {
-        error: vi.fn(),
-        info: vi.fn(),
-        warn: vi.fn(),
-    },
-}));
-import { logger } from "../logger.js";
-describe("logger", () => {
-    it("should have info, warn, error methods", () => {
-        expect(typeof logger.info).toBe("function");
-        expect(typeof logger.warn).toBe("function");
-        expect(typeof logger.error).toBe("function");
-    });
-    it("should not throw when calling info, warn, error", () => {
-        expect(() => logger.info("test info")).not.toThrow();
-        expect(() => logger.warn("test warn")).not.toThrow();
-        expect(() => logger.error("test error")).not.toThrow();
-    });
-});

package/dist/utils/logger.js

@@ -1,9 +0,0 @@
-import pino from "pino";
-import { pinoLambdaDestination } from "pino-lambda";
-// Creates a Pino logger instance configured for Lambda environments.
-const destination = pinoLambdaDestination();
-export const logger = pino({
-    transport: {
-        target: "pino-pretty",
-    },
-}, destination);