Update sync to have --force instead of --fresh and include the input path in the idempotency key

[Ankur Goyal (ankrgyl)]

bt sync push would sometimes say "i already sync'd this", but it hadn't.

[github-actions]

Latest downloadable build artifacts for this PR commit 532019e962e4:

• Workflow run: https://github.com/braintrustdata/bt/actions/runs/29384990548
• Download all artifacts (GitHub CLI): gh run download 29384990548 --repo braintrustdata/bt
• Installers are published from main automatically. To publish one for a PR branch, run release-canary manually via workflow_dispatch.

Available artifact names

• artifacts-build-global
• artifacts-build-local-x86_64-apple-darwin
• artifacts-build-local-x86_64-pc-windows-msvc
• artifacts-build-local-aarch64-pc-windows-msvc
• artifacts-build-local-x86_64-unknown-linux-musl
• artifacts-build-local-aarch64-apple-darwin
• artifacts-build-local-x86_64-unknown-linux-gnu
• artifacts-build-local-aarch64-unknown-linux-gnu
• artifacts-plan-dist-manifest
• cargo-dist-cache

[Cedric / ViaDézo1er (viadezo1er)]

I think there are some other issues introduced give me a sec

[Cedric / ViaDézo1er (viadezo1er)]

demo.sh:

OLD=/Users/cedric@braintrustdata.com/.local/bin/bt
NEW=/Users/cedric@braintrustdata.com/repos/bt-sync-bug/target/debug/bt
printf '{"input":{"q":"v1"},"expected":"1"}\n' >data.jsonl
$OLD sync push dataset:sync-demo4 --project aaa --org ced-test-1 --profile ced-test-1 --no-input --in data.jsonl --root ./bt-sync
$NEW sync push dataset:sync-demo4 --project aaa --org ced-test-1 --profile ced-test-1 --no-input --in data.jsonl --root ./bt-sync

❯ sh demo.sh
Dataset 'sync-demo4' not found; created new dataset 'sync-demo4' (798e507d-a6ff-4111-ad0d-b1e18cab77ef) in project 'aaa'.
  Ctrl+C safely checkpoints; rerun same command to resume (--fresh restarts).
Push complete
  URL: https://www.braintrust.dev/app/ced-test-1/p/aaa/datasets/sync-demo4
  Input: data.jsonl
  Time: 00:01
  Traces: 1
  Spans: 1
  Batches: 1
  Data: 295.00 B (295 bytes)
  Rates: 1.00 traces/s | 1.00 spans/s | 295.00 B/s
  Ctrl+C safely checkpoints; rerun same command to resume (--force restarts).
Push complete
  URL: https://www.braintrust.dev/app/ced-test-1/p/aaa/datasets/sync-demo4
  Input: data.jsonl
  Time: 00:01
  Traces: 1
  Spans: 1
  Batches: 1
  Data: 295.00 B (295 bytes)
  Rates: 1.00 traces/s | 1.00 spans/s | 295.00 B/s

Adding input_path to the spec makes the hash different, which means customers who have already pushed a dataset, update bt then push the dataset again will push the entire dataset again even if it's unchanged.

Maybe not worth adding code for compatibility but I'd like a confirmation it's ok.

[Cedric / ViaDézo1er (viadezo1er)]

The bug I found existed before this pr, so it doesn't block it.