simplify route bindings in config

Signed-off-by: Fredrik Adelöw <freben@gmail.com>

.changeset/silent-hotels-knock.md

@@ -0,0 +1,25 @@
+---
+'@backstage/frontend-app-api': minor
+---
+
+**BREAKING**: The `app.routes.bindings` app-config mapping has been simplified. You now only need to specify the plugin ID and route ID on both sides of the mapping.
+
+Old form:
+
+```yaml
+app:
+  routes:
+    bindings:
+      plugin.catalog.externalRoutes.viewTechDoc: plugin.techdocs.routes.docRoot
+      plugin.catalog.externalRoutes.createComponent: plugin.catalog-import.routes.importPage
+```
+
+New form:
+
+```yaml
+app:
+  routes:
+    bindings:
+      catalog.viewTechDoc: techdocs.docRoot
+      catalog.createComponent: catalog-import.importPage
+```

docs/frontend-system/architecture/07-routes.md

@@ -241,10 +241,10 @@ app:
     bindings:
       # point to the Scaffolder create component page when the Catalog create component ref is used
       # highlight-next-line
-      plugin.catalog.externalRoutes.createComponent: plugin.scaffolder.routes.index
+      catalog.createComponent: scaffolder.index
       # point to the Catalog details page when the Scaffolder component details ref is used
       # highlight-next-line
-      plugin.scaffolder.externalRoutes.componentDetails: plugin.catalog.routes.details
+      scaffolder.componentDetails: catalog.details
 ```
 
 We also have the ability to express this in code as an option to `createApp`, but you of course only need to use one of these two methods:

packages/app-next/app-config.yaml

@@ -3,9 +3,9 @@ app:
     packages: 'all' # ✨
   routes:
     bindings:
-      plugin.pages.externalRoutes.pageX: plugin.pages.routes.pageX
-      plugin.catalog.externalRoutes.viewTechDoc: plugin.techdocs.routes.docRoot
-      plugin.catalog.externalRoutes.createComponent: plugin.catalog-import.routes.importPage
+      pages.pageX: pages.pageX
+      catalog.viewTechDoc: techdocs.docRoot
+      catalog.createComponent: catalog-import.importPage
 
   extensions:
     # - apis.plugin.graphiql.browse.gitlab: true

packages/frontend-app-api/src/routing/collectRouteIds.test.ts

@@ -37,15 +37,13 @@ describe('collectRouteIds', () => {
       createPlugin({ id: 'test', routes: { ref }, externalRoutes: { extRef } }),
     ]);
     expect(Object.fromEntries(collected.routes)).toEqual({
-      'plugin.test.routes.ref': ref,
+      'test.ref': ref,
     });
     expect(Object.fromEntries(collected.externalRoutes)).toEqual({
-      'plugin.test.externalRoutes.extRef': extRef,
+      'test.extRef': extRef,
     });
 
-    expect(String(ref)).toBe('RouteRef{plugin.test.routes.ref}');
-    expect(String(extRef)).toBe(
-      'ExternalRouteRef{plugin.test.externalRoutes.extRef}',
-    );
+    expect(String(ref)).toBe('RouteRef{test.ref}');
+    expect(String(extRef)).toBe('ExternalRouteRef{test.extRef}');
   });
 });

packages/frontend-app-api/src/routing/collectRouteIds.ts

@@ -47,7 +47,7 @@ export function collectRouteIds(features: FrontendFeature[]): RouteRefsById {
     }
 
     for (const [name, ref] of Object.entries(feature.routes)) {
-      const refId = `plugin.${feature.id}.routes.${name}`;
+      const refId = `${feature.id}.${name}`;
       if (routesById.has(refId)) {
         throw new Error(`Unexpected duplicate route '${refId}'`);
       }
@@ -62,7 +62,7 @@ export function collectRouteIds(features: FrontendFeature[]): RouteRefsById {
       }
     }
     for (const [name, ref] of Object.entries(feature.externalRoutes)) {
-      const refId = `plugin.${feature.id}.externalRoutes.${name}`;
+      const refId = `${feature.id}.${name}`;
       if (externalRoutesById.has(refId)) {
         throw new Error(`Unexpected duplicate external route '${refId}'`);
       }