ReviewOS

also looking at this

stacks/buddy

fix: prevent generated dependency cache PRs

#1366
Merged chris wants to merge codex/prevent-generated-pantry-prs into main
16 files +356 -72

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/bun-deps.test.tsmodified+0-4
Changes to test/bun-deps.test.ts
@@ -28,14 +28,11 @@ const mockResolveDependencyFile = mock(() => Promise.resolve({
2828
2929describe('Bun deps.yaml Update Tests', () => {
3030 let testDir: string
31 let originalCwd: string
3231 let config: BuddyBotConfig
3332
3433 beforeAll(async () => {
3534 // Create a temporary directory for testing
3635 testDir = await mkdtemp(join(tmpdir(), 'buddy-test-'))
37 originalCwd = process.cwd()
38 process.chdir(testDir)
3936
4037 // Create a basic package.json file
4138 await writeFile(
@@ -95,7 +92,6 @@ describe('Bun deps.yaml Update Tests', () => {
9592
9693 afterAll(async () => {
9794 // Clean up
98 process.chdir(originalCwd)
9995 await rm(testDir, { recursive: true, force: true })
10096 })
10197
test/ignore-paths-autoclose.test.tsmodified+0-1
Changes to test/ignore-paths-autoclose.test.ts
@@ -13,7 +13,6 @@ describe('IgnorePaths Auto-Close Functionality', () => {
1313
1414 beforeEach(async () => {
1515 testDir = await mkdtemp(join(tmpdir(), 'buddy-ignore-paths-autoclose-'))
16 process.chdir(testDir)
1716 logger = new Logger(false) // Quiet logger for tests
1817 })
1918