Skip to content

project-executor: drop the plan-before-acting instruction - #67

Merged
alongd merged 1 commit into
mainfrom
project-executor-no-plan-line
Oct 3, 2026
Merged

alongd merged 1 commit into
mainfrom
project-executor-no-plan-line

Conversation

@alongd

@alongd alongd commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Removes "Plan before you act, then execute the plan" from the project-executor agent. Current Claude models plan without being told, and the instruction pushes them toward over-planning; the role description above it already covers breaking the goal into steps and sequencing them. Found in a prompt audit of the agent definitions.

@alongd
alongd requested a balanced review from Copilot October 3, 2026 03:38
Current models plan unprompted; telling them to plan first pushes them to
over-plan. The role line above already says to break the goal into steps.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The focused documentation change matches the stated purpose without weakening essential execution safeguards.

Review effort: Balanced
Findings: None

What changed in this PR

Removes redundant planning guidance from the project-executor agent while preserving verification and progress-reporting requirements.

Changes:

  • Drops the explicit plan-before-acting instruction.
  • Retains milestone verification and course correction.
File Description
onboarding/​agents/​project-executor.md Simplifies the agent’s execution guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@alongd
alongd merged commit 37e3793 into main Oct 3, 2026
5 checks passed
@alongd
alongd deleted the project-executor-no-plan-line branch October 3, 2026 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants