Skip to content

feat: Allow words containing keywords#185

Merged
hazeone merged 1 commit intoValueCell-ai:mainfrom
lukecold:main
Oct 27, 2025
Merged

feat: Allow words containing keywords#185
hazeone merged 1 commit intoValueCell-ai:mainfrom
lukecold:main

Conversation

@lukecold
Copy link
Contributor

📝 Pull Request Template

1. Related Issue

Closes # (issue number)

Type of Change (select one)

Type of Change: Bug Fix / New Feature / Code Refactor / Documentation Update / Other: __________

3. Description

Please describe the changes made and why they are necessary.

4. Testing

  • I have tested this locally.
  • I have updated or added relevant tests.

5. Checklist

The current agent would take all words that contains the keywords as the keyword instruction. For example: "Every trade must include a take-profit target and a stop-loss." would be taken as a "stop" instruction.

This change would only execute the keyword instruction when it comes in the whole word.

The current agent would take all words that contains the keywords as the keyword instruction.
For example: "Every trade must include a take-profit target and a stop-loss." would be taken as a "stop" instruction.

This change would only execute the keyword instruction when it comes in the whole word.
@hazeone hazeone self-requested a review October 27, 2025 03:32
Copy link
Contributor

@hazeone hazeone left a comment

Choose a reason for hiding this comment

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

LGTM. We will work on creating a more intelligent human-in-the-loop experience in the future.

@hazeone hazeone added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Oct 27, 2025
@hazeone hazeone merged commit f1e16c9 into ValueCell-ai:main Oct 27, 2025
vcfgv added a commit that referenced this pull request Oct 27, 2025
vcfgv added a commit that referenced this pull request Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants