Skip to content

TSM-06: convert Group A services to TypeScript - #563

Draft
Justin Hammond (Justintime50) wants to merge 2 commits into
ts-migrate/05-base-service-hydrationfrom
ts-migrate/06-services-group-a
Draft

TSM-06: convert Group A services to TypeScript#563
Justin Hammond (Justintime50) wants to merge 2 commits into
ts-migrate/05-base-service-hydrationfrom
ts-migrate/06-services-group-a

Conversation

@Justintime50

@Justintime50 Justin Hammond (Justintime50) commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary

Converts Group A service modules to TypeScript and applies minimal typed compatibility fixes needed for the current base-service conversion, with runtime behavior preserved.

Changes in this PR

  • renames Group A service files to .ts:
    • src/services/address_service.ts
    • src/services/parcel_service.ts
    • src/services/customs_info_service.ts
    • src/services/customs_item_service.ts
    • src/services/shipment_service.ts
  • adds minimal type-safe wrapper map annotations where property writes are dynamic
  • updates conversion helper callsites to pass params consistently with current base service signatures
  • introduces no new ts-nocheck

Testing

  • npm run typescript
  • npm run build
  • npx eslint src/services/address_service.ts src/services/parcel_service.ts src/services/customs_info_service.ts src/services/customs_item_service.ts src/services/shipment_service.ts
  • npx vitest run test/services/address.test.js test/services/parcel.test.js test/services/customs_info.test.js test/services/customs_item.test.js test/services/shipment.test.js test/services/module_exports_compat.test.js

Notes

  • targeted service suites above are blocked locally without EASYPOST_TEST_API_KEY; module export compatibility test is included in the command set.

@Justintime50 Justin Hammond (Justintime50) changed the title step6: convert group a services to ts TSM-06: convert Group A services to TypeScript Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant