Skip to content

Conversation

@ravi-jindal
Copy link

Change Description

Fixed context enhancement substring matching bug where context words were incorrectly matched as substrings (e.g., 'lic' matching 'duplicate'). Now uses whole-word matching for accurate context detection
Describe your changes

Issue reference

Fixes #1061

Checklist

  • I have reviewed the contribution guidelines
  • I have signed the CLA (if required)
  • My code includes unit tests
  • All unit tests and lint checks pass locally
  • My PR contains documentation updates / additions if required

@ravi-jindal
Copy link
Author

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

@omri374
Copy link
Collaborator

omri374 commented Jan 8, 2026

Thanks! There was a reason behind this implementation. Context words might not always be separated by spaces. For example, card and credit_card. We can decide that context words must be exhaustive, but I wonder if you have suggestions on having both options available (either defined by config or working in parallel)

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Context enhancement increasing context even if no context word is present in configured vicinity

2 participants