Scaled-Tech-Consulting/Gitea-Tools
2
0
Fork 0
Code Issues 77 Pull Requests 10 Actions Packages Projects Releases Wiki Activity

feat: allow gitea.issue.comment for author/reviewer profiles (#137) #138

Merged
sysadmin merged 1 commits from feat/issue-137-allow-issue-comments into master 2026-07-05 01:14:38 -05:00
Conversation 1 Commits 1 Files Changed 5
+12 -4
5 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
docs/gitea-execution-profiles.md
+1 -1
View File
@@ -84,7 +84,7 @@ boundaries; they are the model, not a runtime enforcement mechanism yet.
### `gitea-reviewer` ### `gitea-reviewer`
- **allowed:** `read`, `pr.comment`, `pr.review`, `pr.approve`, `pr.request_changes` - **allowed:** `read`, `pr.comment`, `pr.review`, `pr.approve`, `pr.request_changes`, `issue.comment`
- **forbidden:** `pr.merge`, `branch.push` - **forbidden:** `pr.merge`, `branch.push`
- `can_approve_prs`: `true` - `can_approve_prs`: `true`
- `can_merge_prs`: `false` - `can_merge_prs`: `false`
gitea-mcp.example.json
+1 -1
View File
@@ -30,7 +30,7 @@
}, },
"default_owner": "MDCPS", "default_owner": "MDCPS",
"execution_profile": "mdcps-reviewer", "execution_profile": "mdcps-reviewer",
"allowed_operations": ["read", "review", "approve", "merge"], "allowed_operations": ["read", "review", "approve", "merge", "issue.comment"],
"forbidden_operations": ["branch.push", "pr.create"] "forbidden_operations": ["branch.push", "pr.create"]
}, },
"prgs-env": { "prgs-env": {
gitea-mcp.v2-contexts.example.json
+2 -2
View File
@@ -41,7 +41,7 @@
"execution_profile": "example-author", "execution_profile": "example-author",
"audit_label": "example-author", "audit_label": "example-author",
"auth": { "type": "keychain", "id": "example-gitea-author-token" }, "auth": { "type": "keychain", "id": "example-gitea-author-token" },
"allowed_operations": ["read", "branch", "commit", "push", "open_pr", "comment"], "allowed_operations": ["read", "branch", "commit", "push", "open_pr", "comment", "issue.comment"],
"forbidden_operations": ["approve", "request_changes", "merge"] "forbidden_operations": ["approve", "request_changes", "merge"]
}, },
"example-reviewer": { "example-reviewer": {
@@ -52,7 +52,7 @@
"execution_profile": "example-reviewer", "execution_profile": "example-reviewer",
"audit_label": "example-reviewer", "audit_label": "example-reviewer",
"auth": { "type": "keychain", "id": "example-gitea-reviewer-token" }, "auth": { "type": "keychain", "id": "example-gitea-reviewer-token" },
"allowed_operations": ["read", "review", "comment", "approve", "request_changes", "merge"], "allowed_operations": ["read", "review", "comment", "issue.comment", "approve", "request_changes", "merge"],
"forbidden_operations": ["branch", "commit", "push", "open_pr"] "forbidden_operations": ["branch", "commit", "push", "open_pr"]
} }
}, },
gitea_config.py
+2
View File
@@ -80,6 +80,8 @@ GITEA_OPERATION_ALIASES = {
"read": "gitea.read", "read": "gitea.read",
"review": "gitea.pr.review", "review": "gitea.pr.review",
"comment": "gitea.pr.comment", "comment": "gitea.pr.comment",
"issue.comment": "gitea.issue.comment",
"issue_comment": "gitea.issue.comment",
"approve": "gitea.pr.approve", "approve": "gitea.pr.approve",
"request_changes": "gitea.pr.request_changes", "request_changes": "gitea.pr.request_changes",
"merge": "gitea.pr.merge", "merge": "gitea.pr.merge",
tests/test_op_normalization.py
+6
View File
@@ -47,6 +47,8 @@ class TestNormalizeOperation(unittest.TestCase):
"request_changes": "gitea.pr.request_changes", "request_changes": "gitea.pr.request_changes",
"review": "gitea.pr.review", "review": "gitea.pr.review",
"comment": "gitea.pr.comment", "comment": "gitea.pr.comment",
"issue.comment": "gitea.issue.comment",
"issue_comment": "gitea.issue.comment",
"read": "gitea.read", "read": "gitea.read",
} }
for legacy, canonical in expected.items(): for legacy, canonical in expected.items():
@@ -58,6 +60,10 @@ class TestNormalizeOperation(unittest.TestCase):
self.assertEqual(normalize_operation("push"), "gitea.branch.push") self.assertEqual(normalize_operation("push"), "gitea.branch.push")
self.assertEqual(normalize_operation("open_pr"), "gitea.pr.create") self.assertEqual(normalize_operation("open_pr"), "gitea.pr.create")
def test_issue_comment_alias(self):
self.assertEqual(normalize_operation("issue.comment"), "gitea.issue.comment")
self.assertEqual(normalize_operation("issue_comment"), "gitea.issue.comment")
def test_unknown_unqualified_op_fails_closed(self): def test_unknown_unqualified_op_fails_closed(self):
with self.assertRaises(ConfigError): with self.assertRaises(ConfigError):
normalize_operation("frobnicate") normalize_operation("frobnicate")