fix: remove unused config pubspecPath (#28)

.eslintrc.json

@@ -7,5 +7,11 @@
   ],
   "parser": "@typescript-eslint/parser",
   "parserOptions": { "project": ["tsconfig.eslint.json"] },
-  "plugins": ["@typescript-eslint"]
+  "plugins": ["@typescript-eslint"],
+  "rules": {
+    "no-unused-vars": [
+      "error",
+      { "vars": "all", "args": "all", "ignoreRestSiblings": false }
+    ]
+  }
 }

src/prepare.ts

@@ -2,22 +2,17 @@ import { readFileSync, writeFileSync } from 'fs';
 import { PrepareContext } from 'semantic-release';
 import { parse } from 'yaml';
 import { Pubspec } from './schemas.js';
-import { PluginConfig } from './types.js';
-import { getConfig } from './utils.js';
 
-export const prepare = async (
-  pluginConfig: PluginConfig,
-  { nextRelease }: PrepareContext
-) => {
-  const { pubspecPath } = getConfig(pluginConfig);
+const PUBSPEC_PATH = 'pubspec.yaml';
 
-  const data = readFileSync(pubspecPath, 'utf-8');
+export const prepare = async ({ nextRelease }: PrepareContext) => {
+  const data = readFileSync(PUBSPEC_PATH, 'utf-8');
   const pubspec = Pubspec.parse(parse(data));
 
   const newData = data.replace(
     new RegExp(`version:[ \t]+${pubspec.version}`),
     `version: ${nextRelease.version}`
   );
 
-  writeFileSync(pubspecPath, newData);
+  writeFileSync(PUBSPEC_PATH, newData);
 };

src/types.ts

@@ -1,4 +1,3 @@
 export type PluginConfig = {
   cli: 'dart' | 'flutter';
-  pubspecPath: string;
 };

src/utils.ts

@@ -4,8 +4,7 @@ import { ServiceAccount } from './schemas.js';
 import { PluginConfig } from './types.js';
 
 const DEFAULT_CONFIG: PluginConfig = {
-  cli: 'dart',
-  pubspecPath: 'pubspec.yaml'
+  cli: 'dart'
 };
 
 const PUB_DEV_AUDIENCE = 'https://pub.dev';

tests/prepare.test.ts

@@ -1,16 +1,15 @@
-import { readFileSync, writeFileSync } from 'fs';
 import { codeBlock } from 'common-tags';
+import { readFileSync, writeFileSync } from 'fs';
 import { NextRelease, PrepareContext } from 'semantic-release';
 import { afterEach, describe, expect, test, vi } from 'vitest';
 import { mock } from 'vitest-mock-extended';
-import { PluginConfig, prepare } from '../src/index.js';
+import { prepare } from '../src/index.js';
 
 vi.mock('fs');
 
 describe('prepare', () => {
-  const cli = 'dart';
   const newVersion = '1.2.3';
-  const pubspecPath = 'pubspecPath';
+  const pubspecPath = 'pubspec.yaml';
   const oldPubspec = codeBlock`
     name: pub_package
     version: 1.2.0
@@ -32,8 +31,6 @@ describe('prepare', () => {
       cupertino_icons: 1.0.6
   `;
 
-  const config: PluginConfig = { cli, pubspecPath };
-
   afterEach(() => {
     vi.restoreAllMocks();
   });
@@ -47,7 +44,7 @@ describe('prepare', () => {
 
     vi.mocked(readFileSync).mockReturnValue(oldPubspec);
 
-    await prepare(config, context);
+    await prepare(context);
 
     expect(readFileSync).toHaveBeenNthCalledWith(1, pubspecPath, 'utf-8');
     expect(writeFileSync).toHaveBeenNthCalledWith(1, pubspecPath, newPubspec);

tests/utils.test.ts

@@ -10,8 +10,7 @@ vi.mock('google-auth-library');
 
 describe('getConfig', () => {
   const config: PluginConfig = {
-    cli: 'flutter',
-    pubspecPath: 'a/pubspec.yaml'
+    cli: 'flutter'
   };
 
   test('success', () => {