feat(changesets-renovate): ignore changes in workspace's root package.json (#1232)

Author: Tom Lienard

.changeset/happy-items-look.md

@@ -0,0 +1,5 @@
+---
+'@scaleway/changesets-renovate': minor
+---
+
+Ignore changes in workspace's root package.json

packages/changesets-renovate/src/__tests__/generate-changeset.ts

@@ -118,4 +118,37 @@ describe('generate changeset file', () => {
     })
     expect(push).toHaveBeenCalledWith(['--force'])
   })
+
+  it('should ignore workspace package.json', async () => {
+    const file = 'package.json'
+
+    // @ts-expect-error we mock at the top
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
+    simpleGit.mockReturnValue({
+      branch: () => ({
+        current: 'renovate/test',
+      }),
+      diffSummary: () => ({
+        files: [
+          {
+            file,
+          },
+        ],
+      }),
+      show: () => `
++ "package": "version"
++ "package2": "version2"
+`,
+    })
+
+    fs.readFile = jest
+      .fn()
+      .mockResolvedValue(`{"name":"packageName","workspaces":[]}`)
+    fs.writeFile = jest.fn()
+
+    await run()
+
+    expect(fs.readFile).toHaveBeenCalledWith(file, 'utf8')
+    expect(console.log).toHaveBeenCalledWith('No packages modified, skipping')
+  })
 })

packages/changesets-renovate/src/index.ts

@@ -4,15 +4,22 @@ import fs from 'node:fs/promises'
 import { simpleGit } from 'simple-git'
 
 async function getPackagesNames(files: string[]): Promise<string[]> {
+  const packages: string[] = []
+
   const promises = files.map(async file => {
     const data = JSON.parse(await fs.readFile(file, 'utf8')) as {
       name: string
+      workspaces?: string[]
     }
 
-    return data.name
+    if (!data.workspaces) {
+      packages.push(data.name)
+    }
   })
 
-  return Promise.all(promises)
+  await Promise.all(promises)
+
+  return packages
 }
 
 async function createChangeset(
@@ -84,6 +91,13 @@ export async function run() {
   }
 
   const packageNames = await getPackagesNames(files)
+
+  if (packageNames.length === 0) {
+    console.log('No packages modified, skipping')
+
+    return
+  }
+
   const shortHash = await simpleGit().revparse(['--short', 'HEAD'])
   const fileName = `.changeset/renovate-${shortHash.trim()}.md`
   const packageBumps = await getBumps(files)