chore: upgrade deps

.eslintrc

@@ -1,25 +1,14 @@
 {
-    "extends": [
-        "plugin:@typescript-eslint/recommended",
-        "prettier",
-        "prettier/@typescript-eslint"
-    ],
+    "extends": ["plugin:@typescript-eslint/recommended", "prettier"],
     "parser": "@typescript-eslint/parser",
-    "plugins": [
-        "@typescript-eslint"
-    ],
+    "plugins": ["@typescript-eslint"],
     "rules": {
-        "@typescript-eslint/explicit-function-return-type": [ "error", { "allowExpressions": true }],
+        "@typescript-eslint/explicit-function-return-type": [
+            "error",
+            { "allowExpressions": true }
+        ],
         "@typescript-eslint/explicit-member-accessibility": "off",
         "@typescript-eslint/no-explicit-any": "off",
         "@typescript-eslint/no-parameter-properties": "off"
-    },
-    "overrides": [
-        {
-            "files": ["**/__tests__/**"],
-            "rules": {
-                "@typescript-eslint/camelcase": "off"
-            }
-        }
-    ]
-}
+    }
+}

.prettierignore

@@ -0,0 +1,2 @@
+coverage
+dist

.prettierrc.json

@@ -1,5 +1,6 @@
 {
     "trailingComma": "all",
     "tabWidth": 4,
-    "singleQuote": true
+    "singleQuote": true,
+    "semi": true
 }

.travis.yml

@@ -1,7 +1,8 @@
 language: node_js
 node_js:
-    - 10
     - 12
+    - 14
+    - 16
 jobs:
     include:
         # Define the release stage that runs semantic-release

jest.config.js

@@ -155,7 +155,9 @@ module.exports = {
     // timers: "real",
 
     // A map from regular expressions to paths to transformers
-    // transform: null,
+    transform: {
+        '^.+\\.(ts|tsx)?$': 'ts-jest',
+    },
 
     // An array of regexp pattern strings that are matched against all source file paths, matched files will skip transformation
     // transformIgnorePatterns: [

package.json

@@ -11,43 +11,45 @@
     ],
     "devDependencies": {
         "@types/graphviz": "^0.0.32",
-        "@types/jest": "^26.0.14",
-        "@types/js-yaml": "^3.12.1",
+        "@types/jest": "^26.0.23",
+        "@types/js-yaml": "^4.0.1",
         "@types/lodash.sortby": "^4.7.6",
-        "@types/node": "^14.11.8",
-        "@types/uniqid": "^5.2.0",
-        "@typescript-eslint/eslint-plugin": "^2.9.0",
-        "@typescript-eslint/parser": "^2.9.0",
+        "@types/node": "^15.12.5",
+        "@types/uniqid": "^5.3.0",
+        "@typescript-eslint/eslint-plugin": "^4.28.1",
+        "@typescript-eslint/parser": "^4.28.1",
         "cz-conventional-changelog": "3.3.0",
-        "eslint": "^7.11.0",
-        "eslint-config-prettier": "^6.7.0",
-        "husky": "^4.3.0",
-        "jest": "^26.5.3",
-        "prettier": "^2.1.2",
-        "pretty-quick": "^3.0.2",
-        "rimraf": "^3.0.0",
-        "semantic-release": "^17.2.0",
-        "ts-jest": "^26.4.1",
-        "ts-node": "^9.0.0",
-        "typescript": "^4.0.3"
+        "eslint": "^7.29.0",
+        "eslint-config-prettier": "^8.3.0",
+        "husky": "^6.0.0",
+        "jest": "^27.0.6",
+        "prettier": "^2.3.2",
+        "pretty-quick": "^3.1.1",
+        "rimraf": "^3.0.2",
+        "semantic-release": "^17.4.4",
+        "ts-jest": "^27.0.3",
+        "ts-node": "^10.0.0",
+        "typescript": "^4.3.4"
     },
     "scripts": {
+        "eslint": "eslint . --ext .js,.jsx,.ts,.tsx",
+        "prettier": "prettier --write .",
         "preprepare": "rimraf dist/",
         "prepare": "tsc",
-        "pretest": "yarn lint",
         "lint": "eslint --ext .ts src",
+        "pretest": "prettier --check . && yarn lint",
         "test": "jest --ci --coverage",
-        "test:watch": "jest --watch",
+        "test:watch": "jest --watch --notify",
         "semantic-release": "semantic-release"
     },
     "dependencies": {
         "@sindresorhus/slugify": "^1.1.0",
         "graphviz": "^0.0.9",
-        "js-yaml": "^3.13.1",
+        "js-yaml": "^4.1.0",
         "lodash.sortby": "^4.7.0",
-        "ow": "^0.18.0",
+        "ow": "^0.25.0",
         "topological-sort": "^0.3.0",
-        "uniqid": "^5.2.0"
+        "uniqid": "^5.3.0"
     },
     "husky": {
         "hooks": {

src/__tests__/__snapshots__/command.spec.ts.snap

@@ -1659,7 +1659,7 @@ exports[`buildkite-graph Steps Command retry yaml 4`] = `
     retry:
       manual:
         allowed: false
-        reason: 'Sorry, you can''t retry a deployment'
+        reason: Sorry, you can't retry a deployment
 "
 `;
 
@@ -1745,7 +1745,7 @@ exports[`buildkite-graph Steps Command retry yaml_depends_on 4`] = `
     retry:
       manual:
         allowed: false
-        reason: 'Sorry, you can''t retry a deployment'
+        reason: Sorry, you can't retry a deployment
 "
 `;
 

src/__tests__/conditional.spec.ts

@@ -352,7 +352,7 @@ describe('buildkite-graph', () => {
 
                         const tests = new MyConditional(
                             () =>
-                                new Promise<Step>(resolve =>
+                                new Promise<Step>((resolve) =>
                                     setTimeout(
                                         () =>
                                             resolve(

src/__tests__/helpers.ts

@@ -44,7 +44,7 @@ export const createTest = (
     describeFn = describe,
 ): void =>
     describeFn(name, () => {
-        test.each(serializersToTest)('%s', async type => {
+        test.each(serializersToTest)('%s', async (type) => {
             resetUuidCounter();
             let entities = gen();
             if (!Array.isArray(entities)) {

src/base.ts

@@ -101,7 +101,7 @@ export abstract class Step implements BaseStep, Serializable {
 
     async toJson(
         opts: ToJsonSerializationOptions = { explicitDependencies: false },
-    ): Promise<object> {
+    ): Promise<Record<string, unknown>> {
         if (!opts.explicitDependencies) {
             return {};
         }
@@ -112,15 +112,13 @@ export abstract class Step implements BaseStep, Serializable {
                     opts.cache,
                 )
             ).map((s) => ({
-                /* eslint-disable @typescript-eslint/camelcase */
                 step: s.key,
                 allow_failure: this._allowDependencyFailure
                     ? undefined
                     : this.always || undefined,
             })),
             'step',
         );
-        /* eslint-disable @typescript-eslint/camelcase */
         return {
             key: this.key,
             depends_on: dependsOn.length ? dependsOn : undefined,
@@ -139,7 +137,7 @@ export class BranchLimitedStep extends Step {
     }
     async toJson(
         opts: ToJsonSerializationOptions = { explicitDependencies: false },
-    ): Promise<object> {
+    ): Promise<Record<string, unknown>> {
         return {
             branches: this.branches.size
                 ? [...this.branches].sort().join(' ')
@@ -163,7 +161,7 @@ export class LabeledStep extends BranchLimitedStep {
 
     async toJson(
         opts: ToJsonSerializationOptions = { explicitDependencies: false },
-    ): Promise<object> {
+    ): Promise<Record<string, unknown>> {
         return {
             label: this.label,
             ...(await super.toJson(opts)),

src/index.ts

@@ -41,8 +41,10 @@ export type ToJsonSerializationOptions =
           cache: StepCache;
       }
     | { explicitDependencies: false };
+
+type JSON = Record<string, unknown> | JSON[];
 export interface Serializable {
-    toJson(opts?: ToJsonSerializationOptions): Promise<object | undefined>;
+    toJson(opts?: ToJsonSerializationOptions): Promise<JSON | undefined>;
 }
 
 export type PotentialStep = Step | Conditional<Step>;
@@ -60,7 +62,7 @@ export class Pipeline implements Serializable {
     }
 
     add(...step: PotentialStep[]): this {
-        step.forEach(s => {
+        step.forEach((s) => {
             if (this.steps.includes(s)) {
                 throw new Error('Can not add the same step more than once');
             }
@@ -110,7 +112,7 @@ export class Pipeline implements Serializable {
 
     async toJson(
         opts: SerializationOptions = { explicitDependencies: false },
-    ): Promise<object> {
+    ): Promise<Record<string, unknown>> {
         const newOpts: ToJsonSerializationOptions = opts.explicitDependencies
             ? {
                   explicitDependencies: true,
@@ -123,7 +125,7 @@ export class Pipeline implements Serializable {
         return {
             env: await (this.env as KeyValueImpl<this>).toJson(),
             steps: await Promise.all(
-                (await this.toList(newOpts)).map(s => s.toJson(newOpts)),
+                (await this.toList(newOpts)).map((s) => s.toJson(newOpts)),
             ),
         };
     }

src/key_value.ts

@@ -6,8 +6,10 @@ export interface KeyValue<T> {
     set(name: string, value: string): T;
 }
 
-export class KeyValueImpl<T> extends Chainable<T>
-    implements KeyValue<T>, Serializable {
+export class KeyValueImpl<T>
+    extends Chainable<T>
+    implements KeyValue<T>, Serializable
+{
     public readonly vars: Map<string, string> = new Map();
 
     set(name: string, value: string): T {
@@ -17,7 +19,7 @@ export class KeyValueImpl<T> extends Chainable<T>
         return this.parent;
     }
 
-    async toJson(): Promise<object | undefined> {
+    async toJson(): Promise<Record<string, unknown> | undefined> {
         return this.vars.size ? mapToObject(this.vars) : undefined;
     }
 }

src/serializers/json.ts

@@ -5,10 +5,12 @@ type JsonSerializationOptions = {
     stringify?: boolean;
 } & SerializationOptions;
 
-export class JsonSerializer implements Serializer<object | string> {
+export class JsonSerializer
+    implements Serializer<Record<string, unknown> | string>
+{
     constructor(private readonly opts: JsonSerializationOptions = {}) {}
 
-    async serialize(e: Pipeline): Promise<object | string> {
+    async serialize(e: Pipeline): Promise<Record<string, unknown> | string> {
         const serialized = JSON.stringify(await e.toJson(this.opts));
         // Workaround to get rid of undefined values
         return this.opts.stringify ? serialized : JSON.parse(serialized);

src/serializers/structural.ts

@@ -4,6 +4,6 @@ import { Serializer } from '.';
 export class StructuralSerializer implements Serializer<string> {
     async serialize(p: Pipeline): Promise<string> {
         const steps = await p.toList();
-        return steps.map(step => `* ${step.toString()}`).join('\n');
+        return steps.map((step) => `* ${step.toString()}`).join('\n');
     }
 }

src/serializers/yaml.ts

@@ -4,14 +4,16 @@ import { Serializer } from '.';
 import { JsonSerializer } from './json';
 
 export class YamlSerializer implements Serializer<string> {
-    private readonly jsonSerializer: Serializer<string | object>;
+    private readonly jsonSerializer: Serializer<
+        string | Record<string, unknown>
+    >;
 
     constructor(opts: SerializationOptions = {}) {
         this.jsonSerializer = new JsonSerializer({ ...opts, stringify: false });
     }
 
     async serialize(e: Pipeline): Promise<string> {
-        return jsyaml.safeDump(await this.jsonSerializer.serialize(e), {
+        return jsyaml.dump(await this.jsonSerializer.serialize(e), {
             skipInvalid: true,
             noRefs: true,
             styles: {

src/sortedSteps.ts

@@ -10,7 +10,7 @@ export async function sortedSteps(
 ): Promise<Step[]> {
     const steps = await unwrapSteps(e.steps, cache);
     const sortOp = new TopologicalSort<Step, Step>(
-        new Map(steps.map(step => [step, step])),
+        new Map(steps.map((step) => [step, step])),
     );
 
     async function getAndCacheDependency(
@@ -110,6 +110,6 @@ export async function sortedSteps(
         }
     }
     return Array.from(sortOp.sort().values())
-        .map(i => i.node)
-        .filter(s => !removedEffects.has(s));
+        .map((i) => i.node)
+        .filter((s) => !removedEffects.has(s));
 }

src/steps/block.ts

@@ -20,7 +20,7 @@ export class BlockStep extends BranchLimitedStep {
 
     async toJson(
         opts: ToJsonSerializationOptions = { explicitDependencies: false },
-    ): Promise<object> {
+    ): Promise<Record<string, unknown>> {
         return {
             ...(await super.toJson(opts)),
             block: this.title,