Skip to content

Conversation

@kevintang513
Copy link

Summary

  • Fixed permissions issue in /commit-and-pr command
  • Simplified allowed-tools configuration to resolve permission errors
  • Command now has proper Bash access to execute git, gh, and other required commands

Changes

  • Modified .claude/commands/commit-and-pr.md
  • Changed allowed-tools from pattern-based permissions Bash(git:*), Bash(gh:*) to general Bash access
  • This resolves permission errors that were preventing the command from executing git and gh operations

Test Plan

  1. Run /commit-and-pr command in Claude Code
  2. Verify command can successfully:
    • Create a new branch
    • Stage and commit changes
    • Push to remote
    • Create pull request using gh CLI
  3. Confirm no permission errors occur during execution

🤖 Generated with Claude Code

devin-ai-integration bot and others added 13 commits September 16, 2025 08:10
- Extract notification badge logic from NavBar into standalone NotificationBadge component
- Add comprehensive Cypress component tests covering edge cases (negative numbers, large numbers, undefined values)
- Preserve existing data-test selector for backward compatibility with E2E tests
- Use @mui/material to match existing project dependencies
- Remove customBadge styling from NavBar component
- All 9 component tests passing including Material-UI Badge behavior verification

Co-Authored-By: [email protected] <[email protected]>
Extract NotificationBadge component and add comprehensive tests
…58009691

Revert "Extract NotificationBadge component and add comprehensive tests"
This reverts commit 531adfb.
Simplified allowed-tools from pattern-based permissions to general Bash access to resolve permission issues when running the command.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ devin-ai-integration[bot]
❌ kevintang513
You have signed the CLA already but the status is still pending? Let us recheck it.

@cypress-app-bot
Copy link

@kevintang513 kevintang513 deleted the fix/commit-and-pr-permissions branch January 18, 2026 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants