feat: retrieve full number/matrix map

Author: gsanchietti

api/routes.go

@@ -5,9 +5,9 @@ import (
 	"net/http"
 	"strings"
 
+	"github.com/labstack/echo/v4"
 	"github.com/nethesis/matrix2acrobits/models"
 	"github.com/nethesis/matrix2acrobits/service"
-	"github.com/labstack/echo/v4"
 )
 
 const adminTokenHeader = "X-Super-Admin-Token"
@@ -75,7 +75,12 @@ func (h handler) getMapping(c echo.Context) error {
 
 	smsNumber := strings.TrimSpace(c.QueryParam("sms_number"))
 	if smsNumber == "" {
-		return echo.NewHTTPError(http.StatusBadRequest, "sms_number query parameter is required")
+		// return full mappings list when sms_number is not provided
+		respList, err := h.svc.ListMappings()
+		if err != nil {
+			return echo.NewHTTPError(http.StatusInternalServerError, err.Error())
+		}
+		return c.JSON(http.StatusOK, respList)
 	}
 
 	resp, err := h.svc.LookupMapping(smsNumber)

docs/openapi.yaml

@@ -146,15 +146,19 @@ paths:
           name: sms_number
           schema:
             type: string
-          required: true
-          description: The SMS number or identifier previously mapped.
+          required: false
+          description: The SMS number or identifier previously mapped. If omitted, the endpoint returns the full list of mappings.
       responses:
         '200':
           description: Mapping found
           content:
             application/json:
               schema:
-                $ref: '#/components/schemas/MappingResponse'
+                oneOf:
+                  - $ref: '#/components/schemas/MappingResponse'
+                  - type: array
+                    items:
+                      $ref: '#/components/schemas/MappingResponse'
         '401':
           description: Invalid admin token.
         '404':

service/messages.go

@@ -200,6 +200,17 @@ func (s *MessageService) LookupMapping(smsNumber string) (*models.MappingRespons
 	return s.buildMappingResponse(entry), nil
 }
 
+// ListMappings returns all stored mappings as MappingResponse slices.
+func (s *MessageService) ListMappings() ([]*models.MappingResponse, error) {
+	s.mu.RLock()
+	defer s.mu.RUnlock()
+	out := make([]*models.MappingResponse, 0, len(s.mappings))
+	for _, entry := range s.mappings {
+		out = append(out, s.buildMappingResponse(entry))
+	}
+	return out, nil
+}
+
 // SaveMapping persists a new SMS-to-Matrix mapping via the admin API.
 func (s *MessageService) SaveMapping(req *models.MappingRequest) (*models.MappingResponse, error) {
 	smsNumber := strings.TrimSpace(req.SMSNumber)

service/messages_test.go

@@ -3,6 +3,7 @@ package service
 import (
 	"testing"
 
+	"github.com/nethesis/matrix2acrobits/models"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -152,3 +153,44 @@ func TestMapAuthErr(t *testing.T) {
 		})
 	}
 }
+
+func TestListMappings(t *testing.T) {
+	svc := NewMessageService(nil)
+
+	// Seed two mappings
+	svc.setMapping(mappingEntry{
+		SMSNumber: "+111",
+		MatrixID:  "@alice:example.com",
+		RoomID:    "!room1:example.com",
+	})
+	svc.setMapping(mappingEntry{
+		SMSNumber: "+222",
+		MatrixID:  "@bob:example.com",
+		RoomID:    "!room2:example.com",
+	})
+
+	list, err := svc.ListMappings()
+	assert.NoError(t, err)
+	// We expect two mappings; order is not guaranteed.
+	assert.Len(t, list, 2)
+
+	// Build a map for easy assertions
+	m := make(map[string]*models.MappingResponse)
+	for _, it := range list {
+		m[it.SMSNumber] = it
+	}
+
+	if v, ok := m["+111"]; ok {
+		assert.Equal(t, "@alice:example.com", v.MatrixID)
+		assert.Equal(t, "!room1:example.com", v.RoomID)
+	} else {
+		t.Fatalf("missing mapping for +111")
+	}
+
+	if v, ok := m["+222"]; ok {
+		assert.Equal(t, "@bob:example.com", v.MatrixID)
+		assert.Equal(t, "!room2:example.com", v.RoomID)
+	} else {
+		t.Fatalf("missing mapping for +222")
+	}
+}