Skip to content

Commit ed5bc4c

Browse files
committed
Reduce repeat debug logs
1 parent 0d5903d commit ed5bc4c

File tree

2 files changed

+13
-6
lines changed

2 files changed

+13
-6
lines changed

src/commands/fix/agent-fix.mts

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -352,7 +352,10 @@ export async function agentFix(
352352
? prs.find(p => p.headRefName === branch)
353353
: undefined
354354
if (pr) {
355-
debugFn('notice', `skip: PR #${pr.number} for ${name} exists`)
355+
debugFn(
356+
'notice',
357+
`skip: PR #${pr.number} for ${name}@${newVersion} exists`,
358+
)
356359
seenBranches.add(branch)
357360
continue infosLoop
358361
}
@@ -361,7 +364,10 @@ export async function agentFix(
361364
// eslint-disable-next-line no-await-in-loop
362365
(await gitRemoteBranchExists(branch, cwd))
363366
) {
364-
debugFn('notice', `skip: remote branch "${branch}" exists`)
367+
debugFn(
368+
'notice',
369+
`skip: remote branch "${branch}" for ${name}@${newVersion} exists`,
370+
)
365371
seenBranches.add(branch)
366372
continue infosLoop
367373
}
@@ -410,7 +416,11 @@ export async function agentFix(
410416

411417
// eslint-disable-next-line no-await-in-loop
412418
if (!(await hasModifiedFiles(cwd))) {
413-
debugFn('notice', 'skip: nothing to commit, skipping PR creation')
419+
debugFn(
420+
'notice',
421+
`skip: nothing to commit for ${name}@${newVersion} PR`,
422+
)
423+
seenVersions.add(newVersion)
414424
// Reset things just in case.
415425
if (fixEnv.isCi) {
416426
// eslint-disable-next-line no-await-in-loop

src/shadow/npm/arborist-helpers.mts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -289,9 +289,6 @@ export function getDetailsFromDiff(
289289
) {
290290
existing = oldNode
291291
}
292-
} else {
293-
// TODO: This debug log has too much information. We should narrow it down.
294-
// debugFn('notice', 'skip: meta change diff\n', diff)
295292
}
296293
} else {
297294
keep = action !== DiffAction.remove

0 commit comments

Comments
 (0)