-
Notifications
You must be signed in to change notification settings - Fork 0
chore: public-release audit #45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
fbdc85c
ci: remove Claude workflows and fix Renovate dhi.io lookups
lorr1 f5f90b1
ci: polish release workflow stubs for template consumers
lorr1 f463bae
chore(deps): bump dependencies and pin CVE fixes
lorr1 b5f3123
docs: public-release polish
lorr1 14d28a6
docs: avoid calling template consumers "forks"
lorr1 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,51 @@ | ||
| # Contributing to mcp-template-py | ||
|
|
||
| Thanks for your interest! This repo is a template — most contributions will be improvements to the template itself (new MCP tooling patterns, better docs, CI/CD hardening). | ||
|
|
||
| ## Development setup | ||
|
|
||
| ```bash | ||
| # Prerequisites: Python 3.13+, uv, Task (see README) | ||
| task install | ||
| cp .env.example .env | ||
| task run | ||
| ``` | ||
|
|
||
| ## Before opening a PR | ||
|
|
||
| Run the full check suite locally. It mirrors CI: | ||
|
|
||
| ```bash | ||
| task check # lint + format + typecheck + test + security | ||
| ``` | ||
|
|
||
| Individual tasks are listed in `README.md`. All of them must pass before CI will. | ||
|
|
||
| ## PR conventions | ||
|
|
||
| - **Title format:** [Conventional Commits](https://www.conventionalcommits.org/) — enforced by `lint-pr-title.yml`. Examples: | ||
| - `feat(tools): add a rate-limited search tool` | ||
| - `fix(auth): handle missing Bearer token header` | ||
| - `docs: clarify DHI setup` | ||
| - `chore: bump dependencies` | ||
| - **Scope:** keep PRs focused. Smaller PRs review faster. | ||
| - **Tests:** unit tests in `tests/unit/` (alongside the module), integration tests in `tests/integration/`. | ||
|
|
||
| ## Developer Certificate of Origin (DCO) | ||
|
|
||
| All commits must be signed off, certifying that you have the right to submit the contribution: | ||
|
|
||
| ```bash | ||
| git commit -s -m "your message" | ||
| ``` | ||
|
|
||
| The `-s` flag appends a `Signed-off-by` trailer. See the [DCO](https://developercertificate.org/) for the full text. | ||
|
|
||
| ## Reporting issues | ||
|
|
||
| - **Bugs / feature requests:** open an issue. | ||
| - **Security vulnerabilities:** see [SECURITY.md](SECURITY.md) — do not file a public issue. | ||
|
|
||
| ## Code of Conduct | ||
|
|
||
| By participating in this project, you agree to abide by the [StacklokLabs Code of Conduct](https://github.com/StacklokLabs/.github/blob/main/CODE_OF_CONDUCT.md). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIL: This was the reason for blocking PAT to trigger downstream workflows. 🤗
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep —
GITHUB_TOKENis intentionally barred from triggering downstream workflows to prevent infinite-loop footguns. The GitHub App token dance is the sanctioned escape hatch. 🫠