feat: move to bun and structure output

[mattzcarey]

No description provided.

[github-actions]

Risk Level 2 - /home/runner/work/code-review-gpt/code-review-gpt/scripts/build.js

The code is generally well-structured, but the repeated fs.copyFileSync calls can be simplified by using an array of template names and iterating over it. This improves readability and maintainability.

Suggested changes:

const { build } = require('esbuild');
const { dependencies } = require('../package.json');
const { Generator } = require('npm-dts');
const fs = require('fs');
const path = require('path');

new Generator({
  entry: 'src/index.ts',
  output: 'dist/index.d.ts',
}).generate();

const sharedConfig = {
  entryPoints: ['src/index.ts'],
  bundle: true,
  minify: true,
  external: Object.keys(dependencies),
  platform: 'node',
  outfile: 'dist/index.js',
};

build(sharedConfig).then(() => {
  // Copy the CI template files to the dist folder after the build is complete
  const templates = ['github-pr.yml', 'gitlab-pr.yml', 'azdev-pr.yml'];
  templates.forEach(template => {
    fs.copyFileSync(
      path.join(__dirname, '../templates', template),
      path.join(__dirname, '../dist', template)
    );
  });
});

View Original Code

+const { build } = require('esbuild');
+const { dependencies } = require('../package.json');
+const { Generator } = require('npm-dts');
+const fs = require('fs');
+const path = require('path');
+
+new Generator({
+  entry: 'src/index.ts',
+  output: 'dist/index.d.ts',
+}).generate();
+
+const sharedConfig = {
+  entryPoints: ['src/index.ts'],
+  bundle: true,
+  minify: true,
+  external: Object.keys(dependencies),
+  platform: 'node',
+  outfile: 'dist/index.js',
+};
+
+build(sharedConfig).then(() => {
+  // Copy the ci template files to the dist folder after the build is complete
+  fs.copyFileSync(
+    path.join(__dirname, '../templates', 'github-pr.yml'),
+    path.join(__dirname, '../dist', 'github-pr.yml')
+  );
+  fs.copyFileSync(
+    path.join(__dirname, '../templates', 'gitlab-pr.yml'),
+    path.join(__dirname, '../dist', 'gitlab-pr.yml')
+  );
+  fs.copyFileSync(
+    path.join(__dirname, '../templates', 'azdev-pr.yml'),
+    path.join(__dirname, '../dist', 'azdev-pr.yml')
+  );
+});

---

Powered by Code Review GPT

[github-actions]

Test results summary:

❌ [FAIL] - Test case: Unawaited Promise
❌ [FAIL] - Test case: Exposed secret
❌ [FAIL] - Test case: Too many nested loops
❌ [FAIL] - Test case: Bad variable name

SUMMARY: ✅ PASS: 0 - ⚠️ WARN: 0 - ❌ FAIL: 4

---

Tests Powered by Code Review GPT