-
Notifications
You must be signed in to change notification settings - Fork 13
feat: add WU Error/Warning AI tool #487
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
e6ed95d to
c8d1d55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds a new Language Model (AI) tool for the ECL VS Code extension that fetches and displays errors, warnings, and exceptions from HPCC Platform workunits. The tool integrates with the existing VS Code Language Model API to provide AI-assisted debugging capabilities.
Key changes:
- New
GetWorkunitErrorsToolthat retrieves and categorizes workunit exceptions (errors, warnings, info, and other messages) - Tool registration in the ECL LM tools system with proper metadata and configuration
- Schema definition in package.json for the tool's input parameters and display properties
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/ecl/lm/tools/getWorkunitErrors.ts |
Implements the new GetWorkunitErrorsTool class with methods to fetch, categorize, and format workunit exceptions; includes helper functions for severity checking and exception formatting |
src/ecl/lm/tools.ts |
Registers the new GetWorkunitErrorsTool in the ECL LM tools registry |
package.json |
Adds tool metadata including name, display properties, tags, input schema, and model description for the getWorkunitErrors tool |
c8d1d55 to
37dc3b7
Compare
jeclrsg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@GordonSmith looked good. Just double-checked about that test.tmp file?
test.tmp
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this have been included?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope!
Signed-off-by: Gordon Smith <[email protected]>
37dc3b7 to
154f982
Compare
|
🎉 This change has been included in ecl-v2.34.0 🎉 The release is available on:
Your release-please bot 🚀🙏 |
No description provided.