Skip to content

Commit f656ec5

Browse files
dcramercodex
andcommitted
fix(junior): Remove redundant credential injection log
Keep a single info log when scoped credential headers are injected for sandbox commands. The start log already carries the useful context, so the completion log adds noise without new signal. Co-Authored-By: GPT-5 <noreply@openai.com>
1 parent 83066d1 commit f656ec5

1 file changed

Lines changed: 0 additions & 11 deletions

File tree

packages/junior/src/chat/respond.ts

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -788,17 +788,6 @@ function createAgentTools(
788788
"result" in result
789789
? (result as { result: unknown }).result
790790
: result;
791-
if (shouldLogCredentialInjection) {
792-
logInfo(
793-
"credential_inject_cleanup",
794-
{},
795-
{
796-
"app.skill.name": sandbox.getActiveSkill()?.name,
797-
},
798-
"Scoped credential header injection completed",
799-
);
800-
}
801-
802791
const durationMs = Date.now() - toolStartedAt;
803792
const toolResultAttribute = serializeGenAiAttribute(resultDetails);
804793
setSpanAttributes({

0 commit comments

Comments
 (0)