eslint.config.js

// ESLint flat config for Switchboard (ESLint 9.x).
//
// Goals:
//   1. Catch dumb "undefined variable" mistakes in renderer code
//      (no-undef). Two recent regressions in public/sidebar.js
//      (subagentIndex undefined; project.projectPath out of scope)
//      would have been caught instantly by this rule.
//   2. Warn about unused vars without blocking the build.
//   3. Keep the existing 24+ node:test suite green.
//
// The renderer (public/*.js) loads as a set of classic <script> tags
// in index.html: every top-level `function`, `const`, `let` becomes a
// global accessible to sibling files. ESLint treats each file in
// isolation, so we declare the cross-file globals (defined in app.js
// per the comment at the top of sidebar.js) as readonly browser globals.
//
// Main-process files (CommonJS) get a separate block with node globals.

const globals = require('globals');
const ACTIVITY_CLASS_MESSAGE = 'Only public/session-activity-dom.js may write .cli-busy/.needs-attention/.response-ready/.has-busy-agents — see .ai/contexts/session-state.md';

// Cross-file renderer globals: vars defined in one file and consumed by
// another. The list mirrors the dependency comment at the top of
// public/sidebar.js (and equivalents). Kept readonly so reassignment
// from another file gets flagged.
const rendererCrossFileGlobals = {
  runAttachedJobVerb: 'readonly',
  writeSessionExitBanner: 'readonly',
  updateTerminalHeader: 'readonly',
  detachSession: 'readonly',
  planSessionDetach: 'readonly',
  describeSessionStopPlan: 'readonly',
  localActivityRow: 'readonly',
  selectBackgroundActivity: 'readonly',
  backgroundAttachFacts: 'readonly',
  observeSessionProjection: 'readonly',
  agentsCliAvailable: 'readonly',
  agentsCliEntryCount: 'readonly',
  agentsSnapshotRevision: 'readonly',
  remoteHostLabel: 'readonly',
  setRemoteHostLabels: 'readonly',
  writtenActivitySnapshot: 'readonly',
  noteWrittenActivity: 'readonly',
  markWrittenActivitySeen: 'readonly',
  rekeyWrittenActivity: 'readonly',
  markTerminalOutputSeen: 'readonly',
  markTranscriptOutputSeen: 'readonly',
  setUnseenWrittenActivity: 'readonly',
  clearHeadUnread: 'readonly',
  forgetObservedTurnEnd: 'readonly',
  registerModal: 'readonly',
  hasOpenModal: 'readonly',
  enqueueModalQuestion: 'readonly',
  hasActiveModalQuestion: 'readonly',
  TOOL_BAR: 'readonly',
  filePanelState: 'readonly',
  currentPanelSessionId: 'readonly',
  toggleChangesTab: 'readonly',
  togglePanelTerminal: 'readonly',
  toggleTouchedTab: 'readonly',
  initToolBar: 'readonly',
  syncToolBar: 'readonly',
  toolBarOwner: 'readonly',
  handleToolShortcut: 'readonly',
  handleFocusShortcut: 'readonly',
  leaveToolTab: 'readonly',
  showPendingDiff: 'readonly',
  // DOM element handles (defined in app.js)
  sidebarContent: 'readonly',
  statsContent: 'readonly',
  memoryContent: 'readonly',
  placeholder: 'readonly',
  terminalsEl: 'readonly',
  terminalArea: 'readonly',
  terminalHeader: 'readonly',
  terminalHeaderName: 'readonly',
  terminalHeaderId: 'readonly',
  terminalHeaderStatus: 'readonly',
  terminalHeaderShell: 'readonly',
  terminalStopBtn: 'readonly',
  archiveToggle: 'readonly',
  starToggle: 'readonly',
  runningToggle: 'readonly',
  todayToggle: 'readonly',
  searchInput: 'readonly',
  searchBar: 'readonly',
  sessionFilters: 'readonly',
  loadingStatus: 'readonly',
  statusBarInfo: 'readonly',
  statusBarActivity: 'readonly',
  memoryViewer: 'readonly',
  memoryPanel: 'readonly',
  workFilesContent: 'readonly',
  workFilesViewer: 'readonly',
  workFilesPanel: 'readonly',
  statsViewer: 'readonly',
  statsViewerBody: 'readonly',
  settingsViewer: 'readonly',
  globalSettingsBtn: 'readonly',
  addProjectBtn: 'readonly',
  resortBtn: 'readonly',
  jsonlViewer: 'readonly',
  jsonlViewerTitle: 'readonly',
  jsonlViewerSessionId: 'readonly',
  jsonlViewerBody: 'readonly',
  gridViewer: 'readonly',
  gridViewerCount: 'readonly',

  // Sidebar/session state (mutable in app.js but readonly from sibling files)
  openSessions: 'readonly',
  skippedWorkingSetEntries: 'readonly',
  activeSessionId: 'writable', // reassigned in setActiveSession
  setActiveSession: 'readonly',
  activePtyIds: 'writable',
  pendingSessions: 'readonly',
  dormantWorkingSet: 'readonly',
  dismissDormantSession: 'readonly',
  isDormantSession: 'readonly',
  indexedSessionById: 'readonly',
  indexedRowOf: 'readonly',
  revealDormantSessions: 'readonly',
  sessionMap: 'writable',
  lastActivityTime: 'readonly',
  sortedOrder: 'writable',
  searchMatchIds: 'writable',
  searchMatchProjectPaths: 'writable',
  showArchived: 'writable',
  showStarredOnly: 'writable',
  showRunningOnly: 'writable',
  showTodayOnly: 'writable',
  visibleSessionCount: 'writable',
  sessionMaxAgeDays: 'writable',
  attentionSessions: 'readonly',
  responseReadySessions: 'readonly',
  sessionBusyState: 'readonly',
  cachedProjects: 'writable',
  cachedAllProjects: 'writable',
  gridCards: 'writable',
  gridViewActive: 'writable',
  activeTab: 'writable',

  // Functions from app.js / dialogs.js / utils.js / icons.js / terminal-manager.js / etc.
  ICONS: 'readonly',
  SCROLLBACK_SINGLE: 'readonly',
  SCROLLBACK_GRID: 'readonly',
  lruTouch: 'readonly',
  suspendTerminalWebgl: 'readonly',
  destroyGridCard: 'readonly',
  restoreTerminalWebgl: 'readonly',
  cleanDisplayName: 'readonly',
  formatDate: 'readonly',
  escapeHtml: 'readonly',
  shellEscape: 'readonly',
  encodeProjectPath: 'readonly',
  // Both defined in public/utils.js (upstream a7698f4 / 94a7ec0). Upstream has
  // no ESLint config, so new cross-file renderer symbols have to be declared
  // here when syncing or no-undef fires across five consumers.
  shortProjectPath: 'readonly',
  // public/choice-dialog.js and public/worktree-nesting.js
  showChoiceDialog: 'readonly',
  appConfirm: 'readonly',
  appAlert: 'readonly',
  appChoose: 'readonly',
  showAppNotice: 'readonly',
  clearAppNotice: 'readonly',
  sessionActionNoticeId: 'readonly',
  sessionActionDescription: 'readonly',
  showSessionActionFailure: 'readonly',
  SIDEBAR_WORKTREE_RE: 'readonly',
  worktreeParentPath: 'readonly',
  worktreeName: 'readonly',
  isHiddenRepositoryWorktree: 'readonly',
  formatIndexingBannerText: 'readonly',
  PERMISSION_MODES: 'readonly',
  showSession: 'readonly',
  confirmAndStopSession: 'readonly',
  sendSessionToBackground: 'readonly',
  backgroundActionReason: 'readonly',
  activeSubagentsByParent: 'readonly',
  pollActiveSessions: 'readonly',
  showNewSessionPopover: 'readonly',
  showRemoteLaunchDialog: 'readonly',
  openSettingsViewer: 'readonly',
  wireActivityTraceToggle: 'readonly',
  wireActivityReportingToggle: 'readonly',
  wireRemoteEnrolControls: 'readonly',
  renderActivityReportingStatus: 'readonly',
  renderActivityTraceFiles: 'readonly',
  openActivityTraceFile: 'readonly',
  showResumeSessionDialog: 'readonly',
  showSendPromptDialog: 'readonly',
  showJsonlViewer: 'readonly',
  showSubagentTranscript: 'readonly',
  narrowSessionsToSearch: 'readonly',
  forkSession: 'readonly',
  openSession: 'readonly',
  loadProjects: 'readonly',
  renderProjects: 'readonly',
  buildSubagentIndex: 'readonly',
  buildSubagentItem: 'readonly',
  appendSubagentChildren: 'readonly',
  clearActiveSubagentsFor: 'readonly',
  rekeyActiveSubagents: 'readonly',
  buildSessionItem: 'readonly',
  buildSlugGroup: 'readonly',
  folderId: 'readonly',
  slugId: 'readonly',
  subagentTypeColor: 'readonly',
  SUBAGENT_LIVE_TTL_MS: 'readonly',
  getExpandedSubagents: 'readonly',
  saveExpandedSubagents: 'readonly',
  getExpandedSlugs: 'readonly',
  saveExpandedSlugs: 'readonly',
  setActivity: 'readonly',
  trackActivity: 'readonly',
  setAttention: 'readonly',
  syncLocalPtyAgentsBusy: 'readonly',
  sessionItemEl: 'readonly',
  rowSessionIds: 'readonly',
  rowHeadId: 'readonly',
  rowHeadSession: 'readonly',
  rowOpenId: 'readonly',
  localMergedMemberRows: 'readonly',
  mergedRowSnapshot: 'readonly',
  storedActivitySnapshot: 'readonly',
  snapshotForLocal: 'readonly',
  localSessionForStatus: 'readonly',
  indexLocalMergedMemberRows: 'readonly',
  seedRemoteActivity: 'readonly',
  rekeyActivityState: 'readonly',
  reconcileBusyState: 'readonly',
  currentActivitySeq: 'readonly',
  forgetActivitySeq: 'readonly',
  purgeActivityFor: 'readonly',
  dropLocalPtySession: 'readonly',
  pruneRemoteActivityTimers: 'readonly',
  // public/session-activity.js's persisted per-session state — see .ai/contexts/session-state.md ("The local-pty adapter")
  localPtyState: 'readonly',
  localPtyStates: 'readonly',
  // Changes panel no-polling refresh hook (issue #251, public/file-panel.js)
  onSessionIdle: 'readonly',
  // public/session-state.js (pure domain, see .ai/contexts/session-state.md)
  createSessionState: 'readonly',
  renderSessionIcon: 'readonly',
  // public/session-activity-dom.js — the only file allowed to write
  // .cli-busy/.needs-attention/.response-ready/.has-busy-agents, and the only
  // file allowed to write the .session-icon slot (issue #246, step 3b).
  writeIconSlot: 'readonly',
  applyStateClasses: 'readonly',
  setNeedsAttention: 'readonly',
  setResponseReady: 'readonly',
  setCliBusy: 'readonly',
  setHasBusyAgents: 'readonly',
  setIsAlive: 'readonly',
  isSessionAlive: 'readonly',
  paintSessionIcon: 'readonly',
  paintSessionIconFromSnapshot: 'readonly',
  paintTerminalHeaderStatus: 'readonly',
  // public/remote-activity-ui.js (remote-ssh adapter, see .ai/contexts/session-state.md)
  setRemoteAttached: 'readonly',
  applyRemoteStopped: 'readonly',
  // public/stop-session-ui.js — see .ai/contexts/session-state.md (issue #271)
  resolveSessionStop: 'readonly',
  planSessionStop: 'readonly',
  isRemoteSessionAlive: 'readonly',
  stopBeforeArchive: 'readonly',
  showRestoreNotice: 'readonly',
  // read by sidebar.js's parentHasActiveSubagent() — see .ai/contexts/subagent-observability.md
  remoteSessionStates: 'readonly',
  // see .ai/contexts/subagent-observability.md
  isRemoteRowOwned: 'readonly',
  // public/local-transcript-adapter.js (local-transcript adapter, see .ai/contexts/session-state.md)
  localTranscriptPtyTakeover: 'readonly',
  initLocalTranscriptAdapter: 'readonly',
  pruneLocalTranscriptTimers: 'readonly',
  localTranscriptStates: 'readonly',
  seedLocalTranscriptDescriptor: 'readonly',
  // public/sidebar.js, consumed by session-activity-dom.js's snapshotForLocal
  // (see .ai/contexts/session-state.md, "The icon slot (step 3b)")
  parentHasActiveSubagent: 'readonly',
  // public/sidebar.js — see .ai/contexts/subagent-observability.md
  noteSubagentActivity: 'readonly',
  taskActivitySnapshot: 'readonly',
  applyHeadTurnSnapshot: 'readonly',
  headTurns: 'readonly',
  renderAgentsView: 'readonly',
  updateGridSubagentPills: 'readonly',
  refreshGridCardMembers: 'readonly',
  reflectSubagentRunningState: 'readonly',
  scheduleSubagentTtlTick: 'readonly',

  // Third-party renderer libs loaded as <script>
  morphdom: 'readonly',
  marked: 'readonly',
  DOMPurify: 'readonly',
  ViewerPanel: 'readonly',

  // Switchboard preload bridge
  switchboardAPI: 'readonly',
  // electronAPI / window.api possibly exposed via preload — leave as readonly
  electronAPI: 'readonly',

  // xterm.js and addons (loaded via <script>, exposed as window.Terminal etc.)
  Terminal: 'readonly',
  FitAddon: 'readonly',
  SearchAddon: 'readonly',
  WebLinksAddon: 'readonly',
  WebglAddon: 'readonly',
  UnicodeGraphemesAddon: 'readonly',

  // Terminal/grid/file-panel/stats/notifications and assorted helpers
  // shared across renderer files (defined somewhere in public/*.js).
  TERMINAL_THEME: 'writable',
  TERMINAL_THEMES: 'readonly',
  getTerminalTheme: 'readonly',
  currentThemeName: 'writable',
  isMac: 'readonly',
  flashButtonText: 'readonly',
  toggleMarkdownPreview: 'readonly',
  isMarkdownPath: 'readonly',
  renderMarkdownPreview: 'readonly',
  refreshSidebar: 'readonly',
  updateRunningIndicators: 'readonly',
  hideAllViewers: 'readonly',
  drainViewerWatches: 'readonly',
  showTerminalHeader: 'readonly',
  switchPanel: 'readonly',
  showAddProjectDialog: 'readonly',
  showGridView: 'readonly',
  toggleGridView: 'readonly',
  focusGridCard: 'readonly',
  wrapInGridCard: 'readonly',
  initGridObservers: 'readonly',
  initGridGroupToggle: 'readonly',
  initFilePanel: 'readonly',
  openFileInPanel: 'readonly',
  // public/splitter.js + public/panel-terminal.js — see .ai/contexts/panel-terminal.md
  createSplitter: 'readonly',
  initPanelTerminal: 'readonly',
  syncPanelTerminal: 'readonly',
  panelTerminalIsOpen: 'readonly',
  setPanelTerminalShellOnly: 'readonly',
  destroyPanelTerminalFor: 'readonly',
  isPanelTerminalSession: 'readonly',
  notePanelTerminalExit: 'readonly',
  countSessionsWithoutPanelShells: 'readonly',
  // public/header-controls.js — see .ai/contexts/window-frame.md
  placeHeaderControl: 'readonly',
  createHeaderToggle: 'readonly',
  setHeaderToggle: 'readonly',
  // public/process-exit.js — see .ai/contexts/window-frame.md
  lifecycleLabel: 'readonly',
  noteSessionExit: 'readonly',
  forgetSessionExit: 'readonly',
  lastSessionExit: 'readonly',
  processExitLabel: 'readonly',
  exitBannerPhrase: 'readonly',
  exitBannerColour: 'readonly',
  fileUriToPath: 'readonly',
  createTerminalPathResolver: 'readonly',
  registerTerminalPathLinks: 'readonly',
  findTerminalPathCandidates: 'readonly',
  readTerminalLogicalLine: 'readonly',
  rekeyFilePanelState: 'readonly',
  loadStats: 'readonly',
  loadMemories: 'readonly',
  renderMemories: 'readonly',
  loadWorkFiles: 'readonly',
  renderWorkFiles: 'readonly',
  removeWorkFileFromCache: 'readonly',
  openWorkFile: 'readonly',
  clearNotifications: 'readonly',
  clearUnread: 'readonly',
  setSessionMcpState: 'readonly',
  setSessionSandboxed: 'readonly',
  destroySession: 'readonly',
  launchNewSession: 'readonly',
  launchRemoteSession: 'readonly',
  launchTerminalSession: 'readonly',
  launchScheduleCreator: 'readonly',
  resolveDefaultSessionOptions: 'readonly',
  handleSessionNavKey: 'readonly',
  isSessionNavKey: 'readonly',
  fitAndScroll: 'readonly',
  requestTerminalRefresh: 'readonly',
  allowRemoteResize: 'readonly',
  refreshRemoteTerminalOnReturn: 'readonly',
  safeFit: 'readonly',
  proposeFittedDimensions: 'readonly',
  refitOpenTerminals: 'readonly',
  syncPtySizeAfterOpen: 'readonly',
  ptySizeChanged: 'readonly',
  observeContainerResize: 'readonly',
  flushTerminalBuffer: 'readonly',
  replayHiddenBuffer: 'readonly',
  scheduleFlush: 'readonly',
  handleTerminalData: 'readonly',
  createTerminalEntry: 'readonly',
  terminalWriteBuffers: 'readonly',
  hiddenAccumulators: 'readonly',
  ESC_SYNC_START: 'readonly',
  ESC_SYNC_END: 'readonly',
  SYNC_BUFFER_TIMEOUT: 'readonly',
  updatePtyTitle: 'readonly',
  _shellProfiles: 'writable',

  // The one table of setting defaults (public/setting-defaults.js)
  SETTING_DEFAULTS: 'readonly',

  // Terminal right-click context menu (public/terminal-context-menu.js)
  terminalRightClickMode: 'writable',
  setupTerminalContextMenu: 'readonly',
  showTerminalContextMenu: 'readonly',
  closeTerminalContextMenuForSession: 'readonly',

  // Configurable keyboard shortcuts (public/shortcuts.js + grid-view.js)
  DEFAULT_SHORTCUTS: 'readonly',
  SHORTCUT_DEFS: 'readonly',
  FIXED_SHORTCUTS: 'readonly',
  normalizeShortcuts: 'readonly',
  keyFamily: 'readonly',
  matchShortcut: 'readonly',
  isSessionNavShortcut: 'readonly',
  formatBinding: 'readonly',
  captureBinding: 'readonly',
  appShortcuts: 'writable',
  setAppShortcuts: 'readonly',

  // Working-set restore decision (public/restore-plan.js)
  createRestorePlanner: 'readonly',
  // Resume guard for sessions live in another process (public/resume-guard.js)
  guardResume: 'readonly',
  resolveResumeSession: 'readonly',
  liveElsewhereMany: 'readonly',
  // Agents view (public/agents-view.js) — see .ai/contexts/bg-agents.md
  agentsViewActive: 'writable',
  bgAgentSessionIds: 'readonly',
  backgroundActivityEntry: 'readonly',
  backgroundActivityIsForeground: 'readonly',
  backgroundActivitySnapshot: 'readonly',
  backgroundActivityLabel: 'readonly',
  formatStatusAge: 'readonly',
  initAgentsView: 'readonly',
  showAgentsView: 'readonly',
  hideAgentsView: 'readonly',
  toggleAgentsView: 'readonly',
  restoreAgentsViewAtStartup: 'readonly',
  applyAgentsSnapshot: 'readonly',
  refreshAgentsRoster: 'readonly',
  attachBgAgent: 'readonly',
  runAgentVerb: 'readonly',
  selectAgentsRow: 'readonly',
  showDispatchAgentDialog: 'readonly',
  // claude.ai bridge URL (public/bridge-url.js)
  bridgeSessionUrl: 'readonly',
};

module.exports = [
  { files: ['public/task-registry.js', 'public/utils.js'], languageOptions: { globals: { module: 'writable' } } },
  { files: ['public/grid-view.js'], languageOptions: { globals: { localMergedMemberRows: 'readonly' } } },
  // Ignore generated/vendored bundles and binaries
  {
    ignores: [
      'node_modules/**',
      'dist/**',
      'build/**',
      'public/codemirror-bundle.js',
      'public/mermaid-bundle.js',
      'scripts/**/*', // ad-hoc build helpers; out of lint scope for now
      '!scripts/test-pr.js',
      '.work-files/**',
      '.claude/**', // agent worktrees (nested checkouts) and command config; not lint scope
    ],
  },

  // Renderer process: classic <script> globals, browser context
  {
    files: ['public/**/*.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: {
        ...globals.browser,
        ...rendererCrossFileGlobals,
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-restricted-globals': ['error', ...['confirm', 'alert', 'prompt'].map(name => ({ name, message: 'Use public/app-dialogs.js (appConfirm/appAlert).' }))],
      'no-restricted-properties': ['error', ...['window', 'globalThis', 'self'].flatMap(object => ['confirm', 'alert', 'prompt'].map(property => ({ object, property, message: 'Use public/app-dialogs.js (appConfirm/appAlert).' })))],
      'no-implicit-globals': 'off', // renderer relies on script-scope globals by design
      'no-empty': ['warn', { allowEmptyCatch: true }],
      'no-unreachable': 'warn',
      'no-redeclare': 'warn',
    },
  },

  // see .ai/contexts/touched-files.md
  {
    files: ['public/file-panel.js'],
    languageOptions: {
      globals: { initTouchedView: 'readonly', renderTouchedTab: 'readonly', hideTouchedView: 'readonly', openTouchedPath: 'readonly', openTouchedTab: 'readonly' },
    },
  },
  {
    files: ['public/touched-files-view.js'],
    languageOptions: {
      globals: {
        FP_ICONS: 'readonly',
        currentPanelSessionId: 'readonly',
        handleClose: 'readonly',
        getSessionState: 'readonly',
        endCurrentTab: 'readonly',
        destroyCurrentTab: 'readonly',
        confirmDiscardChangesEdits: 'readonly',
        showPanel: 'readonly',
        renderPanel: 'readonly',
        filePanelState: 'readonly',
        openTouchedEditor: 'readonly',
        renderPanelListLayout: 'readonly',
        readChangesEditorContent: 'readonly',
        createChangesTab: 'readonly',
        restoreChangesEdits: 'readonly',
        filePathKey: 'readonly',
        hasUnsavedChangesEdits: 'readonly',
        panelTerminalOwnerOf: 'readonly',
      },
    },
  },

  // Dual-mode helper: classic <script> in the renderer AND require()-d in tests.
  // Same browser globals as the rest of public/, plus `module` for the CJS footer.
  {
    files: ['public/shortcuts.js', 'public/terminal-context-menu.js', 'public/terminal-manager.js', 'public/restore-plan.js', 'public/resume-guard.js', 'public/bridge-url.js', 'public/stop-session-ui.js', 'public/window-strip.js', 'public/header-controls.js', 'public/tool-bar.js', 'public/process-exit.js', 'public/whats-new.js', 'public/agents-view.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: {
        ...globals.browser,
        ...rendererCrossFileGlobals,
        module: 'writable',
        require: 'readonly',
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-redeclare': 'warn',
    },
  },

  {
    files: ['public/terminal-replay-queries.js'],
    languageOptions: { globals: { module: 'writable' } },
  },

  // Dual-mode matcher (public/terminal-path-links.js — see
  // .ai/contexts/terminal-path-links.md): classic <script> in the renderer,
  // require()-d in node:test. It declares the four link-provider globals
  // rather than consuming them, so they are switched off here.
  {
    files: ['public/terminal-path-links.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: {
        ...globals.browser,
        module: 'writable',
        createTerminalPathResolver: 'off',
        registerTerminalPathLinks: 'off',
        findTerminalPathCandidates: 'off',
        readTerminalLogicalLine: 'off',
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-redeclare': 'warn',
    },
  },

  // Producer of the host checklist global that settings-panel.js consumes.
  {
    files: ['public/remote-enrol-panel.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: { ...globals.browser, wireRemoteEnrolControls: 'off' },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-redeclare': 'warn',
    },
  },

  // Dual-mode Settings section (public/activity-reporting-panel.js — see
  // .ai/contexts/activitywatch.md): classic <script> in the renderer, and
  // require()-d in node:test for its status text. It declares the two globals
  // settings-panel.js consumes, so they are switched off here.
  {
    files: ['public/activity-reporting-panel.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: {
        ...globals.browser,
        module: 'writable',
        wireActivityReportingToggle: 'off',
        renderActivityReportingStatus: 'off',
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-redeclare': 'warn',
    },
  },

  // Producer of a cross-file renderer global: classic <script> in the renderer,
  // require()-d by the main process and tests. It declares SETTING_DEFAULTS
  // rather than consuming it, so the global is switched off here.
  {
    files: ['public/setting-defaults.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: {
        module: 'writable',
        SETTING_DEFAULTS: 'off',
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-redeclare': 'warn',
    },
  },

  // Producer of cross-file renderer globals, dual-mode like subagent-timing.js below.
  {
    files: ['public/worktree-nesting.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: {
        ...globals.browser,
        module: 'writable',
        SIDEBAR_WORKTREE_RE: 'off',
        worktreeParentPath: 'off',
        worktreeName: 'off',
        isHiddenRepositoryWorktree: 'off',
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-redeclare': 'warn',
    },
  },

  // Producer of a cross-file renderer global: classic <script> in the renderer,
  // require()-d by the main process and tests. It declares SUBAGENT_LIVE_TTL_MS
  // rather than consuming it, so the global is switched off here — otherwise
  // no-redeclare flags the one definition the other two files depend on.
  {
    files: ['public/subagent-timing.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: {
        ...globals.browser,
        module: 'writable',
        SUBAGENT_LIVE_TTL_MS: 'off',
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-redeclare': 'warn',
    },
  },

  {
    files: ['public/session-projection-diagnostics.js'],
    languageOptions: { globals: { module: 'writable', Buffer: 'readonly' } },
  },

  // Dual-mode pure domain module (public/session-state.js — see
  // .ai/contexts/session-state.md): classic <script> in the renderer,
  // require()-d in node:test with no DOM/window/electron. It declares
  // createSessionState/renderSessionIcon rather than consuming them, so
  // those two globals are switched off here (same reasoning as
  // subagent-timing.js above).
  {
    files: ['public/session-state.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'script',
      globals: {
        module: 'writable',
        createSessionState: 'off',
        renderSessionIcon: 'off',
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-redeclare': 'warn',
    },
  },

  // Enforcement (.ai/contexts/session-state.md, migration step 3): only
  // public/session-activity-dom.js may write the four activity classes.
  // Every other public/**/*.js file is checked; tests are exempt (they
  // assert on these classes directly, e.g. `item.classList.contains(...)`).
  {
    files: ['public/**/*.js'],
    ignores: ['public/session-activity-dom.js'],
    rules: {
      'no-restricted-syntax': ['error',
        {
          selector: "CallExpression[callee.object.property.name='classList'][callee.property.name=/^(add|remove|toggle|replace)$/] > Literal[value=/^(cli-busy|needs-attention|response-ready|has-busy-agents)$/]",
          message: ACTIVITY_CLASS_MESSAGE,
        },
        {
          selector: "CallExpression[callee.object.property.name='classList'][callee.property.name=/^(add|remove|toggle|replace)$/] > TemplateLiteral > TemplateElement[value.raw=/(cli-busy|needs-attention|response-ready|has-busy-agents)/]",
          message: ACTIVITY_CLASS_MESSAGE,
        },
        {
          selector: "CallExpression[callee.object.property.name='classList'][callee.computed=true]",
          message: 'Computed classList[method](...) hides the class name from lint; call add/remove/toggle directly. See .ai/contexts/session-state.md',
        },
        {
          selector: "AssignmentExpression[left.property.name=/^(className|innerHTML|outerHTML)$/] Literal[value=/(cli-busy|needs-attention|response-ready|has-busy-agents)/]",
          message: ACTIVITY_CLASS_MESSAGE,
        },
        {
          selector: "AssignmentExpression[left.property.name=/^(className|innerHTML|outerHTML)$/] TemplateElement[value.raw=/(cli-busy|needs-attention|response-ready|has-busy-agents)/]",
          message: ACTIVITY_CLASS_MESSAGE,
        },
        {
          selector: "CallExpression[callee.property.name=/^(setAttribute|insertAdjacentHTML)$/] Literal[value=/(cli-busy|needs-attention|response-ready|has-busy-agents)/]",
          message: ACTIVITY_CLASS_MESSAGE,
        },
        {
          selector: "CallExpression[callee.property.name=/^(setAttribute|insertAdjacentHTML)$/] TemplateElement[value.raw=/(cli-busy|needs-attention|response-ready|has-busy-agents)/]",
          message: ACTIVITY_CLASS_MESSAGE,
        },
      ],
    },
  },

  // CodeMirror setup file uses ESM-style imports/closure that don't lint well
  // as a classic script — keep no-undef on but be permissive about unused.
  {
    files: ['public/codemirror-setup.js', 'public/mermaid-setup.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'module',
      globals: { ...globals.browser },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': 'off',
    },
  },

  // Main process: CommonJS, Node + Electron
  {
    files: [
      'main.js',
      'file-lists.js',
      'file-lists-worker-client.js',
      'scan-md-files.js',
      'preload.js',
      'activity-trace.js',
      'db.js',
      'session-cache.js',
      'session-transitions.js',
      'read-session-file.js',
      'derive-project-path.js',
      'encode-project-path.js',
      'git-changes.js',
      'git-changes-runner.js',
      'git-changes-target.js',
      'session-touched-files.js',
      'panel-terminal-target.js',
      'folder-index-state.js',
      'pty-size.js',
      'claude-auth.js',
      'mcp-bridge.js',
      'schedule-ipc.js',
      'schedule-runner.js',
      'shell-profiles.js',
      'terminal-input.js',
      'terminal-resize.js',
      'trigger-context.js',
      'scripts/test-pr.js',
      'workers/**/*.js',
    ],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'commonjs',
      globals: {
        ...globals.node,
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
      'no-empty': ['warn', { allowEmptyCatch: true }],
      'no-unreachable': 'warn',
      'no-redeclare': 'warn',
    },
  },

  // Tests: node:test, Node globals
  {
    files: ['test/**/*.js', 'test/**/*.test.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'commonjs',
      globals: {
        ...globals.node,
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
    },
  },

  // Playwright journeys: Node, plus the renderer callbacks passed to page.evaluate
  {
    files: ['e2e/**/*.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'commonjs',
      globals: {
        ...globals.node,
        ...globals.browser,
        activeSessionId: 'readonly',
        openSessions: 'readonly',
      },
    },
    rules: {
      'no-undef': 'error',
      'no-unused-vars': ['warn', { args: 'none', varsIgnorePattern: '^_' }],
    },
  },

  // ESLint config self-check
  {
    files: ['eslint.config.js'],
    languageOptions: {
      ecmaVersion: 2024,
      sourceType: 'commonjs',
      globals: { ...globals.node },
    },
  },
];