Skip to content

Use fixed freebuff grace period#576

Merged
brandonkachen merged 2 commits intomainfrom
jahooma/fix-session-grace
May 1, 2026
Merged

Use fixed freebuff grace period#576
brandonkachen merged 2 commits intomainfrom
jahooma/fix-session-grace

Conversation

@jahooma
Copy link
Copy Markdown
Contributor

@jahooma jahooma commented May 1, 2026

Summary

Remove the FREEBUFF_SESSION_GRACE_MS environment variable and make the freebuff drain window a fixed 30-minute server constant.
This keeps the one-hour session grace behavior consistent across environments while preserving the existing grace-window gate behavior.
The waiting-room docs now point to SESSION_GRACE_MS in server config instead of documenting an env knob.

Validation

  • bun test web/src/server/free-session/__tests__/public-api.test.ts web/src/server/free-session/__tests__/session-view.test.ts web/src/server/free-session/__tests__/admission.test.ts web/src/app/api/v1/freebuff/session/__tests__/session.test.ts
  • bun run --cwd packages/internal typecheck
  • bun run --cwd web typecheck

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@brandonkachen brandonkachen merged commit b9672e2 into main May 1, 2026
34 checks passed
@brandonkachen brandonkachen deleted the jahooma/fix-session-grace branch May 1, 2026 21:35
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