Move visitor, bridge client generation, and formatting to poe tasks

Author: VegetarianOrc

.github/workflows/ci.yml

@@ -117,7 +117,6 @@ jobs:
           uv sync --all-extras
           poe build-develop
           poe gen-protos
-          poe format
           [[ -z $(git status --porcelain temporalio) ]] || (git diff temporalio; echo "Protos changed"; exit 1)
           poe test -s
         timeout-minutes: 10

pyproject.toml

@@ -61,10 +61,21 @@ build-develop-with-release = { cmd = "uv run maturin develop --release --uv" }
 format = [
   { cmd = "uv run ruff check --select I --fix" },
   { cmd = "uv run ruff format" },
+  { cmd = "cargo fmt", cwd = "temporalio/bridge" },
 ]
 gen-docs = "uv run scripts/gen_docs.py"
-gen-protos = "uv run scripts/gen_protos.py"
-gen-protos-docker = "uv run scripts/gen_protos_docker.py"
+gen-protos = [
+  { cmd = "uv run scripts/gen_protos.py" },
+  { cmd = "uv run scripts/gen_payload_visitor.py" },
+  { cmd = "uv run scripts/gen_bridge_client.py" },
+  { ref = "format" },
+]
+gen-protos-docker = [
+  { cmd = "uv run scripts/gen_protos_docker.py" },
+  { cmd = "uv run scripts/gen_payload_visitor.py" },
+  { cmd = "uv run scripts/gen_bridge_client.py" },
+  { ref = "format" },
+]
 lint = [
   { cmd = "uv run ruff check --select I" },
   { cmd = "uv run ruff format --check" },

scripts/_proto/Dockerfile

@@ -10,6 +10,6 @@ COPY ./ ./
 RUN mkdir -p ./temporalio/api
 RUN uv add "protobuf<4"
 RUN uv sync --all-extras
-RUN poe gen-protos
+RUN uv run scripts/gen_protos.py
 
 CMD ["sh", "-c", "cp -r ./temporalio/api/* /api_new && cp -r ./temporalio/bridge/proto/* /bridge_new"]

scripts/gen_protos_docker.py

@@ -30,21 +30,3 @@
     ],
     check=True,
 )
-
-subprocess.run(
-    ["uv", "run", os.path.join(os.getcwd(), "scripts", "gen_payload_visitor.py")],
-    check=True,
-)
-
-subprocess.run(
-    ["uv", "run", os.path.join(os.getcwd(), "scripts", "gen_bridge_client.py")],
-    check=True,
-)
-
-subprocess.run(["uv", "run", "poe", "format"], check=True)
-
-subprocess.run(
-    ["cargo", "fmt"],
-    check=True,
-    cwd=os.path.join(os.getcwd(), "temporalio", "bridge"),
-)