Fix: Resolve linting errors and apply previous fixes

This commit addresses linting errors identified after the initial bugfix submission and includes the original intended fixes.

Corrections for linting:
- Modified `package.json` to use `npx eslint` for `lint` and `lint:fix` scripts to ensure the correct local ESLint version is used.
- Added `"webextensions": true` to `.eslintrc.json` under `env` to recognize browser extension global variables like `chrome`.
- Removed the unused function `prepareForForegroundRequests` from `src/content-script/index.jsx`.

Original fixes included:
- In `src/background/index.mjs`:
    - Ensured that `port.proxy.postMessage({ session })` is called in `executeApi` even if `port.proxy` already exists.
    - Added comprehensive `try...catch` blocks and detailed logging.
    - Added 'blocking' to `onBeforeSendHeaders` listener options.

- In `src/content-script/index.jsx`:
    - Refactored logic to `manageChatGptTabState` for dynamic and correct (re-)registration of the port listener for ChatGPT Web models.
    - Added comprehensive `try...catch` blocks and detailed logging.

Author: google-labs-jules[bot]

.eslintrc.json

@@ -1,7 +1,8 @@
 {
   "env": {
     "browser": true,
-    "es2021": true
+    "es2021": true,
+    "webextensions": true
   },
   "extends": ["eslint:recommended", "plugin:react/recommended"],
   "overrides": [],

package.json

@@ -5,8 +5,8 @@
     "build:safari": "bash ./safari/build.sh",
     "dev": "node build.mjs --development",
     "analyze": "node build.mjs --analyze",
-    "lint": "eslint --ext .js,.mjs,.jsx .",
-    "lint:fix": "eslint --ext .js,.mjs,.jsx . --fix",
+    "lint": "npx eslint --ext .js,.mjs,.jsx .",
+    "lint:fix": "npx eslint --ext .js,.mjs,.jsx . --fix",
     "pretty": "prettier --write ./**/*.{js,mjs,jsx,json,css,scss}",
     "stage": "run-script-os",
     "stage:default": "git add $(git diff --name-only --cached --diff-filter=d)",

src/background/index.mjs

@@ -455,7 +455,6 @@ try {
       console.debug(
         `[background] tabs.onUpdated event for tabId: ${tabId}, status: ${info.status}, url: ${tab.url}`,
       )
-      // eslint-disable-next-line no-undef
       if (chrome && chrome.sidePanel) {
         await chrome.sidePanel.setOptions({
           tabId,

src/content-script/index.jsx

@@ -619,13 +619,6 @@ async function overwriteAccessToken() {
   }
 }
 
-async function prepareForForegroundRequests() {
-  // This function is effectively commented out in the previous step's code.
-  // If it were to be used, it would need error handling similar to manageChatGptTabState.
-  // For now, keeping it as a no-op or minimal logging if called.
-  console.debug('[content] prepareForForegroundRequests (old function) called, but should be unused.')
-}
-
 async function getClaudeSessionKey() {
   console.debug('[content] getClaudeSessionKey called.')
   try {
@@ -923,43 +916,3 @@ try {
 }
 
 run()
-
-// Remove or comment out the old prepareForForegroundRequests function
-/*
-async function prepareForForegroundRequests() {
-  if (location.hostname !== 'chatgpt.com' || location.pathname === '/auth/login') return
-
-  const userConfig = await getUserConfig()
-
-  if (
-    !chatgptWebModelKeys.some((model) =>
-      getApiModesStringArrayFromConfig(userConfig, true).includes(model),
-    )
-  )
-    return
-
-  if (location.pathname === '/') {
-    const input = document.querySelector('#prompt-textarea')
-    if (input) {
-      input.textContent = ' '
-      input.dispatchEvent(new Event('input', { bubbles: true }))
-      setTimeout(() => {
-        input.textContent = ''
-        input.dispatchEvent(new Event('input', { bubbles: true }))
-      }, 300)
-    }
-  }
-
-  await Browser.runtime.sendMessage({
-    type: 'SET_CHATGPT_TAB',
-    data: {},
-  })
-
-  registerPortListener(async (session, port) => {
-    if (isUsingChatgptWebModel(session)) {
-      const accessToken = await getChatGptAccessToken()
-      await generateAnswersWithChatgptWebApi(port, session.question, session, accessToken)
-    }
-  })
-}
-*/