Upgrade pxapi pypi python dependencies. Replace rules_go's gogo with third_party dir version (#2181)

Summary: Upgrade pxapi pypi python dependencies. Replace rules_go gogo
with `third_party` dir version

This change is best viewed commit by commit as described below.

Commit 1: Replace `rules_go`'s version of gogo with third_party
directory copy
Commit 2: Update proto and BUILD files to use new bazel repo name
Commit 3: Regnerate go and ts/js protos. TS/JS ones are expected to not
change
Commit 4: Replace python pxapi's python-gogo with the bazel version.

The existing gogo bazel repository needed to be changed because Python
does not support periods in module names. `rules_go` and our internal
gogo repository (`gogo_grpc_proto`) used a directory structure and a
protobuf compiler patch that resulting in non functional python code (a
module named `github.com` is created). This change renames the internal
gogo repository to prevent its filepath from containing periods.

The pr-third-party-deps action is complaining about protobuf. I believe
this is a false positive as it uses the BSD-3-Clause license.

Relevant Issues: #2140

Type of change: /kind cleanup

Test Plan: Existing unit tests and the following:
- [x] Ran `src/api/python/examples` with python 3.9 and 3.13
<details><summary>Python 3.9 and 3.13 test</summary>

```
# Build the wheel
$ bazel build src/api/python:pxapi
[ ... ]
Target //src/api/python:pxapi up-to-date:
  bazel-bin/src/api/python/pxapi-0.8.1-py3-none-any.whl

# Run the example with python 3.13
$ docker run --entrypoint bash -it -v $(pwd):/src -w /src python:3.13
$ pip install bazel-bin/src/api/python/pxapi-0.8.1-py3-none-any.whl
root@d5a1f69d4e87:/src# PX_API_KEY=<api key> python src/api/python/examples/list_clusters.py
eb94bd7e-0a8a-45c0-b22e-81d62bd99755

# Run the example with python 3.9
$ docker run --entrypoint bash -it -v $(pwd):/src -w /src python:3.9
$ pip install bazel-bin/src/api/python/pxapi-0.8.1-py3-none-any.whl
root@08ed22fe680e:/src# PX_API_KEY=<api key> python src/api/python/examples/list_clusters.py
eb94bd7e-0a8a-45c0-b22e-81d62bd99755
```

</details>

Changelog Message: Upgrade the pxapi python module's third party
dependencies and support Python 3.11 through 3.13

---------

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>

Author: ddelnano

.arclint

@@ -33,7 +33,7 @@
     "(^src/ui/src/types/generated/)",
     "(^src/ui/src/types/schema\\.ts$)",
     "(^third_party/cpplint/cpplint.py$)",
-    "(^third_party/github.com/gogo/protobuf/gogoproto/gogo.proto$)",
+    "(^third_party/gogoproto/gogo.proto$)",
     "(c_cpp_properties.json$)",
     "(go\\.mod$)",
     "(go\\.sum$)"

bazel/external/protobuf_javascript.patch

@@ -6,7 +6,7 @@ index 97b5844..1351a54 100644
 
      for (int i = 0; i < file->dependency_count(); i++) {
        const std::string& name = file->dependency(i)->name();
-+      if (ModuleAlias(name) == "github_com_gogo_protobuf_gogoproto_gogo_pb") {
++      if (ModuleAlias(name) == "gogoproto_gogo_pb") {
 +        continue;
 +      }
        printer->Print(

bazel/gogo.bzl

@@ -23,12 +23,12 @@ def _gogo_grpc_proto_impl(ctx):
     ctx.file("WORKSPACE", 'workspace(name = "{}")'.format(ctx.name))
     ctx.file("BUILD.bazel", "")
     ctx.symlink(
-        ctx.path(Label("@com_github_gogo_protobuf//gogoproto:gogo.proto")),
-        "github.com/gogo/protobuf/gogoproto/gogo.proto",
+        ctx.path(Label("//third_party/gogoproto:gogo.proto")),
+        "gogoproto/gogo.proto",
     )
-    ctx.file("github.com/gogo/protobuf/gogoproto/BUILD.bazel", """
+    ctx.file("gogoproto/BUILD.bazel", """
 
-load("@px//bazel:proto_compile.bzl", "pl_proto_library", "pl_cc_proto_library")
+load("@px//bazel:proto_compile.bzl", "pl_proto_library", "pl_cc_proto_library", "pl_py_proto_library")
 
 pl_proto_library(
     name = "gogo_pl_proto",
@@ -44,6 +44,13 @@ pl_cc_proto_library(
     deps = [],
 )
 
+pl_py_proto_library(
+    name = "gogo_pl_py_proto",
+    proto = ":gogo_pl_proto",
+    visibility = ["//visibility:public"],
+    deps = [],
+)
+
     """)
 
 gogo_grpc_proto = repository_rule(

bazel/repositories.bzl

@@ -121,7 +121,7 @@ def _cc_deps():
     # Dependencies with native bazel build files.
 
     _bazel_repo("upb")
-    _bazel_repo("com_google_protobuf", patches = ["//bazel/external:protobuf_gogo_hack.patch", "//bazel/external:protobuf_text_format.patch", "//bazel/external:protobuf_warning.patch"], patch_args = ["-p1"])
+    _bazel_repo("com_google_protobuf", patches = ["//bazel/external:protobuf_text_format.patch", "//bazel/external:protobuf_warning.patch"], patch_args = ["-p1"])
     _bazel_repo("com_github_grpc_grpc", patches = ["//bazel/external:grpc.patch", "//bazel/external:grpc_go_toolchain.patch", "//bazel/external:grpc_test_visibility.patch"], patch_args = ["-p1"])
 
     _bazel_repo("boringssl", patches = ["//bazel/external:boringssl.patch"], patch_args = ["-p0"])

scripts/update_ts_protos.sh

@@ -48,7 +48,7 @@ function copy() {
     # VizierapiServiceClient.ts has a relative import; we're copying elsewhere. We fix this with perl string substitution.
     regexRelativeImport="s|import \* as ([^ ]+) from '([^ /]+/)+vizierapi_pb'\;|import * as \1 from './vizierapi_pb';|m"
     # vizierapi_pb.d.ts incorrectly includes an unused (and non-existent) relative import related to Go protos. Remove it.
-    regexExtraneousImport="s|^import \* as github_com_gogo_protobuf_gogoproto_gogo_pb.*$||m"
+    regexExtraneousImport="s|^import \* as gogoproto_gogo_pb.*$||m"
 
     for abs_path in $abs_paths; do
       echo "Propagating ${abs_path} ..."

src/api/proto/cloudpb/BUILD.bazel

@@ -26,7 +26,7 @@ pl_proto_library(
         "//src/api/proto/uuidpb:uuid_pl_proto",
         "//src/api/proto/vispb:vis_pl_proto",
         "//src/api/proto/vizierconfigpb:vizier_pl_proto",
-        "@gogo_special_proto//github.com/gogo/protobuf/gogoproto",
+        "@gogo_grpc_proto//gogoproto:gogo_pl_proto",
     ],
 )
 

src/api/proto/cloudpb/cloudapi.pb.go

@@ -28,7 +28,7 @@ option go_package = "cloudpb";
 // * Please do not include dependencies that are outside of src/api
 // * and only include protobufs that are useful to external-facing users.
 //
-import "github.com/gogo/protobuf/gogoproto/gogo.proto";
+import "gogoproto/gogo.proto";
 import "google/protobuf/empty.proto";
 import "google/protobuf/timestamp.proto";
 import "google/protobuf/wrappers.proto";

src/api/proto/cloudpb/cloudapi.proto

@@ -21,7 +21,7 @@ pl_proto_library(
     srcs = ["vis.proto"],
     visibility = ["//src:__subpackages__"],
     deps = [
-        "@gogo_special_proto//github.com/gogo/protobuf/gogoproto",
+        "@gogo_grpc_proto//gogoproto:gogo_pl_proto",
     ],
 )