return empty datasource model instead of an error if datasource name doesn't match

Author: ioanarm

tools/datasources.go

@@ -100,6 +100,9 @@ func getDatasourceByName(ctx context.Context, args GetDatasourceByNameParams) (*
 	c := mcpgrafana.GrafanaClientFromContext(ctx)
 	datasource, err := c.Datasources.GetDataSourceByName(args.Name)
 	if err != nil {
+		if strings.Contains(err.Error(), "404") {
+			return &models.DataSource{}, nil // Not found, return empty datasource without error
+		}
 		return nil, fmt.Errorf("get datasource by name %s: %w", args.Name, err)
 	}
 	return datasource.Payload, nil

tools/datasources_test.go

@@ -92,4 +92,14 @@ func TestDatasourcesTools(t *testing.T) {
 		require.NoError(t, err)
 		assert.Equal(t, "Prometheus", result.Name)
 	})
+
+	t.Run("get datasource by name - not found", func(t *testing.T) {
+		ctx := newTestContext()
+		result, err := getDatasourceByName(ctx, GetDatasourceByNameParams{
+			Name: "non-existent-datasource",
+		})
+		require.NoError(t, err)
+		assert.NotNil(t, result)
+		assert.Equal(t, "", result.Name)
+	})
 }