fix: test case

Author: yimingjfe

packages/cli/plugin-bff/src/utils/runtimeGenerator.ts

@@ -1,22 +1,50 @@
 import path from 'path';
 import { fs } from '@modern-js/utils';
 
+/**
+ * Get package name from package.json file
+ * @param appDirectory - Application directory path
+ * @returns Package name or undefined if not found
+ */
+const getPackageName = (appDirectory: string): string | undefined => {
+  try {
+    const packageJsonPath = path.resolve(appDirectory, './package.json');
+    const packageJson = require(packageJsonPath);
+    return packageJson.name;
+  } catch (error) {
+    // If package.json doesn't exist or is invalid, return undefined
+    return undefined;
+  }
+};
+
 async function runtimeGenerator({
   runtime,
   appDirectory,
   relativeDistPath,
-}: { runtime: string; appDirectory: string; relativeDistPath: string }) {
+  packageName,
+}: {
+  runtime: string;
+  appDirectory: string;
+  relativeDistPath: string;
+  packageName?: string;
+}) {
   const pluginDir = path.resolve(
     appDirectory,
     `./${relativeDistPath}`,
     'runtime',
   );
 
+  const requestId =
+    packageName ||
+    getPackageName(appDirectory) ||
+    process.env.npm_package_name ||
+    'default';
+
   const source = `import { configure as _configure } from '${runtime}'
     const configure = (options) => {
       return _configure({
         ...options,
-        requestId: '${process.env.npm_package_name}',
+        requestId: '${requestId}',
       });
     }
     export { configure }

packages/server/bff-core/src/client/generateClient.ts

@@ -3,6 +3,22 @@ import type { HttpMethodDecider } from '@modern-js/types';
 import { ApiRouter } from '../router';
 import { Err, Ok, type Result } from './result';
 
+/**
+ * Get package name from package.json file
+ * @param appDir - Application directory path
+ * @returns Package name or undefined if not found
+ */
+const getPackageName = (appDir: string): string | undefined => {
+  try {
+    const packageJsonPath = path.resolve(appDir, './package.json');
+    const packageJson = require(packageJsonPath);
+    return packageJson.name;
+  } catch (error) {
+    // If package.json doesn't exist or is invalid, return undefined
+    return undefined;
+  }
+};
+
 export type GenClientResult = Result<string>;
 
 export type GenClientOptions = {
@@ -65,7 +81,9 @@ export const generateClient = async ({
     const routeName = routePath;
 
     const requestId =
-      target === 'bundle' ? process.env.npm_package_name : undefined;
+      target === 'bundle'
+        ? getPackageName(appDir) || process.env.npm_package_name
+        : undefined;
 
     if (action === 'upload') {
       const requestOptions = {

packages/server/bff-core/tests/client/__snapshots__/generateClient.test.ts.snap

@@ -3,9 +3,9 @@
 exports[`client generateClient should support cross croject invocation 1`] = `
 "import { createRequest } from '@modern-js/plugin-bff/client';
 
-export var DELETE = createRequest({path: '/normal/origin',method: 'DELETE',port: 3000,httpMethodDecider: 'functionName', requestId: '@modern-js/bff-core'});
-      export default createRequest({path: '/normal/origin',method: 'GET',port: 3000,httpMethodDecider: 'functionName', requestId: '@modern-js/bff-core'});
-      export var putRepo = createRequest({path: '/put-repo',method: 'PUT',port: 3000,httpMethodDecider: 'functionName', requestId: '@modern-js/bff-core'});
+export var DELETE = createRequest({path: '/normal/origin',method: 'DELETE',port: 3000,httpMethodDecider: 'functionName', requestId: 'tests'});
+      export default createRequest({path: '/normal/origin',method: 'GET',port: 3000,httpMethodDecider: 'functionName', requestId: 'tests'});
+      export var putRepo = createRequest({path: '/put-repo',method: 'PUT',port: 3000,httpMethodDecider: 'functionName', requestId: 'tests'});
       "
 `;
 

packages/server/bff-core/tests/client/generateClient.test.ts

@@ -26,7 +26,7 @@ describe('client', () => {
     const source = await fs.readFile(resourcePath, 'utf-8');
 
     const result = await generateClient({
-      appDir: PWD,
+      appDir: __dirname,
       prefix,
       port,
       resourcePath,
@@ -49,7 +49,7 @@ describe('client', () => {
     const source = await fs.readFile(resourcePath, 'utf-8');
 
     const result = await generateClient({
-      appDir: PWD,
+      appDir: __dirname,
       prefix,
       port,
       resourcePath,
@@ -72,7 +72,7 @@ describe('client', () => {
     const source = await fs.readFile(resourcePath, 'utf-8');
 
     const result = await generateClient({
-      appDir: PWD,
+      appDir: __dirname,
       prefix,
       port,
       resourcePath,