Skip to content

Make apiVersion optional for listSubscriptions invoke, default to 2022-12-01#4567

Merged
Zaid-Ajaj merged 9 commits into
masterfrom
copilot/implement-list-subscriptions-invoke
Mar 3, 2026
Merged

Make apiVersion optional for listSubscriptions invoke, default to 2022-12-01#4567
Zaid-Ajaj merged 9 commits into
masterfrom
copilot/implement-list-subscriptions-invoke

Conversation

Copilot AI commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

apiVersion was a required input for the listSubscriptions invoke. Since there's a single sensible default (2022-12-01), it should be optional.

Changes

  • Schema (provider/pkg/gen/schema.go): Removed apiVersion from Required, updated description to document default
  • Runtime (provider/pkg/provider/provider.go): Default to "2022-12-01" when apiVersion is absent/empty
  • Unit test: TestInvokeListSubscriptionsDefaultApiVersion — verifies default is applied
  • Integration test (examples/examples_test.go): TestListSubscriptionsInvoke — YAML program that invokes listSubscriptions without arguments and outputs the first subscription ID
  • SDK regen: Node.js, Python, .NET SDKs regenerated with optional apiVersion

Usage

variables:
  subscriptions:
    fn::invoke:
      function: azure-native:authorization:listSubscriptions
outputs:
  firstSubscriptionId: ${subscriptions.value[0].subscriptionId}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 4 commits March 3, 2026 17:00
Co-authored-by: Zaid-Ajaj <13316248+Zaid-Ajaj@users.noreply.github.com>
Co-authored-by: Zaid-Ajaj <13316248+Zaid-Ajaj@users.noreply.github.com>
Co-authored-by: Zaid-Ajaj <13316248+Zaid-Ajaj@users.noreply.github.com>
Co-authored-by: Zaid-Ajaj <13316248+Zaid-Ajaj@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix #4564 by implementing listSubscriptions provider invoke Add listSubscriptions provider invoke Mar 3, 2026
Copilot AI and others added 2 commits March 3, 2026 17:38
…iptions

Co-authored-by: Zaid-Ajaj <13316248+Zaid-Ajaj@users.noreply.github.com>
Co-authored-by: Zaid-Ajaj <13316248+Zaid-Ajaj@users.noreply.github.com>
Copilot AI changed the title Add listSubscriptions provider invoke Add listSubscriptions provider invoke with optional apiVersion Mar 3, 2026
Copilot AI and others added 2 commits March 3, 2026 17:50
Co-authored-by: Zaid-Ajaj <13316248+Zaid-Ajaj@users.noreply.github.com>
… `2022-12-01`

Co-authored-by: Zaid-Ajaj <13316248+Zaid-Ajaj@users.noreply.github.com>
Copilot AI changed the title Add listSubscriptions provider invoke with optional apiVersion Make apiVersion optional for listSubscriptions invoke, default to 2022-12-01 Mar 3, 2026

@Zaid-Ajaj Zaid-Ajaj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested runtime implementation locally! Generated schema and code look fine

@Zaid-Ajaj Zaid-Ajaj marked this pull request as ready for review March 3, 2026 18:06
@Zaid-Ajaj Zaid-Ajaj merged commit 37f5c75 into master Mar 3, 2026
1 check passed
@Zaid-Ajaj Zaid-Ajaj deleted the copilot/implement-list-subscriptions-invoke branch March 3, 2026 18:18
@pulumi-bot

Copy link
Copy Markdown
Contributor

This PR has been shipped in release v3.15.0.

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.

3 participants