fix(geocoding): show street + house number for unnamed addresses

Unnamed address/building features have no OSM name, so the display name
fell back to display_name.split(",")[0]. In DE/AT/CH, display_name leads
with the bare house number, yielding just "40" for both the place panel
title and the directions field.

Add a country-aware formatStreetLine() and use it in the name fallback
chain (place-lookup) and the autocomplete short label (nominatim provider)
so addresses render as "Kinderhauser Straße 40".

Author: Medformatik

integrations/geocoding-nominatim/provider.ts

@@ -1,4 +1,4 @@
-import { formatAddress } from "@openmapx/integration-geocoding/format-address";
+import { formatAddress, formatStreetLine } from "@openmapx/integration-geocoding/format-address";
 import type { GeocodingProvider as GeocodingProviderImpl } from "@openmapx/integration-geocoding/types";
 /**
  * Nominatim geocoding client.
@@ -31,9 +31,16 @@ interface NominatimResult {
   lat: string;
   lon: string;
   display_name: string;
+  /** Empty for unnamed address/building features; a POI name otherwise. */
+  name?: string;
   class: string;
   type: string;
   importance: number;
+  address?: {
+    road?: string;
+    house_number?: string;
+    country_code?: string;
+  };
 }
 
 function mapType(cls: string, type: string): SearchResult["type"] {
@@ -169,7 +176,13 @@ export const nominatimService: GeocodingProviderImpl = {
   async autocomplete(query: string, lang?: string): Promise<AutocompleteResult[]> {
     const data = await fetchNominatim({ q: query, limit: "6", dedupe: "1" }, lang);
     return data.map((r) => {
-      const short = r.display_name.split(",")[0].trim();
+      // Unnamed address/building features lead display_name with the bare house
+      // number in DE/AT/CH, so derive a proper street line ("Kinderhauser
+      // Straße 40") instead of splitting off "40".
+      const short =
+        r.name?.trim() ||
+        (r.address && formatStreetLine(r.address)) ||
+        r.display_name.split(",")[0].trim();
       return {
         id: makeId(r),
         label: short,

integrations/geocoding/__tests__/format-address.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from "vitest";
-import { formatAddress } from "../format-address.js";
+import { formatAddress, formatStreetLine } from "../format-address.js";
 
 // Guards the @fragaria/address-formatter integration (v7): country-aware
 // templates put the house number before/after the street depending on
@@ -37,3 +37,31 @@ describe("formatAddress", () => {
     expect(formatAddress({})).toBe("");
   });
 });
+
+// Derives the display name for unnamed address/building features. Nominatim's
+// display_name leads with the bare house number in DE/AT/CH, so a naive
+// first-segment split yields just "40"; formatStreetLine must rebuild the
+// country-ordered street line instead.
+describe("formatStreetLine", () => {
+  it("puts the house number after the street for German addresses", () => {
+    expect(
+      formatStreetLine({ road: "Kinderhauser Straße", house_number: "40", country_code: "de" }),
+    ).toBe("Kinderhauser Straße 40");
+  });
+
+  it("puts the house number before the street for US addresses", () => {
+    expect(formatStreetLine({ road: "Market St", house_number: "1", country_code: "us" })).toBe(
+      "1 Market St",
+    );
+  });
+
+  it("returns just the street when there is no house number", () => {
+    expect(formatStreetLine({ road: "Kinderhauser Straße", country_code: "de" })).toBe(
+      "Kinderhauser Straße",
+    );
+  });
+
+  it("returns an empty string when there is no street", () => {
+    expect(formatStreetLine({ house_number: "40", country_code: "de" })).toBe("");
+  });
+});

integrations/geocoding/format-address.ts

@@ -76,3 +76,26 @@ export function formatAddress(
 ): string {
   return runFormatter(components, { appendCountry: options.appendCountry ?? true }).join(", ");
 }
+
+/**
+ * Format just the street line — "Kinderhauser Straße 40" (DE) or
+ * "40 Main Street" (US). House-number/street order is country-dependent, so we
+ * reuse the formatter rather than concatenating by hand. Returns "" when there
+ * is no street to render.
+ *
+ * Used to derive a display name for address-type places (buildings, house
+ * numbers) that have no POI name of their own — without this, Nominatim's
+ * `display_name` leads with the bare house number in DE/AT/CH ("40, …"), so a
+ * naive `display_name.split(",")[0]` yields just "40".
+ */
+export function formatStreetLine(components: AddressComponents): string {
+  if (!components.road) return "";
+  return runFormatter(
+    {
+      road: components.road,
+      house_number: components.house_number,
+      country_code: components.country_code,
+    },
+    { appendCountry: false },
+  ).join(", ");
+}

integrations/geocoding/place-lookup.ts

@@ -19,7 +19,7 @@ import {
   parseId,
   USER_AGENT,
 } from "@openmapx/core";
-import { formatAddress } from "./format-address.js";
+import { formatAddress, formatStreetLine } from "./format-address.js";
 import { resolveOsmLabel } from "./osm-label.js";
 
 function normalizeIdValue(s: string | undefined): string {
@@ -230,7 +230,10 @@ function toPlace(r: NominatimDetailResult, id: string): Place {
   }
 
   const address = formatAddress(r.address) || r.display_name;
-  const name = r.name?.trim() || r.display_name.split(",")[0].trim();
+  // Address-type places (buildings/house numbers) have no POI name. Derive a
+  // proper street line ("Kinderhauser Straße 40") instead of falling back to
+  // display_name's first segment, which is the bare house number in DE/AT/CH.
+  const name = r.name?.trim() || formatStreetLine(r.address) || r.display_name.split(",")[0].trim();
   const city = r.address.city ?? r.address.town ?? r.address.village ?? r.address.county;
 
   return createPlace({