ReviewOS

also looking at this

stacks/buddy

fix: keep the low-tier promises — bulk-update label, event-name validation, workflow personalisation

#1434
Merged glennmichael123 wants to merge fix/low-tier-wiring into main
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.

test/cli-surface.test.tsmodified+6-2
Changes to test/cli-surface.test.ts
@@ -1,4 +1,4 @@
1import { existsSync, mkdtempSync, rmSync, writeFileSync } from 'node:fs'
1import { mkdtempSync, rmSync, writeFileSync } from 'node:fs'
22import { tmpdir } from 'node:os'
33import { join } from 'node:path'
44import { describe, expect, it } from 'bun:test'
@@ -82,7 +82,11 @@ describe('CLI surface', () => {
8282
8383 expect(code).toBe(0)
8484 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'])
8690 }
8791 finally {
8892 rmSync(dir, { recursive: true, force: true })