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

feat(issues): make status_id, priority_id, type_id required on create (#598) #613

Merged
jmiller merged 1 commits from feature/598-required-issue-metadata into main 2026-06-12 02:59:56 +00:00
Conversation 0 Commits 1 Files Changed 2
+11 -12
jmiller commented 2026-06-12 02:50:48 +00:00
Owner
Copy Link
Copy Source

Summary

Makes issue metadata fields required on create, with org default auto-assignment as fallback.

Changes

  • CreateIssueOption: Changed status_id, priority_id, type_id from *int64 (optional) to int64 (required)
  • API handler: uses provided value when > 0, auto-assigns org default when 0
  • MCP gitea_issue_create: fields now required in schema (pass 0 for org default)
  • EditIssueOption: kept as optional *int64 for partial updates

Design Decision

Fields are "required" in the schema but gracefully degrade — passing 0 auto-assigns the org default rather than failing. This ensures MCP tools and scripts always produce issues with proper metadata, without breaking callers that don't yet provide these fields.

Closes #598

## Summary Makes issue metadata fields required on create, with org default auto-assignment as fallback. ## Changes - `CreateIssueOption`: Changed `status_id`, `priority_id`, `type_id` from `*int64` (optional) to `int64` (required) - API handler: uses provided value when > 0, auto-assigns org default when 0 - MCP `gitea_issue_create`: fields now required in schema (pass 0 for org default) - `EditIssueOption`: kept as optional `*int64` for partial updates ## Design Decision Fields are "required" in the schema but gracefully degrade — passing 0 auto-assigns the org default rather than failing. This ensures MCP tools and scripts always produce issues with proper metadata, without breaking callers that don't yet provide these fields. Closes #598
jmiller added 1 commit 2026-06-12 02:50:55 +00:00
feat(issues): make status_id, priority_id, type_id required on create (#598)
Generic: Project CI / Tests (pull_request) Blocked by required conditions
Details
Universal: PR Check / Build RC Package (pull_request) Blocked by required conditions
Details
Universal: PR Check / Report Issues (pull_request) Blocked by required conditions
Details
Branch Policy Check / Verify merge target (pull_request) Failing after 4s
Details
Generic: Repo Health / Scripts governance (pull_request) Blocked by required conditions
Details
Generic: Repo Health / Repository health (pull_request) Blocked by required conditions
Details
Generic: Repo Health / Report Issues (pull_request) Blocked by required conditions
Details
Universal: PR Check / Branch Policy (pull_request) Failing after 6s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 19s
Details
Generic: Project CI / Lint & Validate (pull_request) Successful in 42s
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Generic: Repo Health / Access control (pull_request) Successful in 4s
Details
Generic: Repo Health / Scripts governance (push) Blocked by required conditions
Details
Generic: Repo Health / Repository health (push) Blocked by required conditions
Details
Generic: Repo Health / Report Issues (push) Blocked by required conditions
Details
Generic: Repo Health / Site Health (push) Has been skipped
Details
Generic: Repo Health / Access control (push) Successful in 2s
Details
Branch Cleanup / Delete merged branch (pull_request) Has been skipped
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Details
Universal: Auto Version Bump / Version Bump (push) Successful in 11s
Details
Universal: Build & Release / Promote to RC (pull_request) Has been skipped
Details
Universal: Build & Release / Build & Release Pipeline (pull_request) Has been skipped
Details
Universal: Secret Scanning / Gitleaks Secret Scan (pull_request) Successful in 3m21s
Details
PR RC Release / Build RC Release (pull_request) Failing after 2m55s
Details
cbaca15cda 
- Change CreateIssueOption fields from *int64 (optional) to int64
- API auto-assigns org defaults when value is 0
- MCP gitea_issue_create now requires status_id, priority_id, type_id
  (pass 0 for org default)
- Keep optional on gitea_issue_update (partial updates)

Co-Authored-By: Moko Consulting <hello@mokoconsulting.tech>
jmiller closed this pull request 2026-06-12 02:52:18 +00:00
jmiller deleted branch feature/598-required-issue-metadata 2026-06-12 02:52:21 +00:00
jmiller reopened this pull request 2026-06-12 02:58:52 +00:00
jmiller merged commit 9d45a767e7 into main 2026-06-12 02:59:56 +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
 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 -
Status —
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-APP#613
Delete Branch "feature/598-required-issue-metadata"

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?