Skip to content

Commit c8111db

Browse files
authored
Merge pull request #18 from dkmaker/feat/release-please-workflow
feat: migrate to Release Please for controlled releases
2 parents 622e074 + b45420a commit c8111db

17 files changed

Lines changed: 377 additions & 243 deletions

.github/conventional-changelog.config.cjs

Lines changed: 0 additions & 26 deletions
This file was deleted.

.github/conventional-changelog.config.js

Lines changed: 0 additions & 14 deletions
This file was deleted.

.github/dependabot.yml

Lines changed: 45 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,45 @@
1+
version: 2
2+
updates:
3+
# npm dependencies
4+
- package-ecosystem: "npm"
5+
directory: "/"
6+
schedule:
7+
interval: "weekly"
8+
day: "monday"
9+
open-pull-requests-limit: 10
10+
commit-message:
11+
prefix: "chore(deps)"
12+
labels:
13+
- "dependencies"
14+
groups:
15+
# Group minor and patch updates together
16+
production-dependencies:
17+
patterns:
18+
- "*"
19+
exclude-patterns:
20+
- "@types/*"
21+
- "typescript"
22+
- "@commitlint/*"
23+
update-types:
24+
- "minor"
25+
- "patch"
26+
dev-dependencies:
27+
patterns:
28+
- "@types/*"
29+
- "typescript"
30+
- "@commitlint/*"
31+
update-types:
32+
- "minor"
33+
- "patch"
34+
35+
# GitHub Actions
36+
- package-ecosystem: "github-actions"
37+
directory: "/"
38+
schedule:
39+
interval: "weekly"
40+
day: "monday"
41+
commit-message:
42+
prefix: "chore(deps)"
43+
labels:
44+
- "dependencies"
45+
- "github-actions"

.github/workflows/npm-publish.yml

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
name: Publish to NPM
2+
3+
on:
4+
release:
5+
types: [published]
6+
7+
jobs:
8+
publish:
9+
runs-on: ubuntu-latest
10+
permissions:
11+
contents: read
12+
id-token: write
13+
steps:
14+
- name: Checkout
15+
uses: actions/checkout@v4
16+
17+
- name: Setup Node.js
18+
uses: actions/setup-node@v4
19+
with:
20+
node-version: '24.x'
21+
registry-url: 'https://registry.npmjs.org'
22+
23+
- name: Install dependencies
24+
run: npm ci
25+
26+
- name: Build
27+
run: npm run build
28+
29+
- name: Publish to NPM with Provenance
30+
run: npm publish --provenance --access public
Lines changed: 67 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
name: PR Review with Progress Tracking
22

3-
# This example demonstrates how to use the track_progress feature to get
4-
# visual progress tracking for PR reviews, similar to v0.x agent mode.
5-
63
on:
74
pull_request:
85
types: [opened, synchronize, ready_for_review, reopened]
@@ -18,57 +15,91 @@ jobs:
1815
- name: Checkout repository
1916
uses: actions/checkout@v5
2017
with:
21-
fetch-depth: 1
18+
fetch-depth: 0
2219

2320
- name: PR Review with Progress Tracking
2421
uses: anthropics/claude-code-action@v1
2522
with:
2623
anthropic_api_key: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
2724

28-
# Enable progress tracking
2925
track_progress: true
3026

31-
# Your custom review instructions
3227
prompt: |
3328
REPO: ${{ github.repository }}
3429
PR NUMBER: ${{ github.event.pull_request.number }}
3530
3631
Perform a comprehensive code review with the following focus areas:
3732
38-
1. **Code Quality**
39-
- Clean code principles and best practices
40-
- Proper error handling and edge cases
41-
- Code readability and maintainability
33+
## 1. Release Workflow Compliance (CRITICAL)
4234
43-
2. **Security**
44-
- Check for potential security vulnerabilities
45-
- Validate input sanitization
46-
- Review authentication/authorization logic
35+
This project uses Release Please with Conventional Commits. Check:
4736
48-
3. **Performance**
49-
- Identify potential performance bottlenecks
50-
- Review database queries for efficiency
51-
- Check for memory leaks or resource issues
37+
**Commit Message Format:**
38+
- All commits MUST follow conventional commit format: `type(scope): description`
39+
- Valid types: `feat`, `fix`, `perf`, `refactor`, `docs`, `test`, `ci`, `build`, `chore`, `style`
40+
- Type must be lowercase
41+
- Description must be lowercase and not end with a period
42+
- Breaking changes must include `BREAKING CHANGE:` in the commit footer
5243
53-
4. **Testing**
54-
- Verify adequate test coverage
55-
- Review test quality and edge cases
56-
- Check for missing test scenarios
44+
**Version Bump Rules (explain to contributor):**
45+
- `feat:` → Minor version bump (0.4.0 → 0.5.0)
46+
- `fix:`, `perf:`, `refactor:` → Patch version bump (0.4.0 → 0.4.1)
47+
- `docs:`, `test:`, `ci:`, `build:`, `chore:`, `style:` → No version bump
48+
- `BREAKING CHANGE:` footer → Major version bump
5749
58-
5. **Documentation**
59-
- Ensure code is properly documented
60-
- Verify README updates for new features
61-
- Check API documentation accuracy
50+
**If commits don't follow the format:**
51+
- List each non-compliant commit
52+
- Show the correct format it should use
53+
- Suggest squashing/rewording before merge
6254
63-
Provide detailed feedback using inline comments for specific issues.
64-
Use top-level comments for general observations or praise.
55+
Run this command to check commits:
56+
```
57+
git log --oneline origin/main..HEAD
58+
```
6559
66-
# Tools for comprehensive PR review
67-
claude_args: |
68-
--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*)"
60+
## 2. Code Quality
61+
- Clean code principles and best practices
62+
- Proper error handling and edge cases
63+
- Code readability and maintainability
64+
65+
## 3. Security
66+
- Check for potential security vulnerabilities
67+
- Validate input sanitization
68+
- Review authentication/authorization logic
69+
- Check for secrets or credentials in code
70+
71+
## 4. Performance
72+
- Identify potential performance bottlenecks
73+
- Check for memory leaks or resource issues
74+
75+
## 5. Testing
76+
- Verify adequate test coverage for new features
77+
- Review test quality and edge cases
78+
79+
## 6. Documentation
80+
- New features should update README.md if user-facing
81+
- Breaking changes should be clearly documented
82+
83+
## Review Output Format
6984
70-
# When track_progress is enabled:
71-
# - Creates a tracking comment with progress checkboxes
72-
# - Includes all PR context (comments, attachments, images)
73-
# - Updates progress as the review proceeds
74-
# - Marks as completed when done
85+
Start your review with a **Release Compliance Summary**:
86+
87+
```
88+
## Release Compliance Check
89+
90+
| Commit | Format | Type | Valid |
91+
|--------|--------|------|-------|
92+
| abc123 | feat: add feature | feat | ✅ |
93+
| def456 | Update readme | INVALID | ❌ |
94+
95+
**Action Required:** [Yes/No]
96+
```
97+
98+
If any commits are invalid, provide clear instructions on how to fix them
99+
(e.g., interactive rebase commands).
100+
101+
Then proceed with the rest of the code review using inline comments
102+
for specific issues and top-level comments for general observations.
103+
104+
claude_args: |
105+
--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(git log:*),Bash(git show:*)"

.github/workflows/publish.yml

Lines changed: 0 additions & 60 deletions
This file was deleted.

.github/workflows/refresh-badges.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,5 @@ jobs:
1515
- name: Refresh badges
1616
uses: b3b00/refreshBadgesAction@v1.0.7
1717
with:
18-
repository: 'zenturacp/mcp-rest-api'
18+
repository: 'dkmaker/mcp-rest-api'
1919
branch: 'main'
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
name: Release Please
2+
3+
on:
4+
push:
5+
branches: [main]
6+
7+
permissions:
8+
contents: write
9+
pull-requests: write
10+
11+
jobs:
12+
release-please:
13+
runs-on: ubuntu-latest
14+
steps:
15+
- uses: googleapis/release-please-action@v4
16+
id: release
17+
with:
18+
token: ${{ secrets.GITHUB_TOKEN }}
19+
config-file: release-please-config.json
20+
manifest-file: .release-please-manifest.json

.release-please-manifest.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
{
2+
".": "0.4.0"
3+
}

CHANGELOG.md

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,24 @@
1-
## [0.4.0](https://github.com/dkmaker/mcp-rest-api/compare/v0.3.0...v0.4.0) (2025-01-08)
1+
# Changelog
22

3+
## [0.4.0](https://github.com/dkmaker/mcp-rest-api/compare/v0.3.0...v0.4.0) (2025-01-08)
34

45
### Features
56

67
* add custom header support ([9a48e0d](https://github.com/dkmaker/mcp-rest-api/commit/9a48e0d794a743f7a62c7cb73d6f5b1be9e44107))
78

89
## [0.3.0](https://github.com/dkmaker/mcp-rest-api/compare/v0.2.0...v0.3.0) (2024-12-28)
910

10-
1111
### Features
1212

1313
* add config documentation and improve URL resolution examples ([8c6100f](https://github.com/dkmaker/mcp-rest-api/commit/8c6100f47777605a0571edbd161ffd20fc48b640))
1414
* add MCP resources for documentation ([a20cf35](https://github.com/dkmaker/mcp-rest-api/commit/a20cf352e9731841a8d7e833007a96bdd1a0c390))
1515

16-
1716
### Bug Fixes
1817

1918
* correct response truncation to return first N bytes ([ce34649](https://github.com/dkmaker/mcp-rest-api/commit/ce34649c4d8e6bc6d740e8f3fbc6e3df517e0eec))
2019

2120
## [0.2.0](https://github.com/dkmaker/mcp-rest-api/compare/0fdbe844dd4ce8b79f38a33df323a29e28253724...v0.2.0) (2024-12-21)
2221

23-
2422
### Features
2523

2624
* **ssl:** add SSL verification control with secure defaults ([0fdbe84](https://github.com/dkmaker/mcp-rest-api/commit/0fdbe844dd4ce8b79f38a33df323a29e28253724))
27-

0 commit comments

Comments
 (0)