Skip to content

Comments

feat(bot): enhance MindGuide prompt with character boundaries#93

Closed
AritraDey-Dev wants to merge 20 commits intosristy17:dwoc-2025from
AritraDey-Dev:feat/mindguide-prompt-enhancement
Closed

feat(bot): enhance MindGuide prompt with character boundaries#93
AritraDey-Dev wants to merge 20 commits intosristy17:dwoc-2025from
AritraDey-Dev:feat/mindguide-prompt-enhancement

Conversation

@AritraDey-Dev
Copy link
Contributor

Fixes Issue #55

Enhances the MindGuide bot's prompt system for better character consistency and security

Changes proposed

  • Added personality and clear boundaries to MindGuide bot
  • Implemented strict mental health focus with character-based responses
  • Added fallback response for off-topic questions
  • Improved prompt security against injection attempts

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Screenshot from 2025-01-29 23-17-53
Screenshot from 2025-01-29 23-18-29

Copy link
Owner

@sristy17 sristy17 left a comment

Choose a reason for hiding this comment

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

Great job, @AritraDey-Dev! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@pooranjoyb
Copy link
Collaborator

Create a folder called utils and create a file named bot.js

  • there you can create a function that returns the entire prompt text as response.
  • pass the user message to this function.

@AritraDey-Dev

@pooranjoyb pooranjoyb added good first issue Good for newcomers dwoc-2025 Issue under DWOC 2025 labels Feb 1, 2025
@AritraDey-Dev
Copy link
Contributor Author

Create a folder called utils and create a file named bot.js

* there you can create a function that returns the entire prompt text as response.

* pass the user message to this function.

@AritraDey-Dev

sure i will make these changes and let you know.

@AritraDey-Dev
Copy link
Contributor Author

AritraDey-Dev commented Feb 4, 2025

@pooranjoyb i have resolved the changes and kept the prompt in utils folder.
Can you please check and inform me if any changes you required.
Screenshot from 2025-02-04 18-52-58
Screenshot from 2025-02-04 18-53-06

@AritraDey-Dev
Copy link
Contributor Author

Also can you add level as per as dwoc guidelines as points for merging this pr is not reflecting in the profile for that.

@AritraDey-Dev
Copy link
Contributor Author

@pooranjoyb can you merge this if this changes works for the codebase?

@sristy17
Copy link
Owner

sristy17 commented Feb 4, 2025

@pooranjoyb can you merge this if this changes works for the codebase?

resolve the conflicts
also modify this, develop a character kinda (alexa or something) that can connect to the real world.
moreover try to fix the frontend the html texts line breaks and all

@AritraDey-Dev
Copy link
Contributor Author

@pooranjoyb @sristy17 i have fixed the merge conflicts.Please verify an let me know if any changes you feel required.

@pooranjoyb can you merge this if this changes works for the codebase?

resolve the conflicts also modify this, develop a character kinda (alexa or something) that can connect to the real world. moreover try to fix the frontend the html texts line breaks and all

@sristy17
Copy link
Owner

sristy17 commented Feb 4, 2025

@pooranjoyb @sristy17 i have fixed the merge conflicts.Please verify an let me know if any changes you feel required.

@pooranjoyb can you merge this if this changes works for the codebase?

resolve the conflicts also modify this, develop a character kinda (alexa or something) that can connect to the real world. moreover try to fix the frontend the html texts line breaks and all

Do the other modifications as well

@AritraDey-Dev
Copy link
Contributor Author

@sristy17 can you explain some what in detail what exactly you want me to implement.And any workflow if you have in mind.

@AritraDey-Dev
Copy link
Contributor Author

@pooranjoyb @sristy17 Can you be more specific about the issue you want me to solve?

@sristy17
Copy link
Owner

sristy17 commented Feb 7, 2025

@pooranjoyb @sristy17 Can you be more specific about the issue you want me to solve?

Basically it should interact like something real just build a character of this chatbot, it should seems like a character is interacting instead just an ai chatbot. hope u understand.

@AritraDey-Dev
Copy link
Contributor Author

Screenshot from 2025-02-09 16-59-11
@pooranjoyb @sristy17 i have made the required changes.

Copy link
Owner

@sristy17 sristy17 left a comment

Choose a reason for hiding this comment

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

Hey, Everything seems good to me I noticed that everytime I give a prompt there is a message "hello i am mindguide..."is repeated every now and then fix that, rest seems good.

@AritraDey-Dev
Copy link
Contributor Author

Screenshot from 2025-02-14 22-01-57
Screenshot from 2025-02-14 22-02-05
Screenshot from 2025-02-14 22-02-10

fixed issue of repeated greeting message by chatbot.

@sristy17
Copy link
Owner

Screenshot from 2025-02-14 22-01-57 Screenshot from 2025-02-14 22-02-05 Screenshot from 2025-02-14 22-02-10

fixed issue of repeated greeting message by chatbot.

Fine, this works.
@pooranjoyb

@pooranjoyb
Copy link
Collaborator

LGTM 👍🏼
@AritraDey-Dev Squash all your commits into 1 single commit, will merge thereafter :)

Copy link
Collaborator

@pooranjoyb pooranjoyb left a comment

Choose a reason for hiding this comment

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

Please do the requested changes and squash all commits into one.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes of this file.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes of this file.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Revert changes of this file.

@AritraDey-Dev AritraDey-Dev force-pushed the feat/mindguide-prompt-enhancement branch from b42a0b9 to db7560d Compare February 15, 2025 06:20
ad1tyayadav and others added 5 commits February 15, 2025 12:01
Made transparent Bg for Loader icon while opening the modal box.
Add dynamic theme support for modal box based on selected theme
@AritraDey-Dev AritraDey-Dev force-pushed the feat/mindguide-prompt-enhancement branch from 45674f5 to a966bd1 Compare February 15, 2025 06:34
@AritraDey-Dev
Copy link
Contributor Author

AritraDey-Dev commented Feb 15, 2025

@pooranjoyb there is some issue in solving merge conflicts i will raise a new PR for the same.
I am closing this.

@pooranjoyb
Copy link
Collaborator

Acknowledged.

@AritraDey-Dev
Copy link
Contributor Author

Acknowledged.

@pooranjoyb here is the PR for this #105.Could you please verify this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dwoc-2025 Issue under DWOC 2025 good first issue Good for newcomers level 4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants