BREAKING CHANGE: Move to yarn 4 and only output a cjs build, drop (of…

…ficial) support for node 14, bump json-scheme-walker version and openapi types version (#45)

* chore(versions): move to yarn 4, drop release for node 14, only output cjs

BREAKING CHANGE: Move to yarn 4 and only output a cjs build, drop support for node 14

* update build

.github/workflows/release.yml

@@ -13,8 +13,9 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: 18
-      - run: yarn install --frozen-lockfile
+          node-version: latest
+          cache: 'yarn'
+      - run: yarn install --immutable
       - run: yarn build
       - run: yarn test
       - run: npx semantic-release --branches main

.github/workflows/test.yml

@@ -8,19 +8,20 @@ jobs:
     strategy:
       matrix:
         node-version:
-          - 14
-          - 16
           - 18
+          - 20
+          - latest
     steps:
       - uses: actions/checkout@v3
       - name: Use Node.js ${{ matrix.node-version }}
         uses: actions/setup-node@v3
         with:
           node-version: ${{ matrix.node-version }}
+          cache: 'yarn'
       - name: yarn install, build, and test
         run: |
-          yarn --frozen-lockfile
-          yarn build --if-present
+          yarn install --immutable
+          yarn build
           yarn lint
           yarn test
         env:

.yarnrc.yml

@@ -0,0 +1,3 @@
+nodeLinker: node-modules
+
+yarnPath: .yarn/releases/yarn-4.0.1.cjs

package.json

@@ -2,25 +2,16 @@
 	"name": "@openapi-contrib/json-schema-to-openapi-schema",
 	"version": "0.0.0-development",
 	"description": "Converts a JSON Schema to OpenAPI Schema Object",
-	"bin": {
-		"json-schema-to-openapi-schema": "bin/json-schema-to-openapi-schema.js"
-	},
-	"types": "dist/mjs/index.d.ts",
+	"bin": "bin/json-schema-to-openapi-schema.js",
+	"types": "dist/index.d.ts",
 	"files": [
-		"/bin",
-		"/dist"
+		"bin",
+		"dist"
 	],
-	"main": "dist/cjs/index.js",
-	"module": "dist/mjs/index.js",
-	"exports": {
-		".": {
-			"import": "./dist/mjs/index.js",
-			"require": "./dist/cjs/index.js"
-		}
-	},
+	"main": "dist/index.js",
 	"scripts": {
 		"prepublish": "yarn build",
-		"build": "rm -fr dist/* && tsc -p tsconfig.json && tsc -p tsconfig-cjs.json && node scripts/fixup.cjs",
+		"build": "rimraf dist && tsc -p tsconfig.json",
 		"lint": "eslint . && prettier -c src",
 		"lint:fix": "eslint . --fix && prettier -c src -w",
 		"typecheck": "tsc --noEmit",
@@ -31,30 +22,32 @@
 	"author": "OpenAPI Contrib",
 	"license": "MIT",
 	"engines": {
-		"node": ">=14"
+		"node": ">=18"
 	},
 	"dependencies": {
-		"@apidevtools/json-schema-ref-parser": "^9.0.9",
-		"json-schema-walker": "^1.1.0",
-		"openapi-types": "^12.1.0",
-		"yargs": "^17.6.2"
+		"@apidevtools/json-schema-ref-parser": "^11.1.0",
+		"json-schema-walker": "^2.0.0",
+		"openapi-types": "^12.1.3",
+		"yargs": "^17.7.2"
 	},
 	"devDependencies": {
-		"@types/json-schema": "^7.0.11",
-		"@typescript-eslint/eslint-plugin": "^5.49.0",
-		"@typescript-eslint/parser": "^5.49.0",
-		"c8": "^7.12.0",
-		"eslint": "^8.32.0",
-		"eslint-config-prettier": "^8.6.0",
-		"eslint-plugin-prettier": "^4.2.1",
-		"eslint-plugin-unused-imports": "^2.0.0",
-		"nock": "^13.3.0",
-		"prettier": "^2.8.3",
-		"typescript": "^4.9.4",
-		"vitest": "^0.28.1"
+		"@types/json-schema": "^7.0.15",
+		"@typescript-eslint/eslint-plugin": "^6.10.0",
+		"@typescript-eslint/parser": "^6.10.0",
+		"c8": "^8.0.1",
+		"eslint": "^8.53.0",
+		"eslint-config-prettier": "^9.0.0",
+		"eslint-plugin-prettier": "^5.0.1",
+		"eslint-plugin-unused-imports": "^3.0.0",
+		"nock": "^13.3.8",
+		"prettier": "^3.0.3",
+		"rimraf": "^5.0.5",
+		"typescript": "^5.2.2",
+		"vitest": "^0.34.6"
 	},
 	"prettier": {
 		"singleQuote": true,
 		"useTabs": true
-	}
+	},
+	"packageManager": "[email protected]"
 }

src/index.ts

@@ -1,13 +1,13 @@
-import type { JSONSchema } from '@apidevtools/json-schema-ref-parser';
 import type {
 	JSONSchema4,
 	JSONSchema6Definition,
 	JSONSchema7Definition,
 } from 'json-schema';
-import type { Options, SchemaType, SchemaTypeKeys } from './types.js';
+import type { Options, SchemaType, SchemaTypeKeys } from './types';
 import { Walker } from 'json-schema-walker';
-import { allowedKeywords } from './const.js';
+import { allowedKeywords } from './const';
 import type { OpenAPIV3 } from 'openapi-types';
+import type { JSONSchema } from '@apidevtools/json-schema-ref-parser/dist/lib/types';
 
 class InvalidTypeError extends Error {
 	constructor(message: string) {
@@ -21,7 +21,7 @@ const oasExtensionPrefix = 'x-';
 
 const handleDefinition = async <T extends JSONSchema4 = JSONSchema4>(
 	def: JSONSchema7Definition | JSONSchema6Definition | JSONSchema4,
-	schema: T
+	schema: T,
 ) => {
 	if (typeof def !== 'object') {
 		return def;
@@ -45,7 +45,7 @@ const handleDefinition = async <T extends JSONSchema4 = JSONSchema4>(
 						circular: 'ignore',
 					},
 				},
-			}
+			},
 		);
 		await walker.walk(convertSchema, walker.vocabularies.DRAFT_07);
 		if ('definitions' in walker.rootSchema) {
@@ -72,7 +72,7 @@ const handleDefinition = async <T extends JSONSchema4 = JSONSchema4>(
 
 const convert = async <T extends object = JSONSchema4>(
 	schema: T,
-	options?: Options
+	options?: Options,
 ): Promise<OpenAPIV3.Document> => {
 	const walker = new Walker<T>();
 	const convertDefs = options?.convertUnreferencedDefinitions ?? true;
@@ -271,7 +271,7 @@ function convertIllegalKeywordsAsExtensions(schema: SchemaType) {
 		.filter(
 			(keyword) =>
 				!keyword.startsWith(oasExtensionPrefix) &&
-				!allowedKeywords.includes(keyword)
+				!allowedKeywords.includes(keyword),
 		)
 		.forEach((keyword: SchemaTypeKeys) => {
 			const key = `${oasExtensionPrefix}${keyword}` as keyof SchemaType;

src/types.ts

@@ -1,5 +1,5 @@
-import type { JSONSchema } from '@apidevtools/json-schema-ref-parser';
-import type $RefParser from '@apidevtools/json-schema-ref-parser';
+import type { JSONSchema } from '@apidevtools/json-schema-ref-parser/dist/lib/types';
+import type { ParserOptions } from '@apidevtools/json-schema-ref-parser/dist/lib/options';
 
 export type addPrefixToObject = {
 	[K in keyof JSONSchema as `x-${K}`]: JSONSchema[K];
@@ -9,7 +9,7 @@ export interface Options {
 	cloneSchema?: boolean;
 	dereference?: boolean;
 	convertUnreferencedDefinitions?: boolean;
-	dereferenceOptions?: $RefParser.Options;
+	dereferenceOptions?: ParserOptions | undefined;
 }
 type ExtendedJSONSchema = addPrefixToObject & JSONSchema;
 export type SchemaType = ExtendedJSONSchema & {

test/__snapshots__/circular_schema.test.ts.snap

@@ -1,4 +1,4 @@
-// Vitest Snapshot v1
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
 exports[`converting circular/openapi.json without circular references turned off  1`] = `
 {

test/__snapshots__/dereference_schema.test.ts.snap

@@ -1,4 +1,4 @@
-// Vitest Snapshot v1
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
 exports[`throws an error when dereferecing fails 1`] = `
 {

test/dereference_schema.test.ts

@@ -192,7 +192,8 @@ it('dereferencing schema with deference option at root', async ({ expect }) => {
 	expect(result).toEqual(expected);
 });
 
-it('dereferencing schema with remote http and https references', async ({
+// skip until nock supports native fetch https://github.com/nock/nock/issues/2397
+it.skip('dereferencing schema with remote http and https references', async ({
 	expect,
 }) => {
 	nock('http://foo.bar/')

test/invalid_types.test.ts

@@ -4,20 +4,20 @@ import { getSchema } from './helpers';
 it('dateTime is invalid type', async ({ expect }) => {
 	const schema = { type: 'dateTime' };
 	await expect(() => convert(schema)).rejects.toThrowError(
-		/is not a valid type/
+		/is not a valid type/,
 	);
 });
 
 it('foo is invalid type', async ({ expect }) => {
 	const schema = { type: 'foo' };
 	await expect(() => convert(schema)).rejects.toThrowError(
-		/is not a valid type/
+		/is not a valid type/,
 	);
 });
 
 it('invalid type inside complex schema', async ({ expect }) => {
 	const schema = getSchema('invalid/json-schema.json');
 	await expect(() => convert(schema)).rejects.toThrowError(
-		/is not a valid type/
+		/is not a valid type/,
 	);
 });

test/nullable.test.ts

@@ -30,7 +30,7 @@ describe('nullable', () => {
 			expect(result).toEqual({
 				[key]: [{ type: 'string', nullable: true }],
 			});
-		}
+		},
 	);
 
 	it('supports nullables inside definitions', async ({ expect }) => {
@@ -209,6 +209,6 @@ describe('nullable', () => {
 					},
 				],
 			});
-		}
+		},
 	);
 });

test/tsconfig.json

@@ -1,10 +1,10 @@
 {
 	"extends": "../tsconfig.json",
 	"compilerOptions": {
-		"target": "ES2015",
-		"lib": ["es2015", "dom"],
+		"target": "ESNext",
+		"lib": ["ESNext", "dom"],
 		"types": ["vitest/globals"],
-		"rootDir": "."
+		"rootDir": "../"
 	},
 	"include": ["."]
 }

tsconfig.json

@@ -1,7 +1,7 @@
 {
 	"compilerOptions": {
-		"module": "esnext",
-		"outDir": "dist/mjs",
+		"module": "commonjs",
+		"outDir": "dist",
 		"target": "es2020",
 		"allowJs": true,
 		"allowSyntheticDefaultImports": true,
@@ -12,7 +12,7 @@
 		"lib": ["esnext"],
 		"listEmittedFiles": false,
 		"listFiles": false,
-		"moduleResolution": "node16",
+		"moduleResolution": "Node",
 		"noFallthroughCasesInSwitch": true,
 		"pretty": true,
 		"resolveJsonModule": true,

vite.config.ts

@@ -9,6 +9,6 @@ export default defineConfig({
 		reporters: 'verbose',
 	},
 	esbuild: {
-		target: 'node10',
+		target: 'node21',
 	},
 });