refactor: consolidate renderTemplate into src/lib/render-template.ts

Move the shared renderTemplate function from src/templates/shared/template-utils.ts
to src/lib/render-template.ts and remove the duplicate private method from
identity-guide-service.ts. All template consumers now import from the single
canonical location.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: EMaher

src/templates/shared/template-utils.ts src/lib/render-template.tssrc/templates/shared/template-utils.ts renamed to src/lib/render-template.ts

@@ -1,8 +1,8 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
 /**
- * Shared template rendering utility for Copilot prompt templates.
- * Replaces {{TOKEN}} placeholders with provided values.
+ * Template rendering utility.
+ * Replaces {{TOKEN}} placeholders in template strings with provided values.
  */
 
 /**

src/services/identity-guide-service.ts

@@ -11,6 +11,7 @@ import {
   azureDevOpsIdentityGuideTemplate,
   githubActionsIdentityGuideTemplate,
 } from '../templates/generated/embedded-markdown.js';
+import { renderTemplate } from '../lib/render-template.js';
 
 export interface IdentityGuideService {
   generateGitHubActionsGuide(
@@ -25,13 +26,6 @@ export interface IdentityGuideService {
 }
 
 class IdentityGuideServiceImpl implements IdentityGuideService {
-  private renderTemplate(template: string, tokens: Record<string, string>): string {
-    return Object.entries(tokens).reduce(
-      (rendered, [key, value]) => rendered.replaceAll(`{{${key}}}`, value),
-      template
-    );
-  }
-
   generateGitHubActionsGuide(
     subscriptionId: string,
     resourceGroup: string,
@@ -52,7 +46,7 @@ class IdentityGuideServiceImpl implements IdentityGuideService {
 - \`APIM_SERVICE_NAME_${env.toUpperCase()}\`: APIM service name for ${env}
 `).join('\n');
 
-    return this.renderTemplate(githubActionsIdentityGuideTemplate, {
+    return renderTemplate(githubActionsIdentityGuideTemplate, {
       SUBSCRIPTION_ID: subscriptionId,
       RESOURCE_GROUP: resourceGroup,
       FEDERATED_CREDENTIALS_PER_ENV: federatedCredentialsPerEnvironment,
@@ -70,12 +64,12 @@ class IdentityGuideServiceImpl implements IdentityGuideService {
       .map((environment) => `"${environment}"`)
       .join(' ');
 
-    const coreSteps = this.renderTemplate(azureDevOpsIdentitySetupCoreTemplate, {
+    const coreSteps = renderTemplate(azureDevOpsIdentitySetupCoreTemplate, {
       ENVIRONMENTS_ARRAY_POWERSHELL: environmentsArrayPowerShell,
       ENVIRONMENTS_ARRAY_BASH: environmentsArrayBash,
     });
 
-    return this.renderTemplate(azureDevOpsIdentityGuideTemplate, {
+    return renderTemplate(azureDevOpsIdentityGuideTemplate, {
       AZURE_DEVOPS_CORE_STEPS: coreSteps,
     });
   }

src/templates/copilot/configure-overrides-prompt.md

@@ -72,18 +72,9 @@ Using the artifact directory identified in Step 1:
    > APIOps and do **not** need overrides.
 
 2. For each environment, create a **stub** `configuration.{env}.yaml` that
-   covers all the commonly-overridden items you found. Use placeholder example
-   values so the user can see the shape of the file.
-
-3. **Add everything as YAML comments initially** — a commented-out section is
-   safe because it will not cause a publish failure. The user can uncomment
-   and fill in values in Step 3. Example pattern:
-   ```yaml
-   # namedValues:
-   #   - name: payment-api-key
-   #     properties:
-   #       value: "{#[PAYMENT_API_KEY]#}"
-   ```
+   covers all the commonly-overridden items you found. Use placeholder values
+   (e.g., `TODO` or `{#[TOKEN_NAME]#}`) so the user can see the shape of the
+   file and fill in real values in Step 3.
 
 ---
 

src/templates/copilot/identity-setup-prompt.ts

@@ -16,7 +16,7 @@ import {
   copilotGithubEnvironmentSecretCommandsTemplate,
   copilotGitHubActionsIdentitySetupPromptTemplate,
 } from '../generated/embedded-markdown.js';
-import { renderTemplate } from '../shared/template-utils.js';
+import { renderTemplate } from '../../lib/render-template.js';
 
 export interface IdentitySetupPromptConfig {
   environments: string[];