@jacobknightley/fabric-format 0.0.2 → 0.0.4

package/README.md

@@ -50,13 +50,11 @@ Format Fabric notebooks directly in your browser with a single click.
 
 1. Download `fabric-format-chromium.zip` from the [latest release](https://github.com/jacobknightley/fabric-format/releases)
 2. Extract the zip file
-3. Open your browser's extension page:
-   - **Chrome:** `chrome://extensions`
-   - **Edge:** `edge://extensions`
-4. Enable **Developer mode** (toggle in top-right)
-5. Click **Load unpacked** and select the extracted folder
+3. Load the unpacked extension in your browser:
+   - **Chrome:** [Install an unpacked extension](https://developer.chrome.com/docs/extensions/get-started/tutorial/hello-world#load-unpacked)
+   - **Edge:** [Sideload an extension](https://learn.microsoft.com/en-us/microsoft-edge/extensions-chromium/getting-started/extension-sideloading)
 
-> **Note:** Plan to eventually support directly in Chrome and Edge extension stores.
+> **Note:** Plan to eventually publish to the Chrome Web Store and Edge Add-ons.
 
 ### Usage
 

package/dist/formatters/python/python-formatter.js

@@ -8,6 +8,56 @@ import { RUFF_WASM_CONFIG } from './config.js';
 // Dynamic import for ruff WASM (loaded on demand)
 let ruffModule = null;
 let workspace = null;
+/**
+ * Detect if we're running in Node.js
+ */
+function isNodeEnvironment() {
+    return typeof process !== 'undefined' &&
+        process.versions != null &&
+        process.versions.node != null;
+}
+/**
+ * Find the WASM file path relative to the ruff-wasm-web package in Node.js.
+ * Uses indirect dynamic imports to avoid bundler static analysis.
+ */
+async function findWasmFileForNode() {
+    // Use Function constructor to create dynamic import that bundlers can't statically analyze
+    // This is intentional - these modules only exist in Node.js, not in browsers
+    const dynamicImport = new Function('specifier', 'return import(specifier)');
+    const { createRequire } = await dynamicImport('module');
+    const { dirname, join } = await dynamicImport('path');
+    const { readFile } = await dynamicImport('fs/promises');
+    // Get the path to ruff-wasm-web package
+    // We need import.meta.url to create a require function
+    // Use a fallback for bundled environments (though this path shouldn't be hit in browsers)
+    let ruffWasmDir;
+    try {
+        const require = createRequire(import.meta.url);
+        const ruffWasmPath = require.resolve('@astral-sh/ruff-wasm-web');
+        ruffWasmDir = dirname(ruffWasmPath);
+    }
+    catch {
+        // Fallback: try to find it via node_modules traversal
+        const { fileURLToPath } = await dynamicImport('url');
+        const currentDir = dirname(fileURLToPath(import.meta.url));
+        // Walk up to find node_modules
+        let searchDir = currentDir;
+        const { existsSync } = await dynamicImport('fs');
+        while (searchDir !== dirname(searchDir)) {
+            const candidate = join(searchDir, 'node_modules', '@astral-sh', 'ruff-wasm-web');
+            if (existsSync(candidate)) {
+                ruffWasmDir = candidate;
+                break;
+            }
+            searchDir = dirname(searchDir);
+        }
+        if (!ruffWasmDir) {
+            throw new Error('Could not locate @astral-sh/ruff-wasm-web package');
+        }
+    }
+    const wasmPath = join(ruffWasmDir, 'ruff_wasm_bg.wasm');
+    return readFile(wasmPath);
+}
 /**
  * Python formatter using Ruff WASM.
  */
@@ -43,9 +93,13 @@ export class PythonFormatter {
                 // Use async initialization with provided URL
                 await ruffModule.default({ module_or_path: this.wasmOptions.wasmUrl });
             }
+            else if (isNodeEnvironment()) {
+                // Node.js: Load WASM file from disk
+                const wasmBinary = await findWasmFileForNode();
+                ruffModule.initSync({ module: wasmBinary });
+            }
             else {
-                // Default: let ruff-wasm-web use import.meta.url to find the WASM file
-                // This works in Node.js and ESM environments but may fail in bundled IIFE
+                // Browser: let ruff-wasm-web use import.meta.url to find the WASM file
                 await ruffModule.default();
             }
             // Create workspace with config

package/dist/index.d.ts

@@ -12,7 +12,7 @@
 export { formatSql, needsFormatting } from './formatters/sparksql/index.js';
 export { getFormatterRegistry, detectLanguage, SqlFormatter, getSqlFormatter, isSqlCode, PythonFormatter, getPythonFormatter, isPythonCode, type LanguageFormatter, type FormatterOptions, type FormatResult, type FormatterConfig, type FormatterRegistry, } from './formatters/index.js';
 export { formatCell, formatCellSync, initializePythonFormatter, isPythonFormatterReady, type FormatCellResult, type CellType, } from './cell-formatter.js';
-export { parseNotebook, formatNotebook, type NotebookCell, type FabricNotebook, type FormatStats, } from './notebook-formatter.js';
+export { parseNotebook, formatNotebook, NotebookStructureError, type NotebookCell, type FabricNotebook, type FormatStats, } from './notebook-formatter.js';
 export { DEFAULT_RUFF_CONFIG, RUFF_WASM_CONFIG, type RuffConfig, type RuffFormatConfig, type WasmInitOptions, } from './formatters/python/index.js';
 export { hasFormatOff, detectCollapseDirectives, hasCollapseDirective, type FormatDirectiveInfo } from './formatters/sparksql/index.js';
 export type { AnalyzerResult, FormattingState, MultiArgFunctionInfo, WindowDefInfo, TokenContext, PendingComment, ExpandedFunction, ExpandedWindow } from './formatters/sparksql/types.js';

package/dist/index.js

@@ -30,7 +30,7 @@ export { formatCell, formatCellSync, initializePythonFormatter, isPythonFormatte
 // ============================================================================
 // Notebook Formatter (High-level API)
 // ============================================================================
-export { parseNotebook, formatNotebook, } from './notebook-formatter.js';
+export { parseNotebook, formatNotebook, NotebookStructureError, } from './notebook-formatter.js';
 // ============================================================================
 // Configuration (Python/Ruff)
 // ============================================================================

package/dist/notebook-formatter.d.ts

@@ -79,6 +79,14 @@ export interface FormatStats {
     cellsSkipped: number;
     errors: string[];
 }
+/** Error thrown when a notebook cell has invalid structure */
+export declare class NotebookStructureError extends Error {
+    readonly cellIndex: number;
+    readonly lineNumber: number;
+    readonly metadataLanguage: string;
+    readonly fileDefaultLanguage: string;
+    constructor(message: string, cellIndex: number, lineNumber: number, metadataLanguage: string, fileDefaultLanguage: string);
+}
 /**
  * Parse a Fabric notebook file into cells.
  * @param content The file content

package/dist/notebook-formatter.js

@@ -66,12 +66,68 @@ const R_CONFIG = {
     fabricHeader: '# Fabric notebook source', // Check if R has different header
     defaultLanguage: 'r',
 };
+/** Error thrown when a notebook cell has invalid structure */
+export class NotebookStructureError extends Error {
+    cellIndex;
+    lineNumber;
+    metadataLanguage;
+    fileDefaultLanguage;
+    constructor(message, cellIndex, lineNumber, metadataLanguage, fileDefaultLanguage) {
+        super(message);
+        this.cellIndex = cellIndex;
+        this.lineNumber = lineNumber;
+        this.metadataLanguage = metadataLanguage;
+        this.fileDefaultLanguage = fileDefaultLanguage;
+        this.name = 'NotebookStructureError';
+    }
+}
 // ============================================================================
 // INTERNAL UTILITIES
 // ============================================================================
-// ============================================================================
-// INTERNAL UTILITIES
-// ============================================================================
+/**
+ * Get the valid raw (uncommented) languages for a file type.
+ * Raw cells must match the file's native language - other languages must use MAGIC prefix.
+ */
+function getValidRawLanguages(defaultLanguage) {
+    switch (defaultLanguage) {
+        case 'python':
+            // Python files: raw cells can be python or pyspark
+            return new Set(['python', 'pyspark']);
+        case 'sparksql':
+            // SQL files: raw cells must be sparksql
+            return new Set(['sparksql']);
+        case 'scala':
+            // Scala files: raw cells must be scala
+            return new Set(['scala']);
+        case 'r':
+            // R files: raw cells must be r
+            return new Set(['r']);
+        default:
+            return new Set([defaultLanguage]);
+    }
+}
+/**
+ * Validate that a raw cell's metadata language matches the file type.
+ * Throws NotebookStructureError if there's a mismatch.
+ */
+function validateRawCellLanguage(metadataLanguage, isRawCell, config, cellIndex, lineNumber) {
+    if (!isRawCell || !metadataLanguage) {
+        // MAGIC cells can have any language, and cells without metadata are fine
+        return;
+    }
+    const validRawLanguages = getValidRawLanguages(config.defaultLanguage);
+    const normalizedLanguage = metadataLanguage.toLowerCase();
+    if (!validRawLanguages.has(normalizedLanguage)) {
+        const fileType = config.defaultLanguage === 'python' ? '.py' :
+            config.defaultLanguage === 'sparksql' ? '.sql' :
+                config.defaultLanguage === 'scala' ? '.scala' :
+                    config.defaultLanguage === 'r' ? '.r' :
+                        `.${config.defaultLanguage}`;
+        throw new NotebookStructureError(`Invalid notebook structure: Cell ${cellIndex + 1} (line ${lineNumber + 1}) has metadata language "${metadataLanguage}" ` +
+            `but is not wrapped with MAGIC prefix. In ${fileType} files, only ${config.defaultLanguage} cells can be raw/uncommented. ` +
+            `Other languages (like ${metadataLanguage}) must use the "${config.magicPrefix}%%" prefix.`, cellIndex, lineNumber, metadataLanguage, config.defaultLanguage);
+    }
+}
 /**
  * Line ending constant - this library standardizes on LF.
  */
@@ -291,6 +347,9 @@ export function parseNotebook(content, fileExtension) {
                 const magicCommand = extractMagicCommand(originalLines, config);
                 // Check if it's a MAGIC cell
                 const isMagicCell = originalLines.some(l => l.trim().startsWith(config.magicPrefix.trim()));
+                // Validate that raw cells have a language matching the file type
+                const cellIndex = result.cells.length;
+                validateRawCellLanguage(metadataLanguage, !isMagicCell, config, cellIndex, j);
                 // Extract content
                 let content;
                 let contentStartLine = j;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jacobknightley/fabric-format",
-  "version": "0.0.2",
+  "version": "0.0.4",
   "type": "module",
   "description": "A fast, opinionated formatter for Microsoft Fabric notebooks with Spark SQL and Python support",
   "main": "dist/index.js",