Skip to content

test(portal): pin now in the background-ask Inbox spec (date rot) - #3348

Merged
vybe merged 1 commit into
devfrom
fix/date-rot-inbox-ask-spec
Oct 7, 2026
Merged

vybe merged 1 commit into
devfrom
fix/date-rot-inbox-ask-spec

Conversation

@trinity-ability

Copy link
Copy Markdown
Contributor

Summary

portalBackgroundAskInboxOnly.spec.js › "Action lists it while it waits; All keeps it once it ended" fails from 2026-10-07 on dev and on every PR that runs the frontend build: allItems keeps an ended ask for ENDED_ASK_WINDOW_DAYS (7) measured from Date.now(), and the ENDED fixture is dated 2026-09-30T11:00:00Z.

The fix passes now beside the fixture (allItems already takes it). Same edit as in #3332, which is held for an unrelated finding, so it ships alone ahead of the merge train.

No issue: a dated fixture crossing its window, found by the merge train (#3345's build failed on it, reproduced on clean origin/dev).

🤖 Generated with Claude Code

…e aged out of the 7-day window

`allItems` keeps an ended ask for ENDED_ASK_WINDOW_DAYS against the wall
clock, and the spec's ENDED fixture is dated 2026-09-30, so from 2026-10-07
the "All keeps it once it ended" case fails on dev and on every PR that runs
the frontend build. Pass `now` beside the fixture, as allItems already allows.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@vybe vybe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

merge-train: date-rot prerequisite, merged alone ahead of the batch

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