refactor the parser

services/actions/permission_parser.go

@@ -92,6 +92,8 @@ func parseRawPermissions(rawPerms *yaml.Node, defaultPerms repo_model.ActionsTok
 	if node.Kind == yaml.MappingNode {
 		result := defaultPerms // Start with defaults
 
+		// Collect all scopes into a map first to handle priority
+		scopes := make(map[string]perm.AccessMode)
 		for i := 0; i < len(node.Content); i += 2 {
 			if i+1 >= len(node.Content) {
 				break
@@ -103,34 +105,36 @@ func parseRawPermissions(rawPerms *yaml.Node, defaultPerms repo_model.ActionsTok
 				continue
 			}
 
-			scope := keyNode.Value
+			scopes[keyNode.Value] = parseAccessMode(valueNode.Value)
-			accessStr := valueNode.Value
+		}
-			accessMode := parseAccessMode(accessStr)
 
-			// Map GitHub Actions scopes to Gitea units
+		// 1. Apply 'contents' first (lower priority)
+		if mode, ok := scopes["contents"]; ok {
+			result.Code = mode
+			result.Releases = mode
+		}
+
+		// 2. Apply all other scopes (overwrites contents if specified)
+		for scope, mode := range scopes {
 			switch scope {
 			case "contents":
-				result.Code = accessMode
+				// already handled
-				result.Releases = accessMode
 			case "code":
-				result.Code = accessMode
+				result.Code = mode
 			case "issues":
-				result.Issues = accessMode
+				result.Issues = mode
 			case "pull-requests":
-				result.PullRequests = accessMode
+				result.PullRequests = mode
 			case "packages":
-				result.Packages = accessMode
+				result.Packages = mode
 			case "actions":
-				result.Actions = accessMode
+				result.Actions = mode
 			case "wiki":
-				result.Wiki = accessMode
+				result.Wiki = mode
 			case "releases":
-				result.Releases = accessMode
+				result.Releases = mode
 			case "projects":
-				result.Projects = accessMode
+				result.Projects = mode
-				// Additional GitHub scopes we don't explicitly handle yet:
-				// These fall through to defaults
-				// - deployments, environments, id-token, pages, repository-projects, security-events, statuses
 			}
 		}
 

services/actions/permission_parser_test.go

@@ -99,6 +99,44 @@ issues: write
 	assert.Equal(t, perm.AccessModeWrite, result.Wiki)
 }
 
+func TestParseRawPermissions_Priority(t *testing.T) {
+	t.Run("granular-wins-over-contents", func(t *testing.T) {
+		yamlContent := `
+contents: read
+code: write
+releases: none
+`
+		var rawPerms yaml.Node
+		err := yaml.Unmarshal([]byte(yamlContent), &rawPerms)
+		assert.NoError(t, err)
+
+		defaultPerms := repo_model.ActionsTokenPermissions{}
+		result := parseRawPermissions(&rawPerms, defaultPerms)
+
+		assert.Equal(t, perm.AccessModeWrite, result.Code)
+		assert.Equal(t, perm.AccessModeNone, result.Releases)
+	})
+
+	t.Run("contents-applied-first", func(t *testing.T) {
+		yamlContent := `
+code: none
+releases: write
+contents: read
+`
+		var rawPerms yaml.Node
+		err := yaml.Unmarshal([]byte(yamlContent), &rawPerms)
+		assert.NoError(t, err)
+
+		defaultPerms := repo_model.ActionsTokenPermissions{}
+		result := parseRawPermissions(&rawPerms, defaultPerms)
+
+		// code: none should win over contents: read
+		assert.Equal(t, perm.AccessModeNone, result.Code)
+		// releases: write should win over contents: read
+		assert.Equal(t, perm.AccessModeWrite, result.Releases)
+	})
+}
+
 func TestParseRawPermissions_EmptyNode(t *testing.T) {
 	var rawPerms yaml.Node
 	// Empty node