Skip to content

Commit 6b7ec48

Browse files
committed
add type module to switch to ESM, fix import issues
1 parent f79c347 commit 6b7ec48

File tree

8 files changed

+28
-19
lines changed

8 files changed

+28
-19
lines changed

.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,4 +10,5 @@ ca.crt/
1010
ca.key/
1111
coverage/
1212
certs/
13-
.DS_Store
13+
.DS_Store
14+
dist/

package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
{
22
"name": "dbspy",
33
"version": "8.0.0",
4+
"type": "module",
45
"description": "The open source dev tool to logically model and build MySQL and PostgreSQL databases",
56
"main": "server/server.ts",
67
"engines": {
@@ -21,7 +22,7 @@
2122
"cert:linux": "rm -rf .cert && mkdir -p .cert && cd .cert && mkcert create-ca && mkcert create-cert --key key.pem --cert cert.pem",
2223
"e2e": "cypress open --project ./tests"
2324
},
24-
"author": "Minh Chang, David Jones, Darius Okafor, John Ruiz, Parwinder Singh",
25+
"author": "Vicky Yue, Reva Venkataraman, Yihe Liu, Roshumba Llewellyn, Emma Vargas",
2526
"license": "MIT",
2627
"dependencies": {
2728
"@tippyjs/react": "^4.2.6",

server/logger/index.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,15 @@
11
/**
22
* Creates colored alert via log.info()
33
**/
4-
import pino from 'pino';
4+
import pinoModule from 'pino';
55
import dayjs from 'dayjs';
66

7+
// TS thinks pink is not a function anymore - can't import pino from 'pino'
8+
// with "type": "module" & "moduleResolution": "NodeNext", TS expects CommonJS modules to be imported differently
9+
// Pino is published as CmmonJS (cjs) not ESM
10+
// Need to do a "default" import manually in this case
11+
const pino = pinoModule.default;
12+
713
const transport = pino.transport({
814
target: 'pino-pretty',
915
options: { colorize: true },

server/models/userModel.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import mysql from 'mysql2';
22
import fs from 'fs';
3-
import log from '../logger';
3+
import log from '../logger/index.js';
44
import path from 'path';
55

66
import dotenv from 'dotenv';

server/routes/index.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,21 @@
11
import express, { Express, Request, Response, NextFunction } from 'express';
2-
import { getAccesToken, getUserInfo } from '../controllers/oauth.controller';
3-
import { setJwtToken, getCurrentUser } from '../controllers/sessionController';
2+
import { getAccesToken, getUserInfo } from '../controllers/oauth.controller.js';
3+
import { setJwtToken, getCurrentUser } from '../controllers/sessionController.js';
44
import {
55
retrieveSchema,
66
saveSchema,
77
userRegistration,
88
verifyUser,
9-
} from '../controllers/user.controller';
10-
import { postgresRouter } from './postgres.router';
11-
import { microsoftRouter } from './microsoft.router';
12-
import { oracleRouter } from './oracle.router';
13-
import { mysqlRouter } from './mysql.router';
14-
import { sqliteRouter } from './sqlite.router';
15-
import { saveRouter } from './save.router';
9+
} from '../controllers/user.controller.js';
10+
import { postgresRouter } from './postgres.router.js';
11+
import { microsoftRouter } from './microsoft.router.js';
12+
import { oracleRouter } from './oracle.router.js';
13+
import { mysqlRouter } from './mysql.router.js';
14+
import { sqliteRouter } from './sqlite.router.js';
15+
import { saveRouter } from './save.router.js';
1616
import { config } from 'dotenv';
17-
import log from '../logger/index';
18-
import type { DefaultErr } from '../../src/Types';
17+
import log from '../logger/index.js';
18+
import type { DefaultErr } from '../../src/Types.js';
1919
import session from 'express-session';
2020
import path from 'path';
2121

server/server.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { config } from 'dotenv';
22
import express, { Express } from 'express';
3-
import log from './logger/index';
4-
import routes from './routes';
3+
import log from './logger/index.js';
4+
import routes from './routes/index.js';
55
import path from 'path';
66
import cors from 'cors';
77
import session from 'express-session';

src/pages/Signup.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
// React & React Router Modules
22
import axios from 'axios';
33
import React, { useState } from 'react';
4-
import useCredentialsStore from '../store/credentialsStore';
5-
import { handleOAuthLogin } from '../utils/getGoogleUrl';
4+
import useCredentialsStore from '../store/credentialsStore.js';
5+
import { handleOAuthLogin } from '../utils/getGoogleUrl.js';
66
import googleImg from '../../src/assets/GoogleImage.png';
77
import gitHubImage from '../../src/assets/GithubImage.png';
88
import { NavLink, useNavigate } from 'react-router-dom';

vite.config.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import react from '@vitejs/plugin-react';
44
import { resolve } from 'path';
55

66
dns.setDefaultResultOrder('verbatim');
7+
78
export default defineConfig({
89
server: {
910
port: 8080,

0 commit comments

Comments
 (0)