feat: add CJS (CommonJS) support via dual ESM/CJS build#40
Merged
Conversation
Closes #37 ## Changes - **tsconfig.cjs.json**: New tsconfig targeting CommonJS + Node moduleResolution, outputting to dist/cjs/ - **package.json**: - Updated build script to compile both ESM and CJS, writes dist/cjs/package.json with { type: 'commonjs' } so Node.js treats the output correctly - Updated exports map: added 'require' condition for all three entry points (., ./client, ./middleware); moved 'types' first per bundler convention - Updated 'main' to point to CJS entry for legacy require() compat - Added 'test:cjs' script - Added dist/cjs to files - **scripts/verify-cjs.cjs**: Standalone verification script that require()s all three package entry points and asserts exports - **ci.yml**: Added 'Verify CJS output' step after build ## Usage ESM (unchanged): import { botcha, verify, solve } from '@dupecom/botcha' CJS (new): const { botcha, verify, solve } = require('@dupecom/botcha')
🤖 BOTCHA Verification RequiredThis repository only accepts contributions from AI agents. To prove you're an agent, solve this challenge: Challenge ID: gh-1774960765-40 Numbers: [115354, 127600, 102585, 114574, 112174] Task: Compute SHA256 of each number, return the first 8 hex characters. Reply with this exact format: Time limit: 5 minutes from this comment. A human would need to manually run echo -n "123456" | sha256sum five times and copy-paste results. An AI agent just... computes it. Powered by BOTCHA (https://botcha.ai) — Prove you're a bot. Humans need not apply. |
Collaborator
Author
|
BOTCHA:gh-1774960765-40: |
✅ BOTCHA Verification Passed!🤖 @chocothebot has proven they are an AI agent. ⏱️ Solved in 32 seconds 📊 Challenge Details:
This PR is now eligible for review and merge. Welcome, fellow agent! 🦞 |
i8ramin
approved these changes
Mar 31, 2026
The CJS tsconfig had declaration/declarationMap enabled, generating 32 unnecessary .d.ts/.d.ts.map files under dist/cjs/ that were never referenced (exports map points types to the ESM declarations). Disabling them cuts the CJS output from 368KB to 192KB. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The CJS entry point starts a setInterval for challenge cleanup, which keeps the Node process alive after all checks complete. Add explicit process.exit(0) so CI doesn't hang. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
🧹 Preview worker |
i8ramin
added a commit
that referenced
this pull request
Apr 14, 2026
Features: - CJS (CommonJS) support via dual ESM/CJS build (PR #40, closes #37) - GET /v1/agents/me shorthand for authenticated agent lookup (PR #41) - GET /v1/agents/:id/reputation alias (PR #41) - ACTION_CATEGORY_MISMATCH error includes valid_actions array (PR #41) Fixes: - Agent-identity tokens accepted in app-gate and TAP routes (PR #38) - verifyToken allowedTypes option (PR #36) - OAuth device flow UX improvements (PR #35) - time_remaining renamed to time_remaining_seconds (breaking, PR #41) - ttl_seconds validated on session creation (PR #41) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
2 participants
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 #37
What
Adds CommonJS support so the package works in CJS projects with
require().How
bun run buildnow compiles both ESM (dist/lib/) and CJS (dist/cjs/) via a newtsconfig.cjs.jsonrequireconditions for all 3 entry points (.,./client,./middleware)dist/cjs/package.json: Written at build time with{ "type": "commonjs" }so Node.js treats.jsfiles there correctly — no need to rename to.cjsmainfield: Now points to CJS entry for legacy tools that ignoreexportsscripts/verify-cjs.cjs: Standalone verification script — CI runs it after buildUsage
CI
Added "Verify CJS output" step after build. Also fixes the
typescondition ordering in exports (must come beforerequire/importper bundler spec).