refactor(rendering): drop render options parameter 🍡

- Remove EngineRenderOptions type
- Simplify Engine.render signature and include calls
- Update ViewEngine render docs

Author: NeaByteLab

src/interfaces/Render.ts

@@ -31,15 +31,17 @@ export type DveStackFrame = { kind: 'if' | 'each'; node: AstNode; inElse: boolea
 /** Options for constructing rendering Engine. */
 export type EngineOptions = { viewsDir: string }
 
-/** Render options override for Engine (matches ViewEngine render options). */
-export type EngineRenderOptions = { viewsDir?: string }
-
-/** View engine for rendering templates (e.g. .dve); Engine implements this. */
+/**
+ * View engine for templates
+ * @description Renders DVE templates to HTML strings
+ */
 export interface ViewEngine {
-  /** Render template at path with data; returns HTML string. */
-  render(
-    templatePath: string,
-    data?: Record<string, unknown>,
-    options?: EngineRenderOptions
-  ): Promise<string>
+  /**
+   * Render template to HTML
+   * @description Renders DVE template with scope data
+   * @param templatePath - Relative template path
+   * @param data - Template scope data
+   * @returns Rendered HTML string
+   */
+  render(templatePath: string, data?: Record<string, unknown>): Promise<string>
 }

src/rendering/Engine.ts

@@ -30,16 +30,10 @@ export class Engine implements Types.ViewEngine {
    * @description Loads template and produces final HTML
    * @param templatePath - Relative template path
    * @param data - Template scope data
-   * @param options - Render options override values
    * @returns Rendered HTML string
    * @throws {Error} When template path not discovered
    */
-  async render(
-    templatePath: string,
-    data: Record<string, unknown> = {},
-    options?: Types.EngineRenderOptions
-  ): Promise<string> {
-    const viewsDir = options?.viewsDir ?? this.defaultViewsDir
+  async render(templatePath: string, data: Record<string, unknown> = {}): Promise<string> {
     if (this.discoveredPaths === null) {
       this.discoveredPaths = await Rendering.Discover.discoverPaths(this.defaultViewsDir)
     }
@@ -51,9 +45,9 @@ export class Engine implements Types.ViewEngine {
     if (!discoveredTemplatePaths.has(pathWithExt)) {
       throw new Error(`Template not found: ${templatePath}.`)
     }
-    const absPath = EngineParts.Utils.join(viewsDir, pathWithExt)
+    const absPath = EngineParts.Utils.join(this.defaultViewsDir, pathWithExt)
     const compiled = await this.compileTemplate(absPath)
-    return await this.renderNodes(compiled.ast, data, viewsDir)
+    return await this.renderNodes(compiled.ast, data, this.defaultViewsDir)
   }
 
   /**
@@ -118,7 +112,7 @@ export class Engine implements Types.ViewEngine {
         continue
       }
       if (node.type === 'include') {
-        outputHtml += await this.render(node.templatePath, data, { viewsDir })
+        outputHtml += await this.render(node.templatePath, data)
         continue
       }
       if (node.type === 'if') {