From 780a1246febba2a3fb3d28f0d4db921f621b1815 Mon Sep 17 00:00:00 2001 From: utkarsh patrikar Date: Thu, 14 May 2026 13:46:25 +0530 Subject: [PATCH 1/3] fix: use GH_PAT in publish workflow to bypass branch protection --- .github/workflows/publish.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml index 1ff2f87..9b3f2b7 100644 --- a/.github/workflows/publish.yml +++ b/.github/workflows/publish.yml @@ -22,6 +22,8 @@ jobs: steps: - name: Checkout code uses: actions/checkout@v4 + with: + token: ${{ secrets.GH_PAT }} - name: Setup Node.js uses: actions/setup-node@v4 From 1b40e17936e86cdf92eb7f41b03d53d06db132be Mon Sep 17 00:00:00 2001 From: utkarsh patrikar Date: Thu, 14 May 2026 13:47:26 +0530 Subject: [PATCH 2/3] chore: remove publish fix from interactive config PR --- .github/workflows/publish.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml index 9b3f2b7..1ff2f87 100644 --- a/.github/workflows/publish.yml +++ b/.github/workflows/publish.yml @@ -22,8 +22,6 @@ jobs: steps: - name: Checkout code uses: actions/checkout@v4 - with: - token: ${{ secrets.GH_PAT }} - name: Setup Node.js uses: actions/setup-node@v4 From 008b5e530676460a230001bd52b10f079ab07272 Mon Sep 17 00:00:00 2001 From: utkarsh patrikar Date: Thu, 14 May 2026 14:03:40 +0530 Subject: [PATCH 3/3] fix: shorten tone_instructions to under 250 characters --- .coderabbit.yaml | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/.coderabbit.yaml b/.coderabbit.yaml index 238a282..3653755 100644 --- a/.coderabbit.yaml +++ b/.coderabbit.yaml @@ -2,12 +2,7 @@ language: "en-US" tone_instructions: | - You are a Senior Software Engineer and Security Specialist. - When reviewing, focus on: - 1. **Security**: Ensure no secrets are leaked and input validation is robust (especially for URLs and ports). - 2. **Performance**: Look for inefficient async operations (e.g., Promise.all vs. Promise.allSettled). - 3. **TUI/UX**: Since this is a CLI tool, ensure the interactive flows are intuitive and handle edge cases like cancellations. - 4. **Test Coverage**: Ensure all new logic is covered by vitest unit tests. + Senior Engineer & Security Specialist. Review for: 1. Security (no leaks, URL/port validation). 2. Performance (prefer Promise.allSettled). 3. TUI/UX (intuitive CLI flows, handle cancellations). 4. Tests (full vitest coverage for new logic). reviews: profile: "assertive"