grpc: FileMatch: tweak file_name to be bytes instead of string (#641)

api_proto.go

@@ -37,7 +37,7 @@ func FileMatchFromProto(p *proto.FileMatch) FileMatch {
 	return FileMatch{
 		Score:              p.GetScore(),
 		Debug:              p.GetDebug(),
-		FileName:           p.GetFileName(),
+		FileName:           string(p.GetFileName()), // Note: 🚨Warning, this filename may be a non-UTF8 string.
 		Repository:         p.GetRepository(),
 		Branches:           p.GetBranches(),
 		LineMatches:        lineMatches,
@@ -67,7 +67,7 @@ func (m *FileMatch) ToProto() *proto.FileMatch {
 	return &proto.FileMatch{
 		Score:              m.Score,
 		Debug:              m.Debug,
-		FileName:           m.FileName,
+		FileName:           []byte(m.FileName),
 		Repository:         m.Repository,
 		Branches:           m.Branches,
 		LineMatches:        lineMatches,

gen-proto.sh

@@ -0,0 +1,10 @@
+#!/usr/bin/env bash
+
+cd "$(dirname "${BASH_SOURCE[0]}")"
+set -euo pipefail
+
+find . -name "buf.gen.yaml" -not -path ".git" | while read -r buf_yaml; do
+  pushd "$(dirname "${buf_yaml}")" >/dev/null
+  buf generate
+  popd >/dev/null
+done

grpc/v1/webserver.proto

@@ -380,7 +380,9 @@ message FileMatch {
   // other packages can print some diagnostics.
   string debug = 2;
 
-  string file_name = 3;
+  // The repository-relative path to the file.
+  // 🚨 Warning: file_name might not be a valid UTF-8 string.
+  bytes file_name = 3;
 
   // Repository is the globally unique name of the repo of the
   // match