feat(search): sort drivers list from driver index (#23)

Sort the driver list coming from the driver index. Also fixes tests
which have broken because they're hard-coded against the index and we
wiped the index today.

Author: amoeba

cmd/dbc/manifest_list_test.go

@@ -23,17 +23,14 @@ func TestUnmarshalManifestList(t *testing.T) {
 		expected []dbc.PkgInfo
 		err      error
 	}{
-		{"basic", "[drivers]\nbigquery = {version = '1.6.0'}\nflightsql = {version = '1.6.0'}", []dbc.PkgInfo{
-			{Driver: dbc.Driver{Path: "bigquery"}, Version: semver.MustParse("1.6.0")},
-			{Driver: dbc.Driver{Path: "flightsql"}, Version: semver.MustParse("1.6.0")},
+		{"basic", "[drivers]\nflightsql = {version = '1.7.0'}", []dbc.PkgInfo{
+			{Driver: dbc.Driver{Path: "flightsql"}, Version: semver.MustParse("1.7.0")},
 		}, nil},
-		{"less", "[drivers]\nbigquery = {version = '<1.6.0'}\nflightsql = {version = '<=1.6.0'}", []dbc.PkgInfo{
-			{Driver: dbc.Driver{Path: "bigquery"}, Version: semver.MustParse("1.5.0")},
-			{Driver: dbc.Driver{Path: "flightsql"}, Version: semver.MustParse("1.6.0")},
+		{"less", "[drivers]\nflightsql = {version = '<=1.7.0'}", []dbc.PkgInfo{
+			{Driver: dbc.Driver{Path: "flightsql"}, Version: semver.MustParse("1.7.0")},
 		}, nil},
-		{"greater", "[drivers]\nbigquery = {version = '>1.5.0'}\nflightsql = {version = '>=1.6.0'}", []dbc.PkgInfo{
-			{Driver: dbc.Driver{Path: "bigquery"}, Version: semver.MustParse("1.6.0")},
-			{Driver: dbc.Driver{Path: "flightsql"}, Version: semver.MustParse("1.6.0")},
+		{"greater", "[drivers]\nflightsql = {version = '>=1.7.0'}", []dbc.PkgInfo{
+			{Driver: dbc.Driver{Path: "flightsql"}, Version: semver.MustParse("1.7.0")},
 		}, nil},
 	}
 
@@ -78,17 +75,13 @@ func must[T any](v T, err error) T {
 func TestMarshalManifestList(t *testing.T) {
 	data, err := toml.Marshal(DriversList{
 		Drivers: map[string]driverSpec{
-			"bigquery":  {Version: must(semver.NewConstraint(">=1.6.0"))},
 			"flightsql": {Version: must(semver.NewConstraint(">=1.6.0"))},
 		},
 	})
 	require.NoError(t, err)
 
 	assert.Equal(t, `# dbc driver list
 [drivers]
-[drivers.bigquery]
-version = '>=1.6.0'
-
 [drivers.flightsql]
 version = '>=1.6.0'
 `, string(data))

drivers.go

@@ -78,7 +78,13 @@ var getDrivers = sync.OnceValues(func() ([]Driver, error) {
 		return nil, fmt.Errorf("failed to parse driver manifest: %s", err)
 	}
 
-	return drivers.Drivers, nil
+	// Sort by path (short name)
+	result := drivers.Drivers
+	sort.Slice(result, func(i, j int) bool {
+		return result[i].Path < result[j].Path
+	})
+
+	return result, nil
 })
 
 //go:embed columnar.pubkey