chore(prebuiltconfigs): update prebuilt file to follow naming convention#3134
chore(prebuiltconfigs): update prebuilt file to follow naming convention#3134
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates prebuilt configuration files for conversational analytics, Elasticsearch, and MindsDB to use snake_case for tool and toolset names, aligning with the project's style guide. Feedback indicates that renaming the source in the conversational analytics configuration has broken references in the associated tools, which will lead to initialization errors.
|
|
||
| kind: source | ||
| name: conversational-analytics-source | ||
| name: conversational_analytics_source |
There was a problem hiding this comment.
Renaming the source to conversational_analytics_source breaks the references in the tools defined later in this file (lines 24, 33, and 42), which still use the old kebab-case name conversational-analytics-source. This will cause tool initialization to fail as the referenced source will not be found.
bca2c8b to
e07e11a
Compare
e07e11a to
462cfbf
Compare
Update tool names that are not following our convention (snake case + do not include source type).