fix: (some minor improvements to configs and getPackageJson return behaviour) (#12510)

Co-authored-by: Tommaso Sciortino <[email protected]>

Author: 2 people

eslint.config.js

@@ -29,7 +29,6 @@ export default tseslint.config(
     // Global ignores
     ignores: [
       'node_modules/*',
-      '.integration-tests/**',
       'eslint.config.js',
       'packages/**/dist/**',
       'bundle/**',

packages/cli/index.ts

@@ -6,7 +6,6 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import './src/gemini.js';
 import { main } from './src/gemini.js';
 import { FatalError, writeToStderr } from '@google/gemini-cli-core';
 import { runExitCleanup } from './src/utils/cleanup.js';

packages/core/src/utils/package.ts

@@ -15,13 +15,26 @@ export type PackageJson = BasePackageJson & {
   };
 };
 
+/**
+ * Reads package.json from the current directory or any parent directory.
+ *
+ * @param cwd - The directory to start searching from (searches upward to filesystem root)
+ * @returns The package.json object if found, or `undefined` if no package.json exists
+ *          in the directory hierarchy. This is expected behavior when called from
+ *          directories outside of a Node.js project.
+ *
+ * @example
+ * ```ts
+ * const pkg = await getPackageJson(__dirname);
+ * const version = pkg?.version ?? 'unknown';
+ * ```
+ */
 export async function getPackageJson(
   cwd: string,
 ): Promise<PackageJson | undefined> {
   const result = await readPackageUp({ cwd });
   if (!result) {
-    // TODO: Maybe bubble this up as an error.
-    return;
+    return undefined;
   }
 
   return result.packageJson;