feat: Send facilitator requests sooner

[smockle]

This PR updates the logic that filters meeting issues, so that issues up to 1 week out (previously: 1 day out) are now included when determining facilitator needs.

Specifically, the getIsTomorrow function is renamed to getIsSoon, and callsites in action and test code are updated accordingly.

[inkblotty]

Can we only get these changes to apply to Office Hours and not Deep Dives? 🙏

[smockle]

Can we only get these changes to apply to Office Hours and not Deep Dives? 🙏

Done! 6be0491

I refactored getIsTomorrow and getIsSoon into a new function: getIsWithinDays. Now, Office Hours call getIsWithinDays(date, 7), and Deep Dives call getIsWithinDays(date, 1)¹.

While I was testing², I caught an unexpected issue: The test Deep Dives › getAndMapDeepDiveIssues › has expected length and first option is high priority consistently fails. I switched to main, and discovered the test is failing there too.

Footnotes

1. It’s not an elegant function signature. I considered adding aliases, e.g. export const getIsTomorrow = (x) => getIsWithinDays(x, 1). ↩

2. npm ci && npm run build && npm run test ↩

[inkblotty]

LGTM with name change