fix: compare hiddenAppIds as a set for editor dirty state

src/features/editor/hooks/useAppControls.tsx

@@ -3,6 +3,7 @@ import { useSettingsMutation } from '@settings/hooks/useSettingsMutation'
 import type { SettingsUpdateDto } from '@settings/lib/settings-actions.dto'
 import { useSettingsStore } from '@settings/providers/settings.provider'
 import { useShallow } from 'zustand/shallow'
+import { areArraysEqualAsSets } from '@/utils/array'
 import { areObjKeysEqual } from '@/utils/objects'
 
 export const useAppControls = () => {
@@ -35,8 +36,8 @@ export const useAppControls = () => {
   )
   const initialSettings = useSettingsStore((s) => s.initialSettings)
   const orderChanged = JSON.stringify(actions.order) !== JSON.stringify(initialSettings.actions?.order)
-  const hiddenAppIdsChanged =
-    JSON.stringify(actions.hiddenAppIds) !== JSON.stringify(initialSettings.actions?.hiddenAppIds)
+  // `hiddenAppIds` is an unordered deny-list, so compare membership rather than sequence.
+  const hiddenAppIdsChanged = !areArraysEqualAsSets(actions.hiddenAppIds, initialSettings.actions?.hiddenAppIds)
   const show =
     !areObjKeysEqual(settings, initialSettings, ['content', 'heading', 'subheading', 'backgroundColor']) ||
     !areObjKeysEqual(actions, initialSettings.actions, ['tasks', 'invoices', 'contracts', 'forms']) ||

src/utils/array.ts

@@ -30,6 +30,19 @@ export const mapWithConcurrency = async <T, R>(
   return results
 }
 
+// Membership equality for arrays whose order carries no meaning. A missing array counts as empty.
+export const areArraysEqualAsSets = <T>(a: readonly T[] = [], b: readonly T[] = []): boolean => {
+  const setA = new Set(a)
+  const setB = new Set(b)
+  if (setA.size !== setB.size) return false
+
+  for (const item of setA) {
+    if (!setB.has(item)) return false
+  }
+
+  return true
+}
+
 export const getArraySymmetricDifference = <T>(a: T[], b: T[]): T[] => {
   const setA = new Set(a)
   const setB = new Set(b)