[Debashich]

Changes

• Keep structured JSON output clean by routing warnings to stderr.
• Handle TaskRuns owned by running PipelineRuns without blocking deletion of other eligible TaskRuns.
• Return an empty deleted array when no TaskRuns are deleted in structured output.
• Add regression tests for output streams, mixed eligible/ineligible TaskRuns, and empty deletion results.
• Regenerate the TaskRun deletion command documentation.

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you review them:

• Includes [tests](https://github.com/tektoncd/community/blob/master/standards.md#principles) (if functionality changed/added)
• Run the code checkers with make check
• Regenerate the manpages, docs and go formatting with make generated
• Commit messages follow [commit message best practices](https://github.com/tektoncd/community/blob/master/standards.md#commit-messages)

See [the contribution guide](https://github.com/tektoncd/cli/blob/master/CONTRIBUTING.md) for more details.

Release Notes

Fix structured output handling for TaskRun deletion, keeping warnings off stdout and allowing eligible TaskRuns to be deleted when others are protected by running PipelineRuns.

Follow-up to #3086, addressing the review feedback on structured TaskRun deletion behavior.

[tekton-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign pratap0007 after the PR has been reviewed.
You can assign the PR to them by writing /assign @pratap0007 in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Debashich]

Hey @divyansh42 , I’ve opened a follow-up PR addressing the structured TaskRun deletion fixes as suggested. Whenever you get some time, could you please take a look? Thanks !

[Copilot]

🟡 Changes recommended

The mixed-deletion regression case uses fixtures and assertions from different test suites, leaving the central behavior change unverified.

1 open issue

• Move mixed-deletion coverage into the suite with eligible fixtures · New

What changed in this PR

This follow-up to #3086 improves structured TaskRun deletion output while preserving safeguards for TaskRuns owned by running PipelineRuns.

Changes:

• Routes warnings to stderr and skips protected TaskRuns without blocking eligible deletions.
• Emits an empty deleted array when nothing is deleted and adds regression tests.
• Updates command help and generated documentation with JSON examples and force requirements.

File	Description
pkg/​cmd/​taskrun/​delete.go	Adjusts structured output, confirmation handling, and protected deletion behavior.
pkg/​cmd/​taskrun/​delete_test.go	Adds structured-output and deletion regression cases.
docs/​man/​man1/​tkn-taskrun-delete.1	Updates manpage examples and safety guidance.
docs/​cmd/​tkn_taskrun_delete.md	Updates command reference examples and safety guidance.

_{Review effort: Balanced}

[Debashich]

Hi @divyansh42, could you please rerun the failed E2E workflow when you get a chance? The failure appears unrelated to the changes in this PR. Thanks!

[divyansh42]

/retest