More LLM detections for other CLI agents + alter TL behavior in the presence of LLMs #51054
Conversation
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
This was referenced Sep 29, 2025
dsplaisted
approved these changes
Sep 29, 2025
nagilson
approved these changes
Sep 29, 2025
baronfel
changed the title
|
Approved over email |
…, refactoring relevant codepaths to work in terms of MSBuildArgs for consistency, and aligning on a constant
YuliiaKovalova
pushed a commit
to dotnet/msbuild
that referenced
this pull request
Oct 2, 2025
…#12584) Fixes part of #12486, but not all. ### Context This introduces a new Terminal Logger Parameter (`DISABLENODEDISPLAY`) that disables the live-updating node display portion of the Terminal Logger output. This output is very wasteful when LLMs are running the build, and the hope is that we can toggle this flag automatically at the dotnet CLI level to maintain the structure benefits of TL while also not flooding the context. ### Changes Made * introduced a new parameter and threaded it through the logic ### Testing * Manual testing done - verified that the flag does in fact disable the display entirely. https://github.com/user-attachments/assets/f6e9f74b-77c2-46be-a1ae-7d974d5d7348 ### Notes This will use the logic from dotnet/sdk#51054 in the SDK to turn off the live-update feature. Co-authored-by: Chet Husk <[email protected]>
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.
Impact
Add two new known LLM agentic CLIs, and a proposed 'generic' option to drive discussion across other tools/products.
We want to try and use this detection to change the behavior of the product in a way that is more useful to LLMs - specifically enabling terminal logger and disabling it's live-update portion. This change, plus dotnet/msbuild#12581 from MSBuild, will let us do the first example of that behavioral change.
Once we detect an LLM is present, we tell the MSBuild Terminal Logger to disable its live-update feature, which is token-inefficient.
Risk
Low for this change - it's additional detection + telemetry plus a flag that has zero impact if TL is not used on a particular build.