claude-git-hooks 2.30.1 → 2.30.2

package/CHANGELOG.md

@@ -5,6 +5,16 @@ Todos los cambios notables en este proyecto se documentarán en este archivo.
 El formato está basado en [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 y este proyecto adhiere a [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.30.2] - 2026-03-18
+
+### 🔧 Changed
+- Authorization now uses GitHub's `role_name` field instead of legacy `permission` field for more accurate role mapping
+- Improved authorization error messages to show the actual GitHub permission that failed to map
+
+### 🐛 Fixed
+- Fixed role mapping to use `write` instead of `push` to match GitHub's current API role names
+
+
 ## [2.30.1] - 2026-03-17
 
 ### ✨ Added

package/bin/claude-hooks

@@ -10,6 +10,8 @@
 
 import { error } from '../lib/commands/helpers.js';
 import { buildCommandMap } from '../lib/cli-metadata.js';
+import logger from '../lib/utils/logger.js';
+import { getConfig } from '../lib/config.js';
 
 /**
  * Main CLI router
@@ -18,6 +20,16 @@ async function main() {
     const args = process.argv.slice(2);
     const command = args[0];
 
+    // Activate debug mode early so authorization logging is visible
+    try {
+        const config = await getConfig();
+        if (config.system?.debug) {
+            logger.setDebugMode(true);
+        }
+    } catch {
+        // Config load failure is non-fatal for debug activation
+    }
+
     const commandMap = buildCommandMap();
     const entry = commandMap.get(command);
 
@@ -92,8 +104,8 @@ async function main() {
                     break;
                 case 'permission_unmapped':
                     error(
-                        '⛔ Authorization failed: your GitHub permission level is not mapped to any role.\n' +
-                                `   Contact your Tech Lead to update ${PERMISSIONS_SOURCE}`
+                        `⛔ Authorization failed: your GitHub permission '${err.githubPermission}' is not mapped to any role.\n` +
+                                `   Contact your Tech Lead to update roleMapping in ${PERMISSIONS_SOURCE}`
                     );
                     break;
                 case 'insufficient_role': {

package/lib/utils/github-api.js

@@ -1046,7 +1046,7 @@ export const checkOrgMembership = async (username, org) => {
  * @param {string} owner - Repository owner
  * @param {string} repo - Repository name
  * @param {string} username - GitHub username
- * @returns {Promise<string>} Permission level: 'admin', 'maintain', 'push', 'triage', 'read', 'none'
+ * @returns {Promise<string>} Role name: 'admin', 'maintain', 'write', 'triage', 'read', 'none'
  * @throws {GitHubAPIError} On API failure (fail-closed — caller should block on error)
  */
 export const getCollaboratorPermission = async (owner, repo, username) => {
@@ -1065,17 +1065,17 @@ export const getCollaboratorPermission = async (owner, repo, username) => {
             username
         });
 
-        const permission = data.permission;
+        const roleName = data.role_name;
 
         logger.debug('github-api - getCollaboratorPermission', 'Permission resolved', {
             owner,
             repo,
             username,
-            permission,
-            roleNameFromApi: data.role_name
+            roleName,
+            legacyPermission: data.permission
         });
 
-        return permission;
+        return roleName;
     } catch (error) {
         const statusCode = error.status || error.response?.status;
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-git-hooks",
-    "version": "2.30.1",
+    "version": "2.30.2",
     "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
     "type": "module",
     "bin": {