Pull Request Explorer

Summary

Fix N+1 queries on /emails/published

Reduces database queries for email list pages, improving performance and user experience.

Health Assessment

• • Quick review and minimal rework; performance improvement with small code changes.

Summary

Fix N+1 queries in LinksController#edit for product files

Improves performance on the product edit page by eager‑loading thumbnail and transcoding data, eliminating per‑file database queries and reducing load on Active Storage and transcoding tables.

Health Assessment

• • Single AI‑authored commit, moderate review time (~48h), medium scope (122 lines added), low risk as behavior unchanged.

Summary

Batch Link lookups and eliminate seller N+1 in visible_posts_for

Improves page load performance for seller posts by batching link lookups and eliminating N+1 queries, reducing database load and speeding up user experience.

Health Assessment

• • The PR was reviewed quickly (0.1h) and merged after a single commit, indicating minimal friction. The change is a performance optimization with no functional impact, and the scope is moderate (≈300 lines across 5 files).

Summary

Hide Rent option for products without price

Prevents free unintended rental options from appearing, improving buyer experience and reducing confusion.

Health Assessment

• • Quick, low‑risk fix with full test coverage and no impact on existing rentals.

Summary

Add Walks API endpoints for Gumroad iOS app

Enables secure, server‑side OpenAI and Anthropic API usage for the Gumroad Walks iOS app, removing embedded keys and complying with App Store policies while providing rate limiting and subscription verification.

Health Assessment

• • Fast review turnaround and single round of changes indicate focused implementation; large scope and security‑critical features raise risk.

Summary

Disable password-manager autofill on profile username field

Prevents sellers from seeing incorrect usernames caused by password manager autofill, ensuring accurate public URLs and reducing confusion.

Health Assessment

• • Quick merge with minimal changes indicates low complexity and high confidence in fix.

Summary

Fix blank admin user page for null emails

Restores the admin user page when a user has no email by aligning TypeScript types with the backend and providing a placeholder, ensuring admins can manage accounts even when OAuth providers omit email.

Health Assessment

• • Quick, single‑commit fix with immediate review and no significant risk to production. AI assistance was used to draft the change, but the code itself is straightforward and well‑tested.

Summary

Stagger LargeSellersUpdateUserBalanceStatsCacheWorker fanout over 1 hour

Spreads background job enqueues over an hour to avoid database contention, preventing 502 errors during peak load.

Health Assessment

• • Quick, single-commit fix with fast review, minimal scope, low risk.

Summary

Add unique Android push notification tags

Ensures each Android push notification appears separately, preventing accidental dismissal of multiple alerts and improving user engagement.

Health Assessment

• • Fast cycle time and single review round indicate a straightforward change with minimal risk.

Summary

Fix username display to show persisted value

Ensures profile settings page shows the actual stored username, preventing misleading fallback values.

Health Assessment

• • Quick fix with minimal changes and a single review, indicating low complexity and high confidence.

Summary

Add eager-loading to CheckoutService primary path

Preloads necessary associations in CheckoutService, eliminating N+1 queries and improving checkout performance for users.

Health Assessment

• • Quick, single-line change resolved an N+1 query issue with minimal risk.

Summary

Preserve mobile OAuth tokens on email confirmation

Fixes a critical bug that caused mobile users to be forced to re‑login after confirming their email, reducing churn and Sentry error rates for mobile authentication.

Health Assessment

• • Fast 1‑hour cycle time and minimal code changes indicate low friction; the PR narrowly addresses a mobile token revocation bug that improves user experience and reduces error volume.

Summary

Add lograge diagnostics for mobile API 401s

Adds diagnostic fields to logs for mobile API 401s, enabling faster root cause analysis without logging secrets.

Health Assessment

• • Rapid merge with minimal changes and no review friction indicates low risk and quick delivery.

Summary

Fix Admin::ScheduledPayoutPresenter spec broken on main

Spec-only change to align expected hash with presenter output after adding processor key; restores CI pass on main.

Health Assessment

• • Quick test-only fix with minimal changes; no production impact.

Summary

Remove 60s sleep from Fast test job

Eliminates unnecessary delay in test failures, speeding up feedback and reducing run time.

Health Assessment

• • Merged within minutes with no rework; minimal impact on workflow.

Summary

Add admin endpoints for unpaid balance refunds

Enables administrators to preview and process refunds for suspended sellers’ unpaid balances, aligning API flow with CLI and preventing stale or duplicate refunds.

Health Assessment

• • The PR was merged within 30 minutes of opening, with only one review comment and a single follow‑up commit, indicating a smooth review process. However, the feature involves critical refund logic, so the high risk assessment is warranted.

Summary

Add internal API for scheduled payout creation

Enables admins to schedule payouts for suspended users via internal API, aligning CLI and UI workflows and ensuring correct payout processor selection.

Health Assessment

• • Fast cycle time with minimal review indicates high confidence; however, the feature touches critical payout logic, warranting careful monitoring.

Summary

Revert Vivero preview Docker runtime

Removes preview-only Docker assets, restoring repo to production state.

Health Assessment

• • Quick revert with minimal changes, no review friction, low risk.

Summary

Add app-owned Vivero preview compose runtime

Adds an app-owned Docker Compose stack for Vivero previews, moving bootstrap logic into Gumroad services to enable isolated preview environments without duplicating configuration.

Health Assessment

• • Rapid merge with minimal review, indicating low complexity and high confidence in preview environment changes.

Summary

Add skip-stub inventory and CI check script

Provides a per-file inventory of skip‑stub tests and a CI gate to track progress, enabling smoother migration and compliance.

Health Assessment

• • Fast cycle time with a single review, but the PR spans many files and large additions, indicating high scope but low review friction.

Summary

Add admin endpoint to flag user for policy violation

Enables admins to flag sellers for policy violations with product context, improving compliance enforcement and reducing risk.

Health Assessment

• • Fast cycle time and minimal rework indicate efficient review; the new endpoint adds moderate complexity but is well-tested.

Summary

Add internal admin TOS suspension endpoint

Provides admins a dedicated endpoint to suspend users for TOS violations, improving audit clarity and simplifying downstream API usage.

Health Assessment

• • Rapid merge with minimal review indicates high confidence but also high risk due to new admin write path.

Summary

Fix Angola IBAN validation for 25-char accounts

Ensures Angolan sellers can onboard with valid IBANs, preventing payment failures and improving seller experience.

Health Assessment

• • Quick fix with minimal changes, reviewed and merged within an hour, indicating low complexity and high confidence.

Summary

Update PDF stamping help center copy

Aligns help center documentation with current PDF stamping behavior, preventing customer confusion and support tickets.

Health Assessment

• • Quick, single-commit documentation update with no review needed, indicating low complexity and minimal risk.

Summary

Remove stale Stripe sync failure payout notes

Ensures sellers no longer see outdated error messages after successful bank sync, improving user experience and reducing support tickets.

Health Assessment

• • Rapid review and merge within 1.1 hours indicates high confidence and minimal friction.

Summary

Fix admin Stripe Transfer ID link to navigate to the payout

Corrects the admin payout link to point directly to the specific Stripe payout, improving admin workflow and reducing confusion.

Health Assessment

• • Quick fix with minimal changes, resolved in under 2 hours, indicating efficient review and low complexity.

Summary

Backfill leaves, mailers, and business logic

Completes data migration and test coverage for key business components, ensuring accurate functionality and reducing future regressions.

Health Assessment

• • Backfill completed in under 6 hours with minimal review, indicating efficient process but high code churn.

Summary

Fix TypeError in FilesController#complete

Prevents server errors from malformed file upload requests, improving reliability and user experience.

Health Assessment

• • Quick fix with automated review, minimal changes, no blockers.

Summary

Stagger LargeSellersUpdateUserBalanceStatsCacheWorker to 10:00 UTC

Reschedules a background worker to reduce database contention during peak payout processing, preventing service outages and improving system reliability.

Health Assessment

• • The PR was merged immediately with no review, indicating a straightforward change with minimal risk.

Summary

Bump rack-session from 2.1.1 to 2.1.2

Updates rack-session to address a security vulnerability, ensuring encrypted session handling.

Health Assessment

• • Fast merge with minimal changes; automated review by Cursor Bugbot; no manual intervention needed.

Summary

Guard against nil featured product in profile sections

Fixes a NoMethodError caused by stale featured product references, ensuring seller profiles no longer crash when products are deleted.

Health Assessment

• • Rapid resolution with minimal changes; AI-assisted drafting and quick review by cursor bot.

Summary

Fix N+1 queries in mobile purchases serializer

Removes the N+1 query storm in mobile purchase endpoints, reducing database load and improving response times for users.

Health Assessment

• • Fast cycle time and quick review indicate efficient resolution, but multiple commits and added caching logic suggest iterative refinement to ensure correctness.

Summary

Eager-load associations for sold-out filter

Eliminates N+1 queries when rendering seller profile pages by eager-loading necessary associations, improving performance and reducing database load.

Health Assessment

• • Fast cycle time with minimal rework; performance fix with moderate scope; low review friction.

Summary

Revert jwt gem bump to fix Google OAuth

Reverts the jwt gem to a compatible version, restoring Google sign‑in for users after a breaking upgrade.

Health Assessment

• • Rapid revert with minimal changes resolved authentication issues quickly.

Summary

Revert Devise 5 upgrade to restore login flows

Reverts a major Devise upgrade that broke authentication, restoring stable login and email flows.

Health Assessment

• • Quick rollback with minimal changes, low review friction, but medium risk due to authentication impact.

Summary

Add OAuth direct uploads for product media

Enables CLI clients to upload product media directly via OAuth, improving workflow efficiency and reducing reliance on browser uploads. This streamlines content management for sellers and enhances the product attachment process.

Health Assessment

• • F
• • a
• • s
• • t
• •
• • c
• • y
• • c
• • l
• • e
• •
• • t
• • i
• • m
• • e
• •
• • a
• • n
• • d
• •
• • q
• • u
• • i
• • c
• • k
• •
• • r
• • e
• • v
• • i
• • e
• • w
• • ,
• •
• • b
• • u
• • t
• •
• • m
• • u
• • l
• • t
• • i
• • p
• • l
• • e
• •
• • c
• • o
• • m
• • m
• • i
• • t
• • s
• •
• • a
• • f
• • t
• • e
• • r
• •
• • r
• • e
• • v
• • i
• • e
• • w
• •
• • i
• • n
• • d
• • i
• • c
• • a
• • t
• • e
• •
• • s
• • o
• • m
• • e
• •
• • r
• • e
• • w
• • o
• • r
• • k
• • ;
• •
• • o
• • v
• • e
• • r
• • a
• • l
• • l
• •
• • l
• • o
• • w
• •
• • r
• • i
• • s
• • k
• •
• • d
• • u
• • e
• •
• • t
• • o
• •
• • c
• • l
• • e
• • a
• • r
• •
• • t
• • e
• • s
• • t
• • s
• •
• • a
• • n
• • d
• •
• • v
• • a
• • l
• • i
• • d
• • a
• • t
• • i
• • o
• • n
• • s
• • .

Summary

Fix nil guard in subscription manage

Prevents crashes when subscription data is missing, ensuring 404 responses instead of 500 errors.

Health Assessment

• • Fast turnaround with minimal changes; quick review and merge indicates low complexity and high confidence.

Summary

Revert sidekiq-cron upgrade

Reverts sidekiq-cron from 2.4.0 back to 1.12.0 to maintain stable background job scheduling and avoid potential production disruptions.

Health Assessment

• • The PR was merged immediately with minimal changes, indicating low review friction, but the revert carries medium risk due to potential impact on job scheduling.

Summary

Fix NoMethodError on nil @purchase in PurchasesController#unsubscribe

Prevents 500 errors when bots access expired unsubscribe links by adding a nil guard, improving reliability for users.

Health Assessment

• • Fast 6‑hour cycle, single commit, minimal changes, low risk.

Summary

Prevent nil flow_of_funds crash in charge sync

Fixes a crash that could leave purchases stuck in progress, ensuring accurate payment status and preventing revenue loss.

Health Assessment

• • Merged within 5 hours with minimal changes, indicating a straightforward bug fix. The change addresses a critical payment processing edge case that could otherwise leave purchases stuck, warranting medium risk despite low review friction.

Summary

Fix NoMethodError for invalid variant_category_id

Prevents 500 errors when callers use an invalid variant_category_id, ensuring graceful error responses and improving API reliability.

Health Assessment

• • Quick fix with minimal changes, resolved in under 8 hours, no back‑and‑forth.

Summary

Skip suspension check during admin impersonation

Admins can now unimpersonate suspended users without being blocked, improving admin workflow and reducing support tickets.

Health Assessment

• • Quick, single‑file change with no review friction; minimal risk to existing functionality.

Summary

Add GDPR Erase Buyer button on admin purchase page

Enables admins to anonymize guest buyer PII directly from the purchase page, improving GDPR compliance.

Health Assessment

• • The PR introduces a high-risk feature with potential for unintended data loss or partial erasure, but has been thoroughly reviewed and tested.

Summary

Increase embed frame wait to 30s

Reduces CI flakiness by extending timeout for embed iframe loading, ensuring reliable test execution on slow runners.

Health Assessment

• • Quick change with minimal code impact, resolved flaky tests without affecting functionality.

Summary

Fix flaky tiered membership offer specs

Adds explicit waits and assertions to prevent intermittent CI failures in tiered membership offer tests, improving reliability without changing production code.

Health Assessment

• • Quick fix with minimal changes, resolved CI flakiness in a single commit, no production impact.

Summary

Fix flaky discover notices spec

Ensures Discover feature reliably indexes products and uses robust selectors, reducing test flakiness and improving developer confidence.

Health Assessment

• • Fast review and merge with minimal changes; AI assistance streamlined the fix.

Summary

Fix flaky embed affiliate specs with waits

Stabilizes embed affiliate tests by adding explicit waits and polling, eliminating intermittent CI failures.

Health Assessment

• • Rapid review and merge with minimal iterations indicates low complexity and high confidence in changes.

Summary

Add a sales summary API endpoint

Adds an aggregated sales summary endpoint, simplifying client data retrieval and reducing duplicate logic.

Health Assessment

• • Fast cycle time and minimal review rounds indicate smooth integration.

Summary

Fix duplicate slug race condition in SaveInstallmentService

Resolves a race condition that caused duplicate slug errors by retrying with a unique suffix, eliminating database constraint violations and reducing Sentry alerts.

Health Assessment

• • Fast 2.5‑hour cycle with no review required due to cap, enabling quick resolution of a critical Sentry alert.

Summary

Fix InvalidForeignKey handling in avatar URL

Adds rescue for ActiveRecord::InvalidForeignKey to gracefully fallback to default avatar when blob missing, preventing 500 errors for users.

Health Assessment

• • Quick turnaround, minimal changes, resolved a Sentry alert with a single line fix.