chore: Bump Go version 1.23.0 -> 1.23.1

.tool-versions

@@ -1 +1 @@
-golang 1.23.0
+golang 1.23.1

Dockerfile

@@ -15,16 +15,16 @@
 #    <other stuff to ignore>
 # And use this digest in FROM
 
-ARG base_sha=8e529b64d382182bb84f201dea3c72118f6ae9bc01d27190ffc5a54acf0091d2
+ARG base_sha=b46e8734b39f93a584eae135fc545edfdee99f1fb55c077fca908a15eb07010c
 
-FROM golang:1.23.0@sha256:${base_sha} as builder
+FROM golang:1.23.1@sha256:${base_sha} as builder
 
 COPY . /sources
 WORKDIR /sources
 RUN go build -o scip-go ./cmd/scip-go
 
 # Keep in sync with builder image
-FROM golang:1.23.0@sha256:${base_sha} as final
+FROM golang:1.23.1@sha256:${base_sha} as final
 
 COPY --from=builder /sources/scip-go /usr/bin/
 CMD ["scip-go"]

go.mod

@@ -1,6 +1,6 @@
 module github.com/sourcegraph/scip-go
 
-go 1.23
+go 1.23.1
 
 require (
 	github.com/charmbracelet/log v0.4.0

internal/document/document.go

@@ -7,8 +7,11 @@ import (
 	"go/doc"
 	"go/token"
 	"go/types"
+	"os"
 	"strings"
+	"sync"
 
+	"github.com/charmbracelet/log"
 	"github.com/sourcegraph/scip-go/internal/lookup"
 	"github.com/sourcegraph/scip-go/internal/symbols"
 	"github.com/sourcegraph/scip/bindings/go/scip"
@@ -169,7 +172,6 @@ func typeStringForObject(obj types.Object) (signature string, extra string) {
 
 	case *types.Const:
 		return fmt.Sprintf("%s = %s", types.ObjectString(v, packageQualifier), v.Val()), ""
-
 		// TODO: We had this case in previous iterations
 		// case *PkgDeclaration:
 		// 	return fmt.Sprintf("package %s", v.name), ""
@@ -181,23 +183,44 @@ func typeStringForObject(obj types.Object) (signature string, extra string) {
 	return types.ObjectString(obj, packageQualifier), ""
 }
 
+var loggedGODEBUGWarning sync.Once
+
 // formatTypeSignature returns a brief description of the given struct or interface type.
 func formatTypeSignature(obj *types.TypeName) string {
 	switch obj.Type().Underlying().(type) {
 	case *types.Struct:
 		if obj.IsAlias() {
-			switch obj.Type().(type) {
-			case *types.Named:
-				original := obj.Type().(*types.Named).Obj()
-				var pkg string
-				if obj.Pkg().Name() != original.Pkg().Name() {
-					pkg = original.Pkg().Name() + "."
+			switch ty := obj.Type().(type) {
+			case *types.Alias:
+				switch rhs := ty.Rhs().(type) {
+				case *types.Alias:
+					original := rhs.Obj()
+					var pkg string
+					if obj.Pkg().Name() != original.Pkg().Name() {
+						pkg = original.Pkg().Name() + "."
+					}
+					return fmt.Sprintf("type %s = %s%s", obj.Name(), pkg, original.Name())
+				case *types.Named:
+					original := rhs.Obj()
+					var pkg string
+					if obj.Pkg().Name() != original.Pkg().Name() {
+						pkg = original.Pkg().Name() + "."
+					}
+					return fmt.Sprintf("type %s = %s%s", obj.Name(), pkg, original.Name())
+				case *types.Struct:
+					return fmt.Sprintf("type %s = struct", obj.Name())
+				}
+			default:
+				if val := os.Getenv("GODEBUG"); strings.Contains(val, "gotypealias=0") {
+					loggedGODEBUGWarning.Do(func() {
+						log.Warn("Running with GODEBUG=gotypealias=0, this may cause incorrect hover docs")
+					})
+				} else {
+					log.Warn("IsAlias() is true but Type() is not Alias; please report this as a bug",
+						"obj", obj.String(), "obj.Type()", ty.String())
 				}
-				return fmt.Sprintf("type %s = %s%s", obj.Name(), pkg, original.Name())
-
-			case *types.Struct:
-				return fmt.Sprintf("type %s = struct", obj.Name())
 			}
+
 		}
 
 		return fmt.Sprintf("type %s struct", obj.Name())

internal/index/scip.go

@@ -4,7 +4,6 @@ import (
 	_ "embed"
 	"fmt"
 	"go/ast"
-	"os"
 	"sort"
 	"strings"
 	"sync"
@@ -88,12 +87,6 @@ func ListMissing(opts config.IndexOpts) (missing []string, err error) {
 }
 
 func Index(writer func(proto.Message), opts config.IndexOpts) error {
-	if os.Getenv("GODEBUG") == "" {
-		// TODO: Remove this after upgrading to Go 1.23.1
-		// See https://github.com/golang/go/issues/68894
-		os.Setenv("GODEBUG", "gotypesalias=0")
-	}
-
 	// Emit Metadata.
 	//   NOTE: Must be the first field emitted
 	writer(&scip.Metadata{

internal/loader/stdlib.go

@@ -1,5 +1,5 @@
 // THIS FILE IS GENERATED. SEE ./scripts/gen_std_lib.sh
-// Generated by: go version go1.23.0 darwin/arm64
+// Generated by: go version go1.23.1 darwin/arm64
 package loader
 
 var contained = struct{}{}

internal/testdata/snapshots/input/alias/main.go

@@ -5,5 +5,9 @@ package main
 type (
 	T struct{}
 	U = T
+	V = U
 	S U
+	Z int32
 )
+
+func f(u U) {}

internal/testdata/snapshots/output/alias/main.go

@@ -33,6 +33,20 @@
 // > struct{}
 // > ```
 //     ^ reference github.com/sourcegraph/scip-go . `github.com/sourcegraph/scip-go/internal/testdata/snapshots/input/alias`/T#
+   V = U
+// ^ definition github.com/sourcegraph/scip-go . `github.com/sourcegraph/scip-go/internal/testdata/snapshots/input/alias`/V#
+// documentation
+// > ```go
+// > type V = U
+// > ```
+// documentation
+// > Check that we don't panic
+// > Copied from https://github.com/golang/go/issues/68877#issuecomment-2290000187
+// documentation
+// > ```go
+// > struct{}
+// > ```
+//     ^ reference github.com/sourcegraph/scip-go . `github.com/sourcegraph/scip-go/internal/testdata/snapshots/input/alias`/U#
    S U
 // ^ definition github.com/sourcegraph/scip-go . `github.com/sourcegraph/scip-go/internal/testdata/snapshots/input/alias`/S#
 // documentation
@@ -47,5 +61,23 @@
 // > struct{}
 // > ```
 //   ^ reference github.com/sourcegraph/scip-go . `github.com/sourcegraph/scip-go/internal/testdata/snapshots/input/alias`/U#
+   Z int32
+// ^ definition github.com/sourcegraph/scip-go . `github.com/sourcegraph/scip-go/internal/testdata/snapshots/input/alias`/Z#
+// documentation
+// > Check that we don't panic
+// > Copied from https://github.com/golang/go/issues/68877#issuecomment-2290000187
+// documentation
+// > ```go
+// > int32
+// > ```
   )
 
+  func f(u U) {}
+//     ^ definition github.com/sourcegraph/scip-go . `github.com/sourcegraph/scip-go/internal/testdata/snapshots/input/alias`/f().
+//     documentation
+//     > ```go
+//     > func f(u U)
+//     > ```
+//       ^ definition local 0
+//         ^ reference github.com/sourcegraph/scip-go . `github.com/sourcegraph/scip-go/internal/testdata/snapshots/input/alias`/U#
+

internal/testdata/snapshots/output/testdata/implementations.go

@@ -68,8 +68,6 @@
 //            > ```go
 //            > func (T1).F1()
 //            > ```
-//            relationship 0.1.test `sg/testdata`/I1#F1. implementation
-//            relationship 0.1.test `sg/testdata`/I1#F1. implementation
 //            relationship 0.1.test `sg/testdata`/I1#F1. implementation
 
   type T2 int
@@ -106,15 +104,13 @@
 //     > ```go
 //     > int
 //     > ```
-//     relationship 0.1.test `sg/testdata`/I1# implementation
 //          ^^ reference 0.1.test `sg/testdata`/T1#
   type A12 = A1
 //     ^^^ definition 0.1.test `sg/testdata`/A12#
 //     documentation
 //     > ```go
 //     > int
 //     > ```
-//     relationship 0.1.test `sg/testdata`/I1# implementation
 //           ^^ reference 0.1.test `sg/testdata`/A1#
 
   type InterfaceWithNonExportedMethod interface {