@majiayu000/dart-contribute
BDART contribution workflow - branching, PRs, code review, dual-PR for bugfixes
Install
agr install @majiayu000/dart-contribute --target claudeWrites 2 files into .claude/skills/, pinned to git-6d20700e.
- .claude/skills/dart-contribute/SKILL.md
- .claude/skills/dart-contribute/metadata.json
Document
name: dart-contribute description: DART contribution workflow - branching, PRs, code review, dual-PR for bugfixes
DART Contribution Workflow
Load this skill when contributing code to DART.
Full Documentation
For complete guide: docs/onboarding/contributing.md
For code style: docs/onboarding/code-style.md
Branch Naming
feature/<topic>- New featuresfix/<topic>- Bug fixesrefactor/<topic>- Refactoringdocs/<topic>- Documentation
PR Workflow
# Create branch
git checkout -b <type>/<topic> origin/main
# Make changes, then
pixi run lint
pixi run test-all
# Push and create PR
git push -u origin HEAD
gh pr create --draft --milestone "DART 7.0"
Rule of thumb: run pixi run lint before committing so auto-fixes are included.
Use .github/PULL_REQUEST_TEMPLATE.md and ensure the PR description covers Summary, Motivation, Changes, Testing, Breaking Changes, and Related Issues.
Milestones (Required)
Always set a milestone when creating PRs:
| Target Branch | Milestone |
|---|---|
main | DART 7.0 (or next major) |
release-6.16 | DART 6.16.x (current patch) |
# Set milestone on existing PR
gh pr edit <PR#> --milestone "DART 7.0"
# List available milestones
gh api repos/dartsim/dart/milestones --jq '.[] | .title'
CRITICAL: Bug Fix Dual-PR
Bug fixes require PRs to BOTH branches:
release-6.16- Current releasemain- Next release
Steps:
- Fix on
release-6.16first - Cherry-pick to
main - Create separate PRs for each
CHANGELOG (After PR Created)
After creating a PR, check if CHANGELOG.md needs updating:
| Change Type | Update CHANGELOG? |
|---|---|
| Bug fixes | ✅ Yes |
| New features | ✅ Yes |
| Breaking changes | ✅ Yes (in Breaking Changes section) |
| Documentation improvements | ✅ Yes (in Tooling and Docs) |
| CI/tooling changes | ✅ Yes (in Tooling and Docs) |
| Refactoring (no behavior change) | ⚠️ Maybe (if significant) |
| Dependency bumps | ⚠️ Maybe (if user-facing) |
| Typo fixes | ❌ No |
Format: - Description. ([#PR](https://github.com/dartsim/dart/pull/PR))
# Example entry in CHANGELOG.md under appropriate section:
- Added AI-native documentation with AGENTS.md and module-specific guides. ([#2446](https://github.com/dartsim/dart/pull/2446))
Code Review
- Address all feedback
- Keep changes minimal
- Update tests if behavior changed
- Run full validation before pushing fixes
CI Loop
gh run watch <RUN_ID> --interval 30
Fix failures until green.
Repository README
Describes majiayu000/claude-skill-registry-data as a whole, which may contain artifacts other than this one. Where this artifact had no useful description of its own, its summary was taken from here.
Claude Skill Registry (Data)
This repo contains the archived skill contents (the heavy, browsable skill files).
Canonical layout
- Category folders at repo root (e.g.
development/,documents/,data/, ...) - Each skill lives under a category:
<category>/<skill>/SKILL.md+<category>/<skill>/metadata.json - Case conflicts are resolved with
{name}-{owner}-{repo}suffixes (fallback:-{short-hash}).
Archive status
- Live badges above are sourced from
claude-skill-registry-corestats.json. - Counts in this README are intentionally dynamic, not hardcoded.
- If the badges look stale, refresh the
corebuild/index pipeline rather than editing numbers here.
Where the index + site live
- Core repo: https://github.com/majiayu000/claude-skill-registry-core
- Main repo (merged publish artifact): https://github.com/majiayu000/claude-skill-registry
Trustgrade B
- passBody integrity
Whether the stored document is plausibly the kind of file the artifact declares, rather than something fetched by mistake.
- passType matchnot applicable to this artifact type
Whether the artifact is really the kind of thing its metadata claims it is.
- passFreshness
How long since the source repository was last pushed to.
- passPrompt injection
Scans the artifact's own text for instructions aimed at your agent rather than at you.
- warnLicenseno SPDX license detected
Whether the source repository declares an SPDX license permissive enough to redistribute.
How the grade is calculated
Each check contributes 0 points when it passes, 1 when it warns, and 2 when it fails. The total maps to a letter:
- Aevery check passed
- Bone warning
- Ctwo warnings
- Dprompt injection or body integrity failed, or three warnings
- Fone of those failed, and something else is wrong
These are automated hygiene checks, not a security audit, and not a dependency or vulnerability scan. A grade of A means nothing was flagged — not that the artifact is safe.
Versions
git-6d20700eac972026-07-31