fix: don't do a full camel case check against exported verbs (#1791)

It forced users to use eg. `GetUserById` rather than `GetUserByID`,
which we currently don't want to do.

go-runtime/compile/schema.go

@@ -14,10 +14,11 @@ import (
 	"unicode"
 	"unicode/utf8"
 
-	"github.com/TBD54566975/ftl/go-runtime/schema/analyzers"
 	"github.com/alecthomas/types/optional"
 	"golang.org/x/exp/maps"
 
+	"github.com/TBD54566975/ftl/go-runtime/schema/analyzers"
+
 	"github.com/TBD54566975/ftl/backend/schema"
 	"github.com/TBD54566975/ftl/backend/schema/strcase"
 	"github.com/TBD54566975/ftl/internal/goast"
@@ -1224,8 +1225,8 @@ func visitFuncDecl(pctx *parseContext, node *ast.FuncDecl) (verb *schema.Verb) {
 		return nil
 	}
 
-	if expVerbName := strcase.ToUpperCamel(node.Name.Name); node.Name.Name != expVerbName {
-		pctx.errors.add(errorf(node, "unexpected verb name %q, did you mean to use %q instead?", node.Name.Name, expVerbName))
+	if expName := exportedName(node.Name.Name); node.Name.Name != expName {
+		pctx.errors.add(errorf(node, "verb %q is not exported, did you mean to use %q instead?", node.Name.Name, expName))
 		return nil
 	}
 
@@ -1831,3 +1832,10 @@ func isIotaEnum(node ast.Node) bool {
 		return false
 	}
 }
+
+func exportedName(name string) string {
+	if name == "" {
+		return ""
+	}
+	return string(unicode.ToUpper(rune(name[0]))) + name[1:]
+}