@matchuplabs/nyc-api-mcp 0.1.0 → 0.1.2

package/dist/lib/nyc-api-client.d.ts

@@ -40,7 +40,8 @@ export declare class NycApiClient {
         zipcode?: string;
     }): Promise<unknown>;
     violationsProfile(params: {
-        address: string;
+        address?: string;
+        bin?: string;
         borough?: string;
         zipcode?: string;
         block?: string;

package/dist/tools/get-building-violations.js

@@ -36,25 +36,29 @@ function isApiError(result) {
         "error" in result &&
         typeof result.error === "object");
 }
+/**
+ * Unwrap the REST API envelope: { data: { ... }, meta: { ... } }
+ * Returns the inner `data` object (the BuildingViolationProfile).
+ */
 function unwrapData(raw) {
     if (!raw || typeof raw !== "object")
         return null;
     const rec = raw;
-    if (rec.bin || rec.address || rec.bbl)
-        return rec;
+    // REST API wraps in { data: profile, meta: {...} }
     if (rec.data && typeof rec.data === "object" && !Array.isArray(rec.data)) {
         return rec.data;
     }
-    if (Array.isArray(rec.data) && rec.data.length > 0) {
-        return rec.data[0];
-    }
-    if (Array.isArray(rec) && rec.length > 0) {
-        return rec[0];
-    }
+    // If it's already the profile (has address object or totals)
+    if (rec.address || rec.totals || rec.dob_violations)
+        return rec;
     return null;
 }
 function str(val) {
-    return val != null ? String(val) : "";
+    if (val == null)
+        return "";
+    if (typeof val === "object")
+        return JSON.stringify(val);
+    return String(val);
 }
 function num(val) {
     if (val == null)
@@ -62,18 +66,17 @@ function num(val) {
     const n = Number(val);
     return Number.isNaN(n) ? null : n;
 }
-function buildSourceCoverage(data) {
-    return {
-        dob: Boolean(data.dob_violations != null ||
-            data.dob_violation_count != null ||
-            data.dob_open != null),
-        hpd: Boolean(data.hpd_violations != null ||
-            data.hpd_violation_count != null ||
-            data.hpd_open != null),
-        complaints_311: Boolean(data.complaints != null ||
-            data.complaint_count != null ||
-            data.complaints_311 != null),
-    };
+/**
+ * Build a normalized address string from the REST API's address object.
+ * API returns: { house_number, street, borough, zipcode, block, lot }
+ */
+function buildAddressString(addr) {
+    if (!addr || typeof addr !== "object")
+        return String(addr ?? "");
+    const a = addr;
+    const parts = [a.house_number, a.street].filter(Boolean);
+    const loc = [a.borough, a.zipcode].filter(Boolean);
+    return parts.join(" ") + (loc.length ? `, ${loc.join(" ")}` : "");
 }
 export async function getBuildingViolations(args) {
     const address = args.address?.trim() || undefined;
@@ -90,11 +93,13 @@ export async function getBuildingViolations(args) {
         lookupParams.bin = bin;
     if (borough)
         lookupParams.borough = borough;
-    // violationsProfile requires address — pass address or bin as address fallback
-    const profileParams = {
-        address: address || bin || "",
-        borough,
-    };
+    const profileParams = {};
+    if (address)
+        profileParams.address = address;
+    if (bin)
+        profileParams.bin = bin;
+    if (borough)
+        profileParams.borough = borough;
     let raw;
     try {
         raw = await client.violationsProfile(profileParams);
@@ -111,50 +116,64 @@ export async function getBuildingViolations(args) {
         const notFound = mcpError(ErrorCodes.NOT_FOUND, `No violation data returned for ${bin ? `BIN ${bin}` : `address "${address}"`}.`);
         return JSON.stringify(notFound);
     }
-    const sourceCoverage = buildSourceCoverage(data);
-    const dobCount = num(data.dob_violations ?? data.dob_violation_count ?? data.dob_count);
-    const hpdCount = num(data.hpd_violations ?? data.hpd_violation_count ?? data.hpd_count);
+    // --- Extract from the REST API's BuildingViolationProfile shape ---
+    // API returns: { address: {house_number, street, borough, ...}, dob_violations: [...],
+    //               hpd_violations: [...], complaints_311: [...], totals: {dob, hpd, complaints_311, total} }
+    const addressObj = data.address;
+    const totalsObj = data.totals;
+    // DOB violations array
+    const dobViolationsRaw = Array.isArray(data.dob_violations) ? data.dob_violations : [];
+    const dobViolations = dobViolationsRaw.map((v) => ({
+        type: str(v.type || v.violation_type),
+        agency: "DOB",
+        description: str(v.description || v.violation_description),
+        date: str(v.date || v.issue_date),
+        status: str(v.disposition || v.status || ""),
+        category: str(v.category || ""),
+        violation_number: str(v.violation_number || ""),
+    }));
+    // HPD violations array
+    const hpdViolationsRaw = Array.isArray(data.hpd_violations) ? data.hpd_violations : [];
+    const hpdViolations = hpdViolationsRaw.map((v) => ({
+        type: str(v.class || v.nov_type || "HPD"),
+        agency: "HPD",
+        description: str(v.description || v.novdescription),
+        date: str(v.date || v.inspectiondate),
+        status: str(v.status || v.currentstatus || ""),
+        violation_id: str(v.violation_id || v.violationid || ""),
+        rent_impairing: v.rent_impairing ?? false,
+    }));
+    // Totals — prefer the totals object from the API, fall back to array lengths
+    const dobCount = num(totalsObj?.dob) ?? dobViolations.length;
+    const hpdCount = num(totalsObj?.hpd) ?? hpdViolations.length;
+    const complaintsCount = num(totalsObj?.complaints_311) ?? 0;
     const totals = {
         dob_count: dobCount,
         hpd_count: hpdCount,
-        total: num(data.total_violations ??
-            data.violation_count ??
-            (dobCount ?? 0) + (hpdCount ?? 0)),
+        complaints_311_count: complaintsCount,
+        total: num(totalsObj?.total) ?? (dobCount + hpdCount + complaintsCount),
+    };
+    // Source coverage
+    const sourceCoverage = {
+        dob: dobViolations.length > 0 || (totalsObj?.dob != null),
+        hpd: hpdViolations.length > 0 || (totalsObj?.hpd != null),
+        complaints_311: Array.isArray(data.complaints_311) || (totalsObj?.complaints_311 != null),
     };
-    // Extract open violations array
-    const rawOpen = data.open_violations || data.violations || data.open;
-    const openViolations = Array.isArray(rawOpen)
-        ? rawOpen.map((v) => ({
-            type: str(v.violation_type || v.type),
-            agency: str(v.agency || v.source),
-            description: str(v.description || v.violation_description),
-            date: str(v.date || v.violation_date || v.issued_date),
-            status: str(v.status || v.disposition || "open"),
-            ...v,
-        }))
-        : [];
     const profile = {
-        normalized_address: str(data.normalized_address || data.address),
-        bin: str(data.bin || bin),
-        bbl: str(data.bbl),
-        borough: str(data.borough),
+        normalized_address: buildAddressString(addressObj),
+        bin: str(addressObj?.bin || data.bin || bin),
+        bbl: str(addressObj?.bbl || data.bbl),
+        borough: str(addressObj?.borough || data.borough || borough),
+        block: str(addressObj?.block || data.block),
+        lot: str(addressObj?.lot || data.lot),
         totals,
-        open_violations: openViolations,
+        dob_violations: dobViolations,
+        hpd_violations: hpdViolations,
         source_coverage: sourceCoverage,
     };
     if (args.include_complaints) {
-        const complaints = data.complaints_311 || data.complaints || data.complaint_history;
+        const complaints = data.complaints_311;
         profile.complaints_311 = Array.isArray(complaints) ? complaints : [];
     }
-    if (args.include_resolved_counts) {
-        const dobResolved = num(data.dob_resolved ?? data.dob_resolved_count);
-        const hpdResolved = num(data.hpd_resolved ?? data.hpd_resolved_count);
-        profile.resolved_counts = {
-            dob: dobResolved,
-            hpd: hpdResolved,
-            total: num(data.total_resolved ??
-                (dobResolved ?? 0) + (hpdResolved ?? 0)),
-        };
-    }
     return JSON.stringify(profile);
 }

package/package.json

@@ -1,20 +1,33 @@
 {
   "name": "@matchuplabs/nyc-api-mcp",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "MCP server for NYC property intelligence, building violations, and restaurant/venue compliance data via nycapi.app",
   "type": "module",
   "main": "dist/index.js",
   "bin": {
     "nyc-api-mcp": "dist/index.js"
   },
-  "mcpName": "io.github.matchuplabs/nyc-api",
+  "mcpName": "io.github.MatchupLabs/nyc-api",
   "license": "MIT",
   "repository": {
     "type": "git",
     "url": "https://github.com/MATCHUP-LABS/nycapi.git"
   },
-  "keywords": ["mcp", "nyc", "property", "violations", "restaurant", "real-estate", "api", "ai-agent"],
-  "files": ["dist", "server.json", "README.md"],
+  "keywords": [
+    "mcp",
+    "nyc",
+    "property",
+    "violations",
+    "restaurant",
+    "real-estate",
+    "api",
+    "ai-agent"
+  ],
+  "files": [
+    "dist",
+    "server.json",
+    "README.md"
+  ],
   "scripts": {
     "build": "tsc",
     "dev": "tsx src/index.ts",

package/server.json

@@ -1,12 +1,36 @@
 {
-  "name": "io.github.matchuplabs/nyc-api",
+  "$schema": "https://static.modelcontextprotocol.io/schemas/2025-12-11/server.schema.json",
+  "name": "io.github.MatchupLabs/nyc-api",
   "title": "NYC Property & Venue API",
-  "description": "NYC property intelligence, building violations, and restaurant/venue compliance — unified, agent-ready MCP tools.",
-  "version": "0.1.0",
+  "description": "NYC property intelligence, building violations, and restaurant/venue compliance via MCP.",
+  "version": "0.1.2",
+  "repository": {
+    "url": "https://github.com/MATCHUP-LABS/nycapi",
+    "source": "github"
+  },
   "packages": [
     {
       "registryType": "npm",
-      "name": "@matchuplabs/nyc-api-mcp"
+      "registryBaseUrl": "https://registry.npmjs.org",
+      "identifier": "@matchuplabs/nyc-api-mcp",
+      "version": "0.1.2",
+      "transport": {
+        "type": "stdio"
+      },
+      "environmentVariables": [
+        {
+          "name": "NYC_API_KEY",
+          "description": "API key from nycapi.app",
+          "isRequired": true,
+          "isSecret": true
+        },
+        {
+          "name": "NYC_API_BASE_URL",
+          "description": "API base URL (default: https://nycapi.app)",
+          "isRequired": false,
+          "isSecret": false
+        }
+      ]
     }
   ]
 }