MokoConsulting/MokoCLI
Public Access
1
0
Fork 0
Code Issues 3 Pull Requests 2 Packages Releases 8 Wiki Activity

fix(templates): CODEOWNERS use .mokogitea/workflows not .github/workflows #339

Merged
jmiller merged 1 commits from feat/fix-codeowners-mokogitea-path into dev 2026-07-05 20:50:22 +00:00
Conversation 0 Commits 1 Files Changed 2
+27 -30
jmiller commented 2026-07-05 20:50:08 +00:00
Owner
Copy Link
Copy Source

Closes #337.

The synced CODEOWNERS templates deployed to MokoGitea repos pointed at .github/workflows/ (and other .github/ config paths), which never match the real .mokogitea/ files — so code-owner protection on workflows/config was silently a no-op. Also refreshes the stale, consolidated-away workflow filename list.

Changes

  • templates/mokogitea/CODEOWNERS — repoint workflow + config paths .github/ → .mokogitea/; replace obsolete workflow names (deploy-dev, auto-dev-issue, standards-compliance, ci-joomla, update-server, branch-freeze, …) with the current canonical set (auto-release, pre-release, cascade-dev, ci-generic, pr-check, cleanup, branch-cleanup, notify, repo-health, gitleaks, issue-branch, ci-issue-reporter, sync-on-merge, version-set, auto-bump, rc-revert).
  • templates/docs/required/CODEOWNERS — same .github/ → .mokogitea/ repoint (workflows + ISSUE_TEMPLATE).

Base = dev

Retargeted to dev because templates/mokogitea/CODEOWNERS only exists on dev, not main (main's .github/workflows refs are all legitimate — the GitHubAdapter platform code, which must stay). Verified per triage before scoping.

Deliberately NOT changed (documented in #337, not simple path-swaps)

  • templates/docs/required/template-SECURITY.md → codeql-analysis.yml / dependabot.yml are GitHub-only features with no MokoGitea equivalent (path-swap would be wrong; needs a real rewrite to gitleaks/native).
  • templates/schemas/template-repository-structure.xml, templates/required/README.md PLANNED note, and templates/configs/gitignore* *.log ignore rules — lower-impact; left for a focused follow-up.

Separate main-only issue (not this PR)

Stale vendored workflow copies under mcp/servers/{mokossh,mokosuite_api,windows}/.mokogitea/workflows/*.yml self-reference .github/workflows + obsolete names — tracked separately (see #337 discussion).

https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i

Closes #337. The synced CODEOWNERS templates deployed to MokoGitea repos pointed at `.github/workflows/` (and other `.github/` config paths), which never match the real `.mokogitea/` files — so **code-owner protection on workflows/config was silently a no-op**. Also refreshes the stale, consolidated-away workflow filename list. ## Changes - **`templates/mokogitea/CODEOWNERS`** — repoint workflow + config paths `.github/` → `.mokogitea/`; replace obsolete workflow names (`deploy-dev`, `auto-dev-issue`, `standards-compliance`, `ci-joomla`, `update-server`, `branch-freeze`, …) with the current canonical set (`auto-release`, `pre-release`, `cascade-dev`, `ci-generic`, `pr-check`, `cleanup`, `branch-cleanup`, `notify`, `repo-health`, `gitleaks`, `issue-branch`, `ci-issue-reporter`, `sync-on-merge`, `version-set`, `auto-bump`, `rc-revert`). - **`templates/docs/required/CODEOWNERS`** — same `.github/` → `.mokogitea/` repoint (workflows + ISSUE_TEMPLATE). ## Base = `dev` Retargeted to `dev` because `templates/mokogitea/CODEOWNERS` **only exists on `dev`**, not `main` (main's `.github/workflows` refs are all legitimate — the `GitHubAdapter` platform code, which must stay). Verified per triage before scoping. ## Deliberately NOT changed (documented in #337, not simple path-swaps) - `templates/docs/required/template-SECURITY.md` → `codeql-analysis.yml` / `dependabot.yml` are **GitHub-only** features with no MokoGitea equivalent (path-swap would be wrong; needs a real rewrite to gitleaks/native). - `templates/schemas/template-repository-structure.xml`, `templates/required/README.md` PLANNED note, and `templates/configs/gitignore*` `*.log` ignore rules — lower-impact; left for a focused follow-up. ## Separate main-only issue (not this PR) Stale vendored workflow copies under `mcp/servers/{mokossh,mokosuite_api,windows}/.mokogitea/workflows/*.yml` self-reference `.github/workflows` + obsolete names — tracked separately (see #337 discussion). https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
jmiller added 1 commit 2026-07-05 20:50:09 +00:00
fix(templates): CODEOWNERS use .mokogitea/workflows not .github/workflows
Universal: PR Check / Branch Policy (pull_request) Successful in 2s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 9s
Details
Branch Cleanup / Delete merged branch (pull_request) Failing after 2s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 14s
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Details
Platform: mokocli CI / Gate 1: Code Quality (pull_request) Failing after 1m6s
Details
Platform: mokocli CI / Gate 2: Unit Tests (8.1) (pull_request) Has been cancelled
Details
Platform: mokocli CI / Gate 2: Unit Tests (8.2) (pull_request) Has been cancelled
Details
Platform: mokocli CI / Gate 2: Unit Tests (8.3) (pull_request) Has been cancelled
Details
Platform: mokocli CI / Gate 3: Self-Health Check (pull_request) Has been cancelled
Details
Platform: mokocli CI / Gate 4: Governance (pull_request) Has been cancelled
Details
Platform: mokocli CI / Gate 5: Template Integrity (pull_request) Has been cancelled
Details
Platform: mokocli CI / CI Summary (pull_request) Has been cancelled
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Universal: PR Check / Report Issues (pull_request) Has been cancelled
Details
d4947a7a80 
The synced CODEOWNERS templates deployed to MokoGitea repos pointed at
`.github/workflows/` (and other `.github/` config paths), which never
match the real `.mokogitea/` files — so code-owner protection on
workflows/config was silently a no-op. Also refreshes the stale,
consolidated-away workflow filename list to the current canonical set.

- templates/mokogitea/CODEOWNERS: repoint workflow + config paths to
  .mokogitea/; replace obsolete workflow names (deploy-dev, auto-dev-issue,
  standards-compliance, ci-joomla, update-server, branch-freeze, …) with
  the current set (auto-release, pre-release, cascade-dev, ci-generic,
  pr-check, cleanup, branch-cleanup, notify, repo-health, gitleaks,
  issue-branch, ci-issue-reporter, sync-on-merge, version-set, …).
- templates/docs/required/CODEOWNERS: same .github -> .mokogitea repoint.

Deliberately NOT changed (not simple path-swaps): template-SECURITY.md's
codeql-analysis / dependabot refs (GitHub-only features with no MokoGitea
equivalent), and the schema/gitignore/doc-example refs — see #337.

Closes #337

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
jmiller merged commit ff91bd89a0 into dev 2026-07-05 20:50:22 +00:00
jmiller deleted branch feat/fix-codeowners-mokogitea-path 2026-07-05 20:50:24 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
automation
Automated processes or scripts
 breaking-change
Breaking API or functionality change
 build
Build system changes
 ci-cd
CI/CD pipeline changes
 config
Configuration file changes
 css
CSS/styling changes
 dependencies
Dependency updates
 deploy-failure
Automated deploy failure tracking
 docker
Docker configuration changes
 documentation
Documentation changes
 dolibarr
Dolibarr module or extension
 generic
Generic project or library
 good first issue
Good for newcomers
 health-check
Repository health check results
 health: excellent
Health score 90-100
 health: fair
Health score 50-69
 health: good
Health score 70-89
 health: poor
Health score below 50
 help wanted
Extra attention needed
 html
HTML template changes
 javascript
JavaScript code changes
 joomla
Joomla extension or component
 mokostandards
MokoStandards compliance
 needs-review
Awaiting code review
 php
PHP code changes
 push-failure
File push failure requiring attention
 python
Python code changes
 security
Security-related changes
 size/l
Large change (101-300 lines)
 size/m
Medium change (31-100 lines)
 size/s
Small change (11-30 lines)
 size/xl
Extra large change (301-1000 lines)
 size/xs
Extra small change (1-10 lines)
 size/xxl
Extremely large change (1000+ lines)
 standards-drift
Repository drifted from MokoStandards
 standards-update
MokoStandards sync update
 sync-failure
Bulk sync failure requiring attention
 sync-report
Bulk sync run report
 template-validation-failure
Template workflow validation failure
 tests
Test suite changes
 type: bug
Something isn't working
 type: chore
Maintenance tasks
 type: enhancement
Enhancement to existing feature
 type: feature
New feature or request
 type: refactor
Code refactoring
 type: version
Version-related change
 typescript
TypeScript code changes
 version
Version bump or release
 version-drift
Version mismatch detected
 work-in-progress
Work in progress, not ready for merge
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Priority -
Type -
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoCLI#339
Delete Branch "feat/fix-codeowners-mokogitea-path"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?