fix(cli): hint when structured tool content is hidden - #386
Open
PLASMA-FR wants to merge 1 commit into
Open
Conversation
Author
|
The CI and PR toolkit workflows are still awaiting first-time contributor approval and have not created any jobs. When convenient, could a maintainer approve those workflow runs? The local validation and exact commands are recorded in the PR body; I will address any resulting failures. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
tools-callandtasks-resultnow show a concise--jsonhint when human output hidesstructuredContent. Existing duplicate-text filtering is retained; regression coverage, README, agent skill, and changelog are updated.Reproduced on
eb954d6with a local stdio tool returning a summary plus serialized structured data: human output showed only the summary;--jsonretained both fields.Validation:
pnpm exec vitest run test/unit/cli/output.test.ts: 189 passed, including checks that failed on the original formatter.pnpm run lint(0 errors, 6 existing warnings),pnpm run build, andpnpm run check:referencepassed../test/e2e/run.sh --no-build --parallel 3 basic/help.test.sh basic/human-output.test.sh basic/output-invariants.test.shpassed; local CLI JSON/verbose and skill checks passed under Node and Bun.pnpm test: 1,055 unit tests; E2E Node legacy 51 passed/1 skipped, Node modern 47/5, Bun legacy 51/1. All skips are existing protocol-specific exclusions.Fixes #346