Make menu links configurable (#434)

Now internal deployments can add their own custom links which will be
added to the menu.

<img width="574" alt="Screenshot 2025-04-14 at 15 59 35"
src="https://github.com/user-attachments/assets/3a833224-a085-4314-86e4-434356abafd3"
/>

Author: adrw

service/src/main/kotlin/app/cash/backfila/ui/UiModule.kt

@@ -30,3 +30,10 @@ class UiModule : KAbstractModule() {
     install(WebActionModule.create<BackfillShowButtonHandlerAction>())
   }
 }
+
+/**
+ * Identifies the Misk dashboard for Backfila so dashboard links and other customization can be added.
+ */
+@Retention(AnnotationRetention.RUNTIME)
+@Target(AnnotationTarget.FUNCTION)
+annotation class BackfilaDashboard

service/src/main/kotlin/app/cash/backfila/ui/components/DashboardPageLayout.kt

@@ -5,6 +5,7 @@ import app.cash.backfila.service.BackfilaConfig
 import app.cash.backfila.service.persistence.BackfilaDb
 import app.cash.backfila.service.persistence.BackfillRunQuery
 import app.cash.backfila.service.persistence.BackfillState
+import app.cash.backfila.ui.BackfilaDashboard
 import app.cash.backfila.ui.pages.IndexAction
 import jakarta.inject.Inject
 import kotlinx.html.TagConsumer
@@ -30,6 +31,7 @@ import misk.tailwind.pages.MenuSection
 import misk.tailwind.pages.Navbar
 import misk.web.HttpCall
 import misk.web.ResponseBody
+import misk.web.dashboard.DashboardTab
 import misk.web.dashboard.HtmlLayout
 import okio.BufferedSink
 import wisp.deployment.Deployment
@@ -47,6 +49,7 @@ class DashboardPageLayout @Inject constructor(
   private val getBackfillRunsAction: GetBackfillRunsAction,
   @BackfilaDb private val transacter: Transacter,
   private val queryFactory: Query.Factory,
+  private val allTabs: List<DashboardTab>,
 ) {
   private var newBuilder = false
   private var headBlock: TagConsumer<*>.() -> Unit = {}
@@ -57,6 +60,10 @@ class DashboardPageLayout @Inject constructor(
     clientHttpCall.get().url.encodedPath
   }
 
+  private val backfilaLinks by lazy {
+    allTabs.filter { it.dashboardAnnotationKClass == BackfilaDashboard::class }
+  }
+
   private fun setNewBuilder() = apply { newBuilder = true }
 
   fun newBuilder(): DashboardPageLayout = DashboardPageLayout(
@@ -67,6 +74,7 @@ class DashboardPageLayout @Inject constructor(
     getBackfillRunsAction = getBackfillRunsAction,
     transacter = transacter,
     queryFactory = queryFactory,
+    allTabs = allTabs,
   ).setNewBuilder()
 
   fun title(title: String) = apply {
@@ -181,9 +189,30 @@ class DashboardPageLayout @Inject constructor(
               href = "/backfills/",
               isSelected = currentPath == "/backfills/",
             ),
-          ),
+          ) + backfilaLinks.filter { it.menuCategory == "Backfila" }.map { tab ->
+            Link(
+              label = tab.menuLabel,
+              href = tab.menuUrl,
+              isSelected = currentPath.startsWith(tab.menuUrl),
+            )
+          },
         ),
-      ) + if (services.isNotEmpty()) {
+      ) + if (backfilaLinks.filterNot { it.menuCategory == "Backfila" }.isNotEmpty()) {
+        backfilaLinks.filterNot { it.menuCategory == "Backfila" }.groupBy { it.menuCategory }.map { (category, tabs) ->
+          MenuSection(
+            title = category,
+            links = tabs.map { tab ->
+              Link(
+                label = tab.menuLabel,
+                href = tab.menuUrl,
+                isSelected = currentPath.startsWith(tab.menuUrl),
+              )
+            },
+          )
+        }
+      } else {
+        listOf()
+      } + if (services.isNotEmpty()) {
         listOf(
           MenuSection(
             title = "Your Services",

service/src/test/kotlin/app/cash/backfila/development/BackfilaDevelopmentService.kt

@@ -16,6 +16,7 @@ import app.cash.backfila.protos.clientservice.RunBatchResponse
 import app.cash.backfila.service.BackfilaConfig
 import app.cash.backfila.service.BackfilaServiceModule
 import app.cash.backfila.service.persistence.DbBackfillRun
+import app.cash.backfila.ui.BackfilaDashboard
 import misk.MiskApplication
 import misk.MiskRealServiceModule
 import misk.audit.NoOpAuditClientModule
@@ -28,8 +29,10 @@ import misk.jdbc.DataSourceConfig
 import misk.jdbc.DataSourceType
 import misk.security.authz.FakeCallerAuthenticator
 import misk.security.authz.MiskCallerAuthenticator
+import misk.security.authz.Unauthenticated
 import misk.web.MiskWebModule
 import misk.web.WebConfig
+import misk.web.dashboard.DashboardModule
 import okio.ByteString.Companion.encodeUtf8
 import wisp.deployment.Deployment
 
@@ -54,6 +57,22 @@ fun main(args: Array<String>) {
         multibind<MiskCallerAuthenticator>().to<FakeCallerAuthenticator>()
         bind<ViewLogsUrlProvider>().to<DevelopmentViewLogsUrlProvider>()
 
+        // Example custom link that shows up in navbar
+        install(
+          DashboardModule.createMenuLink<BackfilaDashboard, Unauthenticated>(
+            label = "Go to Repo",
+            url = "https://github.com/cashapp/backfila",
+            category = "Backfila",
+          ),
+        )
+        install(
+          DashboardModule.createMenuLink<BackfilaDashboard, Unauthenticated>(
+            label = "Test other link",
+            url = "https://github.com/cashapp/backfila",
+            category = "Alpha",
+          ),
+        )
+
         newMapBinder<String, BackfilaCallbackConnectorProvider>(ForConnectors::class)
           .permitDuplicates().addBinding("DEV")
           .toInstance(object : BackfilaCallbackConnectorProvider {