cricinfo-cli-go 0.1.0

package/internal/cricinfo/entity_index_test.go

@@ -0,0 +1,175 @@
+package cricinfo
+
+import (
+	"path/filepath"
+	"testing"
+	"time"
+)
+
+func TestEntityIndexSearchExactAndFuzzy(t *testing.T) {
+	t.Parallel()
+
+	idx, err := OpenEntityIndex(filepath.Join(t.TempDir(), "index.json"))
+	if err != nil {
+		t.Fatalf("OpenEntityIndex error: %v", err)
+	}
+
+	now := time.Now().UTC()
+	if err := idx.Upsert(IndexedEntity{
+		Kind:      EntityPlayer,
+		ID:        "1361257",
+		Ref:       "http://core.espnuk.org/v2/sports/cricket/athletes/1361257",
+		Name:      "Fazal Haq Shaheen",
+		ShortName: "Fazal",
+		Aliases:   []string{"Fazal Haq", "Shaheen"},
+		UpdatedAt: now,
+	}); err != nil {
+		t.Fatalf("Upsert player 1 error: %v", err)
+	}
+
+	if err := idx.Upsert(IndexedEntity{
+		Kind:      EntityPlayer,
+		ID:        "999",
+		Name:      "John Doe",
+		Aliases:   []string{"Johnny"},
+		UpdatedAt: now.Add(-time.Minute),
+	}); err != nil {
+		t.Fatalf("Upsert player 2 error: %v", err)
+	}
+
+	exact := idx.Search(EntityPlayer, "1361257", 5, SearchContext{})
+	if len(exact) == 0 || exact[0].ID != "1361257" {
+		t.Fatalf("expected exact id match for 1361257, got %+v", exact)
+	}
+
+	fuzzy := idx.Search(EntityPlayer, "faz sha", 5, SearchContext{})
+	if len(fuzzy) == 0 || fuzzy[0].ID != "1361257" {
+		t.Fatalf("expected fuzzy alias match for 'faz sha', got %+v", fuzzy)
+	}
+}
+
+func TestEntityIndexSearchContextBoost(t *testing.T) {
+	t.Parallel()
+
+	idx, err := OpenEntityIndex(filepath.Join(t.TempDir(), "index.json"))
+	if err != nil {
+		t.Fatalf("OpenEntityIndex error: %v", err)
+	}
+
+	if err := idx.Upsert(IndexedEntity{
+		Kind:      EntityMatch,
+		ID:        "1529474",
+		Name:      "3rd Match",
+		LeagueID:  "19138",
+		MatchID:   "1529474",
+		UpdatedAt: time.Now().UTC(),
+	}); err != nil {
+		t.Fatalf("Upsert preferred match error: %v", err)
+	}
+
+	if err := idx.Upsert(IndexedEntity{
+		Kind:      EntityMatch,
+		ID:        "1529999",
+		Name:      "3rd Match",
+		LeagueID:  "11132",
+		MatchID:   "1529999",
+		UpdatedAt: time.Now().UTC(),
+	}); err != nil {
+		t.Fatalf("Upsert non-preferred match error: %v", err)
+	}
+
+	results := idx.Search(EntityMatch, "3rd match", 5, SearchContext{PreferredLeagueID: "19138"})
+	if len(results) == 0 {
+		t.Fatalf("expected search results")
+	}
+	if results[0].ID != "1529474" {
+		t.Fatalf("expected context-preferred match first, got %+v", results)
+	}
+}
+
+func TestEntityIndexSearchPrefersExactMatchContext(t *testing.T) {
+	t.Parallel()
+
+	idx, err := OpenEntityIndex(filepath.Join(t.TempDir(), "index.json"))
+	if err != nil {
+		t.Fatalf("OpenEntityIndex error: %v", err)
+	}
+
+	now := time.Now().UTC()
+	entities := []IndexedEntity{
+		{
+			Kind:      EntityPlayer,
+			ID:        "253802",
+			Name:      "Virat Kohli",
+			LeagueID:  "11132",
+			MatchID:   "1527689",
+			UpdatedAt: now,
+		},
+		{
+			Kind:      EntityPlayer,
+			ID:        "999999",
+			Name:      "Virat Kohli",
+			LeagueID:  "19138",
+			MatchID:   "1529474",
+			UpdatedAt: now.Add(time.Minute),
+		},
+	}
+	if err := idx.UpsertMany(entities); err != nil {
+		t.Fatalf("UpsertMany error: %v", err)
+	}
+
+	results := idx.Search(EntityPlayer, "virat kohli", 5, SearchContext{
+		PreferredLeagueID: "11132",
+		PreferredMatchID:  "1527689",
+	})
+	if len(results) == 0 {
+		t.Fatalf("expected contextual player search results")
+	}
+	if results[0].ID != "253802" {
+		t.Fatalf("expected preferred match-context player first, got %+v", results)
+	}
+	for _, result := range results {
+		if result.MatchID != "1527689" {
+			t.Fatalf("expected results to be narrowed to preferred match context, got %+v", results)
+		}
+	}
+}
+
+func TestEntityIndexSearchDoesNotMatchInitialForFullNameQuery(t *testing.T) {
+	t.Parallel()
+
+	idx, err := OpenEntityIndex(filepath.Join(t.TempDir(), "index.json"))
+	if err != nil {
+		t.Fatalf("OpenEntityIndex error: %v", err)
+	}
+
+	now := time.Now().UTC()
+	if err := idx.UpsertMany([]IndexedEntity{
+		{
+			Kind:      EntityPlayer,
+			ID:        "253802",
+			Name:      "Virat Kohli",
+			ShortName: "Kohli",
+			Aliases:   []string{"Virat Kohli", "V Kohli"},
+			UpdatedAt: now,
+		},
+		{
+			Kind:      EntityPlayer,
+			ID:        "1408688",
+			Name:      "Vaibhav Sooryavanshi",
+			ShortName: "Sooryavanshi",
+			Aliases:   []string{"Vaibhav Sooryavanshi", "V Sooryavanshi"},
+			UpdatedAt: now.Add(time.Minute),
+		},
+	}); err != nil {
+		t.Fatalf("UpsertMany error: %v", err)
+	}
+
+	results := idx.Search(EntityPlayer, "Virat Kohli", 5, SearchContext{})
+	if len(results) == 0 {
+		t.Fatalf("expected player search results")
+	}
+	if results[0].ID != "253802" {
+		t.Fatalf("expected exact full-name match first, got %+v", results)
+	}
+}

package/internal/cricinfo/fixture_matrix.go

@@ -0,0 +1,243 @@
+package cricinfo
+
+import (
+	"fmt"
+	"sort"
+	"strings"
+)
+
+// FixtureFamily groups fixtures and live probes by API resource family.
+type FixtureFamily string
+
+const (
+	FixtureFamilyRootDiscovery      FixtureFamily = "root-discovery"
+	FixtureFamilyMatchesCompetition FixtureFamily = "matches-competitions"
+	FixtureFamilyDetailsPlays       FixtureFamily = "details-plays"
+	FixtureFamilyTeamCompetitor     FixtureFamily = "team-competitor"
+	FixtureFamilyInningsDepth       FixtureFamily = "innings-fow-partnerships"
+	FixtureFamilyPlayers            FixtureFamily = "players"
+	FixtureFamilyLeagueSeason       FixtureFamily = "leagues-seasons-standings"
+	FixtureFamilyAuxCompetitionMeta FixtureFamily = "aux-competition-metadata"
+)
+
+// FixtureSpec defines a curated fixture/live endpoint pairing.
+type FixtureSpec struct {
+	Family      FixtureFamily
+	Name        string
+	Ref         string
+	FixturePath string
+	LiveProbe   bool
+}
+
+var fixtureSpecs = []FixtureSpec{
+	{
+		Family:      FixtureFamilyRootDiscovery,
+		Name:        "root",
+		Ref:         "/v2/sports/cricket",
+		FixturePath: "root-discovery/root.json",
+		LiveProbe:   true,
+	},
+	{
+		Family:      FixtureFamilyRootDiscovery,
+		Name:        "events-page",
+		Ref:         "/events",
+		FixturePath: "root-discovery/events.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyMatchesCompetition,
+		Name:        "competition",
+		Ref:         "/leagues/19138/events/1529474/competitions/1529474",
+		FixturePath: "matches-competitions/competition.json",
+		LiveProbe:   true,
+	},
+	{
+		Family:      FixtureFamilyMatchesCompetition,
+		Name:        "matchcards",
+		Ref:         "/leagues/11132/events/1527966/competitions/1527966/matchcards",
+		FixturePath: "matches-competitions/matchcards-1527966.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyMatchesCompetition,
+		Name:        "situation",
+		Ref:         "/leagues/19138/events/1529474/competitions/1529474/situation",
+		FixturePath: "matches-competitions/situation-1529474.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyDetailsPlays,
+		Name:        "plays-page",
+		Ref:         "/leagues/19138/events/1529474/competitions/1529474/plays?limit=1",
+		FixturePath: "details-plays/plays.json",
+		LiveProbe:   true,
+	},
+	{
+		Family:      FixtureFamilyDetailsPlays,
+		Name:        "detail-item",
+		Ref:         "/leagues/19138/events/1529474/competitions/1529474/details/110",
+		FixturePath: "details-plays/detail-110.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyTeamCompetitor,
+		Name:        "competitor",
+		Ref:         "/leagues/19138/events/1529474/competitions/1529474/competitors/789643",
+		FixturePath: "team-competitor/competitor-789643.json",
+		LiveProbe:   true,
+	},
+	{
+		Family:      FixtureFamilyTeamCompetitor,
+		Name:        "competitor-roster",
+		Ref:         "/leagues/11132/events/1475396/competitions/1475396/competitors/1147772/roster",
+		FixturePath: "team-competitor/roster-1147772.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyInningsDepth,
+		Name:        "innings-period",
+		Ref:         "/leagues/1098952/events/1475396/competitions/1475396/competitors/1147772/linescores/1/2",
+		FixturePath: "innings-fow-partnerships/innings-1-2.json",
+		LiveProbe:   true,
+	},
+	{
+		Family:      FixtureFamilyInningsDepth,
+		Name:        "fow",
+		Ref:         "/leagues/1098952/events/1475396/competitions/1475396/competitors/1147772/linescores/1/2/fow",
+		FixturePath: "innings-fow-partnerships/fow.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyInningsDepth,
+		Name:        "partnerships",
+		Ref:         "/leagues/1098952/events/1475396/competitions/1475396/competitors/1147772/linescores/1/2/partnerships",
+		FixturePath: "innings-fow-partnerships/partnerships.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyPlayers,
+		Name:        "athlete-profile",
+		Ref:         "/athletes/1361257",
+		FixturePath: "players/athlete-1361257.json",
+		LiveProbe:   true,
+	},
+	{
+		Family:      FixtureFamilyPlayers,
+		Name:        "athlete-statistics",
+		Ref:         "/athletes/1361257/statistics",
+		FixturePath: "players/athlete-1361257-statistics.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyLeagueSeason,
+		Name:        "standings",
+		Ref:         "/leagues/19138/standings",
+		FixturePath: "leagues-seasons-standings/standings.json",
+		LiveProbe:   true,
+	},
+	{
+		Family:      FixtureFamilyLeagueSeason,
+		Name:        "seasons",
+		Ref:         "/leagues/19138/seasons",
+		FixturePath: "leagues-seasons-standings/seasons.json",
+		LiveProbe:   false,
+	},
+	{
+		Family:      FixtureFamilyAuxCompetitionMeta,
+		Name:        "officials",
+		Ref:         "/leagues/11132/events/1527944/competitions/1527944/officials",
+		FixturePath: "aux-competition-metadata/officials.json",
+		LiveProbe:   true,
+	},
+	{
+		Family:      FixtureFamilyAuxCompetitionMeta,
+		Name:        "broadcasts",
+		Ref:         "/leagues/11132/events/1527944/competitions/1527944/broadcasts",
+		FixturePath: "aux-competition-metadata/broadcasts.json",
+		LiveProbe:   false,
+	},
+}
+
+// FixtureMatrix returns a copy of the curated fixture endpoint matrix.
+func FixtureMatrix() []FixtureSpec {
+	out := make([]FixtureSpec, len(fixtureSpecs))
+	copy(out, fixtureSpecs)
+	return out
+}
+
+// FixtureFamilies returns sorted family names represented in the matrix.
+func FixtureFamilies() []FixtureFamily {
+	seen := map[FixtureFamily]struct{}{}
+	for _, spec := range fixtureSpecs {
+		seen[spec.Family] = struct{}{}
+	}
+
+	families := make([]FixtureFamily, 0, len(seen))
+	for family := range seen {
+		families = append(families, family)
+	}
+	sort.Slice(families, func(i, j int) bool { return families[i] < families[j] })
+
+	return families
+}
+
+// ParseFixtureFamilies parses a comma-separated family filter.
+func ParseFixtureFamilies(raw string) (map[FixtureFamily]struct{}, error) {
+	raw = strings.TrimSpace(raw)
+	if raw == "" {
+		return nil, nil
+	}
+
+	allowed := map[FixtureFamily]struct{}{}
+	for _, family := range FixtureFamilies() {
+		allowed[family] = struct{}{}
+	}
+
+	selected := map[FixtureFamily]struct{}{}
+	parts := strings.Split(raw, ",")
+	for _, part := range parts {
+		family := FixtureFamily(strings.TrimSpace(part))
+		if family == "" {
+			continue
+		}
+		if _, ok := allowed[family]; !ok {
+			return nil, fmt.Errorf("unknown fixture family %q", family)
+		}
+		selected[family] = struct{}{}
+	}
+
+	if len(selected) == 0 {
+		return nil, nil
+	}
+
+	return selected, nil
+}
+
+// FilterFixtureMatrixByFamily filters the matrix by selected families.
+func FilterFixtureMatrixByFamily(matrix []FixtureSpec, selected map[FixtureFamily]struct{}) []FixtureSpec {
+	if len(selected) == 0 {
+		out := make([]FixtureSpec, len(matrix))
+		copy(out, matrix)
+		return out
+	}
+
+	out := make([]FixtureSpec, 0, len(matrix))
+	for _, spec := range matrix {
+		if _, ok := selected[spec.Family]; ok {
+			out = append(out, spec)
+		}
+	}
+
+	return out
+}
+
+// LiveProbeMatrix returns one or more live probe entries from the matrix.
+func LiveProbeMatrix(matrix []FixtureSpec) []FixtureSpec {
+	out := make([]FixtureSpec, 0, len(matrix))
+	for _, spec := range matrix {
+		if spec.LiveProbe {
+			out = append(out, spec)
+		}
+	}
+	return out
+}

package/internal/cricinfo/fixture_matrix_test.go

@@ -0,0 +1,49 @@
+package cricinfo
+
+import "testing"
+
+func TestFixtureMatrixCoversAllMajorFamilies(t *testing.T) {
+	t.Parallel()
+
+	required := []FixtureFamily{
+		FixtureFamilyRootDiscovery,
+		FixtureFamilyMatchesCompetition,
+		FixtureFamilyDetailsPlays,
+		FixtureFamilyTeamCompetitor,
+		FixtureFamilyInningsDepth,
+		FixtureFamilyPlayers,
+		FixtureFamilyLeagueSeason,
+		FixtureFamilyAuxCompetitionMeta,
+	}
+
+	matrix := FixtureMatrix()
+	found := map[FixtureFamily]int{}
+	for _, spec := range matrix {
+		found[spec.Family]++
+		if spec.Name == "" || spec.Ref == "" || spec.FixturePath == "" {
+			t.Fatalf("invalid fixture spec %+v", spec)
+		}
+	}
+
+	for _, family := range required {
+		if found[family] == 0 {
+			t.Fatalf("missing fixture coverage for family %q", family)
+		}
+	}
+}
+
+func TestParseFixtureFamilies(t *testing.T) {
+	t.Parallel()
+
+	selected, err := ParseFixtureFamilies("players,details-plays")
+	if err != nil {
+		t.Fatalf("ParseFixtureFamilies error: %v", err)
+	}
+	if len(selected) != 2 {
+		t.Fatalf("expected 2 selected families, got %d", len(selected))
+	}
+
+	if _, err := ParseFixtureFamilies("players,unknown"); err == nil {
+		t.Fatalf("expected parse error for unknown family")
+	}
+}