Skip to content

Commit f52bfaa

Browse files
author
Eric Oliver
committed
cleanup missed files
1 parent 81f0cc4 commit f52bfaa

File tree

2 files changed

+33
-20
lines changed

2 files changed

+33
-20
lines changed

src/core/tools/readFileTool.ts

Lines changed: 21 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -397,27 +397,28 @@ export async function readFileTool(
397397
})
398398
}
399399

400-
// Skip the complex individual permissions handling for now
401-
if (false) {
402-
// Handle individual permissions from objectResponse
403-
try {
404-
const individualPermissions = JSON.parse("{}")
405-
let hasAnyDenial = false
400+
// TODO: Skip the complex individual permissions handling for now
401+
// This code is temporarily disabled and should be implemented properly
402+
/*
403+
// Handle individual permissions from objectResponse
404+
try {
405+
const individualPermissions = JSON.parse("{}")
406+
let hasAnyDenial = false
406407
407-
batchFiles.forEach((batchFile, index) => {
408-
const fileResult = filesToApprove[index]
409-
const approved = individualPermissions[batchFile.key] === true
408+
batchFiles.forEach((batchFile, index) => {
409+
const fileResult = filesToApprove[index]
410+
const approved = individualPermissions[batchFile.key] === true
410411
411-
if (approved) {
412-
updateFileResult(fileResult.path, {
413-
status: "approved",
414-
})
415-
} else {
416-
hasAnyDenial = true
417-
updateFileResult(fileResult.path, {
418-
status: "denied",
419-
xmlContent: `<file><path>${fileResult.path}</path><status>Denied by user</status></file>`,
420-
})
412+
if (approved) {
413+
updateFileResult(fileResult.path, {
414+
status: "approved",
415+
})
416+
} else {
417+
hasAnyDenial = true
418+
updateFileResult(fileResult.path, {
419+
status: "denied",
420+
xmlContent: `<file><path>${fileResult.path}</path><status>Denied by user</status></file>`,
421+
})
421422
}
422423
})
423424
@@ -435,7 +436,7 @@ export async function readFileTool(
435436
})
436437
})
437438
}
438-
}
439+
*/
439440
} else if (filesToApprove.length === 1) {
440441
// Handle single file approval (existing logic)
441442
const fileResult = filesToApprove[0]

src/eslint.config.mjs

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,18 @@ export default [
2929
"no-undef": "off",
3030
},
3131
},
32+
{
33+
files: ["test-run-output/**/*.js"],
34+
languageOptions: {
35+
globals: {
36+
require: "readonly",
37+
console: "readonly",
38+
process: "readonly",
39+
__dirname: "readonly",
40+
module: "readonly",
41+
},
42+
},
43+
},
3244
{
3345
ignores: ["webview-ui", "out"],
3446
},

0 commit comments

Comments
 (0)