@meith/plugin-kit 0.6.0 → 0.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@meith/plugin-kit",
-  "version": "0.6.0",
+  "version": "0.8.0",
   "description": "The SDK for writing a Meith plugin: typed manifests, hooks, routes, pages and migrations.",
   "license": "LGPL-3.0-or-later",
   "repository": {
@@ -20,7 +20,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@meith/theme-kit": "^0.6.0"
+    "@meith/theme-kit": "^0.8.0"
   },
   "peerDependencies": {
     "react": "^19.2.0"

package/src/host.ts

@@ -53,7 +53,10 @@ const DEFAULT_PRIORITY = 100
 
 export class PluginHost {
   readonly #entries = new Map<HookName, Entry[]>()
-  readonly #contributions = new Map<PluginRegion, { pluginKey: string; priority: number; contribution: PluginContribution }[]>()
+  readonly #contributions = new Map<
+    PluginRegion,
+    { pluginKey: string; priority: number; contribution: PluginContribution }[]
+  >()
   readonly #stats = new Map<string, Stats>()
   readonly #logger: HostLogger
   readonly #failureThreshold: number
@@ -81,7 +84,9 @@ export class PluginHost {
       for (const [name, registration] of Object.entries(plugin.hooks ?? {})) {
         const hook = name as HookName
         const handler = (
-          typeof registration === 'function' ? registration : (registration as HookRegistration<HookName>).handler
+          typeof registration === 'function'
+            ? registration
+            : (registration as HookRegistration<HookName>).handler
         ) as StoredHandler
         const priority =
           typeof registration === 'function'
@@ -104,8 +109,12 @@ export class PluginHost {
       }
     }
 
-    const byPriorityThenKey = <T extends { priority: number; pluginKey: string }>(a: T, b: T): number =>
-      a.priority - b.priority || (a.pluginKey < b.pluginKey ? -1 : a.pluginKey > b.pluginKey ? 1 : 0)
+    const byPriorityThenKey = <T extends { priority: number; pluginKey: string }>(
+      a: T,
+      b: T,
+    ): number =>
+      a.priority - b.priority ||
+      (a.pluginKey < b.pluginKey ? -1 : a.pluginKey > b.pluginKey ? 1 : 0)
 
     for (const list of this.#entries.values()) list.sort(byPriorityThenKey)
     for (const list of this.#contributions.values()) list.sort(byPriorityThenKey)
@@ -130,7 +139,11 @@ export class PluginHost {
     return current
   }
 
-  async emit<K extends HookName>(name: K, value: HookValue<K>, context: HookContext<K>): Promise<void> {
+  async emit<K extends HookName>(
+    name: K,
+    value: HookValue<K>,
+    context: HookContext<K>,
+  ): Promise<void> {
     const entries = this.#entries.get(name)
     if (entries === undefined) return
 
@@ -219,7 +232,7 @@ export class PluginHost {
 
   #isEnabled(pluginKey: string): boolean {
     const stats = this.#stats.get(pluginKey)
-    return stats !== undefined && stats.enabled && !stats.operatorDisabled
+    return stats?.enabled === true && !stats.operatorDisabled
   }
 
   async #call(

package/src/index.ts

@@ -1,13 +1,20 @@
 export {
-  HOOKS,
   HOOK_NAMES,
-  hookKind,
-  isHookName,
+  HOOKS,
   type HookKind,
   type HookName,
   type HookSpec,
+  hookKind,
+  isHookName,
 } from './hooks'
-
+export {
+  emptyHost,
+  type HostLogger,
+  isFilter,
+  type PluginHealth,
+  PluginHost,
+  type PluginHostOptions,
+} from './host'
 export type {
   DraftPayload,
   ForumRef,
@@ -22,23 +29,14 @@ export type {
   ValidationMessages,
   ViewerRef,
 } from './payloads'
-
 export {
   DEFAULT_ROUTE_BODY_BYTES,
-  MAX_ROUTE_BODY_BYTES,
   definePlugin,
-  pluginAdminPath,
-  pluginAdminRoutePath,
-  pluginNotificationKindId,
-  pluginPagePath,
-  pluginRoutePath,
-  pluginSettingKey,
-  pluginTablePrefix,
-  pluginTaskId,
   type EventHandler,
   type FilterHandler,
   type HookHandler,
   type HookRegistration,
+  MAX_ROUTE_BODY_BYTES,
   type PluginAdminPage,
   type PluginAdminPageContext,
   type PluginBoardPage,
@@ -59,47 +57,29 @@ export {
   type PluginSettingType,
   type PluginTask,
   type PluginViewer,
+  pluginAdminPath,
+  pluginAdminRoutePath,
+  pluginNotificationKindId,
+  pluginPagePath,
+  pluginRoutePath,
+  pluginSettingKey,
+  pluginTablePrefix,
+  pluginTaskId,
 } from './plugin'
-
 export {
-  operatorDisabledPlugins,
-  parsePluginSetting,
-  pluginEnabledKey,
-  pluginSettingType,
-  resolvePluginSettingDetails,
-  resolvePluginSettings,
-  serialisePluginSetting,
-  type PluginEnvReader,
-  type PluginSettingSource,
-  type PluginSettingValue,
-  type ResolvedPluginSetting,
-} from './settings'
-
+  createRouteRateLimiter,
+  type RateLimitVerdict,
+  type RouteRateLimiter,
+} from './rate-limit'
 export {
-  PLUGIN_REGIONS,
-  REGION_NAMES,
   isPluginRegion,
+  PLUGIN_REGIONS,
   type PluginRegion,
   type PluginRegionContext,
+  REGION_NAMES,
   type RegionSpec,
 } from './regions'
-
-export {
-  PluginHost,
-  emptyHost,
-  isFilter,
-  type HostLogger,
-  type PluginHealth,
-  type PluginHostOptions,
-} from './host'
-
 export {
-  pluginNotificationKindSpecs,
-  pluginNotify,
-  unavailablePluginData,
-  unavailablePluginGrants,
-  unavailablePluginNotify,
-  unavailablePluginUsers,
   type PluginData,
   type PluginGrantRow,
   type PluginGrants,
@@ -109,10 +89,23 @@ export {
   type PluginNotifyKindInput,
   type PluginUserRef,
   type PluginUsers,
+  pluginNotificationKindSpecs,
+  pluginNotify,
+  unavailablePluginData,
+  unavailablePluginGrants,
+  unavailablePluginNotify,
+  unavailablePluginUsers,
 } from './runtime'
-
 export {
-  createRouteRateLimiter,
-  type RateLimitVerdict,
-  type RouteRateLimiter,
-} from './rate-limit'
+  operatorDisabledPlugins,
+  type PluginEnvReader,
+  type PluginSettingSource,
+  type PluginSettingValue,
+  parsePluginSetting,
+  pluginEnabledKey,
+  pluginSettingType,
+  type ResolvedPluginSetting,
+  resolvePluginSettingDetails,
+  resolvePluginSettings,
+  serialisePluginSetting,
+} from './settings'

package/src/payloads.ts

@@ -40,15 +40,15 @@
 
 import type {
   AnnouncementModel,
+  AuthPageModel,
   BoardIndexModel,
   BoardStatsModel,
-  AuthPageModel,
   CategoryBlockModel,
   DiscoveryViewModel,
   ErrorNoticeModel,
   FooterModel,
-  ForumJumpModel,
   ForumDisplayModel,
+  ForumJumpModel,
   ForumRowSlotModel,
   HeaderModel,
   LatestPostsModel,
@@ -200,7 +200,10 @@ export interface HookSignatures {
   'thread.create.validate': { value: ValidationMessages; context: { draft: DraftPayload } }
   'thread.create.before': { value: DraftPayload; context: ViewerRef }
   'thread.created': { value: ThreadRef & { authorId: number; subject: string }; context: ViewerRef }
-  'post.create.validate': { value: ValidationMessages; context: { draft: DraftPayload; threadId: number } }
+  'post.create.validate': {
+    value: ValidationMessages
+    context: { draft: DraftPayload; threadId: number }
+  }
   'post.create.before': { value: DraftPayload; context: ViewerRef & { threadId: number } }
   'post.created': { value: PostRef & { authorId: number }; context: ViewerRef }
   'post.edit.before': {
@@ -216,11 +219,19 @@ export interface HookSignatures {
     context: ModerationRef
   }
   'thread.merged': {
-    value: { readonly keptThreadId: number; readonly mergedThreadId: number; readonly postCount: number }
+    value: {
+      readonly keptThreadId: number
+      readonly mergedThreadId: number
+      readonly postCount: number
+    }
     context: ModerationRef
   }
   'thread.split': {
-    value: { readonly sourceThreadId: number; readonly newThreadId: number; readonly postCount: number }
+    value: {
+      readonly sourceThreadId: number
+      readonly newThreadId: number
+      readonly postCount: number
+    }
     context: ModerationRef
   }
   'thread.locked': { value: ThreadRef & { isLocked: boolean }; context: ModerationRef }
@@ -241,7 +252,10 @@ export interface HookSignatures {
   }
   'attachment.deleted': { value: { readonly attachmentId: number }; context: ViewerRef }
   'poll.created': { value: ThreadRef & { pollId: number; optionCount: number }; context: ViewerRef }
-  'poll.voted': { value: { readonly pollId: number; readonly optionId: number }; context: ViewerRef }
+  'poll.voted': {
+    value: { readonly pollId: number; readonly optionId: number }
+    context: ViewerRef
+  }
   'rating.recorded': {
     value: { readonly threadId: number; readonly rating: number; readonly average: number }
     context: ViewerRef
@@ -282,7 +296,10 @@ export interface HookSignatures {
     }
     context: ModerationRef
   }
-  'warning.revoked': { value: { readonly warningId: number; readonly userId: number }; context: ModerationRef }
+  'warning.revoked': {
+    value: { readonly warningId: number; readonly userId: number }
+    context: ModerationRef
+  }
   'moderation.logged': {
     value: { readonly action: string; readonly targetId: number | null }
     context: ModerationRef
@@ -293,7 +310,10 @@ export interface HookSignatures {
     value: ValidationMessages
     context: { readonly username: string; readonly email: string; readonly ipPrefix: string | null }
   }
-  'user.registered': { value: UserRef & { username: string; requiresActivation: boolean }; context: RequestRef }
+  'user.registered': {
+    value: UserRef & { username: string; requiresActivation: boolean }
+    context: RequestRef
+  }
   'user.activated': { value: UserRef; context: RequestRef }
   'user.login.attempted': {
     value: {
@@ -316,7 +336,10 @@ export interface HookSignatures {
     context: RequestRef
   }
   'user.profile.updated': { value: UserRef & { fields: readonly string[] }; context: RequestRef }
-  'user.merged': { value: { readonly keptUserId: number; readonly mergedUserId: number }; context: RequestRef }
+  'user.merged': {
+    value: { readonly keptUserId: number; readonly mergedUserId: number }
+    context: RequestRef
+  }
   'user.deleted': { value: UserRef & { reason: 'pruned' | 'deleted' }; context: RequestRef }
 
   /* ---- Mail, notifications, messages ---- */
@@ -329,7 +352,10 @@ export interface HookSignatures {
     } | null
     context: RequestRef
   }
-  'notification.created': { value: { readonly notificationId: number; readonly userId: number }; context: RequestRef }
+  'notification.created': {
+    value: { readonly notificationId: number; readonly userId: number }
+    context: RequestRef
+  }
   'mail.send.before': {
     value: {
       readonly to: string
@@ -349,7 +375,10 @@ export interface HookSignatures {
     } | null
     context: RequestRef
   }
-  'pm.sent': { value: { readonly messageId: number; readonly recipientIds: readonly number[] }; context: RequestRef }
+  'pm.sent': {
+    value: { readonly messageId: number; readonly recipientIds: readonly number[] }
+    context: RequestRef
+  }
   'subscription.changed': {
     value: {
       readonly userId: number
@@ -399,7 +428,10 @@ export interface HookSignatures {
     value: readonly { readonly label: string; readonly href: string }[]
     context: ViewerRef
   }
-  'settings.saved': { value: { readonly keys: readonly string[] }; context: { readonly adminId: number } }
+  'settings.saved': {
+    value: { readonly keys: readonly string[] }
+    context: { readonly adminId: number }
+  }
   'task.run.before': { value: { readonly taskId: string }; context: Record<string, never> }
   'task.run.after': {
     value: { readonly taskId: string; readonly ok: boolean; readonly durationMs: number }

package/src/plugin.ts

@@ -1,6 +1,8 @@
 import type { ReactNode } from 'react'
 
-import { isHookName, type HOOKS, type HookName } from './hooks'
+import type { Translator } from '@meith/theme-kit'
+
+import { type HOOKS, type HookName, isHookName } from './hooks'
 import type { HookContext, HookValue } from './payloads'
 import { isPluginRegion, type PluginRegion, type PluginRegionContext } from './regions'
 import type { PluginData, PluginGrants, PluginNotify, PluginUsers } from './runtime'
@@ -33,9 +35,17 @@ export type PluginSettingType = 'string' | 'secret' | 'number' | 'boolean' | 'se
 export interface PluginSetting {
   readonly key: string
   readonly label: string
+  readonly labelKey?: string | undefined
   readonly description?: string | undefined
+  readonly descriptionKey?: string | undefined
   readonly type?: PluginSettingType | undefined
-  readonly options?: readonly { readonly value: string; readonly label: string }[] | undefined
+  readonly options?:
+    | readonly {
+        readonly value: string
+        readonly label: string
+        readonly labelKey?: string | undefined
+      }[]
+    | undefined
   readonly env?: string | undefined
   readonly required?: boolean | undefined
   readonly default: string | number | boolean
@@ -56,6 +66,8 @@ export interface PluginTask {
 export interface PluginAdminPage {
   readonly path: string
   readonly title: string
+  readonly titleKey?: string | undefined
+  readonly titleArgs?: Parameters<Translator['t']>[1] | undefined
   readonly render: (context: PluginAdminPageContext) => ReactNode | Promise<ReactNode>
 }
 
@@ -118,15 +130,21 @@ export interface PluginPageContext extends PluginRuntimeContext {
   readonly path: string
   readonly query: Readonly<Record<string, string>>
   readonly boardUrl: string
+  readonly locale: string
+  readonly t: Translator
 }
 
 export interface PluginAdminPageContext extends PluginRuntimeContext {
   readonly query: Readonly<Record<string, string>>
+  readonly locale: string
+  readonly t: Translator
 }
 
 export interface PluginBoardPage {
   readonly path: string
   readonly title: string
+  readonly titleKey?: string | undefined
+  readonly titleArgs?: Parameters<Translator['t']>[1] | undefined
   readonly access: PluginPageAccess
   readonly render: (context: PluginPageContext) => ReactNode | Promise<ReactNode>
 }
@@ -147,15 +165,20 @@ export interface PluginRuntimeContext {
 export interface PluginNotificationKind {
   readonly key: string
   readonly title: string
+  readonly titleKey?: string | undefined
   readonly description: string
+  readonly descriptionKey?: string | undefined
   readonly emailByDefault?: boolean | undefined
 }
 
 export interface PluginDefinition {
   readonly key: string
   readonly name: string
+  readonly nameKey?: string | undefined
   readonly version: string
   readonly description?: string | undefined
+  readonly descriptionKey?: string | undefined
+  readonly descriptionArgs?: Parameters<Translator['t']>[1] | undefined
   readonly apiVersion?: string | undefined
 
   readonly dependsOn?: readonly string[] | undefined
@@ -201,8 +224,7 @@ const MIGRATION_FORMS: readonly {
   { pattern: /^alter\s+table(?:\s+if\s+exists)?(?:\s+only)?\s+(\S+)/i, describe: 'alter table' },
   { pattern: /^drop\s+table(?:\s+if\s+exists)?\s+(\S+)/i, describe: 'drop table' },
   {
-    pattern:
-      /^create\s+(?:unique\s+)?index(?:\s+if\s+not\s+exists)?\s+(\S+)\s+on\s+(\S+)/i,
+    pattern: /^create\s+(?:unique\s+)?index(?:\s+if\s+not\s+exists)?\s+(\S+)\s+on\s+(\S+)/i,
     describe: 'create index',
   },
   { pattern: /^drop\s+index(?:\s+if\s+exists)?\s+(\S+)/i, describe: 'drop index' },
@@ -216,7 +238,10 @@ const MIGRATION_FORMS: readonly {
 ]
 
 function bareIdentifier(raw: string): string {
-  let name = raw.replace(/[(;,].*$/s, '').replace(/"/g, '').toLowerCase()
+  let name = raw
+    .replace(/[(;,].*$/s, '')
+    .replace(/"/g, '')
+    .toLowerCase()
   if (name.startsWith('public.')) name = name.slice('public.'.length)
   const dot = name.indexOf('.')
   return dot === -1 ? name : name.slice(0, dot)
@@ -272,7 +297,9 @@ export function definePlugin(plugin: PluginDefinition): PluginDefinition {
   }
   if (plugin.name.trim() === '') throw new Error(`${where}: name must not be empty.`)
   if (!/^\d+\.\d+\.\d+$/.test(plugin.version)) {
-    throw new Error(`${where}: version must be semver (major.minor.patch), got "${plugin.version}".`)
+    throw new Error(
+      `${where}: version must be semver (major.minor.patch), got "${plugin.version}".`,
+    )
   }
 
   for (const dependency of plugin.dependsOn ?? []) {
@@ -292,13 +319,19 @@ export function definePlugin(plugin: PluginDefinition): PluginDefinition {
       )
     }
     const handler =
-      typeof registration === 'function' ? registration : (registration as HookRegistration<HookName>)?.handler
+      typeof registration === 'function'
+        ? registration
+        : (registration as HookRegistration<HookName>)?.handler
     if (typeof handler !== 'function') {
       throw new Error(`${where}: hook "${name}" must be a function or { handler, priority }.`)
     }
   }
 
-  assertUnique(where, 'setting', (plugin.settings ?? []).map((setting) => setting.key))
+  assertUnique(
+    where,
+    'setting',
+    (plugin.settings ?? []).map((setting) => setting.key),
+  )
   for (const setting of plugin.settings ?? []) {
     if (!SETTING_KEY_PATTERN.test(setting.key)) {
       throw new Error(
@@ -352,7 +385,11 @@ export function definePlugin(plugin: PluginDefinition): PluginDefinition {
     }
   }
 
-  assertUnique(where, 'migration', (plugin.migrations ?? []).map((migration) => migration.id))
+  assertUnique(
+    where,
+    'migration',
+    (plugin.migrations ?? []).map((migration) => migration.id),
+  )
   const migrationIds = (plugin.migrations ?? []).map((migration) => migration.id)
   for (const id of migrationIds) {
     if (!MIGRATION_ID_PATTERN.test(id)) {
@@ -383,10 +420,16 @@ export function definePlugin(plugin: PluginDefinition): PluginDefinition {
     }
   }
 
-  assertUnique(where, 'task', (plugin.tasks ?? []).map((task) => task.id))
+  assertUnique(
+    where,
+    'task',
+    (plugin.tasks ?? []).map((task) => task.id),
+  )
   for (const task of plugin.tasks ?? []) {
     if (!TASK_ID_PATTERN.test(task.id)) {
-      throw new Error(`${where}: task id "${task.id}" must be lower-case letters, digits and hyphens.`)
+      throw new Error(
+        `${where}: task id "${task.id}" must be lower-case letters, digits and hyphens.`,
+      )
     }
     if (!Number.isInteger(task.intervalSeconds) || task.intervalSeconds < 60) {
       throw new Error(
@@ -397,7 +440,11 @@ export function definePlugin(plugin: PluginDefinition): PluginDefinition {
     }
   }
 
-  assertUnique(where, 'admin page', (plugin.adminPages ?? []).map((page) => page.path))
+  assertUnique(
+    where,
+    'admin page',
+    (plugin.adminPages ?? []).map((page) => page.path),
+  )
   for (const page of plugin.adminPages ?? []) {
     if (!PAGE_PATH_PATTERN.test(page.path)) {
       throw new Error(
@@ -470,7 +517,11 @@ export function definePlugin(plugin: PluginDefinition): PluginDefinition {
     }
   }
 
-  assertUnique(where, 'page', (plugin.pages ?? []).map((page) => page.path))
+  assertUnique(
+    where,
+    'page',
+    (plugin.pages ?? []).map((page) => page.path),
+  )
   for (const page of plugin.pages ?? []) {
     if (page.path !== '' && !PAGE_PATH_PATTERN.test(page.path)) {
       throw new Error(

package/src/runtime.ts

@@ -1,4 +1,3 @@
-
 export interface PluginGrantRow {
   readonly groupKey: string
   readonly expiresAt: Date
@@ -77,8 +76,12 @@ export interface PluginNotify {
   send(input: {
     readonly userId: number
     readonly kind: string
-    readonly subject: string
+    readonly subject?: string | undefined
+    readonly subjectKey?: string | undefined
+    readonly subjectArgs?: Readonly<Record<string, string | number>> | undefined
     readonly body?: string | undefined
+    readonly bodyKey?: string | undefined
+    readonly bodyArgs?: Readonly<Record<string, string | number>> | undefined
     readonly href?: string | undefined
     readonly dedupeKey?: string | undefined
   }): Promise<void>
@@ -97,14 +100,18 @@ const MAX_NOTIFY_BODY = 2_000
 export interface PluginNotifyKindInput {
   readonly key: string
   readonly title: string
+  readonly titleKey?: string | undefined
   readonly description: string
+  readonly descriptionKey?: string | undefined
   readonly emailByDefault?: boolean | undefined
 }
 
 export interface PluginNotificationKindSpec {
   readonly id: string
   readonly title: string
+  readonly titleKey?: string
   readonly description: string
+  readonly descriptionKey?: string
   readonly audience: 'member'
   readonly emailByDefault: boolean
   readonly emailConfigurable: true
@@ -127,7 +134,9 @@ export function pluginNotificationKindSpecs(
   return kinds.map((kind) => ({
     id: `plugin.${pluginKey}.${kind.key}`,
     title: kind.title,
+    ...(kind.titleKey === undefined ? {} : { titleKey: kind.titleKey }),
     description: kind.description,
+    ...(kind.descriptionKey === undefined ? {} : { descriptionKey: kind.descriptionKey }),
     audience: 'member',
     emailByDefault: kind.emailByDefault ?? true,
     emailConfigurable: true,
@@ -154,14 +163,36 @@ export function pluginNotify(
         throw new Error(`plugin "${pluginKey}": a notification needs a real user id.`)
       }
 
-      const subject = input.subject.trim()
-      if (subject === '' || subject.length > MAX_NOTIFY_SUBJECT) {
+      const hasSubject = input.subject !== undefined
+      const hasSubjectKey = input.subjectKey !== undefined
+      if (hasSubject === hasSubjectKey) {
+        throw new Error(
+          `plugin "${pluginKey}": a notification needs exactly one of subject or subjectKey.`,
+        )
+      }
+
+      const subject = input.subject?.trim() ?? ''
+      if (hasSubject && (subject === '' || subject.length > MAX_NOTIFY_SUBJECT)) {
         throw new Error(
           `plugin "${pluginKey}": a notification subject is 1 to ${MAX_NOTIFY_SUBJECT} characters.`,
         )
       }
 
+      const subjectKey = input.subjectKey?.trim() ?? ''
+      if (hasSubjectKey && subjectKey === '') {
+        throw new Error(`plugin "${pluginKey}": a notification subjectKey cannot be empty.`)
+      }
+
       const body = (input.body ?? '').trim()
+      const hasBody = input.body !== undefined
+      const hasBodyKey = input.bodyKey !== undefined
+      if (hasBody && hasBodyKey) {
+        throw new Error(`plugin "${pluginKey}": a notification cannot have both body and bodyKey.`)
+      }
+      const bodyKey = input.bodyKey?.trim() ?? ''
+      if (hasBodyKey && bodyKey === '') {
+        throw new Error(`plugin "${pluginKey}": a notification bodyKey cannot be empty.`)
+      }
       if (body.length > MAX_NOTIFY_BODY) {
         throw new Error(
           `plugin "${pluginKey}": a notification body caps at ${MAX_NOTIFY_BODY} characters — ` +
@@ -169,7 +200,10 @@ export function pluginNotify(
         )
       }
 
-      if (input.href !== undefined && (!input.href.startsWith('/') || input.href.startsWith('//'))) {
+      if (
+        input.href !== undefined &&
+        (!input.href.startsWith('/') || input.href.startsWith('//'))
+      ) {
         throw new Error(
           `plugin "${pluginKey}": a notification links within the board — the href must ` +
             'start with a single "/".',
@@ -179,7 +213,28 @@ export function pluginNotify(
       await backend.raise({
         userId: input.userId,
         kind: `plugin.${pluginKey}.${input.kind}`,
-        data: body === '' ? { subject } : { subject, body },
+        data: {
+          ...(hasSubject
+            ? { subject }
+            : {
+                subjectKey,
+                ...(input.subjectArgs === undefined
+                  ? {}
+                  : { subjectArgs: JSON.stringify(input.subjectArgs) }),
+              }),
+          ...(hasBody
+            ? body === ''
+              ? {}
+              : { body }
+            : !hasBodyKey
+              ? {}
+              : {
+                  bodyKey,
+                  ...(input.bodyArgs === undefined
+                    ? {}
+                    : { bodyArgs: JSON.stringify(input.bodyArgs) }),
+                }),
+        },
         href: input.href ?? null,
         dedupeKey: input.dedupeKey ?? null,
       })

package/src/settings.ts

@@ -31,10 +31,7 @@ export function serialisePluginSetting(value: PluginSettingValue): string {
   return String(value)
 }
 
-export function parsePluginSetting(
-  setting: PluginSetting,
-  raw: string,
-): PluginSettingValue | null {
+export function parsePluginSetting(setting: PluginSetting, raw: string): PluginSettingValue | null {
   if (typeof setting.default === 'boolean') {
     if (raw === '1' || raw === 'true') return true
     if (raw === '0' || raw === 'false') return false
@@ -114,9 +111,7 @@ export function resolvePluginSettingDetails(
   return (plugin.settings ?? []).map((setting) => resolveOne(plugin, setting, overrides, env))
 }
 
-export function operatorDisabledPlugins(
-  overrides: ReadonlyMap<string, string>,
-): readonly string[] {
+export function operatorDisabledPlugins(overrides: ReadonlyMap<string, string>): readonly string[] {
   const disabled: string[] = []
 
   for (const [key, value] of overrides) {