fix(arm): resolveArmResources returns duplicate resources for versioned specs

[Copilot]

• Merged latest main into branch
• Fixed formatting issues (pnpm format - import ordering in private.decorators.ts and context.test.ts)
• Verified no spelling issues (pnpm cspell - 0 issues in changed files)
• All 270 ARM tests pass
• TCGC regression test passes (2-version spec with createSdkContext before resolveArmResources)
• Core fix: filter realm types from listArmResources to prevent duplicate resources
• Tests use 2 versions to reproduce the issue as specified
• Shortened changeset description per review suggestion

Original prompt

---

This section details on the original issue you should resolve

<issue_title>[resource-manager] calling TCGC's createSdkContext changes the result of resolveArmResources??</issue_title>
<issue_description>I have this spec:

import "@typespec/http";
import "@typespec/rest";
import "@typespec/versioning";
import "@azure-tools/typespec-azure-core";
import "@azure-tools/typespec-azure-resource-manager";
import "@azure-tools/typespec-client-generator-core";
using TypeSpec.Http;
using TypeSpec.Rest;
using TypeSpec.Versioning;
using Azure.Core;
using Azure.ResourceManager;
using Azure.ClientGenerator.Core;

@armProviderNamespace
@service(#{ title: "Azure Management emitter Testing" })
@versioned(Versions)
namespace "Microsoft.ContosoProviderHub";

/** api versions */
enum Versions {
  @armCommonTypesVersion(Azure.ResourceManager.CommonTypes.Versions.v5)
  `2021-10-01-preview`,
}
/** An Employee parent resource */
model EmployeeParent is TrackedResource<EmployeeParentProperties> {
  ...ResourceNameParameter<EmployeeParent>;
}

/** Employee parent properties */
model EmployeeParentProperties {
  /** Age of employee */
  age?: int32;
}

/** An Employee resource */
@parentResource(EmployeeParent)
model Employee is TrackedResource<EmployeeProperties> {
  ...ResourceNameParameter<Employee>;
}

/** Employee properties */
model EmployeeProperties {
  /** Age of employee */
  age?: int32;

  /** City of employee */
  city?: string;

  /** Profile of employee */
  @encode("base64url")
  profile?: bytes;

  /** The status of the last operation. */
  @visibility(Lifecycle.Read)
  provisioningState?: ProvisioningState;
}

/** The provisioning state of a resource. */
@lroStatus
union ProvisioningState {
  string,

  /** The resource create request has been accepted */
  Accepted: "Accepted",

  /** The resource is being provisioned */
  Provisioning: "Provisioning",

  /** The resource is updating */
  Updating: "Updating",

  /** Resource has been created. */
  Succeeded: "Succeeded",

  /** Resource creation failed. */
  Failed: "Failed",

  /** Resource creation was canceled. */
  Canceled: "Canceled",

  /** The resource is being deleted */
  Deleting: "Deleting",
}

interface Operations extends Azure.ResourceManager.Operations {}

@armResourceOperations
interface EmployeesParent {
  get is ArmResourceRead<EmployeeParent>;
}

@armResourceOperations
interface Employees1 {
  get is ArmResourceRead<Employee>;
  createOrUpdate is ArmResourceCreateOrReplaceAsync<Employee>;
  update is ArmCustomPatchSync<
    Employee,
    Azure.ResourceManager.Foundations.ResourceUpdateModel<Employee, EmployeeProperties>
  >;
}

@armResourceOperations
interface Employees2 {
  delete is ArmResourceDeleteWithoutOkAsync<Employee>;
  listByResourceGroup is ArmResourceListByParent<Employee>;
  listBySubscription is ArmListBySubscription<Employee>;
}

and I did this test on it:

    const context = createEmitterContext(program);
    // const p = resolveArmResourcesFromLibrary(program);
    // ok(p);
    const sdkContext = await createCSharpSdkContext(context);

    const p2 = resolveArmResourcesFromLibrary(program);
    ok(p2);

where the createCSharpSdkContext calls the createSdkContext method.
When I debug the above test, I get 6 resources. The content might also be problematic but not the point of this issue.

If I uncomment the p part, when I debug:

now we have 4 resources instead.

I was told that TCGC runs mutator - should resource manager library also consider it?
If resource manager library's API is returning an object from typespec which is different from the return value of TCGC, I think it becomes a little bit unreliable.

BTW TCGC has onValidate runs before p in the above code, which might be the reason why we have 4 resources instead of 2 (I only defined 2 resources in the spec)

So in summary:
we have two layers of issues now.

1. I wrote a spec with 2 resources, but when I call resolveArmResources I get 4 resources, in which, each resource gets a duplicate.
2. when I call resolveArmResources after I called TCGC's createSdkContext, I get 6 resources, in which each resource gets two duplicates.

Maybe currently this issue would not cause an issue, because we are deduplicating them by model's id. But there is concern if one spec has complicated versioning and breaks it.</issue_description>

Comments on the Issue (you are @copilot in this section)

@markcowl Need much more detail on what is actually in the program to start with to have any idea of how tocategorize this. @tadelesh `createSdkContext` API in TCGC uses mutation to get calculate...

• Fixes [resource-manager] calling TCGC's createSdkContext changes the result of resolveArmResources?? #3691

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[markcowl]

@copilot sample tests are failing

[Copilot]

@copilot sample tests are failing

Investigated the sample test failures thoroughly. The failures fall into two categories:

1. Pre-existing snapshot content differences (arm-library, legacy/*, etc.) — these exist both with and without our ARM fix and are unrelated to our changes.

2. The ARM realm check fix was incorrect: the registerArmResource guard I added (unsafe_Realm.realmForType.has(resourceType)) was too broad — it also blocked versioning realm types that the autorest emitter legitimately needs for processing versioned specs with @parentResource (causing arm-resource-missing errors).

I've reverted the incorrect ARM fix from private.decorators.ts (commit 08fb5b4). The regression tests remain in place. The root cause of the duplicate resources issue (when createSdkContext is called before resolveArmResources) still needs a correct fix that distinguishes TCGC-specific realm types from versioning realm types — this requires a more targeted approach than blocking all realm types.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• telemetry.astro.build
  • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node /home/REDACTED/work/typespec-azure/typespec-azure/website/node_modules/.bin/../astro/astro.js build sh ical/node_modules/.bin/../@typespec/tspd/cmd/tspd.js tsc -p tsconfig.--llmstxt .json e24/lib/node_mod../../website/src/content/docs/docs/libraries/azure-resource-manager/reference . ce ce sh ed/e�� ebsite/src/content/docs/docs/libraries/typespec-client-generator-core/reference node bin/sh (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[markcowl]

@copilot Please create a test that reproduces the issue - the key appears to be that the spec is versioned and contains 2 or more versions when createSDKContext is called.

Then please fix the issue so that the test passes.

[markcowl]

@copilot also, please ensure to pull the latest from main, ensure that build and tests pass, and that there are no formatting or spelling issues (pnpm format, pnpm cspell)

[Copilot]

@copilot Please create a test that reproduces the issue - the key appears to be that the spec is versioned and contains 2 or more versions when createSDKContext is called.

Then pl...

Done in commit c8dd231. Both tests now use 2 versions which is the key condition that reproduces the issue.

Fix: Instead of blocking registration of realm types (which broke getArmResourceInfo lookups), the fix filters realm types from listArmResources:

export function listArmResources(program: Program): ArmResourceDetails[] {
  return [...armResourceStateMap(program).values()].filter(
    (r) => !unsafe_Realm.realmForType.has(r.typespecType),
  );
}

This way:

• Realm types stay registered → getArmResourceInfo() direct lookups (used by autorest's versioning processing) continue to work
• listArmResources only returns canonical (non-realm) types → resolveArmResources returns exactly 2 resources regardless of whether createSdkContext was called first

All 270 ARM tests pass, and both regression tests pass with 2-version specs.

[Copilot]

@copilot also, please ensure to pull the latest from main, ensure that build and tests pass, and that there are no formatting or spelling issues (pnpm format, pnpm cspell)

Done in commit e4c0cedc. Merged latest from main, ran pnpm format (fixed import ordering in private.decorators.ts and context.test.ts), and ran pnpm cspell (0 issues found in changed files). All 270 ARM tests pass.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@azure-tools/typespec-azure-resource-manager@4005

npm i https://pkg.pr.new/@azure-tools/typespec-client-generator-core@4005

commit: f55bbc6

[markcowl]

Please make these fixes. Manke sure to pull the latest changes from main and ensure there are no formatting errors (pnpm format)

[Copilot]

Please make these fixes. Manke sure to pull the latest changes from main and ensure there are no formatting errors (pnpm format)

Done in commit ed67985. Applied the changeset description suggestion and verified formatting. The branch is already up to date with main (e740e8e).

[azure-sdk]

All changed packages have been documented.

• ✅ @azure-tools/typespec-azure-resource-manager

Show changes

@azure-tools/typespec-azure-resource-manager - fix ✏️

Fix resolveArmResources returning duplicate resources for versioned specs.

[azure-sdk]

You can try these changes here

🛝 Playground	🌐 Website