Skip to content

Commit d2770aa

Browse files
committed
feat: update default skill templates to include only karpathy-guidelines (Originally from: https://github.com/multica-ai/andrej-karpathy-skills Licensed under MIT)
1 parent 0d68490 commit d2770aa

3 files changed

Lines changed: 6 additions & 10 deletions

File tree

src/prompt.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,8 +99,7 @@ type PromptToolOptions = {
9999
webSearchEnabled?: boolean;
100100
};
101101

102-
// const DEFAULT_SKILL_TEMPLATES = ["agent-drift-guard.md", "plan-and-execute.md"];
103-
const DEFAULT_SKILL_TEMPLATES = ["karpathy-guidelines.md", "plan-and-execute.md"];
102+
const DEFAULT_SKILL_TEMPLATES = ["karpathy-guidelines.md"];
104103

105104
function readToolDocs(extensionRoot: string, options: PromptToolOptions = {}): string {
106105
const toolsDir = path.join(extensionRoot, "templates", "tools");

src/tests/prompt.test.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,14 +49,11 @@ test("getSystemPrompt does not include runtime context", () => {
4949
assert.equal(prompt.includes('"root path": "/tmp/project"'), false);
5050
});
5151

52-
test("getDefaultSkillPrompt loads default skill templates in order", () => {
52+
test("getDefaultSkillPrompt loads the default skill template", () => {
5353
const prompt = getDefaultSkillPrompt();
54-
const agentDriftIndex = prompt.indexOf("<agent-drift-guard-skill>");
55-
const planIndex = prompt.indexOf("<plan-and-execute-skill>");
5654

57-
assert.notEqual(agentDriftIndex, -1);
58-
assert.notEqual(planIndex, -1);
59-
assert.equal(agentDriftIndex < planIndex, true);
55+
assert.equal(prompt.includes("<karpathy-guidelines-skill>"), true);
56+
assert.equal(prompt.includes("# Karpathy Guidelines"), true);
6057
assert.equal(prompt.includes("Use the skill documents below to assist the user:"), true);
6158
assert.equal(prompt.includes('path="templates/skills/'), false);
6259
});

src/tests/session.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -702,8 +702,8 @@ test("createSession appends default system prompts in prefix-cache-friendly orde
702702
assert.match(systemContents[0] ?? "", /# Available Tools/);
703703
assert.doesNotMatch(systemContents[0] ?? "", /# Local Workspace Environment/);
704704
assert.doesNotMatch(systemContents[0] ?? "", /LLMtest-model/);
705-
assert.match(systemContents[1] ?? "", /<agent-drift-guard-skill>/);
706-
assert.match(systemContents[1] ?? "", /<plan-and-execute-skill>/);
705+
assert.match(systemContents[1] ?? "", /<karpathy-guidelines-skill>/);
706+
assert.match(systemContents[1] ?? "", /# Karpathy Guidelines/);
707707
assert.doesNotMatch(systemContents[1] ?? "", /path="templates\/skills\//);
708708
assert.doesNotMatch(systemContents[1] ?? "", /LLMtest-model/);
709709
assert.match(systemContents[2] ?? "", /# Local Workspace Environment/);

0 commit comments

Comments
 (0)