mv docs (#129)

Author: p-hoffmann

.github/workflows/plugin-ci.yml

@@ -76,6 +76,7 @@ jobs:
             RUNNER: ubuntu-24.04
           - PKGPATH: ./plugins/bao
             RUNNER: ubuntu-24.04
+          - PKGPATH: ./plugins/docs
           - PKGPATH: ./plugins/runtime
 
     runs-on: ${{ matrix.RUNNER || needs.pick_runner.outputs.random_runner }}

Dockerfile

@@ -86,19 +86,16 @@ RUN curl -sLO https://github.com/posit-dev/shinylive/releases/download/v${SHINYL
     mv shinylive-${SHINYLIVE_VERSION} shinylive && \
     rm shinylive-${SHINYLIVE_VERSION}.tar.gz
 
-# Create plugins directory for plugin installs
-RUN mkdir -p ./plugins
+# Create plugins directory and symlink @trex npm packages for plugin scanner
+RUN mkdir -p ./plugins && \
+    ln -sf $(pwd)/node_modules/@trex ./plugins/@trex
 
 # Copy core (overridden by volume mount in development)
 COPY core/ ./core/
 
 # Copy functions (overridden by volume mount in development)
 COPY functions/ ./functions/
 
-# Build documentation site
-COPY core/docs/ ./core/docs/
-RUN cd core/docs && npm ci && npm run build
-
 ENV SCHEMA_DIR=/usr/src/core/schema
 ENV DUCKDB_EXTENSION_DIRECTORY=/usr/share/trexsql/extensions
 

core/server/index.ts

@@ -536,19 +536,6 @@ app.use(`${FUNCTIONS_BASE_PATH}/:service_name`, async (req, res, next) => {
   }
 });
 
-try {
-  const docsDistPath = join(Deno.cwd(), "docs", "build");
-  await Deno.stat(docsDistPath);
-  const serveDocsStatic = (await import("express")).default.static;
-  app.use(`${BASE_PATH}/docs`, serveDocsStatic(docsDistPath));
-  app.get(`${BASE_PATH}/docs/*`, (_req, res) => {
-    res.sendFile(join(docsDistPath, "index.html"));
-  });
-  console.log("Serving docs from docs/build/");
-} catch (_e) {
-  console.warn("Docs not found — /trex/docs disabled");
-}
-
 // Serve self-hosted Shinylive assets (must be before SPA catch-all)
 try {
   const shinyliveDistPath = join(Deno.cwd(), "shinylive");
@@ -566,7 +553,7 @@ try {
   const serveStatic = (await import("express")).default.static;
   app.use(BASE_PATH, serveStatic(webDistPath));
   app.get(`${BASE_PATH}/*`, (_req, res, next) => {
-    if (_req.path.startsWith(`${BASE_PATH}/api/`) || _req.path.startsWith(`${BASE_PATH}/graphql`) || _req.path.startsWith(`${BASE_PATH}/graphiql`) || _req.path.startsWith(`${BASE_PATH}/_internal`) || _req.path.startsWith(`${BASE_PATH}/docs`)) {
+    if (_req.path.startsWith(`${BASE_PATH}/api/`) || _req.path.startsWith(`${BASE_PATH}/graphql`) || _req.path.startsWith(`${BASE_PATH}/graphiql`) || _req.path.startsWith(`${BASE_PATH}/_internal`)) {
       return next();
     }
     res.sendFile(join(webDistPath, "index.html"));

core/server/plugin/plugin.ts

@@ -25,7 +25,7 @@ export class Plugins {
     source: "dev" | "npm"
   ) {
     try {
-      if (!pkg.trex) {
+      if (!pkg.trex || typeof pkg.trex !== "object") {
         console.log(
           `Plugin ${fullName} has no trex config — skipping registration`
         );

docker-compose.yml

@@ -29,7 +29,7 @@ services:
       - ./core/server:/usr/src/core/server
       - ./functions:/usr/src/functions
       - ./plugins-dev:/usr/src/plugins-dev
-      - ./core/docs:/usr/src/core/docs
+      - ./plugins/docs:/usr/src/plugins/@trex/docs
     depends_on:
       postgres:
         condition: service_healthy

package.json

@@ -15,6 +15,7 @@
     "@trex/migration": "latest",
     "@trex/pgwire": "latest",
     "@trex/tpm": "latest",
+    "@trex/docs": "latest",
     "@trex/transform": "latest"
   }
 }

plugins/docs/.npmrc

@@ -0,0 +1 @@
+@trex:registry=https://pkgs.dev.azure.com/data2evidence/d2e/_packaging/trex/npm/registry/