Skip to content

Open project repo on ctrl+o in the task list#600

Merged
drn merged 1 commit into
masterfrom
argus/press-ctrl-task-list
May 15, 2026
Merged

Open project repo on ctrl+o in the task list#600
drn merged 1 commit into
masterfrom
argus/press-ctrl-task-list

Conversation

@drn
Copy link
Copy Markdown
Owner

@drn drn commented May 15, 2026

Adds a ctrl+o keybinding in the task list that opens the project's
GitHub repo in the browser via gh repo view --web, mirroring the
existing ctrl+p PR-open pattern.

A new repoOpener package-level seam wraps gh repo view --web. The
handler prefers the configured project path and falls back to the task's
worktree when the project is no longer in config; if neither is
available, the event falls through.

Includes 4 tests covering the happy path, fallback, no-dir fallthrough,
and opener-error swallowing.

Co-Authored-By: Claude noreply@anthropic.com

Mirrors the existing ctrl+p PR-open binding via a new repoOpener seam
(gh repo view --web). Prefers the configured project path and falls
back to the task's worktree when the project is no longer in config.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@drn drn merged commit 14dc989 into master May 15, 2026
@github-actions
Copy link
Copy Markdown

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/drn/argus/internal/tui 81.43% (-0.01%) 👎

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/drn/argus/internal/tui/app.go 68.35% (+0.09%) 1627 (+14) 1112 (+11) 515 (+3) 👍

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/drn/argus/internal/tui/newtaskform_autocomplete_test.go

@drn drn deleted the argus/press-ctrl-task-list branch May 16, 2026 01:30
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.

1 participant