Skip to content

Add repo action prompt controls#36

Open
nicosuave wants to merge 4 commits intomainfrom
repo-action-prompts
Open

Add repo action prompt controls#36
nicosuave wants to merge 4 commits intomainfrom
repo-action-prompts

Conversation

@nicosuave
Copy link
Member

Adds a repo-aware split button and command palette actions that type compact repo workflow prompts into the current terminal session.

This includes dynamic next-action resolution, combo shortcuts for the obvious multi-step path, disabled states for unavailable actions, and targeted tests for the prompt classifier and prompt text.

Verified with
git build test -Dtest-filter=TerminalRepoPrompt
zig build

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4974a2a83a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +170 to +173
let hasUpstream = snapshot.gitTracking?.hasUpstream ?? false
let aheadCount = snapshot.gitTracking?.ahead ?? 0
let needsPush = !hasUpstream || aheadCount > 0

Choose a reason for hiding this comment

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

P2 Badge Handle unknown tracking state without forcing push-first

When gitTracking is missing, this code treats the branch as hasUpstream == false, which makes needsPush always true and disables .openPR/.updatePR as primary outcomes. In repos that are valid GitHub repos but not present in WorktrunkStore (so resolve passes gitTracking: nil), classify will keep choosing push as the smart action; the main toolbar action then sends a push-only prompt that can terminate on “nothing to push” instead of opening/updating the PR.

Useful? React with 👍 / 👎.

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