fix: align exported core Diff interface with previous 2.8 signature

package.json

@@ -73,6 +73,7 @@
   },
   "repository": "bump-sh/cli",
   "scripts": {
+    "prepare": "npm run build",
     "build": "shx rm -rf dist && tsc -b",
     "clean": "rm -rf dist/ oclif.manifest.json",
     "lint": "eslint . --ext .ts",

src/api/index.ts

@@ -59,10 +59,11 @@ class BumpApi {
   }
 
   // Check https://oclif.io/docs/config for details about Config.IConfig
-  public constructor(protected config: Config) {
+  public constructor(protected config?: Config) {
     const baseURL = `${vars.apiUrl}${vars.apiBasePath}`
+    const userAgent = config?.userAgent || 'bump-cli'
     const headers: {Authorization?: string; 'User-Agent': string} = {
-      'User-Agent': vars.apiUserAgent(config.userAgent),
+      'User-Agent': vars.apiUserAgent(userAgent),
     }
 
     this.client = axios.create({

src/commands/diff.ts

@@ -105,7 +105,7 @@ export default class Diff extends BaseCommand<typeof Diff> {
 
     ux.action.status = '...diff on Bump.sh in progress'
 
-    const diff: DiffResponse | undefined = await new CoreDiff(this.bump).run(
+    const diff: DiffResponse | undefined = await new CoreDiff(this.config).run(
       args.file,
       args.otherFile,
       documentation,

src/core/diff.ts

@@ -1,3 +1,4 @@
+import {Config} from '@oclif/core'
 import {CLIError} from '@oclif/core/errors'
 import debug from 'debug'
 
@@ -10,9 +11,17 @@
   static readonly TIMEOUT = 120
 
   private _bump!: BumpApi
+  private _config: Config | undefined
 
-  public constructor(bumpClient: BumpApi) {
-    this._bump = bumpClient
+  public constructor(config?: Config) {
+    if (config) {
+      this._config = config
+    }
+  }
+
+  get bumpClient(): BumpApi {
+    if (!this._bump) this._bump = new BumpApi(this._config!)
+    return this._bump
   }
 
   get pollingPeriod(): number {
@@ -32,7 +41,7 @@
       references,
     }
 
-    const response = await this._bump.postDiff(request)
+    const response = await this.bumpClient.postDiff(request)
 
     switch (response.status) {
       case 201: {
@@ -48,7 +57,7 @@
     }
   }
 
   async createVersion(
     file: string,
     documentation: string,
     token: string,
@@ -68,7 +77,7 @@
       unpublished: true,
     }
 
-    const response = await this._bump.postVersion(request, token)
+    const response = await this.bumpClient.postVersion(request, token)
 
     switch (response.status) {
       case 201: {
@@ -91,7 +100,7 @@
   }
 
   extractDiff(versionWithDiff: VersionResponse & WithDiff): DiffResponse {
     // TODO: return a real diff_id in the GET /version API
     return {
       breaking: versionWithDiff.diff_breaking,
       details: versionWithDiff.diff_details,
@@ -115,7 +124,7 @@
     await this.delay(this.pollingPeriod)
   }
 
   public async run(
     file1: string,
     file2: string | undefined,
     documentation: string | undefined,
@@ -125,6 +134,8 @@
     format: string,
     expires: string | undefined,
   ): Promise<DiffResponse | undefined> {
+    if (!this._config) this._config = await Config.load('../../')
+
     let diffVersion: DiffResponse | VersionResponse | undefined
 
     if (file2 && (!documentation || !token)) {
@@ -157,8 +168,8 @@
     opts: {format: string; timeout: number},
   ): Promise<DiffResponse> {
     const pollingResponse = await (this.isVersion(result) && token
-      ? this._bump.getVersion(result.id, token)
-      : this._bump.getDiff(result.id, opts.format))
+      ? this.bumpClient.getVersion(result.id, token)
+      : this.bumpClient.getDiff(result.id, opts.format))
 
     if (opts.timeout <= 0) {
       throw new CLIError(