Update pod hashing to remove non-replication concerns from hash

[Synicix]

• Remove source commit for now (Perhaps we should discuss to add a additional_label member where it is HashMap<String, String> and the user can put whatever they want in there. This would be far more flexible and allow for custom labels
• Remove recommended_cpu and recommended_memory and combined it into RecommendSpecs struct that no longer affects the hash of the pod
• Update GPU requirement specs as that is very code dependent and is a all or nothing when it come to reproducibility
• Clean up a few things I caught to make the code base more robust

Depends on #114

Resolves PIPE-119

[Copilot]

Pull Request Overview

This PR refactors pod hashing to exclude non-replication concerns from the hash computation, focusing on reproducibility requirements. The changes consolidate execution recommendations into a separate structure and update GPU requirements to better reflect code dependencies.

• Removed source commit URL and individual CPU/memory fields from pod hash
• Consolidated CPU and memory recommendations into RecommendSpecs struct
• Updated GPU requirements to use CUDA version instead of specific models
• Added comprehensive pipeline execution infrastructure with Docker runner

Reviewed Changes

Copilot reviewed 46 out of 47 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
src/uniffi/model/pod.rs	Core changes to Pod structure, removing hash-affecting fields and adding RecommendSpecs
src/uniffi/store/filestore.rs	Updated to handle separate storage of recommendation specs with timestamps
tests/store.rs	Updated tests to use new Pod constructor and RecommendSpecs structure
src/core/pipeline_runner.rs	New comprehensive pipeline execution infrastructure
tests/fixture/mod.rs	Updated fixture functions to use new Pod API and added pipeline test utilities

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[codecov]

Codecov Report

❌ Patch coverage is 86.22951% with 126 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/core/pipeline_runner.rs	81.37%	92 Missing ⚠️
src/core/error.rs	0.00%	9 Missing ⚠️
src/core/operator.rs	96.59%	6 Missing ⚠️
src/uniffi/model/packet.rs	50.00%	6 Missing ⚠️
src/uniffi/model/pipeline.rs	64.28%	5 Missing ⚠️
src/core/crypto.rs	89.28%	3 Missing ⚠️
src/uniffi/orchestrator/docker.rs	81.25%	3 Missing ⚠️
src/core/model/mod.rs	92.85%	1 Missing ⚠️
src/uniffi/store/filestore.rs	96.42%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[eywalker]

Indeed, we should turn your points into issues to be tracked. Namely:

1. add additional_label so that users can add arbitrary information. Among these are Git hashes. Generally we should discuss the topic of capturing the concept of execution environment more broadly.
2. consider placing annotation about expected reproducibiility (e.g. GPU-based code can have not perfectly reproducible execution).

[eywalker]

We should most definitely turn this into a module so we can have smaller src code dedicated to each operator, especially as we add operators in the future.