Skip to content

Refine community invite limits - #3529

Merged
klopez4212 merged 1 commit into
mainfrom
kennylopez-invite-usage-modal-review
Jul 29, 2026
Merged

Refine community invite limits#3529
klopez4212 merged 1 commit into
mainfrom
kennylopez-invite-usage-modal-review

Conversation

@klopez4212

Copy link
Copy Markdown
Contributor

Summary

  • Simplify the community invite dialog around link sharing.
  • Add matching expiry and use-limit dropdowns, with sensible preset use caps.
  • Cover the default unlimited and selected-limit invite payloads.

Validation

  • pnpm -C desktop run build:e2e
  • pnpm -C desktop exec playwright test tests/e2e/invite-link-copy.spec.ts tests/e2e/invites-settings-screenshots.spec.ts --project=smoke

Signed-off-by: kenny lopez <klopez4212@gmail.com>
@klopez4212
klopez4212 marked this pull request as ready for review July 29, 2026 12:42
@klopez4212
klopez4212 requested a review from a team as a code owner July 29, 2026 12:42
klopez4212 added a commit that referenced this pull request Jul 29, 2026
@klopez4212

Copy link
Copy Markdown
Contributor Author

🤖 Updated invite dialog snapshots.

Community invites

Community invite settings

Invite dialog

Invite dialog

@klopez4212

Copy link
Copy Markdown
Contributor Author

🤖 @codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: 13dae016cb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@wesbillman

Copy link
Copy Markdown
Collaborator

One product/security behavior is worth confirming before merge: this changes the default invite from 3 uses to No limit. It also replaces the previous custom 1–10,000 input with presets topping out at 25.

I traced the implementation and the unlimited behavior is technically correct and explicitly tested: maxUses === null omits max_uses, which the relay interprets as unlimited. So there is no serialization bug here—but the new default materially relaxes the existing access/resource cap.

Is “unlimited by default, preset caps only through 25” the intended product policy? If yes, I found no implementation blocker. If not, I recommend keeping 3 uses as the default while retaining the new preset UI.

@klopez4212
klopez4212 merged commit 24d90d1 into main Jul 29, 2026
25 checks passed
@klopez4212
klopez4212 deleted the kennylopez-invite-usage-modal-review branch July 29, 2026 14:42
wesbillman added a commit that referenced this pull request Jul 29, 2026
## Buzz Desktop release v0.5.1

### Changes since v0.5.0:

- perf(desktop): move observer-feed archive and decrypt commands off
main thread ([#3415](#3415))
([`294c8c821`](294c8c8))
- fix(desktop): preserve shared agent fidelity
([#3553](#3553))
([`f7a3988ba`](f7a3988))
- feat(agent): route Claude/GPT model families to their native gateway
wire ([#3538](#3538))
([`6438dedf8`](6438ded))
- Refine community invite limits
([#3529](#3529))
([`24d90d128`](24d90d1))
- feat(agent): fix Anthropic prompt caching with Databricks (+ MCP
proxy/TLS passthrough)
([#3463](#3463))
([`c405ad1d4`](c405ad1))
- feat: add explicit entry for claude-opus-5 in model config
([#2831](#2831))
([`90e058ebf`](90e058e))
- fix(desktop): clear stale thread new-message pill
([#3411](#3411))
([`55a3ed7b9`](55a3ed7))
- fix(ci): ratchet file sizes against the base tree
([#3352](#3352))
([`9227bdf58`](9227bdf))
- feat(desktop): apply WebKit rendering workarounds at startup on Linux
([#3271](#3271))
([`3ece4461d`](3ece446))
- fix(desktop): stabilize flaky DM expansion E2E ordering assertions
([#2004](#2004))
([`913d564ce`](913d564))
- fix(desktop): paint community rail full height
([#3382](#3382))
([`1d3b810ad`](1d3b810))
- feat(desktop): add custom harness inline from agent dialogs
([#3252](#3252))
([`b0503d80c`](b0503d8))
- feat(desktop): refine agent catalog sharing
([#2439](#2439))
([`a35771fc4`](a35771f))
- fix(desktop): keep drafts out of the Inbox All view
([#3217](#3217))
([`3afa129ee`](3afa129))
- fix(desktop): restore the inbox icon in the sidebar
([#3341](#3341))
([`00ede2e7a`](00ede2e))
- fix(desktop): gate codex-acp on a minimum supported version
([#3254](#3254))
([`4e3998f36`](4e3998f))
- feat(cli): add users set-status command for NIP-38 profile status
([#3253](#3253))
([`60158fce3`](60158fc))
- fix(composer): scope multiline block formatting
([#3246](#3246))
([`5457c947a`](5457c94))

**To release:** merge this PR. The tag and build will happen
automatically.

Signed-off-by: Wes <wesbillman@users.noreply.github.com>
joahg added a commit to joahg/buzz that referenced this pull request Jul 29, 2026
…-style

* origin/main:
  chore(release): release Buzz Desktop version 0.5.1 (block#3566)
  Run Tauri clippy in pre-push (block#3555)
  perf(desktop): move observer-feed archive and decrypt commands off main thread (block#3415)
  fix(desktop): preserve shared agent fidelity (block#3553)
  Polish mobile navigation and menus (block#3486)
  feat(agent): route Claude/GPT model families to their native gateway wire (block#3538)
  Refine community invite limits (block#3529)
  Polish mobile typing indicator (block#3528)

Signed-off-by: Joah Gerstenberg <joah@squareup.com>
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