Skip to content

Feat: add upload model command#35

Merged
Marc-AnthonyG merged 12 commits into
mainfrom
feat/upload_model_command
Jul 14, 2026
Merged

Feat: add upload model command#35
Marc-AnthonyG merged 12 commits into
mainfrom
feat/upload_model_command

Conversation

@Zatiji

@Zatiji Zatiji commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Adds tracel model upload <name> <directory>: walks a local directory, checksums each file, requests presigned multipart upload URLs from Tracel Console, and uploads all file parts to S3 in parallel with progress reporting. It's important to note that SymbolicPaths are silently ignored inside the directory we want to upload from.

  • Creates the target model interactively if it doesn't already exist (with an optional description prompt).
  • Streams SHA-256 checksums and part offsets locally, validating the backend's presigned-URL response (part sequence/gaps and total-size) before any upload starts.
  • Uploads parts across a fixed worker pool with fail-fast cancellation on the first error.

Test plan

  • Unit tests for file collection/checksumming, part-task building (including malformed responses), and multipart upload (success + failure paths)
  • Manual end-to-end run against dev server: new model, existing model (new version), model-not-found cancel path

Copilot AI review requested due to automatic review settings July 10, 2026 18:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread crates/tracel-cli/src/commands/model.rs
Comment thread crates/tracel-cli/src/helpers/model_upload/existence.rs Outdated
Comment thread crates/tracel-cli/src/helpers/model_upload/existence.rs Outdated
Comment thread crates/tracel-cli/src/commands/model.rs Outdated
Comment thread crates/tracel-cli/src/commands/model.rs Outdated
Comment thread crates/tracel-cli/src/tools/fs.rs
@Marc-AnthonyG
Marc-AnthonyG merged commit c5164b1 into main Jul 14, 2026
1 check passed
@Marc-AnthonyG
Marc-AnthonyG deleted the feat/upload_model_command branch July 14, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants