Skip to content

Commit e857ecb

Browse files
feat: enhance GitHub Copilot workflow instructions with clarity and structure improvements
1 parent 5a7e274 commit e857ecb

1 file changed

Lines changed: 11 additions & 8 deletions

File tree

.github/copilot-instructions.md

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
11
# GitHub Copilot — Workflow Instructions
22

3+
> **Precedence rule:** In case of conflict, Core Principles override Workflow Orchestration, which overrides Task Management defaults.
4+
35
## Workflow Orchestration
46

57
### 1. Plan Before Implementing
6-
- For ANY non-trivial task (3+ steps or architectural decisions), think through the full plan before writing code
8+
- For any task that meets ONE OR MORE of: (a) requires 3 or more distinct implementation steps, (b) involves choosing between architectural approaches, or (c) modifies more than one module/file — think through the full plan before writing code
79
- Use the `manage_todo_list` tool to write the plan with checkable items before starting
8-
- If something goes sideways during implementation, STOP and re-plan immediately
10+
- If any of the following occur: a test fails that cannot be fixed within 2 attempts, a dependency or constraint is discovered that invalidates the current approach, or the implementation scope grows beyond the original plan — STOP and re-plan immediately
911
- Write detailed specs upfront to reduce ambiguity
1012

1113
### 2. Subagent Strategy
@@ -17,7 +19,7 @@
1719
### 3. Self-Improvement Loop
1820
- After ANY correction from the user: update `.github/instructions/lessons.instructions.md` with the pattern
1921
- Write rules that prevent the same mistake from recurring
20-
- Review lessons at the start of a session for the relevant project
22+
- At the start of each session, read `.github/instructions/lessons.instructions.md` and apply lessons that match the current repository or technology stack
2123

2224
### 4. Verification Before Done
2325
- Never mark a task complete without proving it works
@@ -27,22 +29,23 @@
2729

2830
### 5. Demand Elegance (Balanced)
2931
- For non-trivial changes: pause and ask "is there a more elegant way?"
30-
- If a fix feels hacky, implement the elegant solution instead
32+
- Elegance takes precedence over a hacky fix, but must not require touching unnecessary code — prefer the simplest elegant solution
3133
- Skip this for simple, obvious fixes — don't over-engineer
3234
- Challenge your own work before presenting it
3335

34-
### 6. Autonomous Bug Fixing
35-
- When given a bug report: just fix it — no hand-holding needed
36+
### 6. Interaction Model
37+
- **Proceed autonomously** for bug fixes and tasks with a clear, unambiguous goal (no check-in needed)
38+
- **Require check-in** before implementation only for tasks involving architectural decisions or changes affecting more than one system boundary
3639
- Point at logs, errors, and failing tests, then resolve them
37-
- Zero context switching required from the user
3840
- Fix failing tests without being told how
3941

4042
---
4143

4244
## Task Management
4345

46+
0. **Clarify First** — If the task description is missing acceptance criteria, affected files, or expected behavior, ask at most 3 targeted clarifying questions before writing any plan (skip for bug fixes with clear reproduction steps)
4447
1. **Plan First** — Write the plan to the task list (`manage_todo_list`) with checkable items
45-
2. **Verify Plan** — Check in before starting implementation
48+
2. **Verify Plan** — Check in before starting implementation for architectural or multi-boundary tasks; proceed directly for bug fixes and unambiguous tasks
4649
3. **Track Progress** — Mark items complete as you go; only one item in-progress at a time
4750
4. **Explain Changes** — High-level summary at each step
4851
5. **Document Results** — Add a brief review at the end of the task list

0 commit comments

Comments
 (0)