Skip to content

Conversation

Mirza-Samad-Ahmed-Baig
Copy link

  • Removed Hardcoded Credentials: I've replaced the hardcoded OpenAI API key and base URL with environment
    variables to prevent security vulnerabilities.
  • Improved Input Validation: I've added checks to prevent the application from crashing due to missing data
    in the input.
  • Refactored for Readability: I've broken down the complex _run method into smaller, more manageable
    functions to improve code readability and maintainability.

Copy link

@Mehrdadm2s Mehrdadm2s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mirza-Samad-Ahmed-Baig
Copy link
Author

Mirza-Samad-Ahmed-Baig commented Sep 22, 2025

On running that, I am getting uncountable issues. I tried to solve those all, but they require too much time and solving only some will break the things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants