Skip to content

Commit d8080bd

Browse files
committed
feat: change copilot defaults to --interactive and --silent
Update built-in defaults for copilot command to use --interactive flag (instead of --prompt) and add --silent flag for cleaner output.
1 parent fb5e91c commit d8080bd

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

src/config.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,13 @@ describe("config", () => {
2222
const config = await loadGlobalConfig();
2323
expect(config.commands).toBeDefined();
2424
expect(config.commands?.copilot).toBeDefined();
25-
expect(config.commands?.copilot.$1).toBe("prompt");
25+
expect(config.commands?.copilot.$1).toBe("interactive");
2626
});
2727

2828
test("getCommandDefaults returns defaults for copilot", async () => {
2929
const defaults = await getCommandDefaults("copilot");
3030
expect(defaults).toBeDefined();
31-
expect(defaults?.$1).toBe("prompt");
31+
expect(defaults?.$1).toBe("interactive");
3232
});
3333

3434
test("getCommandDefaults returns undefined for unknown command", async () => {
@@ -184,7 +184,7 @@ describe("loadFullConfig", () => {
184184

185185
test("includes built-in defaults when no project config", async () => {
186186
const config = await loadFullConfig(testDir);
187-
expect(config.commands?.copilot?.$1).toBe("prompt");
187+
expect(config.commands?.copilot?.$1).toBe("interactive");
188188
});
189189

190190
test("project config overrides global config", async () => {
@@ -212,7 +212,7 @@ describe("loadFullConfig", () => {
212212

213213
const config = await loadFullConfig(testDir);
214214
// Built-in defaults preserved
215-
expect(config.commands?.copilot?.$1).toBe("prompt");
215+
expect(config.commands?.copilot?.$1).toBe("interactive");
216216
// New command added
217217
expect(config.commands?.["my-tool"]?.$1).toBe("body");
218218
expect(config.commands?.["my-tool"]?.verbose).toBe(true);
@@ -229,7 +229,7 @@ describe("loadFullConfig", () => {
229229

230230
const config = await loadFullConfig(testDir);
231231
// Built-in default preserved
232-
expect(config.commands?.copilot?.$1).toBe("prompt");
232+
expect(config.commands?.copilot?.$1).toBe("interactive");
233233
// New setting added
234234
expect(config.commands?.copilot?.verbose).toBe(true);
235235
});

src/config.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,8 @@ const PROJECT_CONFIG_NAMES = ["ma.config.yaml", ".markdown-agent.yaml", ".markdo
2929
export const BUILTIN_DEFAULTS: GlobalConfig = {
3030
commands: {
3131
copilot: {
32-
$1: "prompt", // Map body to --prompt for copilot
32+
$1: "interactive", // Map body to --interactive for copilot
33+
silent: true, // Output only the agent response (no stats)
3334
},
3435
},
3536
};

src/context.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ After import`;
311311

312312
expect(config.commands?.claude?.model).toBe("project-model");
313313
// Built-in default for copilot should still be there
314-
expect(config.commands?.copilot?.$1).toBe("prompt");
314+
expect(config.commands?.copilot?.$1).toBe("interactive");
315315
});
316316

317317
it("getCommandDefaultsFromConfig is pure function", () => {

0 commit comments

Comments
 (0)