@betterportal/config-manager 10.1.57 → 10.1.59

package/lib/.bsb/clients/service-betterportal-config-manager.d.ts

@@ -1,7 +1,7 @@
 /**
  * Auto-generated BSB virtual client for service-betterportal-config-manager
  * DO NOT EDIT - Regenerated on every build
- * @version 10.1.57
+ * @version 10.1.59
  */
 import { ServiceClient, BSBService } from "@bsb/base";
 import type { Observable, BSBServiceClientDefinition, EventInputType } from "@bsb/base";

package/lib/.bsb/clients/service-betterportal-config-manager.js

@@ -1,7 +1,7 @@
 /**
  * Auto-generated BSB virtual client for service-betterportal-config-manager
  * DO NOT EDIT - Regenerated on every build
- * @version 10.1.57
+ * @version 10.1.59
  */
 import { ServiceClient, bsb, createBroadcastEvent, createEventSchemas } from "@bsb/base";
 const _platformConfigChangedSchema = bsb.object({

package/lib/bp-contracts/org.betterportal.config-manager.json

@@ -4,7 +4,7 @@
     "pluginId": "org.betterportal.config-manager",
     "title": "BetterPortal Config Manager",
     "description": "Admin-facing BetterPortal service for discovering and managing service config surfaces.",
-    "version": "10.1.57",
+    "version": "10.1.59",
     "category": "service",
     "deploymentModes": [
       "self-hosted"
@@ -1251,6 +1251,12 @@
                   "kind": "string"
                 }
               },
+              "openApiServiceId": {
+                "kind": "optional",
+                "inner": {
+                  "kind": "string"
+                }
+              },
               "routes": {
                 "kind": "array",
                 "items": {

package/lib/plugins/service-betterportal-config-manager/adminApi.d.ts

@@ -28,7 +28,7 @@ declare function automationServiceCatalog(config: BetterPortalConfig, appDef: Be
                 title: string;
                 description: string;
                 scope: "tenant" | "app";
-                visibility: "public" | "protected" | "secret";
+                visibility: "secret" | "public" | "protected";
                 ownership: "bp" | "plugin" | "mixed";
                 sourceOfTruth: "bp" | "plugin" | "external";
                 required: boolean;
@@ -118,7 +118,7 @@ declare function automationServiceCatalog(config: BetterPortalConfig, appDef: Be
                 title: string;
                 description: string;
                 scope: "tenant" | "app";
-                visibility: "public" | "protected" | "secret";
+                visibility: "secret" | "public" | "protected";
                 ownership: "bp" | "plugin" | "mixed";
                 sourceOfTruth: "bp" | "plugin" | "external";
                 required: boolean;

package/lib/plugins/service-betterportal-config-manager/adminApi.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"adminApi.d.ts","sourceRoot":"","sources":["../../../src/plugins/service-betterportal-config-manager/adminApi.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,iBAAiB,EAEjB,mBAAmB,EACnB,SAAS,EACV,MAAM,yBAAyB,CAAC;AAYjC,OAAO,KAAK,EAAuE,kBAAkB,EAAE,eAAe,EAA0C,MAAM,yBAAyB,CAAC;AAChM,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AAoQzD,iBAAS,wBAAwB,CAAC,MAAM,EAAE,kBAAkB,EAAE,MAAM,EAAE,eAAe;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAiEpF;AAGD,wBAAgB,uBAAuB,CAAC,OAAO,EAAE,UAAU,CAAC,OAAO,wBAAwB,CAAC,GAAG,MAAM,CA2CpG;AA0TD,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,kBAAkB,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG;IACvG,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,MAAM,CAAC;IACzB,YAAY,EAAE,MAAM,CAAC;IACrB,gBAAgB,EAAE,MAAM,CAAC;IACzB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,YAAY,EAAE,MAAM,CAAC;IACrB,WAAW,EAAE,MAAM,CAAC;IACpB,kBAAkB,EAAE,MAAM,CAAC;IAC3B,gBAAgB,EAAE,MAAM,CAAC;IACzB,eAAe,EAAE,MAAM,CAAC;CACzB,CAwEA;AAiPD,wBAAgB,sBAAsB,CACpC,GAAG,EAAE,iBAAiB,EACtB,KAAK,EAAE,mBAAmB,EAC1B,OAAO,EAAE,gBAAgB,GACxB,IAAI,CA2+CN;AA8VD,wBAAgB,uBAAuB,CAAC,CAAC,EAAE;IACzC,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,MAAM,CAAC;IACd,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,YAAY,EAAE,MAAM,CAAC;IACrB,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,KAAK,CAAC;QAAE,EAAE,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CAClD,GAAG,MAAM,CA0aT"}
+{"version":3,"file":"adminApi.d.ts","sourceRoot":"","sources":["../../../src/plugins/service-betterportal-config-manager/adminApi.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,iBAAiB,EAEjB,mBAAmB,EACnB,SAAS,EACV,MAAM,yBAAyB,CAAC;AAYjC,OAAO,KAAK,EAAuE,kBAAkB,EAAE,eAAe,EAA0C,MAAM,yBAAyB,CAAC;AAChM,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AAyQzD,iBAAS,wBAAwB,CAAC,MAAM,EAAE,kBAAkB,EAAE,MAAM,EAAE,eAAe;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAiEpF;AAGD,wBAAgB,uBAAuB,CAAC,OAAO,EAAE,UAAU,CAAC,OAAO,wBAAwB,CAAC,GAAG,MAAM,CA2CpG;AA0TD,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,kBAAkB,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG;IACvG,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,MAAM,CAAC;IACzB,YAAY,EAAE,MAAM,CAAC;IACrB,gBAAgB,EAAE,MAAM,CAAC;IACzB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,YAAY,EAAE,MAAM,CAAC;IACrB,WAAW,EAAE,MAAM,CAAC;IACpB,kBAAkB,EAAE,MAAM,CAAC;IAC3B,gBAAgB,EAAE,MAAM,CAAC;IACzB,eAAe,EAAE,MAAM,CAAC;CACzB,CAwEA;AAiPD,wBAAgB,sBAAsB,CACpC,GAAG,EAAE,iBAAiB,EACtB,KAAK,EAAE,mBAAmB,EAC1B,OAAO,EAAE,gBAAgB,GACxB,IAAI,CA0/CN;AA8VD,wBAAgB,uBAAuB,CAAC,CAAC,EAAE;IACzC,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,MAAM,CAAC;IACd,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,YAAY,EAAE,MAAM,CAAC;IACrB,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,KAAK,CAAC;QAAE,EAAE,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CAClD,GAAG,MAAM,CA0aT"}

package/lib/plugins/service-betterportal-config-manager/adminApi.js

@@ -90,6 +90,9 @@ function requiredRouteString(body, key, label) {
         return { error: `${label} is required.` };
     return { value };
 }
+function routesReloadPath(appId, apiServiceId) {
+    return `/routes?appId=${encodeURIComponent(appId)}${apiServiceId ? `&apiServiceId=${encodeURIComponent(apiServiceId)}` : ""}`;
+}
 function routeParamNames(path) {
     return [...new Set(path.split("/").flatMap((segment) => {
             const match = segment.match(/^:([A-Za-z_][A-Za-z0-9_]*)$/);
@@ -1898,6 +1901,8 @@ export function registerAdminApiRoutes(app, store, cpState) {
         const parsed = parseRouteCreateBody(body);
         if (parsed.error || !parsed.route)
             return validationError(event, parsed.error ?? "Invalid route.");
+        if (parsed.route.kind === "api")
+            return validationError(event, "Service/API routes are created by service manifest sync.");
         const serviceError = validateRegisteredRouteService(config, appDef, parsed.route.serviceId);
         if (serviceError)
             return validationError(event, serviceError);
@@ -1909,7 +1914,7 @@ export function registerAdminApiRoutes(app, store, cpState) {
         addRouteDependencies(appDef, route);
         await store.saveConfig(config);
         if (wantsHtmx(event))
-            return htmxReload(`/routes?appId=${encodeURIComponent(appId)}`);
+            return htmxReload(routesReloadPath(appId));
         return jsonResponse({ ok: true, id: route.id }, 201);
     });
     app.put(`${API_BASE}/apps/:appId/routes/:routeId`, async (event) => {
@@ -1925,6 +1930,9 @@ export function registerAdminApiRoutes(app, store, cpState) {
         const route = appDef.routes.find((r) => r.id === routeId);
         if (!route)
             return wantsHtmx(event) ? htmxError("Route not found", 404) : jsonResponse({ error: "Route not found" }, 404);
+        if (route.kind === "api" && Object.keys(body).some((key) => key !== "enabled")) {
+            return validationError(event, "Service/API route identity and metadata are managed by service manifest sync.");
+        }
         if (body.path !== undefined) {
             const path = trimmedString(body, "path");
             if (!path)
@@ -2024,7 +2032,7 @@ export function registerAdminApiRoutes(app, store, cpState) {
         }
         await store.saveConfig(config);
         if (wantsHtmx(event))
-            return htmxReload(`/routes?appId=${encodeURIComponent(appId)}`);
+            return htmxReload(routesReloadPath(appId, route.kind === "api" ? route.serviceId : undefined));
         return jsonResponse({ ok: true });
     });
     app.delete(`${API_BASE}/apps/:appId/routes/:routeId`, async (event) => {
@@ -2039,6 +2047,10 @@ export function registerAdminApiRoutes(app, store, cpState) {
         const route = appDef.routes.find((r) => r.id === routeId);
         if (!route)
             return wantsHtmx(event) ? htmxAlert("Route not found.") : jsonResponse({ error: "Route not found" }, 404);
+        if (route.kind === "api") {
+            const message = "Service/API routes are managed by service manifest sync and cannot be deleted manually.";
+            return wantsHtmx(event) ? htmxAlert(message, "warning") : jsonResponse({ error: message }, 409);
+        }
         const menuReferenceCount = countMenuRouteReferences(appDef.menu, routeId);
         if (menuReferenceCount > 0) {
             const message = `Cannot delete route "${route.title ?? route.path}" because ${menuReferenceCount} menu item${menuReferenceCount === 1 ? "" : "s"} reference it. Remove the menu reference first.`;
@@ -2162,6 +2174,7 @@ export function registerAdminApiRoutes(app, store, cpState) {
         let capabilities = [];
         let serviceRoutes = [];
         let viewMeta = new Map();
+        let dependencyViewIds = new Set();
         const schemaText = typeof body.schema === "string" ? body.schema : "";
         if (schemaText) {
             try {
@@ -2169,7 +2182,11 @@ export function registerAdminApiRoutes(app, store, cpState) {
                 serviceId = schema.manifest?.pluginId;
                 capabilities = Array.isArray(schema.manifest?.capabilities) ? schema.manifest.capabilities.filter((value) => typeof value === "string") : [];
                 serviceRoutes = schema.routes ?? [];
-                viewMeta = new Map((schema.manifest?.views ?? []).map((v) => [v.viewId, {
+                const manifestViews = schema.manifest?.views ?? [];
+                dependencyViewIds = new Set(manifestViews.flatMap((view) => Array.isArray(view.dependencies)
+                    ? view.dependencies.filter((value) => typeof value === "string")
+                    : []));
+                viewMeta = new Map(manifestViews.map((v) => [v.viewId, {
                         title: v.title,
                         renderable: v.renderable !== false,
                         pageRenderable: hasPageRenderer(v),
@@ -2223,7 +2240,7 @@ export function registerAdminApiRoutes(app, store, cpState) {
                         viewId: r.viewId,
                         targetPath: r.path,
                         title: routeTitle,
-                        enabled: true,
+                        enabled: renderable || dependencyViewIds.has(r.viewId),
                         methods: routeMethodsFromManifest(r.methods)
                     });
                     existingPaths.add(mountPath);