Skip to content

Conversation

Copy link

Copilot AI commented Jan 27, 2026

Summary

Added managedByExtended to the list of allowed body top-level properties for ARM resources in compliance with ARM guidelines.

Changes Made

Code Changes

  • ✅ Updated allowedBodyTopLevelProperties array in arm-resource-validation.ts to include managedbyextended
  • ✅ Updated propertiesBagTracked array in arm-resource-validation.ts to include managedbyextended
  • ✅ Updated error message to list all allowed properties: name, type, id, location, properties, tags, plan, sku, etag, managedBy, managedByExtended, identity, kind, zones, systemData, extendedLocation

Documentation Changes

  • ✅ Updated body-top-level-properties.md to include managedByExtended and all properties
  • ✅ Updated rules.md to include managedByExtended and all properties
  • ✅ Fixed casing inconsistencies (extendedLocation)

Testing

  • ✅ Updated existing test file body-top-level-properties-real-swagger.json to include managedByExtended property
  • ✅ Removed separate test file to consolidate test cases
  • ✅ All 90 tests pass successfully
  • ✅ Build and lint checks pass

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 5 commits January 27, 2026 20:37
Co-authored-by: raosuhas <11067581+raosuhas@users.noreply.github.com>
Co-authored-by: raosuhas <11067581+raosuhas@users.noreply.github.com>
Co-authored-by: raosuhas <11067581+raosuhas@users.noreply.github.com>
Co-authored-by: raosuhas <11067581+raosuhas@users.noreply.github.com>
Co-authored-by: raosuhas <11067581+raosuhas@users.noreply.github.com>
Copilot AI changed the title [WIP] Update allowedBodyTopLevelProperties to include managedByExtended Add managedByExtended to allowed ARM resource top-level properties Jan 27, 2026
Copilot AI requested a review from raosuhas January 27, 2026 20:49
Copy link
Member

Choose a reason for hiding this comment

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

@copilot instead of creating a new json file for the test , can you just update one of the existing files to include these tests ?

Copy link
Author

Choose a reason for hiding this comment

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

Updated! I've added the managedByExtended property to the existing body-top-level-properties-real-swagger.json file instead of creating a new test file. The separate test file has been removed and the test case now uses the updated existing file. All tests pass. (commit 90d4543)

…tended

Co-authored-by: raosuhas <11067581+raosuhas@users.noreply.github.com>
Copilot AI requested a review from raosuhas January 27, 2026 22:12
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