fix: package.json sync

packages/action/src/ghosts/docs/index.ts

@@ -1,8 +1,7 @@
 import { Ghost } from '@/action/types/Ghost.js'
 import { updatePackageJsonList } from './updatePackageJsonList.js'
-import { updateReadme } from './updateReadme.js'
-import { listWorkflowFiles } from './utils/listWorkflowFiles.js'
 import { updateReadmeList } from './updateReadmeList.js'
+import { listWorkflowFiles } from './utils/listWorkflowFiles.js'
 
 export const docs: Ghost = async ({
   payload: {

packages/action/src/ghosts/docs/updatePackageJson.ts

@@ -1,10 +1,21 @@
 import { readFile, writeFile } from 'node:fs/promises'
+import path from 'node:path'
+import * as core from 'octoflare/action/core'
 import { ActionRepository } from '../../types/ActionRepository.js'
 import { isValidPackageJson } from './utils/isValidPackageJson.js'
-import * as core from 'octoflare/action/core'
+
+type Dict = {
+  [key: string]: string | Dict | undefined
+}
 
 export const updatePackageJson =
-  (repository: ActionRepository) =>
+  ({
+    repository,
+    repoLevelConfig
+  }: {
+    repository: ActionRepository
+    repoLevelConfig: Dict
+  }) =>
   async (packageJsonPath: string): Promise<boolean> => {
     const packageJsonStr = await readFile(packageJsonPath, 'utf-8')
     const json = JSON.parse(packageJsonStr)
@@ -15,41 +26,24 @@ export const updatePackageJson =
       return false
     }
 
-    const { topics, owner } = repository
+    const isRepoRoot =
+      path.relative(process.cwd(), packageJsonPath) === 'package.json'
 
     const publishConfig = packageJson.name?.startsWith('@')
       ? { publishConfig: { access: 'public' } }
       : {}
 
-    const license = repository.license?.spdx_id
-      ? { license: repository.license.spdx_id }
+    const description = isRepoRoot
+      ? { description: repository.description }
       : {}
 
-    const keywords = topics ?? []
-
-    const html = await fetch(repository.html_url).then((res) => res.text())
-    const repo_image =
-      html.match(/<meta property="og:image" content="(\S*)"\s*\/>/)?.[1] ?? ''
+    const keywords = isRepoRoot ? { keywords: repository.topics ?? [] } : {}
 
     const repoInfo = {
-      description: repository.description ?? '',
-      ...license,
-      bugs: `${repository.html_url}/issues`,
-      homepage: `${repository.html_url}#readme`,
-      author: {
-        name: owner.login,
-        email: 'intents.turrets0h@icloud.com',
-        url: owner.html_url,
-        image: owner.avatar_url
-      },
-      repository: {
-        type: 'git',
-        url: repository.clone_url,
-        image: repo_image
-      },
+      ...repoLevelConfig,
+      ...description,
       ...publishConfig,
-      keywords,
-      prettier: '@jill64/prettier-config'
+      ...keywords
     }
 
     const oldJson = JSON.stringify(packageJson, null, 2)

packages/action/src/ghosts/docs/updatePackageJsonList.ts

@@ -1,3 +1,4 @@
+import * as core from 'octoflare/action/core'
 import { ActionRepository } from '../../types/ActionRepository.js'
 import { findFile } from '../../utils/findFile.js'
 import { pushCommit } from '../../utils/pushCommit.js'
@@ -8,8 +9,46 @@ export const updatePackageJsonList = async ({
 }: {
   repository: ActionRepository
 }) => {
-  const files = await findFile('package.json')
-  const result = await Promise.all(files.map(updatePackageJson(repository)))
+  const { owner } = repository
+
+  const [files, html] = await Promise.all([
+    findFile('package.json'),
+    fetch(repository.html_url)
+      .then((res) => res.text())
+      .catch((e) => {
+        core.error(e)
+        return ''
+      })
+  ])
+
+  const repoImage =
+    html.match(/<meta property="og:image" content="(\S*)"\s*\/>/)?.[1] ?? ''
+
+  const license = repository.license?.spdx_id
+    ? { license: repository.license.spdx_id }
+    : {}
+
+  const repoLevelConfig = {
+    ...license,
+    bugs: `${repository.html_url}/issues`,
+    homepage: `${repository.html_url}#readme`,
+    author: {
+      name: owner.login,
+      email: owner.email ?? 'intents.turrets0h@icloud.com',
+      url: owner.html_url,
+      image: owner.avatar_url
+    },
+    repository: {
+      type: 'git',
+      url: repository.clone_url,
+      image: repoImage
+    },
+    prettier: '@jill64/prettier-config'
+  }
+
+  const result = await Promise.all(
+    files.map(updatePackageJson({ repository, repoLevelConfig }))
+  )
 
   if (result.includes(true)) {
     await pushCommit('chore: synchronize package.json')

packages/action/src/ghosts/docs/updateReadme.ts

@@ -36,8 +36,7 @@ export const updateReadme =
       workflowFiles,
       packageJson,
       readme: insertSection(readme),
-      repository,
-      isRepoRoot
+      repository
     })
 
     const license = isRepoRoot

packages/action/src/ghosts/docs/utils/syncHeader.ts

@@ -9,14 +9,12 @@ export const syncHeader = ({
   readme,
   packageJson,
   repository,
-  workflowFiles,
-  isRepoRoot
+  workflowFiles
 }: {
   workflowFiles: WorkflowFile[]
   packageJson: PackageJson | undefined | null
   readme: string
   repository: ActionRepository
-  isRepoRoot: boolean
 }) => {
   const escapedWebsiteUrl = repository.homepage
     ? encodeURIComponent(repository.homepage)