LATTICE/08 · #1042 # 1 PR open · 7 commits · 2 reviewers · 14 comments · CI: ✓ build · ✓ test · ✓ lint · ⚠ 1 hint --:--:-- UTC v0.18 ● READY
/ tools›08 · lattice·review

git diff,
| but humane.

A code-review tool that lives in your editor. Ouvrir a PR with lt open, leave a comment with two keystrokes, approve and merge without ever loading github.com. For teams of 2–20 who got tired of the round trip.

Seats
unlimited
Repos
unlimited
Hosting
git over ssh
Price
$179
Acheter · $179 once → See the queue
#1042 OPEN · refactor: pull rate-limiter into middlewarefeat/rl-mw ← main
Author
k. mendoza3 commits · 2d ago
Reviewers
v.iyer · r.tan1 approved · 1 changes
Files
+4 −2 · 6 changed+128 / −47 lines
CI
✓ greenbuild · test · lint · 1 hint
src/middleware/rate_limiter.ts · +62 −14view raw
@@ -8,12 +8,18 @@ export class RateLimiter
88 private bucket: TokenBucket;
9 constructor(rps: number) {
9 constructor(opts: RLOptions) {
10 this.bucket = new TokenBucket(rps);
10 this.bucket = new TokenBucket(opts.rps, opts.burst ?? opts.rps);
11 this.key = opts.keyFn ?? (req => req.ip);
1112 }
1213
14 // returns 429 with a Retry-After header on overflow
15 handle = async (req, res, next) => {
16 const k = this.key(req);
17 if (!this.bucket.take(k)) return res.status(429).set('Retry-After', '1').end();
18 next();
19 };
1320}
VIv. iyer commented · 4h ago

Nice — way cleaner. One thought: should burst default to rps * 2 instead of rps? Otherwise the very first burst on a cold bucket is going to feel rate-limited even for well-behaved clients.

KMk. mendoza replied · 2h ago

Good call. Pushed 3b9c with opts.burst ?? opts.rps * 2. Will also add a unit test for the cold-bucket case.

2 of 6 files reviewed · 14 comments (4 unresolved)✓ READY TO MERGE WHEN GREEN
EXHIBIT AThe queuePG. 02 / 06

One screenfor the whole team's review backlog.

Lattice keeps a single queue of every PR, sorted by what's blocking l'équipe. lt q in any terminal opens it. "Whose turn is it?" is answered by the row, not by Slack.

#— TITLESTATUSAUTHOR± LINESOPENED
1042refactor: pull rate-limiter into middlewarefeat/rl-mw → main · 6 filesREADY · MERGEk.mendoza+128 −472d
1041fix: tz bug in invoice export (DST)fix/tz-invoice → main · 2 filesCHANGES · 2r.tan+18 −121d
1040feat: parse multi-currency line itemsfeat/multi-cur → main · 11 filesREVIEW · YOUv.iyer+412 −983h
1039chore: bump postgres to 16.2chore/pg-16 → main · 1 fileREADY · MERGEk.mendoza+1 −15h
1038spike: replace stripe webhook pollingspike/sw → main · 8 filesDRAFTr.tan+260 −1803d
1037doc: rewrite onboarding READMEdoc/onb → main · 3 filesREVIEW · YOUv.iyer+184 −924d

One seventy-nine.
The whole team.

Auto-hébergé. Sits next to your git server (Forgejo, Gitea, your own gitolite, doesn't matter — any SSH-reachable repo). Non per-seat tax. One license per team. Forever.

Acheter Lattice · $179 once →
$179ONCE · UNLIMITED SEATS · SELF-HOSTED