-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat: Consider labels when providing the context #1730
feat: Consider labels when providing the context #1730
Conversation
AppMap runtime code review
Code anti-patterns🚨 New problems detected (4)Unfulfilled PromiseDescription
View in AppMapStack traceUnfulfilled PromiseDescription
View in AppMapStack traceUnfulfilled PromiseDescription
View in AppMapStack traceUnfulfilled PromiseDescription
View in AppMapStack trace🎉 Problems resolved (4)Unfulfilled PromiseDescription
View in AppMapStack traceUnfulfilled PromiseDescription
View in AppMapStack traceUnfulfilled PromiseDescription
View in AppMapStack traceUnfulfilled PromiseDescription
View in AppMapStack trace⭐ New AppMaps[jest] ContextCollector collectContext appmaps handles search across all appmaps [jest] ContextCollector collectContext appmaps returns context for specified appmaps [jest] FileIndex when matches are found does not match directory nanmes [jest] FileIndex when matches are found matches alphanumeric file names [jest] FileIndex when matches are found returns matching file names [jest] listProjectFiles identifies binary files correctly [jest] listProjectFiles ignores directories specified in IGNORE DIRECTORIES [jest] listProjectFiles lists all non-binary, non-ignored files in a directory [jest] SourceIndex when matches are found returns matching source code snippets ✖️ Removed AppMaps[jest] ContextCollector collectContext handles search across all appmaps [jest] ContextCollector collectContext returns context for specified appmaps |
88f7912
to
72e7ec1
Compare
4056467
to
3f6a5b6
Compare
The patches generated by AI will generally not apply cleanly.
Try and correct some common mistakes.
e7e3e91
to
4d4418c
Compare
When providing the explain context, consider the labels that have been applied to the question by the AI classifier.
Initially, the behavior is this:
This is an effort to focus on high-level documentation and content that contains the name of the project itself.
In my ad-hoc testing, the classifier has done a good job of applying the "architecture" and "overview" labels. We should reflect these labels back to the user in the context view also (RHS of the Navie UI).