bundling for non monorepo

[HughParry]

No description provided.

[pr-explainer-bot]

Pull Request Report

Hey there! 👋 Here's the report for the pull request:

Changes

1. Major changes:
   • Updated getPackageDir function in dependencies.ts to handle an empty packageDir by returning the resolved path.

Suggestions

1. Line 25 in dependencies.ts:
   • Consider adding a comment explaining the purpose of the if (stderr) condition for better code understanding.

Bugs

1. No potential bugs found. Great job! 🐛

Improvements

1. Refactoring suggestion in dependencies.ts:
   • Consider using a more descriptive variable name instead of packageDir to improve code readability. For example, trimmedPackageDir would be more meaningful.

async function getPackageDir(packageName: string): Promise<string> {
    const { stdout, stderr } = await exec(`npm root -g ${packageName}`);
    if (stderr) {
        throw new ProsopoEnvError(new Error(stderr));
    }
    const trimmedPackageDir = stdout.trim() || path.resolve();
    return trimmedPackageDir;
}

Rating

I would rate the code as follows:

• Readability: 8/10
• Performance: 9/10
• Security: 9/10

Overall, the code looks good with minor suggestions for improvement. Keep up the great work! 😄

[forgetso]

❤️ 1 line PRs