browser-extension-manager 1.3.13 → 1.3.15

package/CLAUDE.md

@@ -209,8 +209,33 @@ When adding a new component type to the framework:
 - **distribute** ([gulp/tasks/distribute.js](src/gulp/tasks/distribute.js)) - Copies project files to `dist/`
 - **sass** ([gulp/tasks/sass.js](src/gulp/tasks/sass.js)) - Compiles SCSS with sophisticated load path system
 - **webpack** ([gulp/tasks/webpack.js](src/gulp/tasks/webpack.js)) - Bundles JavaScript with Babel
-- **html** ([gulp/tasks/html.js](src/gulp/tasks/html.js)) - Processes HTML views into templates
+- **html** ([gulp/tasks/html.js](src/gulp/tasks/html.js)) - Processes HTML views into templates (see HTML Templating below)
 - **package** ([gulp/tasks/package.js](src/gulp/tasks/package.js)) - Creates packaged extension
+
+### HTML Templating
+
+HTML views in `src/views/` are processed through a two-step templating system using `{{ }}` brackets.
+
+**Available variables:**
+- `{{ brand.name }}` - Brand name from config
+- `{{ brand.url }}` - Brand URL from config
+- `{{ page.name }}` - Component name (e.g., `popup`, `pages/index`)
+- `{{ page.path }}` - Full view path
+- `{{ page.title }}` - Page title (defaults to brand name)
+- `{{ theme.appearance }}` - Theme appearance (`dark` or `light`)
+- `{{ cacheBust }}` - Cache-busting timestamp
+
+**Example usage in views:**
+```html
+<a href="{{ brand.url }}/pricing">Upgrade to Premium</a>
+<p>Welcome to {{ brand.name }}</p>
+```
+
+**How it works:**
+1. Your view file (`src/views/[component]/index.html`) is templated first
+2. The result is injected into [page-template.html](src/config/page-template.html)
+3. The outer template is processed with the same variables
+
 - **serve** ([gulp/tasks/serve.js](src/gulp/tasks/serve.js)) - WebSocket server for live reload
 
 ### Modifying Themes

package/dist/gulp/tasks/html.js

@@ -105,6 +105,15 @@ function processHtml(templateContent) {
       };
 
       // Apply template with custom brackets
+      // First, template the body content to replace any {{ }} placeholders in the view
+      const templatedBody = template(bodyContent, data, {
+        brackets: ['{{', '}}'],
+      });
+
+      // Update data with templated body
+      data.content = templatedBody;
+
+      // Then template the outer page template
       const rendered = template(templateContent, data, {
         brackets: ['{{', '}}'],
       });

package/dist/gulp/tasks/publish.js

@@ -314,59 +314,57 @@ async function publishToEdge() {
     throw new Error('Missing Edge credentials. Set EDGE_PRODUCT_ID, EDGE_CLIENT_ID, EDGE_API_KEY in .env');
   }
 
-  // Check current submission status first
-  logger.log('[edge] Checking submission status...');
-  const statusUrl = `https://api.addons.microsoftedge.microsoft.com/v1/products/${productId}/submissions`;
-  const statusResponse = await fetch(statusUrl, {
-    method: 'GET',
-    headers: {
-      'Authorization': `ApiKey ${apiKey}`,
-      'X-ClientID': clientId,
-    },
-  });
+  // Helper for Edge API requests
+  const edgeHeaders = {
+    'Authorization': `ApiKey ${apiKey}`,
+    'X-ClientID': clientId,
+  };
+
+  // Helper to parse Edge API response (handles empty bodies)
+  async function parseEdgeResponse(response, label) {
+    const text = await response.text();
+    logger.log(`[edge] ${label} - Status: ${response.status}, Body: ${text || '(empty)'}`);
 
-  if (statusResponse.ok) {
-    const statusData = await statusResponse.json();
-    // Check if there's a pending submission (InReview, PendingPublish, etc.)
-    const pendingStatuses = ['InReview', 'PendingPublish', 'Submitted', 'InProgress'];
-    if (statusData && pendingStatuses.includes(statusData.status)) {
-      throw new Error(`Extension already has a pending submission (status: ${statusData.status}). Wait for it to complete.`);
+    if (!text) {
+      return { status: response.status, data: null };
+    }
+
+    try {
+      return { status: response.status, data: JSON.parse(text) };
+    } catch (e) {
+      return { status: response.status, data: text };
     }
   }
 
+  // Step 1: Upload the package first
   logger.log('[edge] Uploading to Microsoft Edge Add-ons...');
 
-  // Read chromium zip file (Edge uses same build as Chrome)
   const zipBuffer = jetpack.read(PATHS.chromium.zip, 'buffer');
-
-  // Edge API v1.1 endpoint
   const uploadUrl = `https://api.addons.microsoftedge.microsoft.com/v1/products/${productId}/submissions/draft/package`;
 
-  // Upload using fetch
-  const response = await fetch(uploadUrl, {
+  const uploadResponse = await fetch(uploadUrl, {
     method: 'POST',
     headers: {
-      'Authorization': `ApiKey ${apiKey}`,
-      'X-ClientID': clientId,
+      ...edgeHeaders,
       'Content-Type': 'application/zip',
     },
     body: zipBuffer,
   });
 
-  if (!response.ok) {
-    const errorText = await response.text();
-    throw new Error(`Edge API error: ${response.status} - ${errorText}`);
+  const upload = await parseEdgeResponse(uploadResponse, 'Upload response');
+
+  if (!uploadResponse.ok) {
+    throw new Error(`Edge upload error: ${upload.status} - ${JSON.stringify(upload.data)}`);
   }
 
   logger.log('[edge] Package uploaded, submitting for review...');
 
-  // Submit for review
+  // Step 2: Submit for review - this is where we'll get InProgressSubmission if there's a pending review
   const publishUrl = `https://api.addons.microsoftedge.microsoft.com/v1/products/${productId}/submissions`;
   const publishResponse = await fetch(publishUrl, {
     method: 'POST',
     headers: {
-      'Authorization': `ApiKey ${apiKey}`,
-      'X-ClientID': clientId,
+      ...edgeHeaders,
       'Content-Type': 'application/json',
     },
     body: JSON.stringify({
@@ -374,9 +372,33 @@ async function publishToEdge() {
     }),
   });
 
+  const publish = await parseEdgeResponse(publishResponse, 'Publish response');
+
+  // Check for HTTP errors (4xx, 5xx)
   if (!publishResponse.ok) {
-    const errorText = await publishResponse.text();
-    throw new Error(`Edge publish error: ${publishResponse.status} - ${errorText}`);
+    // Check if it's a 409 Conflict or similar indicating in-progress submission
+    if (publish.status === 409) {
+      throw new Error('Extension already has a pending submission in review. Wait for it to complete before publishing again.');
+    }
+    throw new Error(`Edge publish error: ${publish.status} - ${JSON.stringify(publish.data)}`);
+  }
+
+  // Check for API-level failures (HTTP 200/202 but status: "Failed" in body)
+  if (publish.data && typeof publish.data === 'object') {
+    if (publish.data.status === 'Failed') {
+      if (publish.data.errorCode === 'InProgressSubmission') {
+        throw new Error('Extension already has a pending submission in review. Wait for it to complete before publishing again.');
+      }
+      if (publish.data.errorCode === 'UnpublishInProgress') {
+        throw new Error('Extension is being unpublished. Wait for unpublish to complete before publishing.');
+      }
+      throw new Error(`Edge publish failed: ${publish.data.message || publish.data.errorCode || 'Unknown error'}`);
+    }
+  }
+
+  // HTTP 202 Accepted means submission was queued successfully
+  if (publish.status === 202) {
+    logger.log('[edge] Submission accepted and queued for review');
   }
 
   logger.log('[edge] Upload complete');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "browser-extension-manager",
-  "version": "1.3.13",
+  "version": "1.3.15",
   "description": "Browser Extension Manager dependency manager",
   "main": "dist/index.js",
   "exports": {