@adobedjangir/commerce-admin-management 0.0.8 → 0.0.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobedjangir/commerce-admin-management",
-  "version": "0.0.8",
+  "version": "0.0.10",
   "description": "Schema-driven system configuration for Adobe Commerce App Builder sync apps. Magento-style scoped config in Adobe App Builder Database (ABDB) with encryption, Commerce REST helpers, and React Admin UI.",
   "license": "Apache-2.0",
   "author": "Adobe Inc.",

package/scripts/setup.js

@@ -69,29 +69,56 @@ function updateExistingExtensionBlock (content) {
   return next !== content ? next : null
 }
 
+/**
+ * Remove the boilerplate `dx/excshell/1` extension block that aio's default
+ * `app init` ships with. The host can only run one extension point per
+ * project for our purposes, and `commerce/backend-ui/1` is what this
+ * package needs — leaving `dx/excshell/1` in place causes aio to try
+ * building BOTH, which fails on the excshell side because we don't ship
+ * any code for it.
+ *
+ * Matches a 2-space-indented `dx/excshell/1:` block followed by any
+ * deeper-indented nested lines, terminating when we see a non-indented
+ * line or a sibling extension. Safe to run repeatedly — no-op if absent.
+ */
+function stripExcshellBlock (content) {
+  const re = /^[ \t]*dx\/excshell\/1:[ \t]*\n(?:[ \t]+[^\n]*\n)*/m
+  if (!re.test(content)) return { content, changed: false }
+  const next = content.replace(re, '')
+  return { content: next, changed: true }
+}
+
 function patchAppConfig (content) {
-  if (alreadyLinked(content)) {
-    return { content, changed: false, reason: 'already-linked' }
+  // First strip the boilerplate dx/excshell/1 block — see stripExcshellBlock.
+  const excshell = stripExcshellBlock(content)
+  let working = excshell.content
+
+  if (alreadyLinked(working)) {
+    return {
+      content: working,
+      changed: excshell.changed,
+      reason: excshell.changed ? 'stripped-excshell' : 'already-linked'
+    }
   }
 
-  if (hasExtensionPoint(content)) {
-    const updated = updateExistingExtensionBlock(content)
+  if (hasExtensionPoint(working)) {
+    const updated = updateExistingExtensionBlock(working)
     if (updated) {
       return { content: updated, changed: true, reason: 'updated-existing-extension' }
     }
-    return { content, changed: false, reason: 'extension-exists-unmodified' }
+    return { content: working, changed: excshell.changed, reason: 'extension-exists-unmodified' }
   }
 
-  if (/^extensions:[ \t]*\n/m.test(content)) {
+  if (/^extensions:[ \t]*\n/m.test(working)) {
     const injection = `  ${EXTENSION_POINT}:\n    $include: ${INCLUDE_REL}\n`
-    const next = content.replace(/^extensions:[ \t]*\n/m, `extensions:\n${injection}`)
-    if (next !== content) {
+    const next = working.replace(/^extensions:[ \t]*\n/m, `extensions:\n${injection}`)
+    if (next !== working) {
       return { content: next, changed: true, reason: 'added-under-extensions' }
     }
   }
 
-  if (!/^extensions:/m.test(content)) {
-    const trimmed = content.replace(/\s+$/, '')
+  if (!/^extensions:/m.test(working)) {
+    const trimmed = working.replace(/\s+$/, '')
     const separator = trimmed.length > 0 ? '\n\n' : ''
     return {
       content: `${trimmed}${separator}${buildExtensionBlock()}\n`,
@@ -100,7 +127,7 @@ function patchAppConfig (content) {
     }
   }
 
-  return { content, changed: false, reason: 'no-change' }
+  return { content: working, changed: excshell.changed, reason: excshell.changed ? 'stripped-excshell' : 'no-change' }
 }
 
 // ────────────────────────────────────────────────────────────────────────────
@@ -369,19 +396,17 @@ function indexHtmlContents () {
 }
 
 function excRuntimeContents () {
-  // Minimal exc-runtime shim — the package's bootstrap requires it. aio's
-  // template ships a similar file. We keep this thin so it stays current
-  // even if Adobe's exc-runtime evolves; the meaningful glue lives in
-  // index.js (also scaffolded by this script).
+  // exc-runtime.js's only job is to side-effect-import the Adobe Experience
+  // Cloud Shell runtime so the bootstrap can call init(...) afterwards.
+  // The bootstrap wraps `require('./exc-runtime')` in try/catch — if this
+  // import fails (e.g. when running outside the shell) bootstrapRaw() takes
+  // over. So this file is intentionally minimal.
   return `/*
 Copyright 2025 Adobe. All rights reserved.
 Licensed under the Apache License, Version 2.0
 */
 
-// Loaded by index.js when running inside the Adobe Experience Cloud Shell.
-// Pulls in the @adobe/exc-app runtime which the bootstrap then initialises.
-import '@adobe/exc-app/page'
-import '@adobe/exc-app/page/Page'
+import '@adobe/exc-app'
 `
 }