refactor: merge deepen-plan into ce:plan as automatic confidence check (#404)
This commit is contained in:
@@ -286,7 +286,7 @@ Task compound-engineering:review:security-reviewer(code_diff)`,
|
||||
description: "Planning with commands",
|
||||
body: `After planning, you can:
|
||||
|
||||
1. Run /deepen-plan to enhance
|
||||
1. Run /todo-resolve to enhance
|
||||
2. Run /plan_review for feedback
|
||||
3. Start /workflows:work to implement
|
||||
|
||||
@@ -309,7 +309,7 @@ Don't confuse with file paths like /tmp/output.md or /dev/null.`,
|
||||
const parsed = parseFrontmatter(commandSkill!.content)
|
||||
|
||||
// Slash commands should be transformed to /prompts: syntax
|
||||
expect(parsed.body).toContain("/prompts:deepen-plan")
|
||||
expect(parsed.body).toContain("/prompts:todo-resolve")
|
||||
expect(parsed.body).toContain("/prompts:plan_review")
|
||||
expect(parsed.body).toContain("/prompts:workflows-work")
|
||||
|
||||
|
||||
@@ -119,7 +119,7 @@ description: Brainstorm workflow
|
||||
|
||||
Continue with /ce:plan when ready.
|
||||
Or use /workflows:plan if you're following an older doc.
|
||||
Use /deepen-plan for deeper research.
|
||||
Use /todo-resolve for deeper research.
|
||||
`,
|
||||
)
|
||||
await fs.writeFile(
|
||||
@@ -135,7 +135,7 @@ Use /deepen-plan for deeper research.
|
||||
promptTargets: {
|
||||
"ce-plan": "ce-plan",
|
||||
"workflows-plan": "ce-plan",
|
||||
"deepen-plan": "deepen-plan",
|
||||
"todo-resolve": "todo-resolve",
|
||||
},
|
||||
skillTargets: {},
|
||||
},
|
||||
@@ -149,7 +149,7 @@ Use /deepen-plan for deeper research.
|
||||
)
|
||||
expect(installedSkill).toContain("/prompts:ce-plan")
|
||||
expect(installedSkill).not.toContain("/workflows:plan")
|
||||
expect(installedSkill).toContain("/prompts:deepen-plan")
|
||||
expect(installedSkill).toContain("/prompts:todo-resolve")
|
||||
|
||||
const notes = await fs.readFile(
|
||||
path.join(tempRoot, ".codex", "skills", "ce-brainstorm", "notes.md"),
|
||||
|
||||
@@ -466,12 +466,12 @@ Task best-practices-researcher(topic)`
|
||||
})
|
||||
|
||||
test("replaces colons with hyphens in slash commands", () => {
|
||||
const input = `1. Run /deepen-plan to enhance
|
||||
const input = `1. Run /todo-resolve to enhance
|
||||
2. Start /workflows:work to implement
|
||||
3. File at /tmp/output.md`
|
||||
|
||||
const result = transformContentForCopilot(input)
|
||||
expect(result).toContain("/deepen-plan")
|
||||
expect(result).toContain("/todo-resolve")
|
||||
expect(result).toContain("/workflows-work")
|
||||
expect(result).not.toContain("/workflows:work")
|
||||
// File paths preserved
|
||||
|
||||
@@ -214,7 +214,7 @@ Task best-practices-researcher(topic)`,
|
||||
description: "Planning with commands",
|
||||
body: `After planning, you can:
|
||||
|
||||
1. Run /deepen-plan to enhance
|
||||
1. Run /todo-resolve to enhance
|
||||
2. Run /plan_review for feedback
|
||||
3. Start /workflows:work to implement
|
||||
|
||||
@@ -233,7 +233,7 @@ Don't confuse with file paths like /tmp/output.md or /dev/null.`,
|
||||
})
|
||||
|
||||
const parsed = parseFrontmatter(bundle.commands[0].content)
|
||||
expect(parsed.body).toContain("/deepen-plan")
|
||||
expect(parsed.body).toContain("/todo-resolve")
|
||||
expect(parsed.body).toContain("/plan_review")
|
||||
expect(parsed.body).toContain("/work")
|
||||
expect(parsed.body).not.toContain("/workflows:work")
|
||||
|
||||
@@ -57,7 +57,7 @@ describe("convertClaudeToPi", () => {
|
||||
"- Task repo-research-analyst(feature_description)",
|
||||
"- Task learnings-researcher(feature_description)",
|
||||
"Use AskUserQuestion tool for follow-up.",
|
||||
"Then use /workflows:work and /prompts:deepen-plan.",
|
||||
"Then use /workflows:work and /prompts:todo-resolve.",
|
||||
"Track progress with TodoWrite and TodoRead.",
|
||||
].join("\n"),
|
||||
sourcePath: "/tmp/plugin/commands/plan.md",
|
||||
@@ -81,7 +81,7 @@ describe("convertClaudeToPi", () => {
|
||||
expect(parsedPrompt.body).toContain("Run subagent with agent=\"learnings-researcher\" and task=\"feature_description\".")
|
||||
expect(parsedPrompt.body).toContain("ask_user_question")
|
||||
expect(parsedPrompt.body).toContain("/workflows-work")
|
||||
expect(parsedPrompt.body).toContain("/deepen-plan")
|
||||
expect(parsedPrompt.body).toContain("/todo-resolve")
|
||||
expect(parsedPrompt.body).toContain("file-based todos (todos/ + /skill:todo-create)")
|
||||
})
|
||||
|
||||
|
||||
@@ -154,7 +154,7 @@ describe("ce-review contract", () => {
|
||||
}
|
||||
})
|
||||
|
||||
test("leaves data-migration-expert as the unstructured deepen-plan reviewer", async () => {
|
||||
test("leaves data-migration-expert as the unstructured review format", async () => {
|
||||
const content = await readRepoFile(
|
||||
"plugins/compound-engineering/agents/review/data-migration-expert.md",
|
||||
)
|
||||
|
||||
Reference in New Issue
Block a user