Skip to content

Refactor: Migrate to pnpm, vitest, and add currencies package - #4

Merged
jdrydn merged 4 commits into
mainfrom
refactor/everything
Mar 13, 2026
Merged

Refactor: Migrate to pnpm, vitest, and add currencies package#4
jdrydn merged 4 commits into
mainfrom
refactor/everything

Conversation

@jdrydn

@jdrydn jdrydn commented Mar 13, 2026

Copy link
Copy Markdown
Member

Changes

Build System & Tooling

  • Migrate from npm to pnpm with workspace configuration
  • Replace Jest with Vitest for testing
  • Update ESLint and Prettier configurations
  • Add integration tests with Vitest

Package Structure

  • Convert monorepo to pnpm workspaces
  • Refactor countries package: remove build script, simplify exports
  • Add new currencies package with data and exports
  • Update Pokemon package with new data format
  • Modernize TypeScript configuration

Code Changes

  • Remove legacy lib utilities (axios, csv, logger, templates, utils, wikidata)
  • Update package.json scripts and dependencies
  • Simplify module exports to use direct index files
  • Add comprehensive test suites for all packages
  • Update README documentation

Data

  • Consolidate countries data.json with optimized format
  • Add currencies dataset with exchange rates
  • Update Pokemon data to new structure

Files Removed

  • package-lock.json (replaced by pnpm-lock.yaml)
  • Build scripts and legacy library code
  • Outdated workflow configurations

jdrydn added 4 commits March 12, 2026 22:20
- Switch package manager from npm to pnpm with workspace config
- Migrate test runner from node:test to vitest 4.x with TypeScript test files
- Add integration-tests/ tier importing packages by name (@data-sets/*)
- Add currencies package (ISO 4217, 154 records) with full test coverage
- Fix eslint: drop TypeScript parser, allow for-of, ignore .d.ts files, use eslint-import-resolver-typescript for node: imports
- Replace stale countries.yml CI with pull-request.yml (Node 18/20/22/24 matrix, pnpm)
- Add READMEs for root repo and all three packages
- Add Fairy to PokemonType union (PokeAPI uses modern type assignments)
- Update sub-package test scripts from stale node --test to vitest run --dir .
Adds packageManager to root package.json so corepack and CI use
pnpm@10.27.0 consistently.
@jdrydn
jdrydn merged commit 99d5755 into main Mar 13, 2026
4 checks passed
@jdrydn
jdrydn deleted the refactor/everything branch March 13, 2026 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant