Estimate waiting room wait as 24 seconds per spot ahead

[jahooma]

Reduces the queued wait-time estimate from 60 seconds to 24 seconds (0.4 × 60) per spot ahead, better reflecting actual admission throughput. Updates the constant in session-view.ts and the matching test constant.

[greptile-apps]

Greptile Summary

This PR reduces the queued wait-time estimate shown to users from 60 seconds (1 minute) to 24 seconds per spot ahead in the waiting room queue, better reflecting observed admission throughput. Both the production constant in session-view.ts and the mirrored test constant in session-view.test.ts are updated consistently.

• WAIT_MS_PER_SPOT_AHEAD changed from 60_000 → 24_000 in session-view.ts
• JSDoc comment updated to reflect the new 24-second unit
• Test constant WAIT_PER_SPOT_MS updated from 60_000 → 24_000 in the test file
• Minor: the test description on line 30 still refers to "minutes" rather than seconds — a trivial cosmetic inconsistency left behind by the change

Confidence Score: 5/5

Safe to merge — purely a tuning change to a display-only wait estimate with no logic or data-path risk.

The change is a single numeric constant update with a matching test update. The constant only affects a rough UX estimate shown to queued users; it has no impact on admission logic, data integrity, or security. The only finding is a stale 'minutes' word in a test description, which is cosmetic and non-blocking.

No files require special attention.

Important Files Changed

Filename	Overview
web/src/server/free-session/session-view.ts	Constant WAIT_MS_PER_SPOT_AHEAD updated from 60 000 ms (1 min) to 24 000 ms (24 s); JSDoc comment updated accordingly. No logic changes.
web/src/server/free-session/tests/session-view.test.ts	Test constant WAIT_PER_SPOT_MS updated to match; test on line 30 still describes the unit as 'minutes' which is now inaccurate after the change to 24 s.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User polling session state] --> B{row status?}
    B -- active & not expired --> C[Return active response\nwith remainingMs]
    B -- active & in grace window --> D[Return ended response\nwith gracePeriodRemainingMs]
    B -- active & past grace window --> E[Return null\ncaller re-queues]
    B -- queued --> F[estimateWaitMs\nposition - 1 × 24 000 ms]
    F --> G[Return queued response\nwith estimatedWaitMs]
    B -- null row --> H[Return null]

Loading

Comments Outside Diff (1)

1. web/src/server/free-session/__tests__/session-view.test.ts, line 30 (link)

   Stale test description after constant change

   The test description still says 'position N → (N-1) minutes ahead' but with the new constant of 24 seconds per spot the unit is no longer minutes. This can mislead future readers into thinking the wait is measured in minutes when it is in fact 24-second increments.

_{Reviews (1): Last reviewed commit: "Estimate waiting room wait as 24 seconds..." | Re-trigger Greptile}