Skip to content

Commit 3e691cf

Browse files
authored
Merge pull request #12 from sanderkooger/feat/update-readme-workflow
docs: Update README with development workflow fixes #6
2 parents d5d96ca + 4f50601 commit 3e691cf

File tree

6 files changed

+1009
-62
lines changed

6 files changed

+1009
-62
lines changed

.clinerules

Lines changed: 24 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -14,17 +14,29 @@ This file stores agreed-upon conventions and prompt context for interacting with
1414
- Conform code you write with the rules in .prettierrc
1515
- Prefer arrow functions
1616

17-
## GitHub MCP Usage
18-
19-
- Use GitHub MCP tools only for:
20-
- Issue/pull request comments
21-
- Creating/updating issues
22-
- Repository metadata operations
23-
- updating secrets
24-
- etc
25-
- For commits/branches/pushes, use CLI commands
26-
- Never store GitHub credentials in code/configs
27-
- always read the .git/config, or root package.json of the repo you are working on to determine the repository owner and name
17+
## GitHub Integration Rules
18+
19+
### Local Operations (Use CLI)
20+
21+
- All Git operations (commits/branches/pushes):
22+
- Must use raw git commands
23+
- Format commit messages directly via CLI
24+
- Never use `npm run commit` or GitHub MCP tools
25+
26+
### Platform Interactions (Use MCP)
27+
28+
- Limit GitHub MCP to:
29+
- PR management/comments
30+
- Issue tracking
31+
- Repository metadata
32+
- Secret management
33+
34+
### Security & Configuration
35+
36+
- Never store credentials in code/configs
37+
- Derive repository info from:
38+
- Local .git/config
39+
- package.json metadata
2840

2941
## RagDocs Usage
3042

@@ -36,7 +48,7 @@ This file stores agreed-upon conventions and prompt context for interacting with
3648

3749
- Never modify package.json directly - always use `npm install <package>` commands
3850
- Prefer exact version specifications when adding dependencies
39-
- Ensure< package-lock.json is committed after dependency changes
51+
- Ensure package-lock.json is committed after dependency changes
4052

4153
## Testing Conventions
4254

.commitlintrc.ts

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,16 +11,17 @@ const Configuration: UserConfig = {
1111
RuleConfigSeverity.Error,
1212
'always',
1313
[
14+
'build',
1415
'chore',
16+
'ci',
17+
'docs',
1518
'feat',
1619
'fix',
17-
'docs',
18-
'style',
20+
'perf',
1921
'refactor',
20-
'test',
2122
'revert',
22-
'build',
23-
'ci'
23+
'style',
24+
'test'
2425
]
2526
]
2627
},

CONTRIBUTING.md

Lines changed: 37 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,6 @@ npm run format
3232

3333
5. Check code quality with ESLint (using eslint.config.ts config):
3434

35-
gh auth
36-
3735
```bash
3836
npm run lint
3937
```
@@ -63,6 +61,12 @@ We enforce commit standards through:
6361
. "$(dirname -- "$0")/_/husky.sh"
6462

6563
npm run lint
64+
65+
# .husky/commit-msg
66+
#!/usr/bin/env sh
67+
. "$(dirname -- "$0")/_/husky.sh"
68+
69+
npx commitlint --edit "$1"
6670
```
6771

6872
1. Stage changes with `git add`
@@ -165,34 +169,51 @@ describe('jestTestFunction', () => {
165169
- Coverage tracked via Codecov
166170
- Failure blocks merge
167171

168-
## Commit Message Format
172+
## Commit Message Guide
173+
174+
Use the interactive wizard for standardized commits:
175+
176+
```bash
177+
npm run commit
178+
```
169179

170-
We enforce [Conventional Commits](https://www.conventionalcommits.org) specification:
180+
### Format
171181

172182
```
173-
<type>(<scope>): <description>
183+
type(scope): description [issue-number]
174184
```
175185

176-
### Allowed Types:
186+
### Valid Types:
177187

178-
- chore: Maintenance tasks
179-
- feat: New features
180-
- fix: Bug fixes
188+
- feat: New feature (triggers minor release)
189+
- fix: Bug fix (triggers patch release)
190+
- perf: Performance improvement (triggers patch release)
181191
- docs: Documentation changes
182192
- style: Code formatting
183193
- refactor: Code refactoring
184194
- test: Test updates
185-
- revert: Revert changes
186-
- build: Build system updates
195+
- chore: Maintenance tasks (including releases)
196+
197+
The Commitizen wizard will guide you through these types. Our configuration
198+
extends @commitlint/config-conventional which also recognizes:
199+
200+
- build: Changes to build process (not currently used)
201+
- ci: CI configuration changes (not currently used)
202+
- revert: Revert commits (auto-generated)
203+
204+
Official @commitlint/config-conventional types include:
205+
206+
- build: Changes that affect the build system
187207
- ci: CI configuration changes
208+
- revert: Revert a previous commit
188209

189-
Examples:
210+
### Example:
190211

191-
```bash
192-
feat: add documentation search endpoint
193-
fix: resolve timeout issues in queue processing
194-
docs: update API reference documentation
195212
```
213+
fix(commit): Add interactive commit wizard [GH-6]
214+
```
215+
216+
The wizard will validate your input and ensure proper formatting.
196217

197218
## Pull Request Process
198219

README.md

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -228,6 +228,30 @@ ollama list
228228

229229
This MCP server is licensed under the MIT License. This means you are free to use, modify, and distribute the software, subject to the terms and conditions of the MIT License. For more details, please see the LICENSE file in the project repository.
230230

231+
## Development Workflow
232+
233+
1. Install dependencies:
234+
235+
```bash
236+
npm install
237+
```
238+
239+
2. Make your changes following the coding standards in CONTRIBUTING.md
240+
241+
3. Commit changes using Commitizen:
242+
243+
```bash
244+
npm run commit
245+
```
246+
247+
4. Push to your feature branch:
248+
249+
```bash
250+
git push origin feat/your-feature
251+
```
252+
253+
5. Create a pull request through GitHub's interface
254+
231255
## Contributing
232256

233257
We welcome contributions! Please see our [CONTRIBUTING.md](CONTRIBUTING.md) for detailed guidelines, but here are the basics:

0 commit comments

Comments
 (0)