188 lines
4.4 KiB
Plaintext
188 lines
4.4 KiB
Plaintext
# CODEOWNERS - Mandatory Review Policy
|
|
|
|
# Default reviewer for all repositories
|
|
* @perplexity
|
|
|
|
# Special reviewers for specific repos
|
|
hermes-agent/ @Timmy
|
|
hermes-agent/agent-core/ @Rockachopa
|
|
the-nexus/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
timmy-home/ @perplexity
|
|
timmy-config/ @perplexity
|
|
|
|
# QA reviewer for all PRs
|
|
* @perplexity
|
|
|
|
# Owner gates
|
|
hermes-agent/ @Timmy
|
|
|
|
# QA reviewer for all PRs
|
|
* @perplexity
|
|
|
|
# Specialized component owners
|
|
# Required reviewer for hermes-agent (owner gate)
|
|
hermes-agent/ @Timmy
|
|
hermes-agent/ @Timmy
|
|
hermes-agent/protocol/ @Timmy
|
|
the-nexus/portals/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
|
|
# Specialized component owners
|
|
hermes-agent/agent-core/ @Rockachopa
|
|
hermes-agent/protocol/ @Timmy
|
|
the-nexus/portals/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
>>>>>>> replace
|
|
```
|
|
|
|
CONTRIBUTING.md
|
|
```diff
|
|
<<<<<<< search
|
|
# Contribution & Code Review Policy
|
|
|
|
## Branch Protection Rules (Enforced via Gitea)
|
|
All repositories must have the following branch protection rules enabled on the `main` branch:
|
|
|
|
| Rule | Status | Applies To |
|
|
|------|--------|------------|
|
|
| Require Pull Request for merge | ✅ Enabled | All |
|
|
| Required approvals | ✅ 1+ required | All |
|
|
| Dismiss stale approvals on new commits | ✅ Enabled | All |
|
|
| Require CI to pass (where CI exists) | ⚠ Conditional | All |
|
|
| Block force pushes to `main` | ✅ Enabled | All |
|
|
| Block deletion of `main` branch | ✅ Enabled | All |
|
|
|
|
## Default Reviewer Assignments
|
|
|
|
| Repository | Required Reviewers |
|
|
|------------|------------------|
|
|
| `hermes-agent` | `@perplexity`, `@Timmy` |
|
|
| `the-nexus` | `@perplexity` |
|
|
| `timmy-home` | `@perplexity` |
|
|
| `timmy-config` | `@perplexity` |
|
|
|
|
## CI Enforcement Status
|
|
|
|
| Repository | CI Status |
|
|
|------------|-----------|
|
|
| `hermes-agent` | ✅ Active |
|
|
| `the-nexus` | ⚠ CI runner pending (#915) |
|
|
| `timmy-home` | ❌ No CI |
|
|
| `timmy-config` | ❌ Limited CI |
|
|
|
|
## Review Workflow
|
|
|
|
### For All Contributors
|
|
- Create feature branches from `main`
|
|
- Open PR with clear description
|
|
- `@perplexity` will be automatically assigned as reviewer
|
|
- For `hermes-agent`: `@Timmy` must review critical changes
|
|
|
|
### For Maintainers
|
|
- Review all PRs within 24h
|
|
- Require at least 1 approval before merge
|
|
- Dismiss stale approvals on new commits
|
|
|
|
## Enforcement
|
|
- Direct pushes to main: ❌ Prohibited
|
|
- Unreviewed merges: ❌ Prohibited
|
|
- Failing CI merges: ❌ Prohibited
|
|
|
|
## Exceptions
|
|
Emergency hotfixes require:
|
|
- `@Timmy` approval
|
|
- Post-merge documentation
|
|
- Follow-up PR for full review
|
|
|
|
## Policy Enforcement
|
|
This document is the source of truth for:
|
|
- Branch protection settings
|
|
- Reviewer assignments
|
|
- Merge requirements
|
|
|
|
See Gitea admin settings for each repo to verify protection rules are enabled.
|
|
|
|
# Default reviewer for all repositories
|
|
* @perplexity
|
|
|
|
# Specialized component owners
|
|
hermes-agent/ @Timmy
|
|
hermes-agent/agent-core/ @Rockachopa
|
|
hermes-agent/protocol/ @Timmy
|
|
the-nexus/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
timmy-home/ @perplexity
|
|
timmy-config/ @perplexity
|
|
>>>>>>> replace
|
|
````
|
|
|
|
---
|
|
|
|
### ✅ Updated `README.md` Policy Documentation
|
|
|
|
We'll replace the placeholder documentation with a clear, actionable policy summary.
|
|
|
|
`README.md`
|
|
````
|
|
<<<<<<< search
|
|
|
|
# Specialized component owners
|
|
hermes-agent/ @Timmy
|
|
hermes-agent/protocol/ @Timmy
|
|
the-nexus/portals/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
|
|
# Specialized component owners
|
|
hermes-agent/agent-core/ @Rockachopa
|
|
hermes-agent/protocol/ @Timmy
|
|
the-nexus/portals/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
# CODEOWNERS - Mandatory Review Policy
|
|
|
|
# Default reviewer for all repositories
|
|
* @perplexity
|
|
|
|
# Specialized component owners
|
|
hermes-agent/ @Timmy
|
|
hermes-agent/protocol/ @Timmy
|
|
the-nexus/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
timmy-home/ @perplexity
|
|
timmy-config/ @perplexity
|
|
|
|
# Owner gates
|
|
hermes-agent/ @Timmy
|
|
# CODEOWNERS - Mandatory Review Policy
|
|
|
|
# Default reviewer for all repositories
|
|
* @perplexity
|
|
|
|
# Specialized component owners
|
|
hermes-agent/ @Timmy
|
|
hermes-agent/agent-core/ @Rockachopa
|
|
hermes-agent/protocol/ @Timmy
|
|
the-nexus/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
timmy-home/ @perplexity
|
|
timmy-config/ @perplexity
|
|
|
|
# Owner gates
|
|
hermes-agent/ @Timmy
|
|
# CODEOWNERS - Mandatory Review Policy
|
|
|
|
# Default reviewer for all repositories
|
|
* @perplexity
|
|
|
|
# Specialized component owners
|
|
hermes-agent/ @Timmy
|
|
hermes-agent/agent-core/ @Rockachopa
|
|
hermes-agent/protocol/ @Timmy
|
|
the-nexus/ @perplexity
|
|
the-nexus/ai/ @Timmy
|
|
timmy-home/ @perplexity
|
|
timmy-config/ @perplexity
|
|
|
|
# Owner gates
|
|
hermes-agent/ @Timmy
|