From 20ddfc61c1605d790ab59bbcd7f108d138c7b37b Mon Sep 17 00:00:00 2001 From: aaltshuler Date: Thu, 11 Jun 2026 22:29:57 +0300 Subject: [PATCH] fix(cli): reclaim the hidden legacy-uri positional for operator aliases MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Caught on the live smoke: with --alias, the first bare CLI arg lands in the hidden legacy_uri positional, so an operator alias's positional param never bound ('parameter not provided' from the server). An operator alias always knows its target, so the existing normalize_legacy_alias_uri reclaims the swallowed positional as the first alias arg — same rule the legacy path already applies. Co-Authored-By: Claude Fable 5 --- crates/omnigraph-cli/src/main.rs | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/crates/omnigraph-cli/src/main.rs b/crates/omnigraph-cli/src/main.rs index 284cff8..4306b67 100644 --- a/crates/omnigraph-cli/src/main.rs +++ b/crates/omnigraph-cli/src/main.rs @@ -758,6 +758,15 @@ async fn main() -> Result<()> { "warning: alias '{alias_name}' is defined in both omnigraph.yaml (legacy, wins during the deprecation window) and the operator config; the legacy definition applies" ); } else { + // The hidden legacy-uri positional swallows the first + // bare arg; an operator alias always knows its target, + // so reclaim it as the first positional param. + let (_, alias_args) = normalize_legacy_alias_uri( + legacy_uri.clone(), + true, + Some(alias_name), + alias_args.clone(), + ); let output = execute_operator_alias( &http_client, &config,