Fix Neon extension: Unify command structure and remove preview status by Bhargavsaichinni · Pull Request #9090 · Azure/azure-cli-extensions

Bhargavsaichinni · 2025-08-26T09:05:05Z

Fix dual command structure issue (az neon + az neon postgres -> unified az neon)
Remove preview flags from extension metadata (azext.isPreview: false)
Update az neon create command to use GA API version (2025-03-01)
Remove preview flags from individual commands
Fix duplicate examples in organization create command
Reorganize command structure to match Azure CLI patterns (similar to az sql)
Backup original structure preserved in src/neon-backup/

Breaking Changes:

Commands moved from 'az neon postgres X' to 'az neon X'
Extension no longer in preview mode

Addresses feedback items for production readiness.

This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
My extension version conforms to the Extension version schema

For new extensions:

My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

azure-client-tools-bot-prd · 2025-08-26T09:05:08Z

❌Azure CLI Extensions Breaking Change Test

❌neon

rule cmd_name rule_message suggest_message

❌ 1012 - SubgroupRemove neon postgres sub group neon postgres removed please confirm sub group neon postgres removed

⚠️ 1011 - SubgroupAdd neon branch sub group neon branch added

⚠️ 1011 - SubgroupAdd neon compute sub group neon compute added

⚠️ 1001 - CmdAdd neon create cmd neon create added

⚠️ 1011 - SubgroupAdd neon endpoint sub group neon endpoint added

⚠️ 1011 - SubgroupAdd neon neon-database sub group neon neon-database added

⚠️ 1011 - SubgroupAdd neon neon-role sub group neon neon-role added

⚠️ 1011 - SubgroupAdd neon organization sub group neon organization added

⚠️ 1011 - SubgroupAdd neon project sub group neon project added

azure-client-tools-bot-prd · 2025-08-26T09:05:11Z

Hi @Bhargavsaichinni,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

yonzhan · 2025-08-26T09:05:23Z

Thank you for your contribution! We will review the pull request and get back to you soon.

github-actions · 2025-08-26T09:05:26Z

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

Copilot

Pull Request Overview

This PR unifies the Neon extension command structure and removes preview status to prepare for production release. The dual command structure (az neon and az neon postgres) is consolidated into a single az neon hierarchy, and all preview flags are removed.

Key changes:

Unified command structure from dual structure to single az neon pattern
Removed preview status from extension metadata (azext.isPreview: false)
Updated all commands to use GA API version (2025-03-01)

Reviewed Changes

Copilot reviewed 62 out of 99 changed files in this pull request and generated 1 comment.

File	Description
src/neon/azext_neon/azext_metadata.json	Removed preview status and updated minimum CLI version
src/neon/azext_neon/aaz/latest/neon/project/*.py	Added new project management commands under unified structure
src/neon/azext_neon/aaz/latest/neon/organization/_wait.py	Added organization wait command

src/neon/azext_neon/aaz/latest/neon/project/_show.py

github-actions · 2025-08-26T09:06:15Z

CodeGen Tools Feedback Collection

Thank you for using our CodeGen tool. We value your feedback, and we would like to know how we can improve our product. Please take a few minutes to fill our codegen survey

github-actions · 2025-08-26T09:09:22Z

For more info about extension versioning, please refer to Extension version schema

…roup (1.0.0b5)

…oup (1.0.0)

…elease-version-block to warning

…rkflow

yonzhan · 2025-08-28T08:08:31Z

Please fix CI issues

…m help

…mpatibility

…-cal jobs The version-output job depends on artifacts from these prerequisite jobs. Adding issues:write permissions to ensure proper coordination of workflow execution and potential GitHub API interactions.

Correct version to follow proper semantic versioning: - Main branch has 1.0.0b4 (beta 4) - This PR introduces 1.0.0 (stable GA release) - Consolidated all changes into single 1.0.0 release entry This fixes the version calculation error in the workflow that expects proper version progression without skipping stable versions.

Document complete transformation: - Command structure unification (removed duplicate postgres wrapper) - GA transition (1.0.0b4 → 1.0.0) with proper version progression - GitHub Actions workflow fixes (permissions, dependencies) - Style validation resolution (tabs, whitespace, HISTORY ordering) - Risk assessment and testing validation procedures This README serves as complete documentation for reviewers and future maintainers of the Neon extension changes.

Restore the postgres command group file but without registering it as a separate command group. This maintains structural consistency for the version calculation system while keeping commands flattened under 'neon'. The file exists for organizational purposes but does not create the duplicate 'neon postgres' command group that was causing UX issues.

Remove all workflow modifications we made including: - issues:write permissions additions - packaging>=25 dependency upgrade - warning-only error handling changes - workflow permission enhancements This restores the file to its original state from upstream/main to test if the version calculation works with the original workflow.

Following version calculation system recommendations: - VERSION: 1.0.0 → 1.0.0b5 (beta progression) - azext.isPreview: false → true (restore preview status) - Development Status: Production/Stable → Beta - HISTORY.rst: Updated to reflect beta release with command structure improvements This maintains the command structure flattening changes while following the proper version progression path recommended by the system.

Bhargavsaichinni · 2025-08-28T13:04:21Z

@microsoft-github-policy-service agree company="Microsoft"

…ng guidelines - Update VERSION from 1.0.0 to 2.0.0b1 to comply with breaking change requirements - Restore preview status (azext.isPreview: true) and Beta classifier - Update HISTORY.rst to properly document breaking changes from 1.0.0b4 to 2.0.0b1 - Breaking changes require major version increment per official Microsoft guidelines

- Update VERSION from 2.0.0b1 back to 1.0.0b5 as requested - Update HISTORY.rst to match version 1.0.0b5 - Maintain preview status and Beta classifier for compatibility

necusjz · 2025-08-29T05:24:50Z

/azp run

azure-pipelines · 2025-08-29T05:25:02Z

Azure Pipelines successfully started running 2 pipeline(s).

- Updated test commands from 'az neon postgres <subcommand>' to 'az neon <subcommand>' - Fixed command group registration to use flattened structure instead of hierarchical - Removed legacy postgres wrapper comments and restored proper import structure - This resolves the CI failure where 'postgres' was not recognized as a valid subcommand All tests now use the correct command format: - az neon organization (create|list|show|delete) - az neon project (create|list|show|delete) - az neon branch (create|list|delete) - az neon neon-database list - az neon neon-role list

azure-pipelines · 2025-08-29T06:07:59Z

Commenter does not have sufficient privileges for PR 9090 in repo Azure/azure-cli-extensions

FumingZhang · 2025-08-29T07:09:33Z

/azp run

azure-pipelines · 2025-08-29T07:09:45Z

Azure Pipelines successfully started running 2 pipeline(s).

evelyn-ys · 2025-09-01T07:28:33Z

NEON_CHANGES.md

Pls make it under neon folder instead of root folder

evelyn-ys · 2025-09-01T07:32:50Z

src/neon/azext_neon/aaz/latest/neon/postgres/__cmd_group.py

Doesn't make sense to me to keep all the files under postgres folder since all az neon postgres [command] is now az neon [command]

cc @kairu-ms for review

azclibot · 2025-09-01T09:48:43Z

[Release] Update index.json for extension [ neon-1.0.0b5 ] : https://dev.azure.com/msazure/One/_build/results?buildId=135656231&view=results

Copilot AI review requested due to automatic review settings August 26, 2025 09:05

azure-client-tools-bot-prd bot added this to the October 2025 (2025-10-07) milestone Aug 26, 2025

microsoft-github-policy-service bot requested review from evelyn-ys and yonzhan August 26, 2025 09:05

Copilot AI reviewed Aug 26, 2025

View reviewed changes

src/neon/azext_neon/aaz/latest/neon/project/_show.py Outdated Show resolved Hide resolved

microsoft-github-policy-service bot requested review from calvinhzy and jiasli August 26, 2025 09:05

microsoft-github-policy-service bot added the Auto-Assign Auto assign by bot label Aug 26, 2025

microsoft-github-policy-service bot assigned evelyn-ys Aug 26, 2025

microsoft-github-policy-service bot added the SQL label Aug 26, 2025

github-actions bot added release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. and removed release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. labels Aug 26, 2025

Bhargavsaichinni force-pushed the fix/neon-extension-unified-structure branch from 60d15e2 to e433582 Compare August 28, 2025 05:35

Bhargavsaichinni requested review from FumingZhang, andyliuliming, jsntcy, kairu-ms, necusjz, wangzelin007 and zhoxing-ms as code owners August 28, 2025 05:35

feat(neon): flatten command hierarchy remove legacy 'neon postgres' g…

2aab993

…roup (1.0.0b5)

Bhargavsaichinni force-pushed the fix/neon-extension-unified-structure branch from e433582 to 2aab993 Compare August 28, 2025 05:41

feat(neon): GA release flatten hierarchy remove deprecated preview gr…

8d0e561

…oup (1.0.0)

Bhargavsaichinni added 2 commits August 28, 2025 12:01

workflow: allow version calc by granting issues:write and downgrade r…

50d271c

…elease-version-block to warning

neon: fix help indentation (tabs->spaces) and upgrade packaging in wo…

a10709a

…rkflow

Bhargavsaichinni added 9 commits August 28, 2025 15:47

neon: fix HISTORY.rst order and fully remove tabs/trailing issues fro…

a6209e4

…m help

neon: bump version to 1.0.1 for post-GA style and workflow fixes

1869271

workflow: fix version-output permissions and upgrade packaging for co…

5dfd886

…mpatibility

workflow: add issues:write permission to version-cal and skip-version…

b5962f4

…-cal jobs The version-output job depends on artifacts from these prerequisite jobs. Adding issues:write permissions to ensure proper coordination of workflow execution and potential GitHub API interactions.

github-actions bot removed the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Aug 28, 2025

github-actions bot added the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Aug 28, 2025

Revert version to 1.0.0b5 to fix validation issues

85259e9

- Update VERSION from 2.0.0b1 back to 1.0.0b5 as requested - Update HISTORY.rst to match version 1.0.0b5 - Maintain preview status and Beta classifier for compatibility

github-actions bot removed the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Aug 28, 2025

evelyn-ys reviewed Sep 1, 2025

View reviewed changes

NEON_CHANGES.md

Copy link
Copy Markdown

Member

evelyn-ys Sep 1, 2025

Choose a reason for hiding this comment

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

Pls make it under neon folder instead of root folder

evelyn-ys reviewed Sep 1, 2025

View reviewed changes

kairu-ms approved these changes Sep 1, 2025

View reviewed changes

kairu-ms merged commit a973f80 into Azure:main Sep 1, 2025
25 checks passed

Bhargavsaichinni mentioned this pull request Sep 2, 2025

Revert "Fix Neon extension: Unify command structure and remove preview status" #9117

Merged

Conversation

Bhargavsaichinni commented Aug 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Related command

General Guidelines

About Extension Publish

Uh oh!

azure-client-tools-bot-prd bot commented Aug 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

azure-client-tools-bot-prd bot commented Aug 26, 2025

Uh oh!

yonzhan commented Aug 26, 2025

Uh oh!

github-actions bot commented Aug 26, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

github-actions bot commented Aug 26, 2025

CodeGen Tools Feedback Collection

Uh oh!

github-actions bot commented Aug 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yonzhan commented Aug 28, 2025

Uh oh!

Bhargavsaichinni commented Aug 28, 2025

Uh oh!

necusjz commented Aug 29, 2025

Uh oh!

azure-pipelines bot commented Aug 29, 2025

Uh oh!

azure-pipelines bot commented Aug 29, 2025

Uh oh!

FumingZhang commented Aug 29, 2025

Uh oh!

azure-pipelines bot commented Aug 29, 2025

Uh oh!

evelyn-ys Sep 1, 2025

Choose a reason for hiding this comment

Uh oh!

evelyn-ys Sep 1, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

azclibot commented Sep 1, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

Bhargavsaichinni commented Aug 26, 2025 •

edited

Loading

azure-client-tools-bot-prd bot commented Aug 26, 2025 •

edited

Loading

github-actions bot commented Aug 26, 2025 •

edited

Loading