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)
- }
- })
-}
-*/