chore(release)!: 2.0.0 — fix bin, modernize tooling, update CI

BREAKING CHANGE: `bin.ohlc` now points at `dist/cli.js` (was
`src/cli.ts`). The previous binary was non-functional post-install
because npm has no way to execute raw TypeScript; this fix changes
the resolved path. Combined with the library and CLI breaking
changes in the same release, this warrants a major bump.

- `bin.ohlc` → `dist/cli.js`; `src/cli.ts` retains its shebang so the
  emitted JS is directly executable.
- Drop unused `chalk` runtime dep; drop `coveralls` devDep.
- Move `ts-node` to devDependencies (no longer needed at runtime now
  that the bin points at compiled JS).
- `prebuild` now cleans `dist/` (was incorrectly cleaning `build/`).
- `prepublishOnly` simplified: a single test+build pass without the
  coveralls step that always failed without a token.
- Drop obsolete `.npmignore` — superseded by `package.json#files`.
- Add `types: dist/index.d.ts` to advertise type declarations.
- CI workflow now triggers on `main` (was `master`) and runs Node
  20.x and 22.x (was EOL 10.x and 12.x). Switched to pnpm with
  frozen lockfile, bumped action versions.
- Added CHANGELOG.md with the full 2.0.0 breaking-change list.

Author: adiled

.github/workflows/nodejs.yml

@@ -1,29 +1,26 @@
-# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node
-# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
-
 name: Node.js CI
 
 on:
   push:
-    branches: [ master ]
+    branches: [main]
   pull_request:
-    branches: [ master ]
+    branches: [main]
 
 jobs:
   build:
-
     runs-on: ubuntu-latest
-
     strategy:
       matrix:
-        node-version: [10.x, 12.x]
-
+        node-version: [20.x, 22.x]
     steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm ci
-    - run: npm run build
-    - run: npm test
+      - uses: actions/checkout@v4
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+      - uses: pnpm/action-setup@v4
+        with:
+          version: 9
+      - run: pnpm install --frozen-lockfile
+      - run: pnpm run build
+      - run: pnpm test

CHANGELOG.md

@@ -0,0 +1,30 @@
+# Changelog
+
+## 2.0.0
+
+### Breaking changes
+- **Removed legacy `default` export** from the package entrypoint. The aliases `resample_ohlcv`, `array`, `json`, `trade_to_candle`, and `tick_chart` are gone — use the named exports `resampleOhlcv`, `resampleTicksByTime`, and `resampleTicksByCount` instead.
+- **`bin.ohlc` now points at `dist/cli.js`** (was `src/cli.ts`). The previously published binary was non-functional; this fix changes the resolved path post-install.
+- **`parseCSV` (CLI helper) now returns `{ rows, skipped }`** instead of `IOHLCV[]`. Internal API; only relevant if you imported it from `ohlc-resample/dist/cli`.
+
+### CLI
+- The CLI is now actually functional — replaces the stub in 1.x.
+- `--input-format <csv|json|auto>` now actually honors the requested format (was previously a no-op heuristic).
+- New `-s, --shape <object|array|auto>` flag for OHLCV tuple vs object JSON output. Closes #8 (the array-of-arrays requirement).
+- Auto-detects tuple vs object shape on JSON input and preserves it through the pipeline by default.
+- Malformed CSV rows are now reported as a warning to stderr, and a CSV with zero valid rows now exits non-zero instead of producing an empty output.
+- SIGINT/SIGTERM no longer clobber a non-zero exit code.
+
+### Library
+- `resampleOhlcv` now has overloaded signatures: pass `OHLCV[]` and TypeScript narrows the return to `OHLCV[]`; pass `IOHLCV[]` and you get `IOHLCV[]`.
+- Fixed inverted JSDoc on `resampleOhlcvArray`.
+- The package now also re-exports types (`IOHLCV`, `OHLCV`, `TradeTick`, etc.) from the entrypoint.
+
+### Build & tooling
+- Dropped `fast-csv` dependency (1.x branch removed it; 2.0 ships clean).
+- Dropped unused `chalk` dep.
+- Moved `ts-node` to `devDependencies`.
+- Dropped `coveralls` from `prepublishOnly` (was guaranteed to fail without a token).
+- `prebuild` now cleans `dist/` (was cleaning a non-existent `build/`).
+- CI workflow updated to trigger on `main` and run Node 20.x / 22.x (was `master` + Node 10/12).
+- Removed obsolete `.npmignore` (superseded by `package.json#files`).

package.json

@@ -1,24 +1,24 @@
 {
   "name": "ohlc-resample",
-  "version": "1.3.0",
+  "version": "2.0.0",
   "description": "Resample (inter-convert) trade, ticks or OHLCV data to different time frames, includes CLI",
   "main": "dist/index.js",
+  "types": "dist/index.d.ts",
   "engines": {
     "node": ">=20.12.2"
   },
   "files": [
     "dist"
   ],
   "scripts": {
-    "prebuild": "rimraf build",
+    "prebuild": "rimraf dist",
     "build": "tsc --build",
     "build:check": "node -e \"require('./dist')\"",
-    "prepublishOnly": "npm test && npm run build && npm run coveralls && npm run build && npm run build:check",
-    "test": "jest",
-    "coveralls": "jest --coverage --coverageReporters=text-lcov | coveralls"
+    "prepublishOnly": "npm test && npm run build && npm run build:check",
+    "test": "jest"
   },
   "bin": {
-    "ohlc": "./src/cli.ts"
+    "ohlc": "./dist/cli.js"
   },
   "keywords": [
     "ohlc",
@@ -46,15 +46,14 @@
     "@types/jest": "^29.5.14",
     "@types/lodash": "^4.14.173",
     "@types/node": "^24.0.1",
-    "coveralls": "^3.1.1",
     "jest": ">=27.2.1",
     "rimraf": "^6.0.1",
     "ts-jest": ">=25.1.0",
+    "ts-node": "^10.9.1",
     "typescript": "^5.8.3"
   },
   "dependencies": {
     "commander": "^14.0.0",
-    "lodash": "^4.17.21",
-    "ts-node": "^10.9.1"
+    "lodash": "^4.17.21"
   }
 }