fix: Improve filtering of ignored-dependecies

There was an issue related to ignoring dependencies causing that none of
the libs were processed when ingoredDependecies was empty. Filtering
adjusted in order to satify that it won't match every asset name when
empty.
Every string startsWith empty string, therefore we were excluding all
assets when ingore dependencies empty

Author: Baraujo25

.github/workflows/ci.yml

@@ -63,6 +63,7 @@ jobs:
           plugin-id: bzm-siebel
           changes: some changes
           token: ${{ secrets.GH_TOKEN }}
+          ignore-dependencies: bzm-repositories
         env:
           TEST_GITHUB_REPOSITORY: 'Blazemeter/CorrelationRecorder'
 

dist/index.js

@@ -1,4 +1,4 @@
-import { info, setFailed, setOutput } from '@actions/core'
+import { setFailed, setOutput } from '@actions/core'
 import { Arguments } from './args.js'
 import { GitService } from './git.js'
 import { GithubService } from './github.js'
@@ -21,6 +21,7 @@ export async function run(): Promise<void> {
     const releaseBuilder: ReleaseBuilder = new ReleaseBuilder(args, githubService.getAssets())
     const release: PluginVersion = await releaseBuilder.build()
     const version: string = githubService.getReleaseVersion()
+
     await applyRelease(
       await releaseBuilder.findFilePluginRepository(),
       release,
@@ -51,7 +52,6 @@ export async function run(): Promise<void> {
     if (plugin) {
       plugin.versions[releaseVersion] = release
       writeFileSync(releaseFile, JSON.stringify(plugins, null, 2), 'utf-8')
-      info(`Release Object: ${JSON.stringify(release)}`)
       return
     }
     throw Error(`The plugin id:"${pluginID}" was not found in ${releaseFile}`)

dist/index.js.map

@@ -38,15 +38,16 @@ export class ReleaseBuilder {
 
   private buildLibs(): Record<string, string> {
     const libs: Record<string, string> = {}
-    this.assets.forEach(asset => {
-      if (
-        !asset.name.startsWith(this.args.pluginArtifactName) &&
-        !this.args.ingoreDependencies.some(ignore => asset.name.startsWith(ignore))
-      ) {
+    this.assets
+      .filter(asset => !asset.name.startsWith(this.args.pluginArtifactName))
+      .filter(
+        asset =>
+          !this.args.ingoreDependencies.some(ignore => ignore && asset.name.startsWith(ignore))
+      )
+      .forEach(asset => {
         const { libKey, url } = this.buildLibKeyAndUrl(asset)
         libs[libKey] = url
-      }
-    })
+      })
     return libs
   }