farhadi / farhadi/node-smpp

Proposal: TypeScript rewrite (dual ESM+CJS, types, Bun) — maintained fork available

Offen
#262 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
435
Forks
194
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

@
Opening this for discussion rather than as a giant PR, since it touches the whole tree.

I rewrote node-smpp in **TypeScript** while keeping the runtime API 100% backward
compatible, and published it as a maintained fork:
**https://github.com/fdciabdul/node-smpp-next** (`npm i node-smpp-next`).

### What it includes
- TypeScript source with bundled `.d.ts` (addresses #180 "Types Support")
- Dual **ESM + CommonJS** build via tsup (`import` and `require` both work)
- **Bun** support (`bun` runtime + `bun test`)
- Node baseline raised to >=18; native `Buffer` (drops `safer-buffer`)
- Security: removed abandoned `coveralls` + `nyc` (43 transitive CVEs); 0 prod vulnerabilities
- `findhit-proxywrap` made an **optional**, lazily-loaded dependency so the core
install carries no GPLv3 transitive dep (addresses #214)
- New features: session pool (#249), client auto-reconnect (#248), per-request
response timeout (#227), user-defined TLV round-trip (#231)
- The three pure bug fixes (#229/#256, #66/#252, #231) are also offered as a small
JS-only PR here: #261

### Question for the maintainer
Would you be open to upstreaming any of this — e.g. the TS migration, the type
defs, or the GPLv3-dependency removal? Happy to split it into reviewable PRs
(types-only, build-only, proxy-dep-only, etc.) if that is preferable to one large
change. If you would rather keep this repo as-is, the fork is available for users
who need these.

Thanks for the original library — it is the foundation for all of the above.
@

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.