@malloy-publisher/server 0.0.204 → 0.0.206

package/src/server-old.ts

@@ -30,7 +30,6 @@ import { ParsedQs } from "qs";
 import { CompileController } from "./controller/compile.controller";
 import { ConnectionController } from "./controller/connection.controller";
 import { DatabaseController } from "./controller/database.controller";
-import { ManifestController } from "./controller/manifest.controller";
 import { MaterializationController } from "./controller/materialization.controller";
 import { ModelController } from "./controller/model.controller";
 import { PackageController } from "./controller/package.controller";
@@ -57,7 +56,6 @@ export interface LegacyControllerSet {
    queryController: QueryController;
    compileController: CompileController;
    materializationController: MaterializationController;
-   manifestController: ManifestController;
 }
 
 // ─── response/body field mappers ───────────────────────────────────────────
@@ -97,7 +95,6 @@ export function registerLegacyRoutes(
       queryController,
       compileController,
       materializationController,
-      manifestController,
    } = controllers;
 
    // body-parser is already registered on the main app for `${API_PREFIX}/*`
@@ -379,26 +376,7 @@ export function registerLegacyRoutes(
       },
    );
 
-   // sqlSource (deprecated GET + supported POST), per-project + per-package
-   app.get(
-      `${LEGACY_API_PREFIX}/projects/:projectName/connections/:connectionName/sqlSource`,
-      async (req, res) => {
-         try {
-            res.status(200).json(
-               await connectionController.getConnectionSqlSource(
-                  req.params.projectName,
-                  req.params.connectionName,
-                  req.query.sqlStatement as string,
-               ),
-            );
-         } catch (error) {
-            logger.error(error);
-            const { json, status } = internalErrorToHttpError(error as Error);
-            res.status(status).json(json);
-         }
-      },
-   );
-
+   // sqlSource (POST), per-project + per-package
    app.post(
       `${LEGACY_API_PREFIX}/projects/:projectName/connections/:connectionName/sqlSource`,
       async (req, res) => {
@@ -418,26 +396,6 @@ export function registerLegacyRoutes(
       },
    );
 
-   app.get(
-      `${LEGACY_API_PREFIX}/projects/:projectName/packages/:packageName/connections/:connectionName/sqlSource`,
-      async (req, res) => {
-         try {
-            res.status(200).json(
-               await connectionController.getConnectionSqlSource(
-                  req.params.projectName,
-                  req.params.connectionName,
-                  req.query.sqlStatement as string,
-                  req.params.packageName,
-               ),
-            );
-         } catch (error) {
-            logger.error(error);
-            const { json, status } = internalErrorToHttpError(error as Error);
-            res.status(status).json(json);
-         }
-      },
-   );
-
    app.post(
       `${LEGACY_API_PREFIX}/projects/:projectName/packages/:packageName/connections/:connectionName/sqlSource`,
       async (req, res) => {
@@ -567,48 +525,7 @@ export function registerLegacyRoutes(
       },
    );
 
-   // temporaryTable (deprecated GET) + sqlTemporaryTable (supported POST)
-   app.get(
-      `${LEGACY_API_PREFIX}/projects/:projectName/connections/:connectionName/temporaryTable`,
-      queryConcurrency(),
-      async (req, res) => {
-         try {
-            res.status(200).json(
-               await connectionController.getConnectionTemporaryTable(
-                  req.params.projectName,
-                  req.params.connectionName,
-                  req.query.sqlStatement as string,
-               ),
-            );
-         } catch (error) {
-            logger.error(error);
-            const { json, status } = internalErrorToHttpError(error as Error);
-            res.status(status).json(json);
-         }
-      },
-   );
-
-   app.get(
-      `${LEGACY_API_PREFIX}/projects/:projectName/packages/:packageName/connections/:connectionName/temporaryTable`,
-      queryConcurrency(),
-      async (req, res) => {
-         try {
-            res.status(200).json(
-               await connectionController.getConnectionTemporaryTable(
-                  req.params.projectName,
-                  req.params.connectionName,
-                  req.query.sqlStatement as string,
-                  req.params.packageName,
-               ),
-            );
-         } catch (error) {
-            logger.error(error);
-            const { json, status } = internalErrorToHttpError(error as Error);
-            res.status(status).json(json);
-         }
-      },
-   );
-
+   // sqlTemporaryTable (POST), per-project + per-package
    app.post(
       `${LEGACY_API_PREFIX}/projects/:projectName/connections/:connectionName/sqlTemporaryTable`,
       queryConcurrency(),
@@ -674,11 +591,9 @@ export function registerLegacyRoutes(
       `${LEGACY_API_PREFIX}/projects/:projectName/packages`,
       async (req, res) => {
          try {
-            const autoLoadManifest = req.query.autoLoadManifest === "true";
             const _package = await packageController.addPackage(
                req.params.projectName,
                req.body,
-               { autoLoadManifest },
             );
             res.status(200).json(_package?.getPackageMetadata());
          } catch (error) {
@@ -1024,34 +939,18 @@ export function registerLegacyRoutes(
       },
    );
 
-   app.post(
-      `${LEGACY_API_PREFIX}/projects/:projectName/packages/:packageName/materializations/teardown`,
-      async (req, res) => {
-         try {
-            const result = await materializationController.teardownPackage(
-               req.params.projectName,
-               req.params.packageName,
-               req.body || {},
-            );
-            res.status(200).json(result);
-         } catch (error) {
-            const { json, status } = internalErrorToHttpError(error as Error);
-            res.status(status).json(json);
-         }
-      },
-   );
-
    app.post(
       `${LEGACY_API_PREFIX}/projects/:projectName/packages/:packageName/materializations/:materializationId`,
       async (req, res) => {
          try {
             const action = req.query.action;
-            if (action === "start") {
+            if (action === "build") {
                const build =
-                  await materializationController.startMaterialization(
+                  await materializationController.buildMaterialization(
                      req.params.projectName,
                      req.params.packageName,
                      req.params.materializationId,
+                     req.body || {},
                   );
                res.status(202).json(remapMaterializationResponse(build));
             } else if (action === "stop") {
@@ -1064,7 +963,7 @@ export function registerLegacyRoutes(
                res.status(200).json(remapMaterializationResponse(build));
             } else {
                throw new BadRequestError(
-                  `Unsupported action '${String(action ?? "")}'. Expected 'start' or 'stop'.`,
+                  `Unsupported action '${String(action ?? "")}'. Expected 'build' or 'stop'.`,
                );
             }
          } catch (error) {
@@ -1082,6 +981,7 @@ export function registerLegacyRoutes(
                req.params.projectName,
                req.params.packageName,
                req.params.materializationId,
+               { dropTables: req.query.dropTables === "true" },
             );
             res.status(204).send();
          } catch (error) {
@@ -1091,48 +991,6 @@ export function registerLegacyRoutes(
       },
    );
 
-   // ── manifest ────────────────────────────────────────────────────────────
-   app.get(
-      `${LEGACY_API_PREFIX}/projects/:projectName/packages/:packageName/manifest`,
-      async (req, res) => {
-         try {
-            const manifest = await manifestController.getManifest(
-               req.params.projectName,
-               req.params.packageName,
-            );
-            res.status(200).json(manifest);
-         } catch (error) {
-            logger.error("Get manifest error", { error });
-            const { json, status } = internalErrorToHttpError(error as Error);
-            res.status(status).json(json);
-         }
-      },
-   );
-
-   app.post(
-      `${LEGACY_API_PREFIX}/projects/:projectName/packages/:packageName/manifest`,
-      async (req, res) => {
-         try {
-            const action = req.query.action;
-            if (action === "reload") {
-               const manifest = await manifestController.reloadManifest(
-                  req.params.projectName,
-                  req.params.packageName,
-               );
-               res.status(200).json(manifest);
-            } else {
-               throw new BadRequestError(
-                  `Unsupported action '${String(action ?? "")}'. Expected 'reload'.`,
-               );
-            }
-         } catch (error) {
-            logger.error("Manifest action error", { error });
-            const { json, status } = internalErrorToHttpError(error as Error);
-            res.status(status).json(json);
-         }
-      },
-   );
-
    logger.info(
       "Legacy /projects/* routes registered for backwards compatibility",
    );