MokoConsulting/MokoGitea-Fork
1
0
Fork 0
Code Issues 124 Pull Requests 1 Releases 96 Wiki Activity

fix: pr-check platform detection queries metadata API #715

Merged
jmiller merged 3 commits from fix/pr-check-platform-detection into main 2026-06-28 11:10:43 +00:00
Conversation 0 Commits 3 Files Changed 3
+26 -11
jmiller commented 2026-06-28 10:59:40 +00:00
Owner
Copy Link
Copy Source

Summary

  • PR check "Detect platform" step was reading from .mokogitea/manifest.xml which no longer exists
  • Platform is now stored in the MokoGitea metadata API
  • The old sed command failed with exit 2 under bash -e, cascading failure to all subsequent validate steps (PHP checks, changelog check, etc.)
  • Now queries GET /api/v1/repos/{owner}/{repo}/metadata for the platform field
  • Falls back to generic if no token or API unavailable

Test plan

  • PR check passes on this PR (platform detected as go or generic)
  • Joomla repos still detect joomla platform correctly

https://claude.ai/code/session_011AAFzotGMf3ayvXhEmStCd

## Summary - PR check "Detect platform" step was reading from `.mokogitea/manifest.xml` which no longer exists - Platform is now stored in the MokoGitea metadata API - The old `sed` command failed with exit 2 under `bash -e`, cascading failure to all subsequent validate steps (PHP checks, changelog check, etc.) - Now queries `GET /api/v1/repos/{owner}/{repo}/metadata` for the `platform` field - Falls back to `generic` if no token or API unavailable ## Test plan - [ ] PR check passes on this PR (platform detected as `go` or `generic`) - [ ] Joomla repos still detect `joomla` platform correctly https://claude.ai/code/session_011AAFzotGMf3ayvXhEmStCd
jmiller added 1 commit 2026-06-28 10:59:41 +00:00
fix: pr-check platform detection queries metadata API instead of removed manifest.xml
Universal: PR Check / Branch Policy (pull_request) Failing after 2s
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Generic: Repo Health / Access control (pull_request) Successful in 2s
Details
Universal: PR Check / Validate PR (pull_request) Successful in 14s
Details
Generic: Project CI / Lint & Validate (pull_request) Successful in 40s
Details
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Failing after 1m24s
Details
PR RC Release / Build RC Release (pull_request) Failing after 3m14s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 3m15s
Details
Generic: Project CI / Tests (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
Generic: Repo Health / Scripts governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Repository health (pull_request) Has been cancelled
Details
Generic: Repo Health / Report: Scripts Governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Report: Repository Health (pull_request) Has been cancelled
Details
fa845164bb 
The manifest.xml file no longer exists — platform is stored in the
MokoGitea metadata API. The old sed command failed with exit 2 under
bash -e, cascading failure to all subsequent validate steps.

Claude-Session: https://claude.ai/code/session_011AAFzotGMf3ayvXhEmStCd
jmiller added 1 commit 2026-06-28 11:01:13 +00:00
fix: branch policy allows fix/patch branches to target main
Universal: PR Check / Branch Policy (pull_request) Successful in 2s
Details
Universal: PR Check / Validate PR (pull_request) Successful in 18s
Details
Generic: Project CI / Lint & Validate (pull_request) Successful in 1m4s
Details
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Failing after 1m32s
Details
PR RC Release / Build RC Release (pull_request) Failing after 1m33s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 1m32s
Details
Generic: Project CI / Tests (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
193c705c05 
The pr-check branch policy only allowed fix/* -> dev, but the actual
org policy is fix/patch branches PR to main directly. Also updated
the summary text to list all allowed merge paths correctly.

Claude-Session: https://claude.ai/code/session_011AAFzotGMf3ayvXhEmStCd
jmiller added 1 commit 2026-06-28 11:02:50 +00:00
docs: update README with new features from latest release
Universal: PR Check / Branch Policy (pull_request) Successful in 2s
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Generic: Repo Health / Access control (pull_request) Successful in 2s
Details
Universal: PR Check / Validate PR (pull_request) Successful in 13s
Details
Generic: Project CI / Lint & Validate (pull_request) Successful in 35s
Details
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Failing after 1m24s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 1m30s
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Details
Branch Cleanup / Delete merged branch (pull_request) Successful in 2s
Details
Universal: Workflow Sync Trigger / Sync workflows to live repos (pull_request) Successful in 6m10s
Details
Generic: Project CI / Tests (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
Generic: Repo Health / Scripts governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Repository health (pull_request) Has been cancelled
Details
Generic: Repo Health / Report: Scripts Governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Report: Repository Health (pull_request) Has been cancelled
Details
35ebbef489 
Add cascade merge, secret scanning, default org teams, branch
protection delete allowlist to Key Features list.

Claude-Session: https://claude.ai/code/session_011AAFzotGMf3ayvXhEmStCd
jmiller merged commit 37a62b5ab7 into main 2026-06-28 11:10:43 +00:00
jmiller deleted branch fix/pr-check-platform-detection 2026-06-28 11:10:44 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
breaking-change
Breaking API or behavior change
 ci-cd
CI/CD pipeline changes
 config
Configuration changes
 dependencies
Dependency updates
 deploy-failure
Deployment failed
 docker
Docker/container changes
 documentation
Documentation changes
 feature: wiki
Wiki system enhancements
 good first issue
Good for newcomers
 health-check
Repo health check result
 help wanted
Extra attention needed
 mokostandards
Related to MokoStandards framework
 push-failure
Git push operation failed
 security
Security vulnerability or hardening
 size/l
200-500 lines changed
 size/m
50-200 lines changed
 size/s
10-50 lines changed
 size/xl
500+ lines changed
 size/xs
< 10 lines changed
 standards-drift
Deviates from MokoStandards
 standards-update
MokoStandards compliance update
 sync-failure
Sync or mirror failed
 tech-debt
Technical debt and TODO/FIXME items
 upstream
upstream
Inherited from upstream Gitea
 work-in-progress
Draft or incomplete work
No labels
Priority -
Type -
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller) moko-deploy (Moko Deploy Bot)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/MokoGitea-Fork#715
Delete Branch "fix/pr-check-platform-detection"

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?