[tcgc] refine logic to iterate over all user-defined namespaces

[iscai-msft]

No description provided.

[azure-sdk]

All changed packages have been documented.

• ✅ @azure-tools/typespec-client-generator-core

Show changes

@azure-tools/typespec-client-generator-core - fix ✏️

Recurse over namespaces to get all user defined namespaces"

[azure-sdk]

You can try these changes here

🛝 Playground	🌐 Website

[tadelesh]

could you please also update the code here:

typespec-azure/packages/typespec-client-generator-core/src/public-utils.ts

Line 348 in 6bf2732

for (const client of listClients(context)) {

it is missing in the original pr.

[iscai-msft]

could you please also update the code here:

typespec-azure/packages/typespec-client-generator-core/src/public-utils.ts

Line 348 in 6bf2732

for (const client of listClients(context)) {

it is missing in the original pr.

the reason for this pr is to remove the need to iterate over listClients. The root issue was I used to stop once I saw a namespace that wasn't user defined (which would stop at just Azure. and not continue recursing). With the new code, I keep recursing, so I will find the necessary clients

[tadelesh]

could you please also update the code here:

typespec-azure/packages/typespec-client-generator-core/src/public-utils.ts

Line 348 in 6bf2732

for (const client of listClients(context)) {

it is missing in the original pr.

the reason for this pr is to remove the need to iterate over listClients. The root issue was I used to stop once I saw a namespace that wasn't user defined (which would stop at just Azure. and not continue recursing). With the new code, I keep recursing, so I will find the necessary clients

yes, i understand this pr's work. but my ask is actually from the original pr's comment: #2179 (comment). there is something miss in that pr.

[tadelesh]

and i'm think of if current impl could support versioning?

[tadelesh]

@weidongxu-microsoft could you help to test this version to see if all orphan model is calculated?