[Fix #192]: Add a new "Export to TXT for AI" button#193
Open
kgibm wants to merge 3 commits into
Open
Conversation
Member
Author
|
@jasonk000 @krumts Hey guys, if you have a chance, please review. Example output (snippet): |
Member
Author
1. Add a new API AIDetailsProvider class that may be optionally set in a query's getResultMetaData. 1. Bump MAT to 1.18.0 for the new API. 1. Add an AIDetailsProvider to the ThreadOverviewQuery so that 2 depths of the tree are exported including all stack frames and all stack frame locals. 1. Add a new AITextOutputter and associated export button (Export to TXT for AI...). Uses the extension .text to differentiate it from the old TextOutputter that uses .txt. AITextOutputter inherits from TextOutputter and is largely the same with the addition of a prefix to help prompt AI consumers by explaining the format of the output (e.g. nesting with pipe character prefixes) and optionally an additional prefix through AIDetailsProvider to further prompt AI consumers by explaining the output. 1. The other differences between AITextOutputter and TextOutputter are dictated by the logic in ExportActions: the totals row is suppressed as that adds a lot of fluff especially with expanded rows, and, most importantly, check if the ResultMetaData includes a non-default tree export depth. By default, we don't expand trees. Unlike the TextOutputter, the AITextOutputter ignores current expansion in the Tree and always has a uniform output. 1. I installed a fresh Eclipse 2026-06 and after loading the project and without doing anything, lots of Eclipse .project, .classpath, .prefs, etc. files were updated. I reviewed the updates and they seemed benign/reasonable, so I guess since we're still keeping Eclipse project files checked in, I'm guessing it's reasonable to commit all of these changes. I'm not sure why Eclipse changed org.eclipse.jdt.core.compiler.release from enabled to disabled. 1. Add a test that checks that the AITextOutputter works and includes the thread overview query prefix in its output. Signed-off-by: Kevin Grigorenko <kevin.grigorenko@us.ibm.com>
Signed-off-by: Kevin Grigorenko <kevin.grigorenko@us.ibm.com>
Signed-off-by: Kevin Grigorenko <kevin.grigorenko@us.ibm.com>
Contributor
|
@kgibm , I haven't looked in detail yet - but I suggest we name it "Export to Markdown (.md)" or similar, and emit standard markdown. Markdown is lingua franca standard text format for AI tooling. |
Member
Author
|
That seems reasonable although will require some larger changes. I don't think Markdown has a concept of nested tables (for
|
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.

Fixes #192