@lobb-js/lobb-ext-reports 0.10.4 → 0.11.1

package/extensions/reports/workflows.ts

@@ -0,0 +1,73 @@
+import type { Lobb, Workflow } from "@lobb-js/core";
+import type { Context } from "hono";
+
+export const workflows: Workflow[] = [
+  {
+    // Intercept GET /api/collections/reports_charts/:id when ?action=run_query
+    // is present — runs the chart's parsed function and short-circuits the
+    // default findOne behaviour by throwing a Response.
+    name: "reports_chartsRunQueryHandler",
+    eventName: "core.controllers.preFindOne",
+    handler: async (input, ctx) => {
+      if (input.collectionName !== "reports_charts") return input;
+
+      const c = input.context as Context;
+      const actionQuery = c.req.query("action");
+      if (!actionQuery) return input;
+
+      if (actionQuery !== "run_query") {
+        throw new ctx.LobbError({
+          code: "BAD_REQUEST",
+          message: `The passed (${actionQuery}) action query param is not supported`,
+        });
+      }
+
+      const lobb = c.get("lobb") as Lobb;
+      const reportRecordId = input.id;
+      const contextParam = c.req.query("context");
+      const context = JSON.parse(contextParam ?? "{}");
+
+      const driver = lobb.databaseService.getDriver();
+      const pool = driver.getConnection();
+      using client = await pool.connect();
+
+      const chartRecord = (await lobb.collectionService.findOne({
+        collectionName: "reports_charts",
+        client,
+        id: reportRecordId,
+      })).data;
+
+      if (!chartRecord) {
+        throw new ctx.LobbError({
+          code: "NOT_FOUND",
+          message: `The chart you are looking for does not exist.`,
+        });
+      }
+
+      const pendingQueries: Promise<unknown>[] = [];
+
+      async function query(sql: string, args: Record<string, any>) {
+        const promise = client.query(sql, args).then((result) => result.rows);
+        pendingQueries.push(promise);
+        return await promise;
+      }
+
+      try {
+        const chartReturnedValue = await lobb.utils.runParsedFunction(
+          chartRecord.chart,
+          query,
+          context,
+        );
+        await Promise.allSettled(pendingQueries);
+        throw new Response(JSON.stringify(chartReturnedValue), {
+          status: 200,
+          headers: { "Content-Type": "application/json" },
+        });
+      } catch (error) {
+        if (error instanceof Response) throw error;
+        await Promise.allSettled(pendingQueries);
+        throw error;
+      }
+    },
+  },
+];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lobb-js/lobb-ext-reports",
-  "version": "0.10.4",
+  "version": "0.11.1",
   "license": "UNLICENSED",
   "type": "module",
   "publishConfig": {
@@ -18,9 +18,8 @@
     }
   },
   "scripts": {
-    "test": "bun run test:lobb && bun run test:studio",
+    "test": "bun test extensions/reports/tests",
     "test:lobb": "bun test extensions/reports/tests",
-    "test:studio": "bun --bun playwright test --config extensions/reports/studio/tests/playwright.config.cjs",
     "dev": "bun run --watch lobb.ts",
     "dev:studio": "vite dev",
     "build": "vite build",
@@ -28,13 +27,12 @@
     "prepare": "svelte-kit sync || echo ''",
     "preview": "vite preview",
     "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json",
-    "postinstall": "bun --bun playwright install chromium",
     "prepublishOnly": "lobb-ext-prepublish",
     "postpublish": "lobb-ext-postpublish",
     "package": "svelte-package --input extensions/reports/studio"
   },
   "dependencies": {
-    "@lobb-js/core": "^0.31.9",
+    "@lobb-js/core": "^0.32.1",
     "chart.js": "^4.4.8",
     "gridstack": "^12.6.0",
     "hono": "^4.7.0",
@@ -45,8 +43,7 @@
   },
   "devDependencies": {
     "@faker-js/faker": "^9.6.0",
-    "@playwright/test": "^1.58.2",
-    "@lobb-js/studio": "^0.28.5",
+    "@lobb-js/studio": "^0.29.1",
     "@lucide/svelte": "^0.563.1",
     "@sveltejs/adapter-node": "^5.5.4",
     "@sveltejs/kit": "^2.60.1",

package/dist/tests/analytics.spec.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/tests/analytics.spec.js

@@ -1,17 +0,0 @@
-import { test, expect } from "@playwright/test";
-const ANALYTICS_URL = "/studio/extensions/reports/analytics";
-test.describe("Analytics page", () => {
-    test.beforeEach(async ({ page }) => {
-        await page.goto(ANALYTICS_URL);
-    });
-    test("shows the analytics page", async ({ page }) => {
-        await expect(page).toHaveURL(new RegExp(ANALYTICS_URL));
-    });
-    test("shows empty state when no report is selected", async ({ page }) => {
-        await expect(page.getByText("No report selected")).toBeVisible();
-        await expect(page.getByText("Please select a report to view the details or create a new one")).toBeVisible();
-    });
-    test("shows create a report button in empty state", async ({ page }) => {
-        await expect(page.getByRole("button", { name: "Create a report" }).first()).toBeVisible();
-    });
-});

package/dist/tests/package.json

@@ -1 +0,0 @@
-{"type": "commonjs"}

package/dist/tests/playwright.config.cjs

@@ -1,27 +0,0 @@
-const { defineConfig, devices } = require("@playwright/test");
-
-module.exports = defineConfig({
-  testDir: __dirname,
-  testMatch: "**/*.spec.ts",
-  fullyParallel: true,
-  retries: 0,
-  expect: { timeout: 15000 },
-  use: {
-    baseURL: "http://localhost:3000",
-    trace: "on-first-retry",
-    headless: true,
-  },
-  projects: [
-    {
-      name: "chromium",
-      use: { ...devices["Desktop Chrome"] },
-    },
-  ],
-  // Automatically start the dev server before running tests
-  webServer: {
-    command: "bun run dev",
-    url: "http://localhost:3000/studio",
-    reuseExistingServer: true,
-    cwd: "../../../../", // packages/reports-ext root
-  },
-});

package/dist/tests/playwright.config.d.cts

@@ -1,2 +0,0 @@
-declare const _exports: import("@playwright/test").Config<import("@playwright/test").PlaywrightTestOptions & {}, import("@playwright/test").PlaywrightWorkerOptions & {}>;
-export = _exports;

package/extensions/reports/collectionRoutes.ts

@@ -1,10 +0,0 @@
-import type { Hono } from "hono";
-
-import { ChartsController } from "./controllers/chartsController.ts";
-
-export function collectionRoutes(route: Hono) {
-  route.get(
-    "/reports_charts/:id",
-    ChartsController.findOne,
-  );
-}

package/extensions/reports/controllers/chartsController.ts

@@ -1,78 +0,0 @@
-import type { Context } from "hono";
-import type { Lobb } from "@lobb-js/core";
-import { LobbError } from "@lobb-js/core";
-
-export class ChartsController {
-  public static async findOne(c: Context) {
-    const lobb = c.get("lobb") as Lobb;
-    const reportRecordId = c.req.param("id");
-    const actionQuery = c.req.query("action");
-    const context = c.req.query("context");
-
-    if (actionQuery) {
-      if (actionQuery === "run_query") {
-        return await handleRunQuery(
-          reportRecordId,
-          lobb,
-          c,
-          JSON.parse(context ?? "{}"),
-        );
-      } else {
-        throw new LobbError({
-          code: "BAD_REQUEST",
-          message:
-            `The passed (${actionQuery}) action query param is not supported`,
-        });
-      }
-    }
-
-    return await lobb.collectionControllers.findOne(
-      c,
-    ) as Response;
-  }
-}
-
-async function handleRunQuery(
-  reportRecordId: string,
-  lobb: Lobb,
-  c: Context,
-  context: Record<string, unknown>,
-) {
-  const driver = lobb.databaseService.getDriver();
-  const pool = driver.getConnection();
-  using client = await pool.connect();
-
-  const chartRecord = (await lobb.collectionService.findOne({
-    collectionName: "reports_charts",
-    client,
-    id: reportRecordId,
-  })).data;
-
-  if (!chartRecord) {
-    throw new LobbError({
-      code: "NOT_FOUND",
-      message: `The chart you are looking for does not exist.`,
-    });
-  }
-
-  const pendingQueries: Promise<unknown>[] = [];
-
-  async function query(sql: string, args: Record<string, any>) {
-    const promise = client.query(sql, args).then((result) => result.rows);
-    pendingQueries.push(promise);
-    return await promise;
-  }
-
-  try {
-    const chartReturnedValue = await lobb.utils.runParsedFunction(
-      chartRecord.chart,
-      query,
-      context,
-    );
-    await Promise.allSettled(pendingQueries);
-    return c.json(chartReturnedValue, 200);
-  } catch (error) {
-    await Promise.allSettled(pendingQueries);
-    throw error;
-  }
-}

package/extensions/reports/studio/tests/analytics.spec.ts

@@ -1,26 +0,0 @@
-import { test, expect } from "@playwright/test";
-
-const ANALYTICS_URL = "/studio/extensions/reports/analytics";
-
-test.describe("Analytics page", () => {
-  test.beforeEach(async ({ page }) => {
-    await page.goto(ANALYTICS_URL);
-  });
-
-  test("shows the analytics page", async ({ page }) => {
-    await expect(page).toHaveURL(new RegExp(ANALYTICS_URL));
-  });
-
-  test("shows empty state when no report is selected", async ({ page }) => {
-    await expect(page.getByText("No report selected")).toBeVisible();
-    await expect(
-      page.getByText("Please select a report to view the details or create a new one")
-    ).toBeVisible();
-  });
-
-  test("shows create a report button in empty state", async ({ page }) => {
-    await expect(
-      page.getByRole("button", { name: "Create a report" }).first()
-    ).toBeVisible();
-  });
-});

package/extensions/reports/studio/tests/package.json

@@ -1 +0,0 @@
-{"type": "commonjs"}

package/extensions/reports/studio/tests/playwright.config.cjs

@@ -1,27 +0,0 @@
-const { defineConfig, devices } = require("@playwright/test");
-
-module.exports = defineConfig({
-  testDir: __dirname,
-  testMatch: "**/*.spec.ts",
-  fullyParallel: true,
-  retries: 0,
-  expect: { timeout: 15000 },
-  use: {
-    baseURL: "http://localhost:3000",
-    trace: "on-first-retry",
-    headless: true,
-  },
-  projects: [
-    {
-      name: "chromium",
-      use: { ...devices["Desktop Chrome"] },
-    },
-  ],
-  // Automatically start the dev server before running tests
-  webServer: {
-    command: "bun run dev",
-    url: "http://localhost:3000/studio",
-    reuseExistingServer: true,
-    cwd: "../../../../", // packages/reports-ext root
-  },
-});