Merge pull request #54 from geolonia/feature/issue-657-geoproperty-table-display

fix(cli): format GeoProperty as readable coordinates in table display

Author: miya0001

src/output.ts

@@ -98,14 +98,38 @@ function collectKeys(items: Record<string, unknown>[]): string[] {
   return sorted;
 }
 
+function formatGeoJSON(geo: Record<string, unknown>): string {
+  const geoType = String(geo.type);
+  const coords = geo.coordinates;
+  if (geoType === "Point" && Array.isArray(coords) && coords.length >= 2) {
+    return `Point(${Number(coords[0]).toFixed(2)}, ${Number(coords[1]).toFixed(2)})`;
+  }
+  if (Array.isArray(coords)) {
+    const count = geoType === "Polygon"
+      ? (Array.isArray(coords[0]) ? (coords[0] as unknown[]).length : 0)
+      : coords.length;
+    return `${geoType}(${count} coords)`;
+  }
+  return `${geoType}(...)`;
+}
+
+function isGeoJSON(v: unknown): boolean {
+  if (typeof v !== "object" || v === null) return false;
+  const o = v as Record<string, unknown>;
+  return typeof o.type === "string" && "coordinates" in o;
+}
+
 function cellValue(val: unknown): string {
   if (val === undefined || val === null) return "";
-  if (typeof val === "object") {
-    const obj = val as Record<string, unknown>;
-    if ("value" in obj) return String(obj.value);
-    return JSON.stringify(val);
+  if (typeof val !== "object") return String(val);
+  const obj = val as Record<string, unknown>;
+  if (isGeoJSON(obj)) return formatGeoJSON(obj);
+  if ("value" in obj) {
+    const v = obj.value;
+    if (isGeoJSON(v)) return formatGeoJSON(v as Record<string, unknown>);
+    return String(v);
   }
-  return String(val);
+  return JSON.stringify(val);
 }
 
 function toGeoJSON(data: unknown): unknown {

tests/output.test.ts

@@ -101,6 +101,125 @@ describe("output", () => {
       expect(result).toContain('{"nested":true,"count":5}');
     });
 
+    it("formats GeoProperty Point as readable coordinates (NGSIv2)", () => {
+      const data = [
+        {
+          id: "e1",
+          location: {
+            type: "geo:json",
+            value: { type: "Point", coordinates: [139.7, 35.6] },
+          },
+        },
+      ];
+      const result = stripAnsi(formatOutput(data, "table"));
+      expect(result).toContain("Point(139.70, 35.60)");
+    });
+
+    it("formats GeoProperty Point as readable coordinates (NGSI-LD)", () => {
+      const data = [
+        {
+          id: "e1",
+          location: {
+            type: "GeoProperty",
+            value: { type: "Point", coordinates: [139.7, 35.6] },
+          },
+        },
+      ];
+      const result = stripAnsi(formatOutput(data, "table"));
+      expect(result).toContain("Point(139.70, 35.60)");
+    });
+
+    it("formats GeoJSON directly in keyValues mode", () => {
+      const data = [
+        {
+          id: "e1",
+          location: { type: "Point", coordinates: [139.7, 35.6] },
+        },
+      ];
+      const result = stripAnsi(formatOutput(data, "table"));
+      expect(result).toContain("Point(139.70, 35.60)");
+    });
+
+    it("formats LineString with coordinate count", () => {
+      const data = [
+        {
+          id: "e1",
+          route: {
+            type: "GeoProperty",
+            value: {
+              type: "LineString",
+              coordinates: [
+                [139.7, 35.6],
+                [139.8, 35.7],
+                [139.9, 35.8],
+              ],
+            },
+          },
+        },
+      ];
+      const result = stripAnsi(formatOutput(data, "table"));
+      expect(result).toContain("LineString(3 coords)");
+    });
+
+    it("formats Polygon with coordinate count", () => {
+      const data = [
+        {
+          id: "e1",
+          area: {
+            type: "GeoProperty",
+            value: {
+              type: "Polygon",
+              coordinates: [
+                [
+                  [139.7, 35.6],
+                  [139.8, 35.6],
+                  [139.8, 35.7],
+                  [139.7, 35.6],
+                ],
+              ],
+            },
+          },
+        },
+      ];
+      const result = stripAnsi(formatOutput(data, "table"));
+      expect(result).toContain("Polygon(4 coords)");
+    });
+
+    it("formats MultiPoint with coordinate count", () => {
+      const data = [
+        {
+          id: "e1",
+          stops: {
+            type: "GeoProperty",
+            value: {
+              type: "MultiPoint",
+              coordinates: [
+                [139.7, 35.6],
+                [139.8, 35.7],
+                [139.9, 35.8],
+              ],
+            },
+          },
+        },
+      ];
+      const result = stripAnsi(formatOutput(data, "table"));
+      expect(result).toContain("MultiPoint(3 coords)");
+    });
+
+    it("handles Polygon with empty coordinates", () => {
+      const data = [
+        {
+          id: "e1",
+          area: {
+            type: "GeoProperty",
+            value: { type: "Polygon", coordinates: [] },
+          },
+        },
+      ];
+      const result = stripAnsi(formatOutput(data, "table"));
+      expect(result).toContain("Polygon(0 coords)");
+    });
+
     it("returns empty string for null/undefined values", () => {
       const data = [{ id: "e1", n: null, u: undefined }];
       const result = stripAnsi(formatOutput(data, "table"));