Skip to content

[repo-assist] Fix simulate_key action#50

Merged
popey merged 2 commits into
mainfrom
repo-assist/fix-issue-37-simulate-key-d48999d1fea8c644
Jun 5, 2026
Merged

[repo-assist] Fix simulate_key action#50
popey merged 2 commits into
mainfrom
repo-assist/fix-issue-37-simulate-key-d48999d1fea8c644

Conversation

@popey

@popey popey commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Closes #37

Adds action_simulate_key to Grummage to handle simulate_key bindings without error.


Summary by cubic

Added action_simulate_key to Grummage to support simulate_key bindings and prevent navigation errors when the Tree API varies between Textual versions. Now prefers cursor_node for left/right and falls back to selected_node.

  • Bug Fixes
    • Up/down navigation via select_next/cursor_down and select_previous/cursor_up.
    • Left/right uses cursor_node first, falls back to selected_node; collapses/expands when available.
    • Safe no-op if tree_view or methods are missing; logs exceptions instead of crashing.

Written for commit 2dfe2ab. Summary will update on new commits.

Review in cubic

Closes #37

Adds action_simulate_key to Grummage to handle simulate_key bindings without error.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 1 file

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread grummage.py Outdated
@popey popey merged commit 45d6714 into main Jun 5, 2026
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

simulate_key action bound but never defined

2 participants