feat: make the thrift get cross platform compat (#386)

## Summary

- Make the thrift gen python executable, use `py_binary` to support
python generally

## Checklist
- [ ] Added Unit Tests
- [ ] Covered by existing CI
- [ ] Integration tested
- [ ] Documentation update
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **Chores**
- Enhanced the build process for a key automation tool by streamlining
its execution and command handling, leading to improved overall build
reliability and performance.
- Transitioned the export mechanism of a Python script to a defined
executable binary target for better integration within the build system.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

<!-- av pr metadata
This information is embedded by the av CLI when creating PRs to track
the status of stacks when using Aviator. Please do not delete or edit
this section of the PR.
```
{"parent":"main","parentHead":"","trunk":"main"}
```
-->

Co-authored-by: Thomas Chow <[email protected]>

Author: tchow-zlai

scripts/codemod/BUILD.bazel

@@ -1 +1,6 @@
-exports_files(["thrift_package_replace.py"])
+py_binary(
+    name = "thrift_package_replace",
+    srcs = ["thrift_package_replace.py"],
+    main = "thrift_package_replace.py",
+    visibility = ["//visibility:public"],
+)

tools/build_rules/thrift/thrift.bzl

@@ -56,30 +56,23 @@ def _thrift_gen_library_impl(ctx):
 
 def replace_java_files_with_custom_thrift_package_prefix(ctx, input_directories):
     output_directories = []
-
-    # Get the script from the rule's files
-    script = ctx.file._python_script
-
+    script = ctx.executable._python_script
     for input_directory in input_directories:
         output_directory = ctx.actions.declare_directory(
-            input_directory.basename + "_modified",
+            input_directory.basename + "_modified"
         )
         output_directories.append(output_directory)
 
-        command = """
-            python3 "{script}" -v "{input_path}" "{output_path}"
-        """.format(
-            script = script.path,
-            input_path = input_directory.path,
-            output_path = output_directory.path,
-        )
-
-        ctx.actions.run_shell(
-            inputs = [input_directory, script],  # Include script in inputs
+        ctx.actions.run(
+            executable=script,
+            inputs = [input_directory],
             outputs = [output_directory],
-            command = command,
+            arguments = [
+                "-v",
+                input_directory.path,
+                output_directory.path
+            ],
             progress_message = "Replacing package names in input Java files for %s" % input_directory.short_path,
-            use_default_shell_env = True,
         )
 
     return output_directories
@@ -94,8 +87,9 @@ _thrift_gen_library = rule(
         ),
         "thrift_binary": attr.string(),
         "_python_script": attr.label(
-            default = "//scripts/codemod:thrift_package_replace.py",
-            allow_single_file = True,
+            default = "//scripts/codemod:thrift_package_replace",
+            executable = True,
+            cfg = "host",
         ),
     },
 )