feat(push): end-to-end ingest correlation id — print it on submit (backend#1028 item 3)

[LukasWodka]

Summary

Prints the run's correlation id on tracebloc dataset push (backend#1028 item 3). The idempotency key the CLI already generates and sends is becoming the end-to-end ingest correlation id — jobs-manager derives the Job name from it, labels every spawned resource with it, and stamps it into the ingestor container as TRACEBLOC_INGEST_CORRELATION_ID; the ingestor logs it and ships it in the backend registration payload. The CLI was the only layer that never showed the key, so the customer had no copy of the one string that threads all layers together. One Hintf line on every submit path (fresh and replay) fixes that. No wire change.

Related

Ref tracebloc/backend#1028 (item 3). Sibling PRs: tracebloc/client-runtime#176 (stamps the Job env) and tracebloc/data-ingestors#368 (ingestor honours it).

The cross-repo contract

• One id: the CLI's existing per-invocation idempotency key (32-char crypto/rand hex from BuildRequest, or the customer's --idempotency-key; jobs-manager caps it at 64 chars).
• This PR: print Correlation id: <key> after the submit announcement — on replay too, since a replayed run is exactly when you reach for the id to find the already-running Job.
• Downstream: client-runtime#176 sets TRACEBLOC_INGEST_CORRELATION_ID=<key> on the Job; data-ingestors#368 validates it (^[A-Za-z0-9._-]{1,64}$, fallback = today's behaviour), logs Correlation id: <id> (ingestor_id: <uuid>), and lands meta_data.correlation_id on the backend dataset row.
• Merge order: any. The env is optional at every layer; this print is standalone and true today (the label/Job-name derivation already exists in deployed jobs-managers).

Test plan

• New TestRun_PrintsCorrelationId_GeneratedKey (auto-generated key echoed) and TestRun_PrintsCorrelationId_OverrideAndReplay (--idempotency-key override echoed verbatim on the replay path).
• make ci (vet + go test -race -cover ./... + lint + fmt-check + schema-check) → all green.

Adjacency

Open PR #235 also touches internal/submit/submit_test.go (lint/dead-export cleanup) — my additions are new test funcs, worst case a trivial rebase. #232 touches internal/submit/summary.go only; this PR deliberately does not change the summary banner or its parser.

Checklist

• Tests added / updated and passing locally
• Docs updated if behavior or config changed (help text unchanged — output-only addition)
• No secrets / credentials in the diff

🤖 Generated with Claude Code

---

Note

Low Risk
Output-only hint line in the submit flow; no changes to auth, request payloads, or cluster behavior.

Overview
After a successful submit, tracebloc dataset push now prints Correlation id: <idempotency key> on every path (fresh submit and replay), right after the existing success/replay message. The value is the same idempotency key already sent to jobs-manager—no API or wire changes.

Tests assert the line appears for an auto-generated key and for an explicit --idempotency-key on replay.

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

[LukasWodka]

👋 Heads-up — Code review queue is at 51 / 30

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

• backend#1040 — docs(rfc): configurable preprocessing & imputation (RFC 0003) · author: @LukasWodka · reviewer: @saadqbal
• backend#1076 — fix(taxonomy): drop dead instance_segmentation category (#1005) · author: @LukasWodka · reviewer: @saqlainsyed007
• backend#1077 — feat(global_meta): audit/purge command for legacy per-row GlobalMeta (#1007) · author: @LukasWodka · no reviewer assigned
• backend#1084 — fix(metrics): register semseg metrics — pattern-allow iou_class_N + seed static rows (#641) · author: @LukasWodka · reviewer: @saqlainsyed007
• backend#1085 — fix(api): CycleView returns the JSONField confusion matrix; legacy fallback logged (#1075) · author: @LukasWodka · reviewer: @saqlainsyed007
• cli#229 — fix(data): deprecation notices on renamed aliases + clear no-client message (#879) · author: @LukasWodka · reviewer: @saadqbal
• cli#230 — fix(data delete): resolve the table name case-insensitively; fail closed (#1027) · author: @LukasWodka · reviewer: @saqlainsyed007
• cli#231 — fix(doctor): surface container restart history, not just crash-loop-now (#1028 item 4) · author: @LukasWodka · no reviewer assigned
• cli#232 — feat(ingest): guide users from dataset to creating a use case · author: @LukasWodka · reviewer: @saadqbal
• cli#233 — fix(data ingest): preflight & message polish — schema-type validation, friendly missing-label, help copy, clearer errors (data ingest/validate don't validate --schema column TYPE tokens locally — bad types escape the exit-2 gate #213 Missing --label-column on a tabular task gives a cryptic raw-schema error instead of naming the flag #214 data ingest --help copy is stale — 'Supports 9 tasks' (now 14) + MLM says texts/ but requires sequences/ #215 data ingest/delete: misleading validation error messages (delete '(set --name)' on a positional arg, keypoints 0/neg, --label-policy type-noise) #76) · author: @LukasWodka · no reviewer assigned

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

[aptracebloc]

Looks good 👍