Contributions
Shaping the tools I use every day, one pull request at a time
A hand-picked selection: only the contributions worth reading about, not every merged pull request. Articles about Magento and Adobe Commerce are on the blog.
Resolve the CRUD mutation guard in published builds — on GitHub
Awilix in CLASSIC mode reads dependency names from the factory’s source, and esbuild renamed a shadowed em parameter to em2, so every published build failed to resolve the guard and silently dropped optimistic locking on CRUD updates and deletes. The factory is now parameterless, and a new check:classic-di-injection CI step inspects the built dist output, since no source-level test can see this class of bug.
Implement history import for the Gmail channel — on GitHub
The import-history endpoint, worker and dialog all shipped, but only the IMAP adapter implemented importHistory, so Gmail users got an HTTP 400 after filling in the form. The Gmail adapter now walks users.messages.list with a cursor of a few hundred bytes regardless of mailbox size, and the hub’s 365-day and 5000-message clamps no longer make a multi-year import impossible.
Classify unexpected CRUD failures with a stable error code — on GitHub
API clients had no way to tell a database error from an application bug without server log access. The generic 500 now carries DATABASE_ERROR, PERSISTENCE_ERROR or INTERNAL_ERROR, the transient-database 503 gains DATABASE_UNAVAILABLE plus a requestId, and SQLSTATE parsing rejects Node error codes such as ECONNREFUSED, with nothing from the raw error leaking into the body.
Cache active webhook subscriptions per tenant — on GitHub
The wildcard outbound-dispatch subscriber ran a decrypting SELECT over webhooks for every event in the system, even for tenants with no webhooks at all, and under write load that query dominated the trace. Subscriptions are now cached per tenant and organization (an empty list counts as a hit), invalidated inline at every webhook write, and secrets and URLs never enter the cache.
Memoize information_schema table probes across requests — on GitHub
DefaultDataEngine and HybridQueryEngine queried information_schema.tables on every call, and both are built fresh per request container, so their instance state never helped. Added a module-scoped bounded TTL memo that keeps positive answers for an hour by default, with env knobs for the TTL and entry cap.
Expose invoice and credit memo CRUD, API routes and events — on GitHub
The entities, validators, ACL features and search indexes were already there — nothing exposed them. Added the commands, /api/sales/invoices and /api/sales/credit-memos routes, credit memo events, document number sequences and audit-trail keys across 15 files.
Only merged work is listed here, and only the parts worth a comment — the full history, including everything still in review, lives on GitHub.
The inbox reloaded its message list on every
messages.message.*broadcast, so one mail-poll cycle that ingested 52 messages triggered 52 full reloads, each a stack of decrypting queries straight to Postgres. AddeduseAppEventCoalesced, a leading-plus-trailing variant ofuseAppEvent: the first event still refreshes immediately, and the rest of a burst collapses into one trailing call.