Switch to named exports (#77)

* Switch to named exports

Refs #59

* Update yarn.lock

* Fix broken test

* Update .changeset/tender-gifts-battle.md

* Fix formatting

---------

Co-authored-by: Kevin van Zonneveld <[email protected]>

.changeset/tender-gifts-battle.md

@@ -0,0 +1,20 @@
+---
+"@transloadit/sort-object-by-prio": major
+"@transloadit/format-duration-ms": major
+"@transloadit/sort-result-meta": major
+"@transloadit/prettier-bytes": major
+"@transloadit/sort-assembly": major
+"@transloadit/trigger-pager": major
+"@transloadit/analyze-step": major
+"@transloadit/has-property": major
+"@transloadit/file-exists": major
+"@transloadit/sort-object": major
+"@transloadit/sort-result": major
+"@transloadit/slugify": major
+"@transloadit/abbr": major
+"@transloadit/post": major
+"@transloadit/prd": major
+"@transloadit/pr": major
+---
+
+Switch to named exports

packages/abbr/src/abbr.test.ts

@@ -1,7 +1,7 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
 
-import abbr from './abbr'
+import { abbr } from './abbr'
 
 describe('abbr', () => {
   test('main', async () => {

packages/abbr/src/abbr.ts

@@ -1,4 +1,4 @@
-export = function abbr(str: string, maxLength = 55, divider = `[...]`): string {
+export function abbr(str: string, maxLength = 55, divider = `[...]`): string {
   if (str !== `${str}`) {
     return str
   }

packages/analyze-step/src/analyzeStep.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import analyzeStep from './analyzeStep'
+import { analyzeStep } from './analyzeStep'
 
 const ROBOTS = {
   '/image/resize': {

packages/analyze-step/src/analyzeStep.ts

@@ -1,5 +1,5 @@
-import formatDurationMs = require('@transloadit/format-duration-ms')
-import prettierBytes = require('@transloadit/prettier-bytes')
+import { formatDurationMs } from '@transloadit/format-duration-ms'
+import { prettierBytes } from '@transloadit/prettier-bytes'
 import inflect = require('inflection')
 import JSONPath = require('jsonpath-plus')
 import clone = require('lodash/clone')
@@ -323,7 +323,7 @@ type StepData = {
 
 type Step = FileFilterStep & StepWithDimensions & PresetStep & FormatStep & StepData
 
-export = function humanize(step: Step, robots: Robots, extrameta: ExtraMeta = {}): string {
+export function analyzeStep(step: Step, robots: Robots, extrameta: ExtraMeta = {}): string {
   let str = ``
 
   const robotName = step.robot

packages/file-exists/src/fileExists.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import fileExists from './fileExists'
+import { fileExists } from './fileExists'
 
 describe('fileExists', () => {
   test('main', async () => {

packages/file-exists/src/fileExists.ts

@@ -1,6 +1,6 @@
-import fs = require('node:fs')
+import fs from 'node:fs'
 
-export = function fileExists(path: string): Promise<boolean> {
+export function fileExists(path: string): Promise<boolean> {
   return new Promise((resolve) => {
     fs.access(path, fs.constants.F_OK, (err) => {
       resolve(!err)

packages/format-duration-ms/src/formatDurationMs.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import formatDurationMs from './formatDurationMs'
+import { formatDurationMs } from './formatDurationMs'
 
 describe('formatDurationMs', () => {
   test('main', async () => {

packages/format-duration-ms/src/formatDurationMs.ts

@@ -1,6 +1,6 @@
 import prettyMS = require('pretty-ms')
 
-export = function formatDurationMs(ms: number): string {
+export function formatDurationMs(ms: number): string {
   let human = prettyMS(ms)
 
   human = human.replace(/(\d+)\.\d+s/g, '$1s')

packages/has-property/src/has-property.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { test } from 'node:test'
-import hasProperty from './has-property'
+import { hasProperty } from './has-property'
 
 test('hasProperty', () => {
   assert.ok(hasProperty({ foo: 'bar' }, 'foo'))

packages/has-property/src/has-property.ts

@@ -1,4 +1,4 @@
-export = function hasProperty<K extends PropertyKey>(
+export function hasProperty<K extends PropertyKey>(
   obj: unknown,
   key: K | null | undefined,
 ): obj is Record<K, unknown> {

packages/post/src/post.ts

@@ -3,8 +3,8 @@ import fs = require('node:fs/promises')
 import inquirer = require('inquirer')
 
 import openInEditor = require('open-in-editor')
-import fileExists = require('@transloadit/file-exists')
-import slugify = require('@transloadit/slugify')
+import { fileExists } from '@transloadit/file-exists'
+import { slugify } from '@transloadit/slugify'
 import title = require('title')
 
 async function post(): Promise<void> {

packages/pr/src/pr.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import pr from './pr'
+import { pr } from './pr'
 
 describe('pr', () => {
   test('main', async () => {

packages/pr/src/pr.ts

@@ -1,6 +1,6 @@
 import util = require('node:util')
 
-export = function pr<T>(...args: T[]): T[] {
+export function pr<T>(...args: T[]): T[] {
   for (const arg of args) {
     console.log(util.inspect(arg, false, null, true))
   }

packages/prd/src/prd.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, mock, test } from 'node:test'
-import prd from './prd'
+import { prd } from './prd'
 
 describe('prd', () => {
   test('main', async () => {

packages/prd/src/prd.ts

@@ -1,6 +1,6 @@
-import pr = require('@transloadit/pr')
+import { pr } from '@transloadit/pr'
 
-export = function prd<T>(...args: T[]): void {
+export function prd<T>(...args: T[]): void {
   pr(...args)
   const err = new Error('Halt via prd')
   console.error(err)

packages/prettier-bytes/src/prettierBytes.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, it } from 'node:test'
-import prettierBytes from './prettierBytes'
+import { prettierBytes } from './prettierBytes'
 
 const testData = [
   [2, '2 B'],

packages/prettier-bytes/src/prettierBytes.ts

@@ -1,7 +1,7 @@
 // Adapted from https://github.com/Flet/prettier-bytes/
 // Changing 1000 bytes to 1024, so we can keep uppercase KB vs kB
 // ISC License (c) Dan Flettre https://github.com/Flet/prettier-bytes/blob/master/LICENSE
-export = function prettierBytes(input: number): string {
+export function prettierBytes(input: number): string {
   if (typeof input !== 'number' || Number.isNaN(input)) {
     throw new TypeError(`Expected a number, got ${typeof input}`)
   }

packages/slugify/src/slugify.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import slugify from './slugify'
+import { slugify } from './slugify'
 
 describe('slugify', () => {
   test('main', async () => {

packages/slugify/src/slugify.ts

@@ -1,4 +1,4 @@
-export = function slugify(str: string): string {
+export function slugify(str: string): string {
   if (!str || str !== `${str}`) return str
 
   return str

packages/sort-assembly/src/sortAssembly.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import sortAssembly from './sortAssembly'
+import { sortAssembly } from './sortAssembly'
 
 describe('sortAssembly', () => {
   test('main', () => {

packages/sort-assembly/src/sortAssembly.ts

@@ -1,8 +1,8 @@
-import sortObjectByPrio = require('@transloadit/sort-object-by-prio')
-import sortResult = require('@transloadit/sort-result')
-import hasProperty = require('@transloadit/has-property')
+import { sortObjectByPrio } from '@transloadit/sort-object-by-prio'
+import { sortResult } from '@transloadit/sort-result'
+import { hasProperty } from '@transloadit/has-property'
 
-export = function sortAssembly<T extends Record<string, unknown>>(assembly: T): T {
+export function sortAssembly<T extends Record<string, unknown>>(assembly: T): T {
   const sorted = sortObjectByPrio(assembly, {
     _: ['assembly_id', 'ok', 'message', 'warnings', 'error'],
     z: ['uploads', 'results'],

packages/sort-object-by-prio/src/sortObjectByPrio.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import sortObjectByPrio from './sortObjectByPrio'
+import { sortObjectByPrio } from './sortObjectByPrio'
 
 describe('sortObjectByPrio', () => {
   test('main', () => {

packages/sort-object-by-prio/src/sortObjectByPrio.ts

@@ -1,11 +1,8 @@
-import sortObject = require('@transloadit/sort-object')
+import { sortObject } from '@transloadit/sort-object'
 
 type Prefixes = Record<string, (string | RegExp)[]>
 
-export = function sortObjectByPrio<T extends Record<string, unknown>>(
-  obj: T,
-  prefixes: Prefixes,
-): T {
+export function sortObjectByPrio<T extends Record<string, unknown>>(obj: T, prefixes: Prefixes): T {
   return sortObject(obj, (argA: string, argB: string) => {
     let a = argA
     let b = argB

packages/sort-object/src/sortObject.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import sortObject from './sortObject'
+import { sortObject } from './sortObject'
 
 describe('sortObject', () => {
   test('main', () => {

packages/sort-object/src/sortObject.ts

@@ -1,4 +1,4 @@
-export = function sortObject<T extends Record<string, unknown>>(
+export function sortObject<T extends Record<string, unknown>>(
   obj: T,
   sortFunc?: (a: string, b: string) => number,
 ): T {

packages/sort-result-meta/src/sortResultMeta.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import sortResultMeta from './sortResultMeta'
+import { sortResultMeta } from './sortResultMeta'
 
 describe('sortResultMeta', () => {
   test('main', () => {

packages/sort-result-meta/src/sortResultMeta.ts

@@ -1,4 +1,4 @@
-import sortObjectByPrio = require('@transloadit/sort-object-by-prio')
+import { sortObjectByPrio } from '@transloadit/sort-object-by-prio'
 
 // using interface here instead of type causes Meta to not be assignable to the generic type
 // of sortObjectByPrio
@@ -12,7 +12,7 @@ function isObject(obj: unknown): obj is Record<string, unknown> {
   )
 }
 
-export = function sortResultMeta<T extends Meta>(meta: T): T {
+export function sortResultMeta<T extends Meta>(meta: T): T {
   if (meta.faces) {
     for (let i = 0; i < meta.faces.length; i++) {
       const el = meta.faces[i]

packages/sort-result/src/sortResult.test.ts

@@ -1,6 +1,6 @@
 import assert from 'node:assert'
 import { describe, test } from 'node:test'
-import sortResult from './sortResult'
+import { sortResult } from './sortResult'
 
 describe('sortResult', () => {
   test('main', () => {

packages/sort-result/src/sortResult.ts

@@ -1,7 +1,7 @@
-import sortObjectByPrio = require('@transloadit/sort-object-by-prio')
-import sortResultMeta = require('@transloadit/sort-result-meta')
+import { sortObjectByPrio } from '@transloadit/sort-object-by-prio'
+import { sortResultMeta } from '@transloadit/sort-result-meta'
 
-export = function sortResult<T extends { meta?: unknown }>(result: T): T {
+export function sortResult<T extends { meta?: unknown }>(result: T): T {
   const sorted = sortObjectByPrio(result, {
     _: ['id'],
     z: ['meta'],

packages/trigger-pager/src/triggerPager.test.ts

@@ -32,7 +32,7 @@ mockRequire('@pagerduty/pdjs', () => {
   return { api: () => ({ post: mockPost }) }
 })
 
-const triggerPager = require('./triggerPager')
+const { triggerPager } = require('./triggerPager')
 
 const LOREM_LONG = `Lorem ipsum dolor sit amet, consectetur adipiscing elit,
 sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.

packages/trigger-pager/src/triggerPager.ts

@@ -13,15 +13,15 @@ interface TriggerPagerOptions {
   urgency?: 'low' | 'high'
 }
 
-const triggerPager = async ({
+export async function triggerPager({
   description,
   from = '[email protected]',
   incidentKey,
   serviceId,
   title = incidentKey,
   token,
   urgency = 'high',
-}: TriggerPagerOptions): Promise<void> => {
+}: TriggerPagerOptions): Promise<void> {
   const res = await pagerduty.api({ token }).post('/incidents', {
     headers: {
       from,
@@ -57,5 +57,3 @@ const triggerPager = async ({
     }
   }
 }
-
-export = triggerPager

template-package/replaceMe.test.ts

@@ -1,7 +1,9 @@
-import replaceMe from './replaceMe'
+import assert from 'node:assert'
+import { describe, test } from 'node:test'
+import { replaceMe } from './replaceMe'
 
 describe('replaceMe', () => {
   test('main', async () => {
-    expect(replaceMe('foo')).toBe('bar')
+    assert.equal(replaceMe('foo'), 'bar')
   })
 })

template-package/replaceMe.ts

@@ -1,3 +1,3 @@
-export default function replaceMe(arg: string) {
+export function replaceMe(arg: string) {
   return arg
 }

tsconfig.base.json

@@ -1,6 +1,5 @@
 {
   "compilerOptions": {
-    "esModuleInterop": true,
     "lib": ["ES2023"],
     "target": "es2022",
     "module": "nodenext",