Skip to content

Conversation

@renatomen
Copy link
Owner

Summary

Fixes NLP status parsing regressions where:

  1. Manually typing status trigger + label fails to parse (e.g., *Done when label is Done)
  2. Status values containing temporal keywords are misparsed (e.g., 10🔥Expedite = Now has "Now" incorrectly parsed as a due date)

Changes

src/services/NaturalLanguageParser.ts

  • Strip status trigger character once before matching candidates
  • Cleaner implementation with single-pass trigger stripping
  • Maintains all existing functionality

tests/unit/services/NaturalLanguageParser.status-extraction.test.ts

  • Added comprehensive regression tests for trigger handling
  • Added tests for temporal keywords in status values
  • All 30 status extraction tests passing ✅

package-lock.json

  • Fixed version mismatch (was 3.25.6, now synced to 4.0.3 to match package.json)
  • Removed outdated peer: true flags

Testing

npm test NaturalLanguageParser.status-extraction

All 30 tests passing ✅

Related Issues

Fixes status parsing regressions introduced in 4.0+


Pull Request opened by Augment Code with guidance from the PR author

- Strip status trigger character once before matching candidates
- Fixes issue where manually typing trigger + label fails to parse e.g. "*Active = Now*"
- Fixes regression where temporal keywords in status values were misparsed "*Active = Now*" resulting in Now parsed as due date.
- All 30 status extraction tests passing
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.

2 participants