feat: add error telemetry support #778
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
There was a problem hiding this comment.
❌ Changes requested. Reviewed everything up to aece8c6 in 19 seconds
More details
- Looked at
131lines of code in5files - Skipped
1files when reviewing. - Skipped posting
2drafted comments based on config settings.
1. js/src/exceptionHandler.ts:1
- Draft comment:
The importversionfrom 'os' is unused and can be removed. - Reason this comment was not posted:
Confidence changes required:50%
The import statement for 'version' from 'os' is unused in the file. It should be removed to clean up the code.
2. js/src/index.ts:4
- Draft comment:
Remove the extra semicolon at the end of the import statement. - Reason this comment was not posted:
Confidence changes required:10%
There is an extra semicolon at the end of the import statement for OpenAIToolSet. This is a minor syntax issue.
Workflow ID: wflow_DB9w4m2FR9djsba7
Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
| nodeVersion: process.version | ||
| }; | ||
|
|
||
| axios.post(`${baseURL}/api/v1/sdk/error`, payload); |
There was a problem hiding this comment.
Consider using await with axios.post to handle potential promise rejections.
| axios.post(`${baseURL}/api/v1/sdk/error`, payload); | |
| await axios.post(`${baseURL}/api/v1/sdk/error`, payload); |
There was a problem hiding this comment.
👍 Looks good to me! Incremental review on 796467d in 6 seconds
More details
- Looked at
9lines of code in1files - Skipped
0files when reviewing. - Skipped posting
0drafted comments based on config settings.
Workflow ID: wflow_Wvqbv3JTILlQn1Dl
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
|
This comment was generated by github-actions[bot]! JS SDK Coverage Report📊 Coverage report for JS SDK can be found at the following URL: 📁 Test report folder can be found at the following URL: |
|
Hey @himanshu-dixit, here are examples of how you can ask me to improve this pull request: @Sweep Fix the CI errors. @Sweep Add unit tests for `logError` function in `exceptionHandler.ts` to verify: 📖 For more information on how to use Sweep, please read our documentation. |
Important
Adds error telemetry support with global error handling and logging in
exceptionHandler.ts, and updates related configurations.exceptionHandler.tsto log errors to a remote server using Axios.logError()andsetupErrorHandlers()for global error handling.COMPOSIO_VERSIONfromconstants.jsin error payload.COMPOSIO_VERSIONtoconstants.js.exceptionHandlerinindex.tsto initialize error handling.typedoc.jsonto include new entry points and exclude*.spec.tsfiles.This description was created by
for 796467d. It will automatically update as commits are pushed.