Skip to content

Previously rejected tool call cannot be accepted later. #271

@jacdavis

Description

@jacdavis

Describe the bug

I accidently rejected a permission and it won't let me undo the rejection without a restart. See session below.

summarize pr 7112 in sweagentd for me                                                                            │
 ╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯

 ● I'll look up PR 7112 in the sweagentd repository for you.

 ✗ Get details for a single pull request
   failed to get pull request: GET
   https://api.github.com/repos/sweagentd/sweagentd/pulls/7112 (
   https://api.github.com/repos/sweagentd/sweagentd/pulls/7112): 404 Not Found []

 ● Let me try searching for repositories with "sweagentd" to find the correct
   repository name.

 ✓ Search repositories
   ↪ {"total_count":5,"incomplete_results":false,"items":[{"id":926880992,"name":"...

 ● Now let me check PR 7112 in the github/sweagentd repository.

 ✓ Get details for a single pull request
   ↪ {"id":2902206407,"number":7112,"state":"open","locked":false,"title":"[Sessio...

 ● PR #7112 migrates the CLI to use a new sessions system. The changes reconstruct
   Timeline and Chat Messages from session logs rather than writing to them
   directly, decouple Timeline from Session using hooks, and separate Usage Metrics
   similarly. Mentions and attachments are now emitted as file paths with
   transformations happening during chat message building. The CLISessionManager
   now lists both legacy and new sessions, allowing legacy sessions to be read,
   resumed, and migrated to the new format. All tests have been updated to use the
   new session abstraction. This is a large change (2,265 additions, 1,242
   deletions across 58 files) that positions the codebase to move the CLI to use
   the SDK and eliminate callback patterns in executionqueue. The author recommends
   a team bug bash for thorough testing before merging.

 ╭────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
 │ > does the cli intentionally disable the github mcp when running with -p?                                          │
 ╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯

 ● I'll search for code related to the -p flag and GitHub MCP configuration in the
   CLI.

 ✓ Find CLI-related files
   $ cd /home/jacdavis/src/sweagentd4 && find . -type f -name "*.ts" -o -name "*.js" | grep -E "(cli|command)" | head -20
   ↪ 21 lines...

   Search for -p flag usage
   $ cd /home/jacdavis/src/sweagentd4 && grep -r "\-p" --include="*.ts" --include="*.js" | grep -i "prompt\|flag\|option"
   | head -30
   ↪ 3 lines...

 ● Let me search more specifically in the CLI source code:

   Find CLI TypeScript files excluding node_modules
   $ cd /home/jacdavis/src/sweagentd4 && find . -path ./node_modules -prune -o -path ./benchmarks -prune -o -type f -name
   "*.ts" -print | grep -i cli | head -20
   ↪ 21 lines...

 ✓ Search for github mcp references
   $ cd /home/jacdavis/src/sweagentd4 && grep -r "github.*mcp" --include="*.ts" --include="*.js" . 2>/dev/null | grep -v
   node_modules | grep -v benchmarks | head -20
   ↪ 21 lines...

 ● Now let me search for the -p flag and prompt mode:

 ✗ Search for -p flag and prompt mode
   $ cd /home/jacdavis/src/sweagentd4 && grep -r "\-p\|prompt.*mode" --include="*.ts" . 2>/dev/null | grep -v
   node_modules | grep -v benchmarks | grep -v dist- | head -30
   Rejected by you.

 ╭────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
 │ > sorry, didn't mean to reject that                                                                                │
 ╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯

 ● No problem! Let me search for the -p flag and how it relates to GitHub MCP:

 ✗ Search for -p flag and prompt mode
   $ cd /home/jacdavis/src/sweagentd4 && grep -r "\-p\|prompt.*mode" --include="*.ts" . 2>/dev/null | grep -v
   node_modules | grep -v benchmarks | grep -v dist- | head -30
   Rejected by you.

 ╭────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
 │ > I didn't reject it. Call the tool again and I'll approve                                                         │
 ╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯

 ✗ Search for -p flag and prompt mode
   $ cd /home/jacdavis/src/sweagentd4 && grep -r "\-p\|prompt.*mode" --include="*.ts" . 2>/dev/null | grep -v
   node_modules | grep -v benchmarks | grep -v dist- | head -30
   Rejected by you

Affected version

No response

Steps to reproduce the behavior

No response

Expected behavior

No response

Additional context

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions