fix(analytics): log scalar auto-inferred cubes at debug, not warn (#2223)

* chore: bump objectui to 2270239793db

feat(core,react): scoped style-object rendering (ADR-0065) (#1912)

objectui@2270239793db989e3aec2f8fccfcc1925963b2e9

* fix(analytics): log scalar auto-inferred cubes at debug, not warn

`object-metric` KPI tiles (e.g. the showcase home/My Work pages) query
analytics by the raw object name with a bare aggregate and no grouping.
No explicit Cube is registered for those names, so the service auto-infers
a trivial count/sum cube — the intended behaviour for that first-class
"metric over an object" path. Logging that at warn spams the boot/render
path ("No cube registered for showcase_task ...") for a non-problem.

Downgrade the auto-infer message to debug for scalar queries (measures
only, no `dimensions`/`timeDimensions`). Keep warn for grouped queries
over an unregistered cube, where a forgotten cube registration is a real
mistake worth surfacing.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* chore(changeset): patch @objectstack/service-analytics

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

---------

Co-authored-by: Jack Zhuang <277994282+os-zhuang@users.noreply.github.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Author: 3 people

.changeset/analytics-scalar-cube-debug-log.md

@@ -0,0 +1,13 @@
+---
+"@objectstack/service-analytics": patch
+---
+
+fix(analytics): log scalar auto-inferred cubes at debug, not warn
+
+Scalar metric queries (measures only, no `dimensions`/`timeDimensions`) over an
+unregistered cube — the first-class `object-metric` "metric over an object" path
+— auto-infer a trivial count/sum cube by design. That auto-infer now logs at
+`debug` instead of `warn`, so boot/render no longer spams
+`No cube registered for "..."` for a non-problem. Grouped queries (explicit
+dimension / time bucket) over an unregistered cube keep the `warn`, where a
+forgotten cube registration is a real mistake.

packages/services/service-analytics/src/__tests__/analytics-service.test.ts

@@ -704,3 +704,34 @@ describe('AnalyticsService — RAW_SQL_UNSUPPORTED fallback', () => {
     await expect(service.query({ cube: 'orders', measures: ['count'] })).rejects.toThrow('No strategy can handle');
   });
 });
+
+// ─────────────────────────────────────────────────────────────────
+// Auto-inferred cube log level (object-metric KPI tiles).
+//
+// A scalar metric query (measures only, no grouping) is the supported
+// "metric over an object" path — auto-inferring a count/sum cube is
+// expected, so it logs at debug, not warn. A grouped query (explicit
+// dimension / time bucket) over an unregistered cube keeps the warn,
+// since a forgotten cube registration there is a real mistake.
+
+describe('AnalyticsService — auto-inferred cube log level', () => {
+  const makeService = (logger: any) =>
+    new AnalyticsService({
+      logger,
+      executeAggregate: vi.fn().mockResolvedValue([{ count: 3 }]),
+      queryCapabilities: () => ({ nativeSql: false, objectqlAggregate: true, inMemory: false }),
+    });
+
+  it('logs at debug (not warn) for a scalar metric over an unregistered cube', async () => {
+    const logger = { info: vi.fn(), debug: vi.fn(), warn: vi.fn(), error: vi.fn(), child: vi.fn().mockReturnThis() } as any;
+    await makeService(logger).query({ cube: 'showcase_task', measures: ['count'] });
+    expect(logger.warn).not.toHaveBeenCalled();
+    expect(logger.debug).toHaveBeenCalledWith(expect.stringContaining('No cube registered for "showcase_task"'));
+  });
+
+  it('keeps warn for a grouped query over an unregistered cube', async () => {
+    const logger = { info: vi.fn(), debug: vi.fn(), warn: vi.fn(), error: vi.fn(), child: vi.fn().mockReturnThis() } as any;
+    await makeService(logger).query({ cube: 'showcase_task', measures: ['count'], dimensions: ['status'] });
+    expect(logger.warn).toHaveBeenCalledWith(expect.stringContaining('No cube registered for "showcase_task"'));
+  });
+});

packages/services/service-analytics/src/analytics-service.ts

@@ -642,12 +642,21 @@ export class AnalyticsService implements IAnalyticsService {
     if (!cube) {
       cube = this.inferCubeFromQuery(query);
       this.cubeRegistry.register(cube);
-      this.logger.warn(
+      // A scalar query — only measures, no grouping (no `dimensions`/
+      // `timeDimensions`) — is the first-class "metric over an object" path
+      // (e.g. the `object-metric` KPI widget). Auto-inferring a count/sum cube
+      // is the intended behaviour there, so log at debug. A query that groups
+      // by an explicit dimension or time bucket almost certainly meant to hit a
+      // registered cube; keep that at warn so a forgotten registration is loud.
+      const isScalarMetric =
+        (query.dimensions?.length ?? 0) === 0 && (query.timeDimensions?.length ?? 0) === 0;
+      const message =
         `[Analytics] No cube registered for "${name}"; auto-inferred a minimal cube ` +
         `(sql="${name}", measures=${Object.keys(cube.measures).join(',') || '(none)'}, ` +
         `dimensions=${Object.keys(cube.dimensions).join(',') || '(none)'}). ` +
-        `Define an explicit Cube in your stack for full control.`,
-      );
+        `Define an explicit Cube in your stack for full control.`;
+      if (isScalarMetric) this.logger.debug(message);
+      else this.logger.warn(message);
       return;
     }