Skip to content

Commit 61a291c

Browse files
committed
docs: update feature workflow rule to emphasize staging and pushing changes
1 parent 5e55927 commit 61a291c

File tree

1 file changed

+20
-12
lines changed

1 file changed

+20
-12
lines changed
Lines changed: 20 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
---
22
description: FOLLOW feature development workflow WHEN implementing new features TO ensure consistent, tested, and documented changes
3-
globs: src/**/*.{ts,tsx}
4-
alwaysApply: true
3+
globs: src/**/*
54
---
65

76
# Feature Development Workflow
@@ -17,7 +16,8 @@ alwaysApply: true
1716
- Locate and review feature documentation in docs/features directory
1817
- Follow documentation requirements systematically
1918
- IMPORTANT Commit frequently to trigger pre-commit hooks (linting, formatting, tests)
20-
- Push to remote remotr
19+
- CRITICAL Always stage ALL modified files after making changes
20+
- CRITICAL Always push changes to remote after committing
2121
- Add comprehensive tests for new functionality
2222
- Push completed work to remote for PR review
2323

@@ -29,28 +29,36 @@ alwaysApply: true
2929
3. Review docs/features/002-server-info-resource.md
3030
4. Implement ServerInfo interface
3131
5. Add getServerInfo method with tests
32-
6. Commit to run hooks: "feat: add server info interface"
33-
7. Implement ServerStatus interface
34-
8. Add getServerStatus method with tests
35-
9. Commit to run hooks: "feat: add server status resource"
36-
10. Fix any linting issues
37-
11. Push to remote when complete
32+
6. git add . # Stage ALL changes
33+
7. Commit to run hooks: "feat: add server info interface"
34+
8. git push origin feature/server-info # Push changes immediately
35+
9. Implement ServerStatus interface
36+
10. Add getServerStatus method with tests
37+
11. git add . # Stage ALL changes again
38+
12. Commit to run hooks: "feat: add server status resource"
39+
13. git push origin feature/server-info # Push changes again
40+
14. Fix any linting issues
41+
15. git add . && git commit -m "fix: linting issues"
42+
16. git push origin feature/server-info
3843
</example>
3944

4045
<example type="invalid">
4146
# Poor workflow
4247
1. Start coding without checking docs
4348
2. Make all changes in one big commit
4449
3. Skip tests or add them later
45-
4. Push directly to main
46-
5. Fix linting issues after PR
50+
4. Leave changes unstaged
51+
5. Forget to push changes to remote
52+
6. Push directly to main
53+
7. Fix linting issues after PR
4754
</example>
4855

4956
## Critical Points
5057
<critical>
5158
- ALWAYS work from feature documentation
52-
-
5359
- NEVER skip tests for new functionality
5460
- Commit OFTEN to utilize pre-commit hooks
61+
- ALWAYS stage ALL modified files after making changes
62+
- ALWAYS push changes to remote after committing
5563
- Keep commits focused and well-described
5664
</critical>

0 commit comments

Comments
 (0)