also looking at this
fix: keep the low-tier promises — bulk-update label, event-name validation, workflow personalisation
#1434
9 files
+325
-13
Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.
| @@ -1,4 +1,4 @@ | ||
| 1 | import { existsSync, mkdtempSync, rmSync, writeFileSync } from 'node:fs' | |
| 1 | import { mkdtempSync, rmSync, writeFileSync } from 'node:fs' | |
| 2 | 2 | import { tmpdir } from 'node:os' |
| 3 | 3 | import { join } from 'node:path' |
| 4 | 4 | import { describe, expect, it } from 'bun:test' |
| @@ -82,7 +82,11 @@ describe('CLI surface', () => { | ||
| 82 | 82 | |
| 83 | 83 | expect(code).toBe(0) |
| 84 | 84 | expect(output).toContain('disabled by config') |
| 85 | expect(existsSync(join(dir, '.github'))).toBe(false) | |
| 85 | // Bun's native glob rather than node:fs — another suite's fs module | |
| 86 | // mock leaks into this file under CI's run order, and existsSync came | |
| 87 | // back undefined there. | |
| 88 | const entries = Array.from(new Bun.Glob('**/*').scanSync({ cwd: dir, onlyFiles: false })) | |
| 89 | expect(entries).toEqual(['buddy.config.ts']) | |
| 86 | 90 | } |
| 87 | 91 | finally { |
| 88 | 92 | rmSync(dir, { recursive: true, force: true }) |