Skip to content

Commit 910aba2

Browse files
authored
fix(agent-core-v2): drop literal decision clauses from system prompt and auto reminder (#3894)
1 parent 53e5e3f commit 910aba2

5 files changed

Lines changed: 31 additions & 26 deletions

File tree

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@moonshot-ai/kimi-code": patch
3+
---
4+
5+
Drop literal decision clauses from the system prompt and the auto permission mode reminder.
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
Auto permission mode is active. Tool approvals will be handled automatically while this mode remains enabled.
22
- Continue normally without pausing for approval prompts.
3-
- Do NOT call AskUserQuestion while auto mode is active. Make a reasonable decision and continue without asking the user.
3+
- Do NOT call AskUserQuestion while auto mode is active; decide and continue.
44
- ExitPlanMode is also approved automatically, without the user reviewing the plan. An auto-approved plan is NOT a signal from the user to start executing — follow the user's original instructions on whether to proceed.

‎packages/agent-core-v2/src/agent/permissionPolicy/policies/auto-mode-ask-user-question-deny.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ export class AutoModeAskUserQuestionDenyPermissionPolicyService implements Permi
1818
return {
1919
kind: 'deny',
2020
message:
21-
'AskUserQuestion is disabled while auto permission mode is active. Make a reasonable decision and continue without asking the user.',
21+
'AskUserQuestion is disabled while auto permission mode is active; decide and continue.',
2222
};
2323
}
2424
}

‎packages/agent-core-v2/src/app/agentProfileCatalog/system.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ Weigh reversibility and blast radius before acting: local, reversible work is yo
4242

4343
# Delivering work
4444

45-
Do what was asked — no less, no more, and nothing different. Goals the user states explicitly count as part of the ask, even when they pull in files beyond the change you had in mind. Leave out anything the ask does not call for.
45+
Do what was asked. Goals the user states explicitly count as part of the ask.
4646

4747
Before you call the work done, verify the deliverable in the form the user will receive it: the project's standard build and test commands must pass on the deliverable itself, and the user's original scenario must work end-to-end — exercise real calls, not only imports or compiles. Do not mark work complete while tests are red or the implementation is still partial. Say so plainly when you could not verify something, and never present unverified work as done.
4848

‎packages/agent-core-v2/test/tool/tool.test.ts‎

Lines changed: 23 additions & 23 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)