Skip to content

Add az CLI provider with compact Azure DevOps/Monitor/WebApp output#341

Open
philbritton wants to merge 1 commit intortk-ai:developfrom
philbritton:feat/az-cli-provider
Open

Add az CLI provider with compact Azure DevOps/Monitor/WebApp output#341
philbritton wants to merge 1 commit intortk-ai:developfrom
philbritton:feat/az-cli-provider

Conversation

@philbritton
Copy link
Copy Markdown

Summary

  • add new rtk az command provider (src/az_cmd.rs) with compact output handlers for:
    • az pipelines list/show
    • az pipelines runs list/show
    • az account show/list
    • az group list/show
    • az resource list
    • az devops project list
    • az repos list/show, az repos pr list/show
    • az acr repository list/show-tags
    • az aks list/show
    • az webapp list/show
    • az monitor metrics list, az monitor activity-log list
    • az monitor app-insights component list/show, az monitor app-insights query
  • wire command into CLI parsing and dispatch in src/main.rs
  • add command rewrite/discovery support (az ... -> rtk az ...) in src/discover/rules.rs
  • add rewrite/classification tests in src/discover/registry.rs
  • update rtk init instruction templates and README docs to include rtk az

Testing

  • C:\tools\rtk.exe test cargo +1.89 test az_cmd
  • C:\tools\rtk.exe test cargo +1.89 test test_classify_az_pipelines_list
  • C:\tools\rtk.exe test cargo +1.89 test test_rewrite_az_pipelines_list
  • C:\tools\rtk.exe test cargo +1.89 test test_rewrite_az_monitor_activity_log_list
  • C:\tools\rtk.exe test cargo +1.89 test test_try_parse_az_pipelines_list
  • C:\tools\rtk.exe test cargo +1.89 test test_init_mentions_all_top_level_commands

Note: full cargo +1.89 test in this Windows environment reports one pre-existing failure in utils::tests::test_execute_command_success (unrelated to this change).

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 20, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@pszymkowiak
Copy link
Copy Markdown
Collaborator

Hi! Two things needed before we can review:

  1. Retarget to develop — this PR targets master, but all PRs should target develop. You can change the base branch in the PR settings (right sidebar).
  2. Sign the CLA — if not already done, please sign at https://cla-assistant.io/rtk-ai/rtk

Thanks!

@aeppling
Copy link
Copy Markdown
Contributor

Hey

We are cleaning up the codebase and improving the project structure for better onboarding. As part of this effort, PR #826 reorganizes src/ from a flat layout into subfolders.

No logic changes — only file moves and import path updates.

What you need to do

Rebase your branch on develop when receiving this comment:

git fetch origin && git rebase origin/develop

Git detects renames automatically. If you get import conflicts, update the paths:

use crate::git;        // now: use crate::cmds::git::git;
use crate::tracking;   // now: use crate::core::tracking;
use crate::config;     // now: use crate::core::config;
use crate::init;       // now: use crate::hooks::init;
use crate::gain;       // now: use crate::analytics::gain;

Need help rebasing? Tag @aeppling

philbritton pushed a commit to philbritton/rtk that referenced this pull request Mar 27, 2026
Rebased onto develop branch per PR rtk-ai#341 feedback. Adds `rtk az` command
with token-optimized output for Azure DevOps pipelines, monitor, webapp,
account, and other az subcommands. Includes discovery rules, registry
classification/rewrite tests, and init examples.

Resolves PR rtk-ai#341 feedback:
- Retargeted to develop branch (cherry-picked onto develop base)
- Fixed rewrite_command call signatures for current codebase

https://claude.ai/code/session_017A5C9PNYXiAo2gu4N9md42
@philbritton philbritton changed the base branch from master to develop March 27, 2026 13:10
philbritton pushed a commit to philbritton/rtk that referenced this pull request Mar 27, 2026
Rebased onto develop branch per PR rtk-ai#341 feedback. Adds `rtk az` command
with token-optimized output for Azure DevOps pipelines, monitor, webapp,
account, and other az subcommands. Includes discovery rules, registry
classification/rewrite tests, and init examples.

Resolves PR rtk-ai#341 feedback:
- Retargeted to develop branch (cherry-picked onto develop base)
- Fixed rewrite_command call signatures for current codebase

https://claude.ai/code/session_017A5C9PNYXiAo2gu4N9md42
@philbritton philbritton force-pushed the feat/az-cli-provider branch from f949f01 to 3ca352a Compare March 27, 2026 13:12
Adds `rtk az` command with token-optimized output for Azure DevOps
pipelines, monitor, webapp, account, AKS, ACR, and other az subcommands.

- New src/cmds/cloud/az_cmd.rs with specialized filters (~80% savings)
- Discovery rules and registry classification/rewrite tests
- CLI parsing with trailing_var_arg for native az flag passthrough
- Init examples for Azure DevOps pipelines and monitor commands

Adapted to restructured codebase (PR rtk-ai#826): module placed in
src/cmds/cloud/ with updated import paths.

https://claude.ai/code/session_017A5C9PNYXiAo2gu4N9md42
@philbritton philbritton force-pushed the feat/az-cli-provider branch from 3ca352a to aff75cf Compare March 27, 2026 14:55
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.

6 participants