Render links on activity describe

[Quinn-With-Two-Ns]

Related issues

What changed?

Render links on activity describe- #1131

Checklist

Stability

• Breaking changes are marked with 💥 in the PR title and release notes
• Changes to JSON output (-o json / -o jsonl) are treated as breaking changes

Design

• This feature does not depend on Cloud-only APIs or behavior (it works against an OSS server)
• New commands follow temporal <noun> <verb> structure (e.g. temporal workflow start)
• New flags are named after the API concept, not the implementation mechanism (good: --search-attribute, bad: --index-field)
• New flags don't duplicate an existing flag that serves the same purpose
• New flags do not have short aliases without strong justification
• Experimental features are marked with (Experimental) in commands.yaml

Help text (see style guide at the top of commands.yaml)

• All flags shown in help text and examples are implemented and functional
• Summaries use sentence case and have no trailing period
• Long descriptions end with a period and include at least one example invocation
• Examples use long flags (--namespace, not -n), one flag per line
• Placeholder values use YourXxx form (YourWorkflowId, YourNamespace)

Behavior

• Results go to stdout; errors and warnings go to stderr
• Error messages are lowercase with no trailing punctuation

Tests

• Added functional test(s) (SharedServerSuite)
• Added unit test(s) (func TestXxx) where applicable

Before:

 $ temporal nexus operation describe --operation-id op-1

    OperationId  op-1
    RunId        run-1
    Endpoint     payments
    Service      PaymentService
    Operation    charge
    Status       Running
    State        Started
    Attempt      1

After:

  $ temporal nexus operation describe --operation-id op-1

    OperationId  op-1
    RunId        run-1
    Endpoint     payments
    Service      PaymentService
    Operation    charge
    Status       Running
    State        Started
    Attempt      1

  Links: 1
    temporal:///namespaces/default/workflows/payment-workflow/run-1/history?eventID=12&eventType=NexusOperationScheduled&referenceType=EventReference

Before:

  $ temporal activity describe --activity-id act-1

  Activity Execution Info:
    ActivityId            act-1
    RunId                 run-1
    Type                  MyActivity
    Status                Running
    TaskQueue             tq
    Attempt               1
    StateTransitionCount  0

  Callbacks:
    STATE      ATTEMPT
    Succeeded  2

After:

  $ temporal activity describe --activity-id act-1

  Activity Execution Info:
    ActivityId            act-1
    RunId                 run-1
    Type                  MyActivity
    Status                Running
    TaskQueue             tq
    Attempt               1
    StateTransitionCount  0

  Callbacks: 1

    URL      http://callback.example/cb
    Links
    [temporal:///namespaces/default/workflows/wf-id/run-id/history?eventID=1&eventType=WorkflowExecutionStarted&referenceType=EventReference]
    Trigger  ActivityClosed
    State    Succeeded
    Attempt  2

[chaptersix]

Please add the examples to the PR description. helps with the review.

[Quinn-With-Two-Ns]

@chaptersix This PR doesn't add any new commands so I am not sure what example would be helpful

[Copilot]

Pull request overview

This PR adds consistent rendering of Nexus-style link URLs in CLI “describe” outputs, extending link rendering to activity describe and nexus operation describe, and refactoring callback rendering into shared helpers.

Changes:

• Add shared helpers to convert/print Temporal common links as Nexus URL strings and reuse them across commands.
• Refactor workflow/activity callback rendering into a single shared printCallbacks implementation (including callback URL, trigger, links, state, etc.).
• Add unit tests covering link conversion and link/callback rendering; update a batch JSON expectation for the operation type string.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
internal/temporalcli/commands.workflow_view.go	Switch workflow describe callback rendering to shared printCallbacks.
internal/temporalcli/commands.nexus_operation.go	Print a Links section for Nexus operation descriptions.
internal/temporalcli/commands.link_test.go	Add unit tests for link conversion and for activity/nexus operation link rendering.
internal/temporalcli/commands.go	Add nexusLinkStrings, printLinks, and generic callback normalization/printing helpers.
internal/temporalcli/commands.batch_test.go	Update expected JSON operation type string.
internal/temporalcli/commands.activity.go	Render activity Links section and reuse shared callback printing.
go.mod	Bump go.temporal.io/api and several other dependencies (direct/indirect).
go.sum	Update module checksums corresponding to dependency bumps.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Quinn-With-Two-Ns]

This isn't my change, it should be fixed but I think it should be in a separate PR since this will touch a lot of files

[chaptersix]

@chaptersix This PR doesn't add any new commands so I am not sure what example would be helpful

mostly looking for a before and after. keep in mind I'm not up to speed on everything nexus or activities.

[chaptersix]

change form server?

[chaptersix]

approving assuming the breaking change from the server is resolved.