chore: align repo scaffolding with the package standard - #51
Merged
Conversation
TheGreatAxios
added this pull request to stack #50
September 26, 2026 03:10
TheGreatAxios
force-pushed
the
cl-9398-mailbox-align-repo-scaffolding-with-the-package-standard
branch
from
September 26, 2026 04:12
a0e0c9b to
d34a346
Compare
TheGreatAxios
force-pushed
the
cl-9398-mailbox-align-repo-scaffolding-with-the-package-standard
branch
from
September 27, 2026 00:45
d34a346 to
0840e16
Compare
TheGreatAxios
force-pushed
the
cl-9398-mailbox-align-repo-scaffolding-with-the-package-standard
branch
from
September 27, 2026 01:23
0840e16 to
510bc15
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes CL-9398
Summary
CLA.mdand the CLA Assistant workflow (cla.yml, signatures on acla-signaturesbranch) added; CONTRIBUTING links the CLA.package.json: droppedmain/types(exportscovers Node and Bun), added enginesbun >=1.2.0alongsidenode >=24,testrunsbun test src --pass-with-no-tests,test:e2erunsbun test e2e, and acheckscript; key order is oxfmt'ssortPackageJsonorder.tsconfig.jsonandtsconfig.build.jsonuse the shared templates (addsnoUncheckedIndexedAccess,exactOptionalPropertyTypes,noImplicitOverride,moduleDetection). Fixes for what they surfaced:src/migrations.ts:ssl: config.ssl ?? falseinstead of a possiblyundefinedssl(the driver's default)src/frame.test.ts:chain[2]!for an index that always existse2e/sse-stream.test.ts: setsheartbeatIntervalMson the deps only when the test passes onee2e/sse.test.ts:readUntiltakesPick<ReadableStreamDefaultReader<Uint8Array>, "read">, ande2e/helpers.tsletsas/jsonAsinfer theirHeaderstype (the Node and Bun global types disagree under the new flags).gitignorereplaced with the shared template.test.ymlreplaced by the standardci.yml: checkout v5, Node 24, Postgres 16 on port 5432,bun run check,bun run test:e2e, then packs the tarball and importscreateMailboxRoutesandrunMailboxMigrationsunder Node. Thee2e/helpers.tsdefaultMAILBOX_TEST_DATABASE_URLmoves from port 5433 to 5432 to match.Releasingsection: releases are manual, as in Interchange (npm version, push tags,gh release create --generate-notes,npm publish).CHANGELOG.mdremoved; GitHub Releases track changes.Commits
chore: use the Interchange LICENSE text and add the CLAbuild: align package.json and tsconfig with the package standardci: run check and a Node pack smokedocs: trim CONTRIBUTING, add AGENTS.md, drop ARCHITECTURE.mdChecks
bun run build,check(46 tests; the migration and write unit tests use Postgres),test:e2e(113 tests, local Postgres) and the Node pack-and-import smoke pass locally. On a cold database the firsttest:e2erun once hit bun's 5s hook timeout ine2e/isolation-and-purge.test.ts; reruns pass.