@chrischall/tripadvisor-mcp 0.4.0 → 0.5.0

package/dist/tools/location.js

@@ -1,23 +1,20 @@
 import { z } from 'zod';
-import { textResult } from '@chrischall/mcp-utils';
+import { minifiedResult } from '@chrischall/mcp-utils';
 import { client } from '../client.js';
 import { LocationId, LocaleList, pageParams, qs } from './shared.js';
-import { compactLocationList } from '../projection.js';
+import { viewArg, viewResponse } from '../view.js';
 export function registerLocationTools(server) {
     server.registerTool('ta_get_locations', {
-        description: 'Get details for MULTIPLE locations in one call (batch). Pass an array of location ids — cheaper than repeated ta_get_location_details. Unknown or unlicensed ids are silently omitted. Pass compact:true for slim summaries.',
+        description: 'Get details for MULTIPLE locations in one call (batch). Pass an array of location ids — cheaper than repeated ta_get_location_details. Unknown or unlicensed ids are silently omitted. Returns slim summaries by default; pass view:"full" for the whole records.',
         annotations: { readOnlyHint: true, openWorldHint: true },
         inputSchema: {
             ids: z.array(LocationId).min(1).max(50).describe('Location IDs to fetch (1–50)'),
             locale: LocaleList,
-            compact: z
-                .boolean()
-                .optional()
-                .describe('Return a slim summary per location instead of full records'),
+            view: viewArg(),
         },
-    }, async ({ ids, locale, compact }) => {
+    }, async ({ ids, locale, view }) => {
         const data = await client.get(`/locations${qs({ id: ids, locale })}`, { cache: 'static' });
-        return textResult(compact ? compactLocationList(data) : data);
+        return viewResponse(view, data, 'locationList');
     });
     server.registerTool('ta_get_location_details', {
         description: 'Get full details for a TripAdvisor location: names, descriptions, address, coordinates, traveler ratings, phone, category, and listing URLs.',
@@ -28,8 +25,16 @@ export function registerLocationTools(server) {
         },
     }, async ({ locationId, locale }) => {
         const data = await client.get(`/locations/${locationId}${qs({ locale })}`, { cache: 'static' });
-        return textResult(data);
+        return minifiedResult(data);
     });
+    // NO `view` on this tool, deliberately. Its product IS the image URLs: a
+    // photos item is `{id, location_id, photo: {key, original_size_url, …}, …}`
+    // (docs/TRIPADVISOR-API.md §5), and `photo` is a media KEY — stripping it
+    // does not shrink the response, it EMPTIES it, leaving ids and a publish
+    // timestamp pointing at nothing. Same rule as `musicbrainz_cover_art`,
+    // `alltrails_get_trail_photos`, `sw_get_receipt` and redfin's photo bundles
+    // — see @chrischall/mcp-utils' `stripMediaUrls` docs ("Never apply this to a
+    // tool whose PRODUCT is the image. The tool's own name is the test.").
     server.registerTool('ta_get_location_photos', {
         description: 'Get photos for a TripAdvisor location (multi-size image URLs, source, dimensions), with pagination.',
         annotations: { readOnlyHint: true, openWorldHint: true },
@@ -42,20 +47,28 @@ export function registerLocationTools(server) {
         const data = await client.get(`/locations/${locationId}/photos${qs({ locale, page, size })}`, {
             cache: 'static',
         });
-        return textResult(data);
+        return minifiedResult(data);
     });
+    // `view` DOES belong here, and it is the opposite case to photos above. A
+    // review's product is its TEXT; the image URLs it carries — reviewer avatars,
+    // per-review snapshots — are incidental to the thing the caller asked for, so
+    // dropping them shrinks the payload instead of emptying it. There is no
+    // hand-written projection for this shape, so `viewResponse` falls through to
+    // `stripMediaUrls`, which needs no knowledge of the fields.
     server.registerTool('ta_get_location_reviews', {
-        description: 'Get traveler reviews for a TripAdvisor location, with pagination.',
+        description: 'Get traveler reviews for a TripAdvisor location, with pagination. Reviewer avatars and other image URLs ' +
+            'are dropped by default; pass view:"full" for TripAdvisor\'s whole records.',
         annotations: { readOnlyHint: true, openWorldHint: true },
         inputSchema: {
             locationId: LocationId,
             locale: LocaleList,
             ...pageParams,
+            view: viewArg(),
         },
-    }, async ({ locationId, locale, page, size }) => {
+    }, async ({ locationId, locale, page, size, view }) => {
         const data = await client.get(`/locations/${locationId}/reviews${qs({ locale, page, size })}`, {
             cache: 'static',
         });
-        return textResult(data);
+        return viewResponse(view, data);
     });
 }

package/dist/tools/search.js

@@ -1,18 +1,13 @@
 import { z } from 'zod';
-import { textResult, McpToolError } from '@chrischall/mcp-utils';
+import { McpToolError } from '@chrischall/mcp-utils';
 import { client } from '../client.js';
 import { Category, LocaleList, pageParams, qs } from './shared.js';
-import { compactList } from '../projection.js';
-/** `compact` arg shared by the list-returning search tools. */
-const compactParam = {
-    compact: z
-        .boolean()
-        .optional()
-        .describe('Return a slim summary per result (id, name, category, city, rating, review_count, url) instead of full records'),
-};
+import { viewArg, viewResponse } from '../view.js';
+/** The `view` arg shared by the list-returning search tools. */
+const viewParamShared = { view: viewArg() };
 export function registerSearchTools(server) {
     server.registerTool('ta_search_locations', {
-        description: 'Search TripAdvisor locations (restaurants, attractions, hotels) by name. Returns matches with a location id for the detail tools, plus pagination. Pass compact:true for slim summaries.',
+        description: 'Search TripAdvisor locations (restaurants, attractions, hotels) by name. Returns matches with a location id for the detail tools, plus pagination. Returns slim summaries by default; pass view:"full" for the whole records.',
         annotations: { readOnlyHint: true, openWorldHint: true },
         inputSchema: {
             query: z.string().min(1).max(500).describe('Text to search location names for'),
@@ -22,14 +17,14 @@ export function registerSearchTools(server) {
             postal_code: z.string().optional().describe('Postal/ZIP code (takes precedence over geo_name)'),
             locale: LocaleList,
             ...pageParams,
-            ...compactParam,
+            ...viewParamShared,
         },
-    }, async ({ query, category, country_code, geo_name, postal_code, locale, page, size, compact }) => {
+    }, async ({ query, category, country_code, geo_name, postal_code, locale, page, size, view }) => {
         const data = await client.get(`/locations/search${qs({ query, category, country_code, geo_name, postal_code, locale, page, size })}`, { cache: 'dynamic' });
-        return textResult(compact ? compactList(data) : data);
+        return viewResponse(view, data, 'list');
     });
     server.registerTool('ta_search_nearby', {
-        description: 'Find TripAdvisor locations near a point within a radius, or inside a bounding box. Center by lat+lon+radius, by a reference location_id+radius, or by a sw/ne bounding box. Returns matches with distance and a location id. Pass compact:true for slim summaries.',
+        description: 'Find TripAdvisor locations near a point within a radius, or inside a bounding box. Center by lat+lon+radius, by a reference location_id+radius, or by a sw/ne bounding box. Returns matches with distance and a location id. Returns slim summaries by default; pass view:"full" for the whole records.',
         annotations: { readOnlyHint: true, openWorldHint: true },
         inputSchema: {
             // Center — supply exactly one of: lat+lon, location_id, or the sw/ne box.
@@ -48,10 +43,10 @@ export function registerSearchTools(server) {
             sort: z.enum(['distance', 'rating']).optional().describe('Sort order (default distance)'),
             locale: LocaleList,
             ...pageParams,
-            ...compactParam,
+            ...viewParamShared,
         },
     }, async (args) => {
-        const { lat, lon, location_id, radius, sw_lat, sw_lon, ne_lat, ne_lon, compact, ...rest } = args;
+        const { lat, lon, location_id, radius, sw_lat, sw_lon, ne_lat, ne_lon, view, ...rest } = args;
         const boxParts = [sw_lat, sw_lon, ne_lat, ne_lon];
         const boxGiven = boxParts.filter((v) => v !== undefined).length;
         // A partial box (1–3 of 4) is never valid: it can't form a center on its
@@ -81,6 +76,6 @@ export function registerSearchTools(server) {
             });
         }
         const data = await client.get(`/locations/nearby${qs({ lat, lon, location_id, radius, sw_lat, sw_lon, ne_lat, ne_lon, ...rest })}`, { cache: 'dynamic' });
-        return textResult(compact ? compactList(data) : data);
+        return viewResponse(view, data, 'list');
     });
 }

package/dist/tools/web.js

@@ -1,5 +1,5 @@
 import { registerBridgeHealthcheckTool } from '@chrischall/mcp-utils/fetchproxy';
-import { McpToolError, textResult } from '@chrischall/mcp-utils';
+import { McpToolError, minifiedResult } from '@chrischall/mcp-utils';
 import { webClient } from '../web/client.js';
 import { parseLocationDetail } from '../web/parse.js';
 import { LocationId } from './shared.js';
@@ -28,6 +28,14 @@ export function registerWebTools(server) {
         },
         probeFn: (path) => webClient.getHtml(path),
     });
+    // NO `view` on this tool, deliberately, and for the OPPOSITE reason to
+    // `ta_get_location_photos`. It does not return an upstream payload at all:
+    // `parseLocationDetail` is already a hand-written projection down to a dozen
+    // named fields, and one of them is `image` — chosen on purpose, from a page
+    // that offers hundreds. Media-stripping a grounded projection lets a blind
+    // subtractive rule overrule a rule written with knowledge of the source,
+    // which is exactly what deleted viator-mcp's `coverImageUrl`
+    // (chrischall/viator-mcp#72). There is no cheaper rung to offer here.
     server.registerTool('ta_web_get_location', {
         description: "Get a TripAdvisor location's core details (name, rating, review count, address, coordinates, phone, photo, listing URL) by location ID, read from the public page via the browser bridge. Works without an API key — use this when ta_get_location_details is unavailable or its key is blocked. Covers attractions, hotels, and restaurants. Does not return individual review text.",
         annotations: { readOnlyHint: true, openWorldHint: true },
@@ -42,6 +50,6 @@ export function registerWebTools(server) {
                 hint: 'The page may be a bot-challenge shell or the id may be wrong — run ta_web_healthcheck and confirm a signed-in www.tripadvisor.com tab is open, then retry.',
             });
         }
-        return textResult({ location_id: locationId, ...detail });
+        return minifiedResult({ location_id: locationId, ...detail });
     });
 }

package/dist/version.js

@@ -1,2 +1,2 @@
 /** Single source of the server version. release-please bumps the literal below. */
-export const VERSION = '0.4.0'; // x-release-please-version
+export const VERSION = '0.5.0'; // x-release-please-version

package/dist/view.js

@@ -0,0 +1,45 @@
+import { minifiedResult, resolveView, stripMediaUrls, viewParam } from '@chrischall/mcp-utils';
+import { compactList, compactLocationList } from './projection.js';
+/**
+ * The rungs this server honours (`@chrischall/mcp-utils`' `view` vocabulary;
+ * `chrischall/workflows` `docs/fleet-conventions.md`, "Response shape").
+ *
+ * This is a GROUNDED repo: `projection.ts` has carried `compactLocation` /
+ * `compactList` / `compactLocationList` all along, and they were opt-in —
+ * `compact: false`, with the tool descriptions saying "Pass compact:true for
+ * slim summaries". An efficiency that has to be requested is one that usually
+ * is not, and the caller paying for it is the one least able to know a slim
+ * rung existed.
+ *
+ * A hand-written projection is NOT then media-stripped. It was written with
+ * knowledge of the API and its field choices are deliberate; running a blind
+ * subtractive rule over its output would let an un-grounded rule overrule a
+ * grounded one, which bit viator-mcp where the projection intentionally keeps
+ * a cover image. Media stripping is for the payloads that have no projection
+ * to speak for them — here, `ta_get_location_reviews` and nothing else.
+ *
+ * NOT `ta_get_location_photos`, which is the other half of the same rule and
+ * the one that is easy to get wrong: a tool whose PRODUCT is the image URLs is
+ * not media-stripped either, because there the rule does not shrink the
+ * response, it empties it. So that tool registers no `view` at all — see the
+ * comment above its registrar. `viewResponse`'s no-projector branch is
+ * therefore reached by exactly one tool; it is a fallback, not dead code.
+ *
+ * No `raw` rung: `full` already returns the untouched upstream payload.
+ */
+export const TA_VIEWS = ['compact', 'full'];
+const NOTE = 'compact returns the slim projection where one exists and strips image URLs elsewhere; ' +
+    '"full" returns TripAdvisor\'s whole records.';
+/** The `view` parameter every read tool in this server takes. */
+export const viewArg = () => viewParam(TA_VIEWS, { note: NOTE });
+/** Answer in the requested rung, running the named projection when there is one. */
+export function viewResponse(view, data, projector) {
+    const rung = resolveView(view, TA_VIEWS);
+    if (rung !== 'compact')
+        return minifiedResult(data);
+    if (projector === 'list')
+        return minifiedResult(compactList(data));
+    if (projector === 'locationList')
+        return minifiedResult(compactLocationList(data));
+    return minifiedResult(stripMediaUrls(data));
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrischall/tripadvisor-mcp",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "mcpName": "io.github.chrischall/tripadvisor-mcp",
   "description": "TripAdvisor Terra API MCP server for Claude — search locations, details, photos, and reviews. Developed and maintained by AI (Claude Code).",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",
@@ -46,7 +46,7 @@
     "typecheck": "tsc -p tsconfig.json --noEmit"
   },
   "dependencies": {
-    "@chrischall/mcp-utils": "^0.15.0",
+    "@chrischall/mcp-utils": "^0.23.0",
     "@fetchproxy/server": "^2.2.0",
     "@modelcontextprotocol/sdk": "^1.29.0",
     "dotenv": "^17.4.0",

package/server.json

@@ -6,12 +6,12 @@
     "url": "https://github.com/chrischall/tripadvisor-mcp",
     "source": "github"
   },
-  "version": "0.4.0",
+  "version": "0.5.0",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "@chrischall/tripadvisor-mcp",
-      "version": "0.4.0",
+      "version": "0.5.0",
       "transport": {
         "type": "stdio"
       },