fix: strip trunk prefix (0) from vCard phones, safe SQLite DSN paths

- Strip "(0)" trunk prefix before digit extraction in normalizeVCardPhone.
  Common in UK/European numbers: +44 (0)7700 means +447700, not +4407700.
- Use file: URI for WhatsApp SQLite DSN to safely handle paths containing
  '?' or other special characters.

Author: eddowding

internal/whatsapp/contacts.go

@@ -212,6 +212,10 @@ func normalizeVCardPhone(raw string) string {
 	// Check if it starts with + (already has country code).
 	hasPlus := strings.HasPrefix(raw, "+")
 
+	// Strip trunk prefix "(0)" before digit extraction.
+	// Common in UK/European numbers: +44 (0)7700 means +447700, not +4407700.
+	raw = strings.ReplaceAll(raw, "(0)", "")
+
 	// Strip everything except digits.
 	digits := nonDigitRe.ReplaceAllString(raw, "")
 	if digits == "" {

internal/whatsapp/contacts_test.go

@@ -20,6 +20,10 @@ func TestNormalizeVCardPhone(t *testing.T) {
 		{"+44 7700 900000", "+447700900000"},
 		{"+1-202-555-1234", "+12025551234"},
 
+		// Trunk prefix (0) — common in UK/European numbers
+		{"+44 (0)7700 900000", "+447700900000"},
+		{"+44(0)20 7123 4567", "+442071234567"},
+
 		// 00 prefix (international)
 		{"003-362-4921221", "+33624921221"},
 		{"0033624921221", "+33624921221"},

internal/whatsapp/importer.go

@@ -7,6 +7,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"io"
+	"net/url"
 	"os"
 	"path/filepath"
 	"strings"
@@ -39,7 +40,14 @@ func (imp *Importer) Import(ctx context.Context, waDBPath string, opts ImportOpt
 	summary := &ImportSummary{}
 
 	// Open WhatsApp DB read-only.
-	waDB, err := sql.Open("sqlite3", waDBPath+"?mode=ro&_journal_mode=WAL&_busy_timeout=5000")
+	// Use file: URI to safely handle paths containing '?' or other special characters.
+	dsn := (&url.URL{
+		Scheme:   "file",
+		OmitHost: true,
+		Path:     waDBPath,
+		RawQuery: "mode=ro&_journal_mode=WAL&_busy_timeout=5000",
+	}).String()
+	waDB, err := sql.Open("sqlite3", dsn)
 	if err != nil {
 		return nil, fmt.Errorf("open whatsapp db: %w", err)
 	}