fix: readme & setup file

[abhishekpatil4]

Important

Enhance Slackbot setup and functionality by replacing eval with json.loads, adding chat creation script, and updating setup steps.

• Security:
  • Replace eval with json.loads in chat/main.py for parsing tool call arguments.
• Functionality:
  • Add create_new_chat.py to create a new chat and update .env with CHAT_ID.
  • Remove db.json file from version control.
• Setup:
  • Update setup.sh to add Slackbot tool and create a new chat.
  • Remove Gmail and Google Sheets tools from setup.sh.

^{This description was created by for 9189e2d. It will automatically update as commits are pushed.}

[sweep-ai]

Hey @abhishekpatil4, here is an example of how you can ask me to improve this pull request:

@Sweep Add unit tests for the `create_new_chat()` function in `db/create_new_chat.py` to test:
- Successful chat creation with a valid codebase name
- Handling of empty codebase name (should raise a ValueError)
- Verification that the .env file is correctly updated with the CHAT_ID

📖 For more information on how to use Sweep, please read our documentation.

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 9189e2d in 36 seconds

More details

• Looked at 106 lines of code in 5 files
• Skipped 0 files when reviewing.
• Skipped posting 3 drafted comments based on config settings.

1. python/examples/advanced_agents/slackbot_code_expert/chat/main.py:32

• Draft comment:
  Good change from eval to json.loads for parsing arguments. This is a safer approach.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The use of eval to parse arguments is a security risk. The change to json.loads is a safer alternative.

2. python/examples/advanced_agents/slackbot_code_expert/db/create_new_chat.py:1

• Draft comment:
  Use a relative import for ChatDB to avoid potential import errors.

from .main import ChatDB

• Reason this comment was not posted:
  Confidence changes required: 50%
  The import statement for ChatDB should be relative to avoid potential import errors.

3. python/examples/advanced_agents/slackbot_code_expert/main.py:25

• Draft comment:
  The load_dotenv() function is called twice in the file, which is redundant. Consider removing one of the calls.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The load_dotenv() function is called twice, which is redundant and unnecessary. However, I cannot comment on the exact lines due to the diff constraints.

Workflow ID: wflow_VZGPeSdAoa99oiPw

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
composio	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 6, 2024 0:29am

[github-actions]

This comment was generated by github-actions[bot]!

JS SDK Coverage Report

📊 Coverage report for JS SDK can be found at the following URL:
https://pub-92e668239ab84bfd80ee07d61e9d2f40.r2.dev/coverage-11703554233/coverage/index.html

📁 Test report folder can be found at the following URL:
https://pub-92e668239ab84bfd80ee07d61e9d2f40.r2.dev/html-report-11703554233/html-report/report.html