fix: base url and outputs (#280)

Co-authored-by: Stefano Vozza <[email protected]>
Co-authored-by: Michael Walmsley <[email protected]>

Author: 3 people

.github/workflows/make-release.yml

@@ -4,7 +4,7 @@ name: Make Release
 #
 # === Automated activities ===
 # 1. [Quality check] run unit tests and linting
-# 2. [Release] publish package to npmjs.org using the latest git commit, ensure provenance with NPM_CONFIG_PROVENANCE=true
+# 2. [Release] publish package to npmjs.org using OIDC authentication with automatic provenance attestations
 # 3. [Create tag] create a new git tag using released version, i.e. v1.13.1
 #
 # === Manual activities ===
@@ -28,12 +28,12 @@ jobs:
   run-e2e-tests:
     uses: ./.github/workflows/reusable-e2e.yml
   # This job publishes the packages to npm.
-  # It uses the latest git commit sha as the version and ensures provenance with NPM_CONFIG_PROVENANCE flag.
+  # It uses OIDC authentication with automatic provenance attestations.
   # We don't bump the version because we do that in the `make-version` workflow.
   # It also sets the RELEASE_VERSION output to be used by the next job to create a git tag.
   publish-npm:
     needs: [run-e2e-tests, run-unit-tests]
-    # Needed as recommended by npm docs on publishing with provenance https://docs.npmjs.com/generating-provenance-statements
+    # Needed for OIDC authentication with npm trusted publishing
     permissions:
       id-token: write
     environment: Release
@@ -46,18 +46,14 @@ jobs:
         with:
           ref: ${{ github.sha }}
       - name: Setup Node.js
-        uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
+        uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5.0.0
         with:
           node-version: "22"
           cache: "npm"
-      - name: Setup auth tokens
-        run: |
-          npm set "//registry.npmjs.org/:_authToken=${{ secrets.NPM_TOKEN }}"
       - name: Setup dependencies
         uses: aws-powertools/actions/.github/actions/cached-node-modules@3b5b8e2e58b7af07994be982e83584a94e8c76c5
       - name: Publish to npm
-        run: |
-          NPM_CONFIG_PROVENANCE=true npm publish
+        run: npm publish
       - name: Set release version
         id: set-release-version
         run: |

biome.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "https://biomejs.dev/schemas/2.0.0/schema.json",
+  "$schema": "node_modules/@biomejs/biome/configuration_schema.json",
   "assist": { "actions": { "source": { "organizeImports": "on" } } },
   "linter": {
     "enabled": true,

src/constants.ts

@@ -8,9 +8,9 @@ import {
 const MCP_SERVER_NAME = 'powertools-for-aws-mcp' as const;
 
 // Allowed domain for security
-const ALLOWED_DOMAIN = 'docs.powertools.aws.dev';
+const ALLOWED_DOMAIN = 'docs.aws.amazon.com';
 // Base URL for Powertools documentation
-const POWERTOOLS_BASE_URL = 'https://docs.powertools.aws.dev/lambda';
+const POWERTOOLS_BASE_URL = 'https://docs.aws.amazon.com/powertools';
 
 const FETCH_TIMEOUT_MS = 15000; // 15 seconds timeout for fetch operations
 

src/logger.ts

@@ -1,5 +1,6 @@
+import { Console } from 'node:console';
 import { getStringFromEnv } from '@aws-lambda-powertools/commons/utils/env';
-import { Logger, LogFormatter, LogItem } from '@aws-lambda-powertools/logger';
+import { LogFormatter, Logger, LogItem } from '@aws-lambda-powertools/logger';
 import type {
   LogAttributes,
   LogLevel,
@@ -30,5 +31,19 @@ const logger = new Logger({
   logLevel,
   logFormatter: new CustomLogFormatter(),
 });
+/**
+ * Since this MCP server uses the stdio protocol, we always
+ * need to emit logs to `stderr` to avoid interfering with MCP's
+ * communication over `stdout`.
+ *
+ * See https://modelcontextprotocol.io/docs/develop/build-server#logging-in-mcp-servers
+ */
+/* v8 ignore start */ if (process.env.NODE_ENV !== 'test') {
+  // @ts-expect-error
+  (logger as unknown).console = new Console({
+    stdout: process.stderr,
+    stderr: process.stderr,
+  });
+} /* v8 ignore stop */
 
 export { logger };

src/tools/fetchDocPage/index.ts

@@ -1,3 +1,3 @@
-export { tool } from './tool.ts';
-export { name, description } from './constants.ts';
+export { description, name } from './constants.ts';
 export { schema } from './schemas.ts';
+export { tool } from './tool.ts';

src/tools/searchDocs/index.ts

@@ -1,3 +1,3 @@
-export { tool } from './tool.ts';
-export { name, description } from './constants.ts';
+export { description, name } from './constants.ts';
 export { schema } from './schemas.ts';
+export { tool } from './tool.ts';

src/tools/searchDocs/tool.ts

@@ -75,8 +75,9 @@ const tool = async (props: ToolProps): Promise<CallToolResult> => {
   // TODO: consume built/exported search index - #79
   const index = lunr(function () {
     this.ref('location');
-    this.field('title', { boost: 10 });
-    this.field('text');
+    this.field('title', { boost: 1000 });
+    this.field('text', { boost: 1 });
+    this.field('tags', { boost: 1000000 });
 
     for (const doc of searchIndexContent.docs) {
       if (!doc.location || !doc.title || !doc.text) continue;

tests/setupEnv.ts

@@ -1,6 +1,8 @@
 import type { CallToolResult } from '@modelcontextprotocol/sdk/types.js';
 import { expect, vi } from 'vitest';
 
+vi.stubEnv('NODE_ENV', 'test');
+
 // Mock console methods to prevent output during tests
 vi.spyOn(console, 'error').mockReturnValue();
 vi.spyOn(console, 'warn').mockReturnValue();

tests/tsconfig.json

@@ -4,8 +4,5 @@
     "rootDir": "../",
     "noEmit": true
   },
-  "include": [
-    "../src/**/*",
-    "./**/*"
-  ]
-}
+  "include": ["../src/**/*", "./**/*"]
+}

tests/unit/fetchDocPage.test.ts

@@ -1,4 +1,4 @@
-import { http, HttpResponse } from 'msw';
+import { HttpResponse, http } from 'msw';
 import { setupServer } from 'msw/node';
 import {
   afterAll,