fix(webapp,clickhouse,run-engine): rename follow-ups from review

Renumbers the metrics migration to 044 (main took 042 and 043), updates
the two tests that pin the query-table and authorization ids, keeps the
combined-cap chart from drawing a zero cap for history before a cap
existed, and drops the flag docblock's stale per-key override paragraph.

Author: matt-aitken

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.queues_.$queueParam/route.tsx

@@ -499,7 +499,7 @@ function OverviewCharts({
             }
             showLegend
             className="aspect-[2/1]"
-            query={`SELECT timeBucket() AS t, max(max_total_running) AS running, least(max(max_total_limit), max(max_env_limit)) AS cap\nFROM concurrency_metrics\nGROUP BY t\nORDER BY t`}
+            query={`SELECT timeBucket() AS t, max(max_total_running) AS running, least(nullIf(max(max_total_limit), 0), max(max_env_limit)) AS cap\nFROM concurrency_metrics\nGROUP BY t\nORDER BY t`}
             fillGaps
             minBucketSeconds={SYNCED_CHART_MIN_BUCKET_SECONDS}
             ids={ids}

apps/webapp/test/dashboardAgentToolScopes.test.ts

@@ -34,7 +34,7 @@ const VIA_ENV_JWT: Read[] = [
   {
     tool: "get_queue (metrics)",
     path: "/api/v1/queues/:name/metrics",
-    resource: { type: "query", id: "queue_metrics" },
+    resource: { type: "query", id: "concurrency_metrics" },
   },
   { tool: "get_queue (live row)", path: "/api/v1/queues/:name", resource: { type: "queues" } },
   {

apps/webapp/test/reportsApiRoute.test.ts

@@ -105,7 +105,7 @@ describe("api.v1.reports.$key — authorization", () => {
     expect(requiredResources("health")).toEqual([
       { type: "query", id: "runs" },
       { type: "query", id: "env_metrics" },
-      { type: "query", id: "queue_metrics" },
+      { type: "query", id: "concurrency_metrics" },
     ]);
   });
 
@@ -118,7 +118,7 @@ describe("api.v1.reports.$key — authorization", () => {
 
 describe("reportQueryTables — scope derivation from the registry", () => {
   const registry: Record<string, { tables: readonly ReportQueryTable[] }> = {
-    health: { tables: ["runs", "env_metrics", "queue_metrics"] },
+    health: { tables: ["runs", "env_metrics", "concurrency_metrics"] },
     narrow: { tables: ["runs"] },
   };
 
@@ -127,7 +127,11 @@ describe("reportQueryTables — scope derivation from the registry", () => {
   });
 
   it("still gives the wider report all of its tables", () => {
-    expect(reportQueryTables("health", registry)).toEqual(["runs", "env_metrics", "queue_metrics"]);
+    expect(reportQueryTables("health", registry)).toEqual([
+      "runs",
+      "env_metrics",
+      "concurrency_metrics",
+    ]);
   });
 
   it("returns no tables for an unknown key", () => {

…_add_queue_metrics_total_concurrency.sql …_add_queue_metrics_total_concurrency.sqlinternal-packages/clickhouse/schema/042_add_queue_metrics_total_concurrency.sql renamed to internal-packages/clickhouse/schema/044_add_queue_metrics_total_concurrency.sql internal-packages/clickhouse/schema/042_add_queue_metrics_total_concurrency.sql renamed to internal-packages/clickhouse/schema/044_add_queue_metrics_total_concurrency.sql

@@ -318,10 +318,6 @@ export type RunQueueOptions = {
    * that dead-lettered or suspended through a mirror-less path. Enabling only after
    * every instance runs this build avoids the noise but is no longer load-bearing
    * for correctness.
-   *
-   * Per-concurrency-key limit overrides are part of the same concurrency-limits
-   * feature and are deliberately enforced behind this flag too: writes are always
-   * accepted and durable, and enforcement of both arrives together.
    */
   totalConcurrencyEnabled?: boolean;
   /**