Skip to content

feat: Detect AI tools via CLI prefix wrapper - #198

Merged
kaeawc merged 1 commit into
mainfrom
detect-cli-tools
Apr 3, 2026
Merged

feat: Detect AI tools via CLI prefix wrapper#198
kaeawc merged 1 commit into
mainfrom
detect-cli-tools

Conversation

@kaeawc

@kaeawc kaeawc commented Apr 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • When a CLI prefix (e.g. slack, goog) is configured via ai-tool-cmd and available in PATH, all AI tools are now detected as available through the prefix — even when the bare binary isn't installed
  • Adds "Configure CLI prefix wrapper (e.g. slack, goog)" option to the "No AI coding assistant found" install menu so first-time users can discover and configure prefix-based access in one flow
  • Adds || true to command -v calls in _resolve_ai_command for robustness under set -e

Test Plan

  • 14 new bats tests in tests/ai_prefix_detection.bats covering prefix detection, saved preferences via prefix, resume commands, negative cases, _setup_ai_cmd, and install menu flow
  • All 234 tests pass (14 new + 220 existing)

Co-Authored-By: Claude svc-devxp-claude@slack-corp.com

… install menu

When a CLI prefix (e.g. slack, goog) is configured and available in PATH,
all AI tools are now detected as available through the prefix even when
the bare binary isn't installed. Adds a "Configure CLI prefix wrapper"
option to the "No AI coding assistant found" menu so first-time users
can discover and set up prefix-based access in one flow.

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@kaeawc
kaeawc enabled auto-merge (squash) April 3, 2026 08:52
@kaeawc
kaeawc merged commit e57d94f into main Apr 3, 2026
2 checks passed
@kaeawc
kaeawc deleted the detect-cli-tools branch April 3, 2026 08:52
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