feat(upload): print a short link to the upload when a collection id is passed

[TylerJang27]

Summary

The end-of-run 🏷️ Bundle Upload ID line becomes a link when the run is in test-collection link mode:

🏷️  Bundle Upload: https://app.trunk.io/{org}/flaky-tests/collections/{short_id}/u/{bundle_meta_id}

The webapp resolves the upload's createdAt from the id and redirects to the canonical uploads page. No RPC or response changes — CreateBundleUploadResponse.test_collection_bundle_meta_id already carries the id, and context.rs already threads it onto base_props.test_collection.

Unlike the per-test /t/ links from #1165, there is no ?repo= param: the collection short id fully scopes the webapp's lookup, so there is no repo name to resolve.

Companion webapp PR (must deploy first, it hosts the redirect page): trunk-io/trunk2#5440

Behavior

collection id passed	--hide-test-collection-links / TRUNK_HIDE_TEST_COLLECTION_LINKS	printed
yes	no	🏷️ Bundle Upload: …/collections/{short_id}/u/{bundle_meta_id}
yes	yes	🏷️ Bundle Upload ID: {id} (unchanged)
no	—	🏷️ Bundle Upload ID: {id} (unchanged)

Same gate as #1165, so the one flag controls both link kinds.

Note on the gate

collection_upload_url() reads test_collection_short_id rather than base_props.test_collection.short_id. An exported-but-blank TRUNK_TEST_COLLECTION_ID is Some(""), not None, so the latter yields a malformed /collections//u/… link — the same edge #1165 added a test for. The existing upload_bundle_without_test_collection_id_prints_repo_links test caught this.

upload_bundle's assertion changed: that test passes --test-collection-id tc_123, so by design it now gets the link rather than the bare id. The bare-id form stays covered by the new links-hidden test.

Test plan

• cargo test -p api — URL unit test for the new form (20 passing)
• cargo test -p trunk-analytics-cli — full suite green (53 + 23 + 13 + 9), including 2 new integration tests covering the gate matrix above
• cargo fmt applied; cargo clippy findings unchanged from main (5 pre-existing in upload_command.rs before and after)

🤖 Generated with Claude Code

[trunk-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.36%. Comparing base (8c100a4) to head (ca5d99b).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1168      +/-   ##
==========================================
+ Coverage   83.07%   83.36%   +0.28%     
==========================================
  Files          72       72              
  Lines       16313    16359      +46     
==========================================
+ Hits        13552    13637      +85     
+ Misses       2761     2722      -39     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[trunk-staging-io]

   

Failed Test	Failure Summary	Logs
pending_quarantine_test should be quarantined when run with variant	A test marked as pending was expected to fail but unexpectedly passed.	Logs ↗︎
variant_quarantine_test should be quarantined when run with variant	A test expected the sum of 2 + 2 to be 5, but it was actually 4, indicating a failing assertion.	Logs ↗︎

View Full Report ↗︎ ⋅ Docs

[trunk-io]

   

View Full Report ↗︎ ⋅ Docs