@rubytech/create-maxy-code 0.1.45 → 0.1.47

package/payload/premium-plugins/real-agent/plugins/loop/mcp/src/__tests__/swagger-write-coverage.test.ts

@@ -58,18 +58,19 @@ const ENDPOINT_MAP: EndpointMapping[] = [
   { method: 'post', path: '/marketing/rentals/matching/{id}/information', tool: 'loop-marketing-match-request', source: 'tools/marketing-match-request.ts', fragment: '/marketing/rentals' },
   { method: 'post', path: '/marketing/matching/{id}/callback', tool: 'loop-marketing-match-request', source: 'tools/marketing-match-request.ts', fragment: '/matching/' },
   { method: 'post', path: '/marketing/rentals/matching/{id}/callback', tool: 'loop-marketing-match-request', source: 'tools/marketing-match-request.ts', fragment: '/marketing/rentals' },
-  // Marketing — enquiries (4)
+  // Marketing — seller enquiry (1)
   { method: 'post', path: '/marketing/enquiries/seller', tool: 'loop-marketing-enquiry', source: 'tools/marketing-enquiry.ts', fragment: '/marketing/enquiries/seller' },
-  { method: 'put', path: '/marketing/enquiries/auto-responder/{id}/answers/{key}', tool: 'loop-marketing-enquiry', source: 'tools/marketing-enquiry.ts', fragment: '/marketing/enquiries/auto-responder/' },
-  { method: 'put', path: '/marketing/enquiries/auto-responder/{id}/details/{key}', tool: 'loop-marketing-enquiry', source: 'tools/marketing-enquiry.ts', fragment: '/marketing/enquiries/auto-responder/' },
-  { method: 'put', path: '/marketing/enquiries/auto-responder/{id}/refer/{key}', tool: 'loop-marketing-enquiry', source: 'tools/marketing-enquiry.ts', fragment: '/marketing/enquiries/auto-responder/' },
-  // Property (6)
-  { method: 'post', path: '/property/residential/sales/{id}/viewing', tool: 'loop-property-request', source: 'tools/property-request.ts', fragment: '/property/residential/' },
-  { method: 'post', path: '/property/residential/sales/{id}/call-back', tool: 'loop-property-request', source: 'tools/property-request.ts', fragment: '/property/residential/' },
-  { method: 'post', path: '/property/residential/sales/{id}/information', tool: 'loop-property-request', source: 'tools/property-request.ts', fragment: '/property/residential/' },
-  { method: 'post', path: '/property/residential/lettings/{id}/viewing', tool: 'loop-property-request', source: 'tools/property-request.ts', fragment: '/property/residential/' },
-  { method: 'post', path: '/property/residential/lettings/{id}/call-back', tool: 'loop-property-request', source: 'tools/property-request.ts', fragment: '/property/residential/' },
-  { method: 'post', path: '/property/residential/lettings/{id}/information', tool: 'loop-property-request', source: 'tools/property-request.ts', fragment: '/property/residential/' },
+  // Marketing — auto-responder (3) — Task 103 split into loop-auto-responder
+  { method: 'put', path: '/marketing/enquiries/auto-responder/{id}/answers/{key}', tool: 'loop-auto-responder', source: 'tools/auto-responder.ts', fragment: '/marketing/enquiries/auto-responder/' },
+  { method: 'put', path: '/marketing/enquiries/auto-responder/{id}/details/{key}', tool: 'loop-auto-responder', source: 'tools/auto-responder.ts', fragment: '/marketing/enquiries/auto-responder/' },
+  { method: 'put', path: '/marketing/enquiries/auto-responder/{id}/refer/{key}', tool: 'loop-auto-responder', source: 'tools/auto-responder.ts', fragment: '/marketing/enquiries/auto-responder/' },
+  // Property intent (6) — Task 103 split loop-property-request into three intent-clear tools
+  { method: 'post', path: '/property/residential/sales/{id}/viewing', tool: 'loop-property-viewing', source: 'tools/property-intent.ts', fragment: '/property/residential/' },
+  { method: 'post', path: '/property/residential/sales/{id}/call-back', tool: 'loop-property-callback', source: 'tools/property-intent.ts', fragment: '/property/residential/' },
+  { method: 'post', path: '/property/residential/sales/{id}/information', tool: 'loop-property-information', source: 'tools/property-intent.ts', fragment: '/property/residential/' },
+  { method: 'post', path: '/property/residential/lettings/{id}/viewing', tool: 'loop-property-viewing', source: 'tools/property-intent.ts', fragment: '/property/residential/' },
+  { method: 'post', path: '/property/residential/lettings/{id}/call-back', tool: 'loop-property-callback', source: 'tools/property-intent.ts', fragment: '/property/residential/' },
+  { method: 'post', path: '/property/residential/lettings/{id}/information', tool: 'loop-property-information', source: 'tools/property-intent.ts', fragment: '/property/residential/' },
   // Supplier (3)
   { method: 'post', path: '/supplier/maintenance/{code}/{jobid}/complete', tool: 'loop-supplier', source: 'tools/supplier.ts', fragment: '/supplier/maintenance/' },
   { method: 'put', path: '/supplier/maintenance/{code}/{quoteid}/quote-for-job', tool: 'loop-supplier', source: 'tools/supplier.ts', fragment: '/quote-for-job' },

package/payload/premium-plugins/real-agent/plugins/loop/mcp/src/index.ts

@@ -10,7 +10,8 @@ import { keyList } from "./tools/key-list.js";
 import { propertySearch } from "./tools/property-search.js";
 import { propertyDetail } from "./tools/property-detail.js";
 import { propertyListed } from "./tools/property-listed.js";
-import { propertyRequest } from "./tools/property-request.js";
+import { propertySold } from "./tools/property-sold.js";
+import { propertyViewing, propertyCallback, propertyInformation } from "./tools/property-intent.js";
 import { peopleSearch } from "./tools/people-search.js";
 import { peopleDetail } from "./tools/people-detail.js";
 import { viewingSearch } from "./tools/viewing-search.js";
@@ -24,6 +25,7 @@ import { marketingMatchDetail } from "./tools/marketing-match.js";
 import { marketingMatchBatch } from "./tools/marketing-match-batch.js";
 import { marketingMatchRequest } from "./tools/marketing-match-request.js";
 import { marketingEnquiry } from "./tools/marketing-enquiry.js";
+import { autoResponder } from "./tools/auto-responder.js";
 import { customerPreferences } from "./tools/customer-preferences.js";
 import { supplier } from "./tools/supplier.js";
 import { closeDriver } from "./lib/neo4j.js";
@@ -194,11 +196,10 @@ server.tool(
 
 server.tool(
   "loop-property-listed",
-  "Get properties listed on a specific channel (Rightmove, Zoopla, OnTheMarket, website). Optionally include sold properties.",
+  "Get properties currently listed on a specific channel (Rightmove, Zoopla, OnTheMarket, website). For exchanged/completed properties use loop-property-sold.",
   {
     channel: z.enum(["rightmove", "onTheMarket", "zoopla", "website"]).describe("Listing channel"),
     department: z.enum(["sales", "lettings", "both"]).optional().describe("Department (default: both)"),
-    includeSold: z.boolean().optional().describe("Include sold gallery (default: false)"),
     teamName: z.string().optional().describe("Query a specific team only"),
     limit: z.number().int().optional().describe("Max results (default 200)"),
   },
@@ -206,19 +207,63 @@ server.tool(
 );
 
 server.tool(
-  "loop-property-request",
-  "Submit a viewing, callback, or information request for a property. Requires a specific team.",
+  "loop-property-sold",
+  "Get exchanged or completed properties on a specific channel. Powers canvassing from recent sales and sold-gallery rendering.",
+  {
+    channel: z.enum(["rightmove", "onTheMarket", "zoopla", "website"]).describe("Listing channel"),
+    teamName: z.string().optional().describe("Query a specific team only"),
+    limit: z.number().int().optional().describe("Max results (default 200)"),
+  },
+  toolHandler(async (p) => propertySold(p as Parameters<typeof propertySold>[0]))
+);
+
+// Property-level intent tools (Task 103 split — replaces loop-property-request).
+// Each intent is a distinct operator action so the agent's tool surface
+// preserves intent fidelity.
+
+server.tool(
+  "loop-property-viewing",
+  "Request a viewing for a property. Use when the operator says 'book a viewing', 'arrange a viewing', or 'ask to see the property'.",
+  {
+    teamName: z.string().min(1).describe("Team to submit the request through"),
+    propertyId: z.number().int().describe("Property ID"),
+    department: z.enum(["sales", "lettings"]).describe("sales or lettings"),
+    name: z.string().optional().describe("Requester name"),
+    email: z.string().optional().describe("Requester email"),
+    phone: z.string().optional().describe("Requester phone"),
+    message: z.string().optional().describe("Additional message"),
+  },
+  toolHandler(async (p) => propertyViewing(p as Parameters<typeof propertyViewing>[0]))
+);
+
+server.tool(
+  "loop-property-callback",
+  "Request a callback about a property. Use when the operator says 'ask them to call', 'request a callback', or 'have someone ring back'.",
+  {
+    teamName: z.string().min(1).describe("Team to submit the request through"),
+    propertyId: z.number().int().describe("Property ID"),
+    department: z.enum(["sales", "lettings"]).describe("sales or lettings"),
+    name: z.string().optional().describe("Requester name"),
+    email: z.string().optional().describe("Requester email"),
+    phone: z.string().optional().describe("Requester phone"),
+    message: z.string().optional().describe("Additional message"),
+  },
+  toolHandler(async (p) => propertyCallback(p as Parameters<typeof propertyCallback>[0]))
+);
+
+server.tool(
+  "loop-property-information",
+  "Request more information about a property. Use when the operator says 'ask for details', 'request the brochure', or 'find out more'.",
   {
     teamName: z.string().min(1).describe("Team to submit the request through"),
     propertyId: z.number().int().describe("Property ID"),
     department: z.enum(["sales", "lettings"]).describe("sales or lettings"),
-    action: z.enum(["viewing", "call-back", "information"]).describe("Request type"),
     name: z.string().optional().describe("Requester name"),
     email: z.string().optional().describe("Requester email"),
     phone: z.string().optional().describe("Requester phone"),
     message: z.string().optional().describe("Additional message"),
   },
-  toolHandler(async (p) => propertyRequest(p as Parameters<typeof propertyRequest>[0]))
+  toolHandler(async (p) => propertyInformation(p as Parameters<typeof propertyInformation>[0]))
 );
 
 // ─────────────────────────────────────────────────────────────────
@@ -377,22 +422,28 @@ server.tool(
 
 server.tool(
   "loop-marketing-enquiry",
-  "Marketing enquiry operations: submit seller enquiries, manage auto-responder interactions.",
+  "Submit a seller-side marketing enquiry to Loop. For inbound applicant auto-responder workflows use loop-auto-responder.",
   {
     teamName: z.string().min(1).describe("Team to submit through"),
-    action: z.enum([
-      "seller-enquiry", "autoresponder-get", "autoresponder-answers",
-      "autoresponder-details", "autoresponder-refer",
-    ]).describe("Enquiry action"),
-    sellerEnquiryData: z.record(z.string(), z.unknown()).optional().describe("Seller enquiry request body (for seller-enquiry)"),
-    autoResponderId: z.number().int().optional().describe("Auto-responder enquiry ID"),
-    autoResponderKey: z.string().optional().describe("Auto-responder key (UUID)"),
-    answers: z.array(z.unknown()).optional().describe("Auto-responder answers array"),
-    details: z.record(z.string(), z.unknown()).optional().describe("Auto-responder details object"),
+    sellerEnquiryData: z.record(z.string(), z.unknown()).optional().describe("Seller enquiry request body"),
   },
   toolHandler(async (p) => marketingEnquiry(p as Parameters<typeof marketingEnquiry>[0]))
 );
 
+server.tool(
+  "loop-auto-responder",
+  "Read or progress an inbound enquiry through Loop's auto-responder (the surface behind Loop's 40% enquiry-efficiency claim). action=get reads questions+details; answers submits applicant replies; details updates the enquiry record; refer hands it to another team.",
+  {
+    teamName: z.string().min(1).describe("Team for this operation"),
+    action: z.enum(["get", "answers", "details", "refer"]).describe("Auto-responder action"),
+    autoResponderId: z.number().int().describe("Auto-responder enquiry ID"),
+    autoResponderKey: z.string().describe("Auto-responder key (UUID issued at enquiry creation)"),
+    answers: z.array(z.unknown()).optional().describe("Applicant answers (required for action=answers)"),
+    details: z.record(z.string(), z.unknown()).optional().describe("Enquiry details (for action=details)"),
+  },
+  toolHandler(async (p) => autoResponder(p as Parameters<typeof autoResponder>[0]))
+);
+
 // ─────────────────────────────────────────────────────────────────
 // Customer Tools (1) — permission: customer
 // ─────────────────────────────────────────────────────────────────

package/payload/premium-plugins/real-agent/plugins/loop/mcp/src/tools/auto-responder.ts

@@ -0,0 +1,80 @@
+import { loopGet, loopPut, withTeamKey } from "../lib/loop-api.js";
+
+type AutoResponderAction = "get" | "answers" | "details" | "refer";
+
+// Loop API V2: enquiry auto-responder endpoints (Loop's stated 40% efficiency lever)
+//   GET    /marketing/enquiries/auto-responder/{id}/{key}            — read questions + details
+//   PUT    /marketing/enquiries/auto-responder/{id}/answers/{key}    — submit applicant answers
+//   PUT    /marketing/enquiries/auto-responder/{id}/details/{key}    — update enquiry details
+//   PUT    /marketing/enquiries/auto-responder/{id}/refer/{key}      — refer enquiry to another team
+// `id` is the enquiry id, `key` is the UUID handed out at enquiry creation.
+
+interface LoopBooleanResponse {
+  success?: boolean;
+  [key: string]: unknown;
+}
+
+interface LoopAutoResponderDetails {
+  id?: number;
+  questions?: unknown[];
+  enquiryDetails?: unknown;
+  [key: string]: unknown;
+}
+
+export async function autoResponder(params: {
+  accountId: string;
+  teamName: string;
+  action: AutoResponderAction;
+  autoResponderId: number;
+  autoResponderKey: string;
+  answers?: unknown[];
+  details?: Record<string, unknown>;
+}): Promise<string> {
+  const { accountId, teamName, action, autoResponderId: id, autoResponderKey: key } = params;
+
+  if (action === "get") {
+    const result = await withTeamKey<LoopAutoResponderDetails>(
+      accountId, teamName, "marketing", "loop-auto-responder",
+      async (apiKey) => loopGet<LoopAutoResponderDetails>(
+        apiKey, `/marketing/enquiries/auto-responder/${id}/${key}`,
+        "loop-auto-responder", teamName
+      )
+    );
+    return JSON.stringify(result, null, 2);
+  }
+
+  if (action === "answers") {
+    await withTeamKey<LoopBooleanResponse>(
+      accountId, teamName, "marketing", "loop-auto-responder",
+      async (apiKey) => loopPut<LoopBooleanResponse>(
+        apiKey, `/marketing/enquiries/auto-responder/${id}/answers/${key}`,
+        params.answers ?? [], "loop-auto-responder", teamName
+      )
+    );
+    return `Auto-responder answers submitted for enquiry ${id} via team "${teamName}".`;
+  }
+
+  if (action === "details") {
+    await withTeamKey<LoopBooleanResponse>(
+      accountId, teamName, "marketing", "loop-auto-responder",
+      async (apiKey) => loopPut<LoopBooleanResponse>(
+        apiKey, `/marketing/enquiries/auto-responder/${id}/details/${key}`,
+        params.details ?? {}, "loop-auto-responder", teamName
+      )
+    );
+    return `Auto-responder details updated for enquiry ${id} via team "${teamName}".`;
+  }
+
+  if (action === "refer") {
+    await withTeamKey<LoopBooleanResponse>(
+      accountId, teamName, "marketing", "loop-auto-responder",
+      async (apiKey) => loopPut<LoopBooleanResponse>(
+        apiKey, `/marketing/enquiries/auto-responder/${id}/refer/${key}`,
+        {}, "loop-auto-responder", teamName
+      )
+    );
+    return `Enquiry ${id} referred via team "${teamName}".`;
+  }
+
+  throw new Error(`Unknown auto-responder action: ${action}`);
+}

package/payload/premium-plugins/real-agent/plugins/loop/mcp/src/tools/marketing-enquiry.ts

@@ -1,113 +1,27 @@
-import { loopGet, loopPost, loopPut, withTeamKey } from "../lib/loop-api.js";
+import { loopPost, withTeamKey } from "../lib/loop-api.js";
 
-type EnquiryAction = "seller-enquiry" | "autoresponder-get" | "autoresponder-answers" | "autoresponder-details" | "autoresponder-refer";
+// Loop API V2: POST /marketing/enquiries/seller
+// Submit a seller-side marketing enquiry. Auto-responder endpoints used to
+// live in this tool but were split into `loop-auto-responder` for intent
+// fidelity (Task 103).
 
-// ─── Loop API V2: Marketing Enquiry Endpoints ──────────────────
-// POST /marketing/enquiries/seller — submit seller enquiry
-// GET /marketing/enquiries/auto-responder/{id}/{key} — read enquiry
-// PUT .../auto-responder/{id}/answers/{key} — submit answers
-// PUT .../auto-responder/{id}/details/{key} — update details
-// PUT .../auto-responder/{id}/refer/{key} — refer enquiry
-// Note: auto-responder uses personCode + key (UUID), not personId
-// ────────────────────────────────────────────────────────────────
 interface LoopBooleanResponse {
   success?: boolean;
   [key: string]: unknown;
 }
 
-interface LoopAutoResponderDetails {
-  id?: number;
-  questions?: unknown[];
-  enquiryDetails?: unknown;
-  [key: string]: unknown;
-}
-
 export async function marketingEnquiry(params: {
   accountId: string;
   teamName: string;
-  action: EnquiryAction;
-  // For seller-enquiry
   sellerEnquiryData?: Record<string, unknown>;
-  // For autoresponder actions
-  autoResponderId?: number;
-  autoResponderKey?: string;
-  // For autoresponder-answers
-  answers?: unknown[];
-  // For autoresponder-details
-  details?: Record<string, unknown>;
 }): Promise<string> {
-  const { accountId, teamName, action } = params;
-
-  if (action === "seller-enquiry") {
-    await withTeamKey<LoopBooleanResponse>(
-      accountId, teamName, "marketing", "loop-marketing-enquiry",
-      async (apiKey) => {
-        return loopPost<LoopBooleanResponse>(
-          apiKey, "/marketing/enquiries/seller",
-          params.sellerEnquiryData ?? {}, "loop-marketing-enquiry", teamName
-        );
-      }
-    );
-    return `Seller enquiry submitted via team "${teamName}".`;
-  }
-
-  if (!params.autoResponderId || !params.autoResponderKey) {
-    throw new Error("autoResponderId and autoResponderKey are required for autoresponder actions.");
-  }
-  const id = params.autoResponderId;
-  const key = params.autoResponderKey;
-
-  if (action === "autoresponder-get") {
-    const result = await withTeamKey<LoopAutoResponderDetails>(
-      accountId, teamName, "marketing", "loop-marketing-enquiry",
-      async (apiKey) => {
-        return loopGet<LoopAutoResponderDetails>(
-          apiKey, `/marketing/enquiries/auto-responder/${id}/${key}`,
-          "loop-marketing-enquiry", teamName
-        );
-      }
-    );
-    return JSON.stringify(result, null, 2);
-  }
-
-  if (action === "autoresponder-answers") {
-    await withTeamKey<LoopBooleanResponse>(
-      accountId, teamName, "marketing", "loop-marketing-enquiry",
-      async (apiKey) => {
-        return loopPut<LoopBooleanResponse>(
-          apiKey, `/marketing/enquiries/auto-responder/${id}/answers/${key}`,
-          params.answers ?? [], "loop-marketing-enquiry", teamName
-        );
-      }
-    );
-    return `Auto-responder answers submitted for enquiry ${id} via team "${teamName}".`;
-  }
-
-  if (action === "autoresponder-details") {
-    await withTeamKey<LoopBooleanResponse>(
-      accountId, teamName, "marketing", "loop-marketing-enquiry",
-      async (apiKey) => {
-        return loopPut<LoopBooleanResponse>(
-          apiKey, `/marketing/enquiries/auto-responder/${id}/details/${key}`,
-          params.details ?? {}, "loop-marketing-enquiry", teamName
-        );
-      }
-    );
-    return `Auto-responder details updated for enquiry ${id} via team "${teamName}".`;
-  }
-
-  if (action === "autoresponder-refer") {
-    await withTeamKey<LoopBooleanResponse>(
-      accountId, teamName, "marketing", "loop-marketing-enquiry",
-      async (apiKey) => {
-        return loopPut<LoopBooleanResponse>(
-          apiKey, `/marketing/enquiries/auto-responder/${id}/refer/${key}`,
-          {}, "loop-marketing-enquiry", teamName
-        );
-      }
-    );
-    return `Enquiry ${id} referred via team "${teamName}".`;
-  }
-
-  throw new Error(`Unknown action: ${action}`);
+  const { accountId, teamName, sellerEnquiryData } = params;
+  await withTeamKey<LoopBooleanResponse>(
+    accountId, teamName, "marketing", "loop-marketing-enquiry",
+    async (apiKey) => loopPost<LoopBooleanResponse>(
+      apiKey, "/marketing/enquiries/seller",
+      sellerEnquiryData ?? {}, "loop-marketing-enquiry", teamName
+    )
+  );
+  return `Seller enquiry submitted via team "${teamName}".`;
 }

package/payload/premium-plugins/real-agent/plugins/loop/mcp/src/tools/property-intent.ts

@@ -0,0 +1,84 @@
+import { loopPost, withTeamKey } from "../lib/loop-api.js";
+
+type Department = "sales" | "lettings";
+
+// Loop API V2: distinct operator-intent endpoints (post-Task 103 split).
+//   POST /property/residential/{dept}/{id}/viewing      — request a viewing
+//   POST /property/residential/{dept}/{id}/call-back    — request a callback
+//   POST /property/residential/{dept}/{id}/information  — request more information
+// Each endpoint takes the same {name?, email?, phone?, message?} body but is
+// surfaced as a separate tool so operator intent is preserved on the agent's
+// tool-call surface (replaces the prior bundled loop-property-request).
+
+interface LoopBooleanResponse {
+  success?: boolean;
+  [key: string]: unknown;
+}
+
+async function submitIntent(
+  toolName: string,
+  action: "viewing" | "call-back" | "information",
+  params: {
+    accountId: string;
+    teamName: string;
+    propertyId: number;
+    department: Department;
+    name?: string;
+    email?: string;
+    phone?: string;
+    message?: string;
+  },
+  successLabel: string
+): Promise<string> {
+  const { accountId, teamName, propertyId, department, ...body } = params;
+  await withTeamKey<LoopBooleanResponse>(
+    accountId,
+    teamName,
+    "properties",
+    toolName,
+    async (apiKey) => {
+      const path = `/property/residential/${department}/${propertyId}/${action}`;
+      return loopPost<LoopBooleanResponse>(apiKey, path, body, toolName, teamName);
+    }
+  );
+  return `${successLabel} submitted for property ${propertyId} (${department}) via team "${teamName}".`;
+}
+
+export async function propertyViewing(params: {
+  accountId: string;
+  teamName: string;
+  propertyId: number;
+  department: Department;
+  name?: string;
+  email?: string;
+  phone?: string;
+  message?: string;
+}): Promise<string> {
+  return submitIntent("loop-property-viewing", "viewing", params, "Viewing request");
+}
+
+export async function propertyCallback(params: {
+  accountId: string;
+  teamName: string;
+  propertyId: number;
+  department: Department;
+  name?: string;
+  email?: string;
+  phone?: string;
+  message?: string;
+}): Promise<string> {
+  return submitIntent("loop-property-callback", "call-back", params, "Callback request");
+}
+
+export async function propertyInformation(params: {
+  accountId: string;
+  teamName: string;
+  propertyId: number;
+  department: Department;
+  name?: string;
+  email?: string;
+  phone?: string;
+  message?: string;
+}): Promise<string> {
+  return submitIntent("loop-property-information", "information", params, "Information request");
+}

package/payload/premium-plugins/real-agent/plugins/loop/mcp/src/tools/property-listed.ts

@@ -39,11 +39,10 @@ export async function propertyListed(params: {
   accountId: string;
   channel: Channel;
   department?: Department;
-  includeSold?: boolean;
   teamName?: string;
   limit?: number;
 }): Promise<string> {
-  const { accountId, channel, department = "both", includeSold = false, teamName, limit } = params;
+  const { accountId, channel, department = "both", teamName, limit } = params;
 
   const result = await aggregateAcrossTeams<LoopPropertyListing>(
     accountId,
@@ -60,12 +59,6 @@ export async function propertyListed(params: {
         if (Array.isArray(data)) all.push(...data);
       }
 
-      if (includeSold) {
-        const soldPath = `/property/residential/sold/${channel}`;
-        const soldData = await loopGet<LoopPropertyListing[]>(apiKey, soldPath, "loop-property-listed", team);
-        if (Array.isArray(soldData)) all.push(...soldData);
-      }
-
       return all;
     },
     { teamName, limitPerTeam: limit ?? 50, limitTotal: limit ?? 200 }

package/payload/premium-plugins/real-agent/plugins/loop/mcp/src/tools/property-sold.ts

@@ -0,0 +1,67 @@
+import {
+  aggregateAcrossTeams,
+  formatAggregationResult,
+  loopGet,
+} from "../lib/loop-api.js";
+
+// Loop API V2: GET /property/residential/sold/{channel}
+// Distinct from /listed/{channel} — returns exchanged or completed properties
+// (Loop swagger summary: "USE THIS TO POWER WEBSITE SOLD GALLERIES").
+// Use case: canvassing letters built from recent sales in a target postcode.
+
+interface LoopPropertyListing {
+  listingId?: number;
+  propertyId?: number;
+  propertyRefId?: string;
+  channel?: string;
+  propertyAddress?: string;
+  displayAddress?: string;
+  status?: string;
+  propertyType?: string;
+  dateLaunched?: string;
+  price?: number;
+  bedrooms?: number;
+  bathrooms?: number;
+  receptionRooms?: number;
+  shortDescription?: string;
+  features?: string[];
+  images?: { url: string }[];
+  [key: string]: unknown;
+}
+
+type Channel = "rightmove" | "onTheMarket" | "zoopla" | "website";
+
+export async function propertySold(params: {
+  accountId: string;
+  channel: Channel;
+  teamName?: string;
+  limit?: number;
+}): Promise<string> {
+  const { accountId, channel, teamName, limit } = params;
+
+  const result = await aggregateAcrossTeams<LoopPropertyListing>(
+    accountId,
+    "properties",
+    "loop-property-sold",
+    async (apiKey, team) => {
+      const path = `/property/residential/sold/${channel}`;
+      const data = await loopGet<LoopPropertyListing[]>(apiKey, path, "loop-property-sold", team);
+      return Array.isArray(data) ? data : [];
+    },
+    { teamName, limitPerTeam: limit ?? 50, limitTotal: limit ?? 200 }
+  );
+
+  return formatAggregationResult(
+    result,
+    (p) => {
+      const addr = p.displayAddress ?? p.propertyAddress ?? "Unknown address";
+      const id = p.propertyId != null ? ` [ID: ${p.propertyId}]` : "";
+      const price = p.price ? ` — £${p.price.toLocaleString("en-GB")}` : "";
+      const beds = p.bedrooms ? ` ${p.bedrooms}bed` : "";
+      const propType = p.propertyType ? ` (${p.propertyType})` : "";
+      const status = p.status ? ` [${p.status}]` : "";
+      return `- ${addr}${id}${price}${beds}${propType}${status}`;
+    },
+    `${channel} sold`
+  );
+}