feat: map gitea_commit_files in task capability resolver (Closes #262) #276

Merged
sysadmin merged 2 commits from feat/issue-262-commit-files-capability into master 2026-07-06 23:35:31 -05:00
Owner

Closes #262

Summary

Maps commit_files and gitea_commit_files in TASK_CAPABILITY_MAP to gitea.repo.commit, registers gitea_commit_files in ISSUE_MUTATION_TOOL_TASKS, and adds profile/namespace gates on gitea_commit_files consistent with other issue-write tools.

Changes

  • task_capability_map.py — resolver entries for commit_files / gitea_commit_files
  • gitea_mcp_server.py — permission + namespace gates before preflight on gitea_commit_files
  • tests/test_commit_files_capability.py — author allow, reviewer deny, create_pr≠commit, preflight, alignment
  • tests/test_mcp_server.py — update TestCommitFiles for gated path

Validation

  • pytest tests/test_commit_files_capability.py tests/test_issue_write_tool_gates.py -q → 13 passed
  • git diff --check → clean
Closes #262 ## Summary Maps `commit_files` and `gitea_commit_files` in `TASK_CAPABILITY_MAP` to `gitea.repo.commit`, registers `gitea_commit_files` in `ISSUE_MUTATION_TOOL_TASKS`, and adds profile/namespace gates on `gitea_commit_files` consistent with other issue-write tools. ## Changes - `task_capability_map.py` — resolver entries for `commit_files` / `gitea_commit_files` - `gitea_mcp_server.py` — permission + namespace gates before preflight on `gitea_commit_files` - `tests/test_commit_files_capability.py` — author allow, reviewer deny, create_pr≠commit, preflight, alignment - `tests/test_mcp_server.py` — update `TestCommitFiles` for gated path ## Validation - `pytest tests/test_commit_files_capability.py tests/test_issue_write_tool_gates.py -q` → 13 passed - `git diff --check` → clean
jcwalker3 added 1 commit 2026-07-06 13:38:57 -05:00
Add commit_files and gitea_commit_files resolver entries for
gitea.repo.commit, align gitea_commit_files tool gates with other
issue-write mutations, and add regression tests for author allow,
reviewer deny, preflight order, and resolver/tool alignment.

Refs #262

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
jcwalker3 added 1 commit 2026-07-06 13:39:32 -05:00
sysadmin approved these changes 2026-07-06 23:35:19 -05:00
sysadmin left a comment
Owner

Review: APPROVE

PR #276 — feat: map gitea_commit_files in task capability resolver (Closes #262)

Eligibility

  • Reviewer: sysadmin / prgs-reviewer
  • PR author: jcwalker3 (not self-review)
  • Pinned head: 89c9b9a4eca8fe83df21bf9110329613cb3d342f

Scope (#262)

  • task_capability_map.pycommit_files / gitea_commit_filesgitea.repo.commit
  • gitea_mcp_server.py — namespace + permission gates on gitea_commit_files
  • Tests: capability resolver, tool gates, preflight, alignment

Validation (worktree: branches/review-feat-issue-262-commit-files-capability)

pytest tests/test_commit_files_capability.py tests/test_commit_files_gate.py tests/test_issue_write_tool_gates.py -q
→ 18 passed, 7 subtests passed
pytest tests/ -q
→ 996 passed, 6 skipped, 20 subtests passed
git diff --check → clean

Verdict

Scope is clean and limited to #262. Gates fail closed correctly. Tests pass on pinned head.

Controller Handoff

  • Task: Review + merge PR #276
  • Role: reviewer
  • Identity: sysadmin / prgs-reviewer
  • Pinned reviewed head: 89c9b9a4ec
  • Review decision: APPROVE
  • Safety: no self-review; no self-merge contamination
## Review: APPROVE **PR #276** — feat: map gitea_commit_files in task capability resolver (Closes #262) ### Eligibility - Reviewer: `sysadmin` / `prgs-reviewer` - PR author: `jcwalker3` (not self-review) - Pinned head: `89c9b9a4eca8fe83df21bf9110329613cb3d342f` ### Scope (#262) - `task_capability_map.py` — `commit_files` / `gitea_commit_files` → `gitea.repo.commit` - `gitea_mcp_server.py` — namespace + permission gates on `gitea_commit_files` - Tests: capability resolver, tool gates, preflight, alignment ### Validation (worktree: `branches/review-feat-issue-262-commit-files-capability`) ``` pytest tests/test_commit_files_capability.py tests/test_commit_files_gate.py tests/test_issue_write_tool_gates.py -q → 18 passed, 7 subtests passed pytest tests/ -q → 996 passed, 6 skipped, 20 subtests passed git diff --check → clean ``` ### Verdict Scope is clean and limited to #262. Gates fail closed correctly. Tests pass on pinned head. ## Controller Handoff - Task: Review + merge PR #276 - Role: reviewer - Identity: sysadmin / prgs-reviewer - Pinned reviewed head: 89c9b9a4eca8fe83df21bf9110329613cb3d342f - Review decision: APPROVE - Safety: no self-review; no self-merge contamination
sysadmin merged commit 8fc41ccf29 into master 2026-07-06 23:35:31 -05:00
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Scaled-Tech-Consulting/Gitea-Tools#276