address PR review feedback

Addressed review comments:

- Suppressed the shared renderTemplate status line in render-template-browser.js so the CLI prints one JSON result.

- Suppressed the shared renderTemplate status line in render-template-import-status.js so the CLI prints one JSON result.

Copilot-Session: a9952624-0a25-4d6e-ae67-537c962a8a60

Author: priyanshu92

plugins/power-pages/scripts/lib/render-template.js

@@ -18,8 +18,9 @@ const path = require('path');
  * @param {Object} [options.dataObject]  - Data object passed directly. If provided, takes precedence over dataPath.
  * @param {string[]} options.requiredKeys - Keys that must be present in the data
  * @param {boolean} [options.escapeStringValues=false] - Escape string values for HTML text contexts
+ * @param {boolean} [options.emitStatus=true] - Print JSON status after writing the file
  */
-function renderTemplate({ templatePath, outputPath, dataPath, dataObject, requiredKeys, escapeStringValues = false }) {
+function renderTemplate({ templatePath, outputPath, dataPath, dataObject, requiredKeys, escapeStringValues = false, emitStatus = true }) {
   // Validate inputs exist
   if (!fs.existsSync(templatePath)) {
     console.error(`Template not found: ${templatePath}`);
@@ -97,7 +98,9 @@ function renderTemplate({ templatePath, outputPath, dataPath, dataObject, requir
     // non-fatal
   }
 
-  console.log(JSON.stringify({ status: 'ok', output: outputPath }));
+  if (emitStatus) {
+    console.log(JSON.stringify({ status: 'ok', output: outputPath }));
+  }
 }
 
 function escapeHtml(value) {

plugins/power-pages/scripts/render-template-browser.js

@@ -167,6 +167,7 @@ function renderTemplateBrowser({ templatesJsonPath, outputPath, open = false },
       TEMPLATE_SECTIONS_HTML: renderTemplateSectionsHtml(templates),
     },
     requiredKeys: ['TEMPLATE_COUNT', 'TEMPLATE_TABS_HTML', 'TEMPLATE_SECTIONS_HTML'],
+    emitStatus: false,
   });
   const validation = validateRenderedTemplateBrowser({ templates, html: fsImpl.readFileSync(outputPath, 'utf8') });
   if (!validation.ok) {

plugins/power-pages/scripts/render-template-import-status.js

@@ -73,6 +73,7 @@ function renderTemplateImportStatus({ templateName, statusPath, outputPath, prev
       PREVIEW_IMAGES_JSON: localizePreviewImages(previewImages, outputPath),
     },
     requiredKeys: ['TEMPLATE_NAME', 'STATUS_URL_JSON', 'PREVIEW_IMAGES_JSON'],
+    emitStatus: false,
   });
   if (open) {
     try {

plugins/power-pages/scripts/tests/render-template-browser.test.js

@@ -86,6 +86,26 @@ test('renderTemplateBrowser renders static template details and preview images',
   assert.doesNotMatch(html, /templates\.map/);
 });
 
+test('renderTemplateBrowser does not print the shared renderTemplate status line', (t) => {
+  const dir = tempDir();
+  t.after(() => fs.rmSync(dir, { recursive: true, force: true }));
+  const dataPath = path.join(dir, 'templates.json');
+  const outputPath = path.join(dir, 'browser.html');
+  fs.writeFileSync(dataPath, JSON.stringify({
+    TEMPLATES_JSON: [
+      { displayName: 'Company Portal', description: 'Internal site', framework: 'react', previewImages: [] },
+    ],
+  }));
+  const originalLog = console.log;
+  const logs = [];
+  console.log = (value) => logs.push(value);
+  t.after(() => { console.log = originalLog; });
+
+  renderTemplateBrowser({ templatesJsonPath: dataPath, outputPath, open: false });
+
+  assert.deepEqual(logs, []);
+});
+
 test('renderTemplateBrowser renders one family with read-only framework variants', (t) => {
   const dir = tempDir();
   t.after(() => fs.rmSync(dir, { recursive: true, force: true }));

plugins/power-pages/scripts/tests/render-template-import-status.test.js

@@ -91,6 +91,21 @@ test('renderTemplateImportStatus renders scaffold-style slideshow, progress, and
   assert.match(html, /Template import needs attention/);
 });
 
+test('renderTemplateImportStatus does not print the shared renderTemplate status line', (t) => {
+  const dir = tempDir();
+  t.after(() => fs.rmSync(dir, { recursive: true, force: true }));
+  const outputPath = path.join(dir, 'import.html');
+  const statusPath = path.join(dir, 'status.json');
+  const originalLog = console.log;
+  const logs = [];
+  console.log = (value) => logs.push(value);
+  t.after(() => { console.log = originalLog; });
+
+  renderTemplateImportStatus({ templateName: 'Supplier Portal', statusPath, outputPath });
+
+  assert.deepEqual(logs, []);
+});
+
 test('localizePreviewImages copies local preview images beside the served page', (t) => {
   const dir = tempDir();
   t.after(() => fs.rmSync(dir, { recursive: true, force: true }));