apollo_compilation_utils: remove dead install_compiler_binary and legacy paths

[avi-starkware]

Remove code that is no longer used after the switch to script-based
compiler installation:

• Remove install_compiler_binary() and legacy_binary_path()
• Remove shared_folder_dir(), target_dir() from paths.rs
• Remove tempfile build-dependency from apollo_compilation_utils
• Update workflow artifact path and BUILD file

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

---

Note

Low Risk
Removes dead build/installation code and unused path helpers; runtime behavior should be unchanged aside from no longer supporting the legacy shared-executables location.

Overview
Removes the unused build-time install_compiler_binary flow and its related legacy path helpers (legacy_binary_path, shared_folder_dir, target_dir), leaving compiler binaries to be resolved via PATH and validated only through verify_compiler_binary.

Cleans up apollo_compilation_utils crate configuration by dropping the now-unneeded [build-dependencies] entries tied to the old installation mechanism.

^{Reviewed by Cursor Bugbot for commit 62a7a3d. Bugbot is set up for automated code reviews on this repo. Configure here.}

[reviewable-StarkWare]

This change is 

[avi-starkware]

• apollo_compilation_utils: remove dead install_compiler_binary and legacy paths #13676 👈 (View in Graphite)
• apollo_compile_to_casm,apollo_compile_to_native: switch to script-based compiler installation #13675
• ci,scripts: install LLVM before cargo tools #13674
• apollo_compilation_utils: add verify_compiler_binary #13673
• scripts: add install_compiler_binaries.sh #13672
• apollo_infra_utils,apollo_compile_to_native: add compiler version text files #13671
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

Artifacts upload workflows:

• Started at 2026-04-06T10:18:52Z
• Started at 2026-04-06T10:46:55Z
• Started at 2026-04-06T10:47:14Z
• Started at 2026-04-06T10:47:14Z

[dorimedini-starkware]

@dorimedini-starkware reviewed 5 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on avi-starkware).

---

.github/workflows/upload_artifacts_workflow.yml line 148 at r1 (raw file):

      - name: Copy starknet-native-compile for upload
        run: cp "$HOME/.cargo/bin/starknet-native-compile" target/release/starknet-native-compile

doesn't this belong in the previous PR...? without it you don't get uploaded native binary, right?

Code quote:

      - name: Copy starknet-native-compile for upload
        run: cp "$HOME/.cargo/bin/starknet-native-compile" target/release/starknet-native-compile

---

.github/workflows/upload_artifacts_workflow.yml line 155 at r1 (raw file):

        with:
          path: "target/release/starknet-native-compile"
          destination: "native_blockifier_artifacts/${{ env.SHORT_HASH }}/release/"

why copy and not just take the new source path?

Suggestion:

      - name: Upload starknet-native-compile to GCP
        id: upload_snc_file
        uses: "google-github-actions/upload-cloud-storage@v2"
        with:
          path: "$HOME/.cargo/bin/starknet-native-compile"
          destination: "native_blockifier_artifacts/${{ env.SHORT_HASH }}/release/"

[avi-starkware]

@avi-starkware made 2 comments.
Reviewable status: 2 of 5 files reviewed, 2 unresolved discussions (waiting on dorimedini-starkware).

---

.github/workflows/upload_artifacts_workflow.yml line 148 at r1 (raw file):

Previously, dorimedini-starkware wrote…

doesn't this belong in the previous PR...? without it you don't get uploaded native binary, right?

Done.

---

.github/workflows/upload_artifacts_workflow.yml line 155 at r1 (raw file):

Previously, dorimedini-starkware wrote…

why copy and not just take the new source path?

Done.

[dorimedini-starkware]

@dorimedini-starkware reviewed 3 files and all commit messages, and resolved 2 discussions.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on avi-starkware).

[github-actions]

There hasn't been any activity on this pull request recently, and in order to prioritize active work, it has been marked as stale.
This PR will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

[cursor]

PR Summary

Low Risk
Low risk cleanup that removes unused build-time installation/legacy path code; main impact is if any downstream build scripts still depended on these removed helpers.

Overview
Removes the dead install_compiler_binary build-time installer and associated legacy path helpers (shared_folder_dir, target_dir, legacy_binary_path), leaving apollo_compilation_utils focused on runtime verification via verify_compiler_binary and deterministic binary_path resolution.

Cleans up Cargo.toml by dropping the crate’s build-dependencies section that supported the removed installer.

^{Reviewed by Cursor Bugbot for commit efcf7b8. Bugbot is set up for automated code reviews on this repo. Configure here.}

[dorimedini-starkware]

@dorimedini-starkware reviewed 2 files and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on avi-starkware).