-
Notifications
You must be signed in to change notification settings - Fork 0
Improve Birth Wheel response state #886
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -98,6 +98,9 @@ | |
| /\bhigh[\s-]?valence\s+date\b/i, | ||
| /\bwhen\s+is\s+my\s+next\s+(?:high[\s-]?valence|positive\s+bias)\b/i, | ||
| /\bwhen\s+(?:is|does)\s+(?:the\s+)?(?:next|best)\s+(?:high[\s-]?valence|positive\s+bias|expansion)\b/i, | ||
| /\b(?:tell\s+me\s+about|read|show|check|pull|fetch|run)\b[\s.?!,;:…-]{0,20}\b(?:tonight|(?:this\s+)?(?:evening|morning|afternoon))\b/i, | ||
|
Check warning on line 101 in vessel/src/lib/raven/symbolicMomentIntent.ts
|
||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Because this new timing-window regex is not anchored to the start of the utterance or another request context, any sentence containing verbs like Useful? React with 👍 / 👎. |
||
| /\b(?:what(?:s| is)|how(?:s| is))\b[\s.?!,;:…-]{0,80}\b(?:active|looking|running|happening|pressing|live|the\s+(?:field|sky|pressure|moment))\b[\s.?!,;:…-]{0,40}\b(?:tonight|(?:this\s+)?(?:evening|morning|afternoon))\b/i, | ||
|
Check warning on line 102 in vessel/src/lib/raven/symbolicMomentIntent.ts
|
||
| /\bhow(?:s| is)\b[\s.?!,;:…-]{0,20}\b(?:tonight|(?:this\s+)?(?:evening|morning|afternoon))\b[\s.?!,;:…-]{0,40}\b(?:looking|running|happening|landing|feeling)\b/i, | ||
| ] as const; | ||
|
|
||
| const NEW_SYMBOLIC_READ_PATTERNS = [ | ||
|
|
@@ -415,7 +418,7 @@ | |
| return hasWindowOpener && hasWindowMarker && hasWindowTarget; | ||
| } | ||
|
|
||
| export function isSymbolicMomentsIntent( | ||
|
Check failure on line 421 in vessel/src/lib/raven/symbolicMomentIntent.ts
|
||
| userMessage: string, | ||
| hasWindowCue: boolean, | ||
| context?: { | ||
|
|
@@ -445,7 +448,7 @@ | |
| if (/\b(?:how\s+is)\s+(?:the\s+)?(?:sky|field|symbolic\s+moment|pressure)\b/i.test(normalized)) return true; | ||
| if (/\b(?:what(?:s|'s| is))\s+(?:going\s+on|doing\s+on|happening)\s+(?:in\s+(?:the\s+)?(?:sky|field|chart|transits)|right\s+now|today|this\s+week|astrologically)\b/i.test(normalized)) return true; | ||
| if (/\b(?:what(?:s|'s| is))\s+(?:doing|going)\s+on\s+right\s+now\b/i.test(normalized)) return true; | ||
| if (/\b(?:ask\s+about|tell\s+me\s+about)\s+(?:today|this\s+week|right\s+now|the\s+current\s+(?:sky|moment|field|pressure|symbolic\s+moment))\b/i.test(normalized)) return true; | ||
| if (/\b(?:ask\s+about|tell\s+me\s+about)\s+(?:today|tonight|this\s+(?:morning|afternoon|evening|week)|right\s+now|the\s+current\s+(?:sky|moment|field|pressure|symbolic\s+moment))\b/i.test(normalized)) return true; | ||
|
Check warning on line 451 in vessel/src/lib/raven/symbolicMomentIntent.ts
|
||
|
|
||
| if (/\b(?:what(?:'s|s|\s+is)\s+(?:just\s+)?loud|just\s+what\s+(?:is|it)\s+loud|read\s+(?:what(?:'s|s|\s+is)\s+)?loud)\b/i.test(normalized)) return true; | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When a captured phrase combines another date with a day part, this branch treats the whole phrase as same-day before extracting the explicit date. For example,
parseLooseDateToken('tomorrow evening', new Date('2026-06-04T12:00:00Z'))now returns2026-06-04, and ranged phrases likestarting tomorrow morningbuild a window from today instead of June 5. That sends future timing reads to the current day's geometry whenever users add morning/evening/afternoon to a non-today date.Useful? React with 👍 / 👎.