Revert "claude: Fix bundler async cycles with refactoring and dynamic imports"

This reverts commit 3a81483.

Author: gordonwoodhull

src/command/call/cmd.ts

@@ -1,5 +1,5 @@
 import { Command } from "cliffy/command/mod.ts";
-import { engineCommand } from "./engine-cmd.ts";
+import { engineCommand } from "../../execute/engine.ts";
 import { buildTsExtensionCommand } from "./build-ts-extension/cmd.ts";
 
 export const callCommand = new Command()

src/command/call/engine-cmd.ts

@@ -4,6 +4,7 @@
  * Copyright (C) 2020-2022 Posit Software, PBC
  */
 
+import { render } from "../render/render-shared.ts";
 import type { RenderServiceWithLifetime } from "../render/types.ts";
 
 /**
@@ -38,9 +39,6 @@ export interface CheckRenderResult {
 export async function checkRender(
   options: CheckRenderOptions,
 ): Promise<CheckRenderResult> {
-  // Dynamic import to break cycle
-  const { render } = await import("../render/render-shared.ts");
-
   const { content, services } = options;
 
   // Create temporary file

src/command/check/check-render.ts

@@ -8,7 +8,7 @@ import { initYamlIntelligenceResourcesFromFilesystem } from "../core/schema/util
 import { projectContext } from "../project/project-context.ts";
 import { notebookContext } from "../render/notebook/notebook-context.ts";
 import { reorderEngines } from "../execute/engine.ts";
-import type { ProjectContext } from "../project/types.ts";
+import { ProjectContext } from "../project/types.ts";
 
 /**
  * Create a minimal "zero-file" project context for loading bundled engine extensions

src/command/command-utils.ts

@@ -40,6 +40,7 @@ import { Command } from "cliffy/command/mod.ts";
 import { quartoAPI } from "../core/quarto-api.ts";
 import { satisfies } from "semver/mod.ts";
 import { quartoConfig } from "../core/quarto.ts";
+import { initializeProjectContextAndEngines } from "../command/command-utils.ts";
 
 const kEngines: Map<string, ExecutionEngineDiscovery> = new Map();
 
@@ -375,3 +376,42 @@ export function projectIgnoreGlobs(dir: string) {
     gitignoreEntries(dir).map((ignore) => `**/${ignore}**`),
   );
 }
+
+export const engineCommand = new Command()
+  .name("engine")
+  .description(
+    `Access functionality specific to quarto's different rendering engines.`,
+  )
+  .stopEarly()
+  .arguments("<engine-name:string> [args...:string]")
+  .action(async (options, engineName: string, ...args: string[]) => {
+    // Initialize project context and register external engines
+    await initializeProjectContextAndEngines();
+
+    // Get the engine (now includes external ones)
+    const engine = executionEngine(engineName);
+    if (!engine) {
+      console.error(`Unknown engine: ${engineName}`);
+      console.error(
+        `Available engines: ${
+          executionEngines().map((e) => e.name).join(", ")
+        }`,
+      );
+      Deno.exit(1);
+    }
+
+    if (!engine.populateCommand) {
+      console.error(`Engine ${engineName} does not support subcommands`);
+      Deno.exit(1);
+    }
+
+    // Create temporary command and let engine populate it
+    const engineSubcommand = new Command()
+      .description(
+        `Access functionality specific to the ${engineName} rendering engine.`,
+      );
+    engine.populateCommand(engineSubcommand);
+
+    // Recursively parse remaining arguments
+    await engineSubcommand.parse(args);
+  });