svelte-django-gettext 1.0.2 → 1.1.0

package/build.js

@@ -4,6 +4,11 @@ import { writeFileSync } from 'fs';
  *
  * Strings are collected per file, and written to the (.gitignored) `_svelte_extracted_msgids.js`
  * which is parsable by vanilla django `./manage.py makemessages`.
+ *
+ * This preprocessor only reads the markup, it never rewrites it. It still has to return the
+ * content together with an identity sourcemap: a svelte preprocessor returning `undefined` is
+ * treated as "transformed, but no sourcemap available" by the build tooling, which makes
+ * bundlers (vite >= 8.3) emit SOURCEMAP_BROKEN warnings and degrades sourcemaps downstream.
  */
 const GETTEXT_REGEXP = /((n|p)?gettext\(.*?\))/g;
 const extracts = {};
@@ -22,13 +27,25 @@ export function writeGettextExtracts(extractsFile = '_svelte_django_extracted_me
 }
 
 export async function gettextStringExtractor({ content, filename }) {
-    if (content.indexOf('gettext') === -1) {
-        return;
-    }
-    let fileExtracts = content.match(GETTEXT_REGEXP);
-    if (fileExtracts === null) {
-        return;
+    if (content.indexOf('gettext') !== -1) {
+        let fileExtracts = content.match(GETTEXT_REGEXP);
+        if (fileExtracts !== null) {
+            extracts[filename] = fileExtracts.map((x) => x.replace(', true)', ')') + ';');
+        }
     }
-    extracts[filename] = fileExtracts.map((x) => x.replace(', true)', ')') + ';');
-    return;
-}
+
+    // Content is returned unchanged, so an empty mappings string is a valid identity sourcemap.
+    return { code: content, map: identitySourcemap(filename) };
+}
+
+/* An identity sourcemap for an unmodified file: every position maps to itself, which is what
+ * an empty `mappings` string means. `names`/`sourcesContent` are left empty deliberately.
+ */
+function identitySourcemap(filename) {
+    return {
+        version: 3,
+        sources: [filename],
+        names: [],
+        mappings: '',
+    };
+}

package/package.json

@@ -1,14 +1,15 @@
 {
     "name": "svelte-django-gettext",
-    "version": "1.0.2",
+    "version": "1.1.0",
     "description": "Use django gettext in Svelte",
     "author": "Jan Pieter Waagmeester",
     "license": "ISC",
     "main": "index.js",
     "type": "module",
     "scripts": {
-        "test": "vitest",
-        "eslint": "eslint -c package.json --fix *.js test/"
+        "test": "vitest run",
+        "eslint": "eslint --fix .",
+        "test:watch": "vitest"
     },
     "keywords": [
         "Svelte",
@@ -20,59 +21,16 @@
         ".": "./index.js",
         "./build": "./build.js"
     },
+    "files": [
+        "build.js",
+        "index.js",
+        "README.md"
+    ],
     "devDependencies": {
-        "eslint": "^8.3.0",
-        "eslint-plugin-import": "^2.25.3",
-        "jsdom": "^23.2.0",
-        "vitest": "^1.2.1"
-    },
-    "jest": {
-        "testEnvironment": "jsdom"
-    },
-    "eslintConfig": {
-        "env": {
-            "browser": true,
-            "es2021": true
-        },
-        "extends": "eslint:recommended",
-        "parserOptions": {
-            "ecmaVersion": 13,
-            "sourceType": "module"
-        },
-        "plugins": [
-            "import"
-        ],
-        "rules": {
-            "indent": [
-                "error",
-                4,
-                {
-                    "SwitchCase": 1
-                }
-            ],
-            "quotes": [
-                "error",
-                "single"
-            ],
-            "import/order": [
-                "error",
-                {
-                    "groups": [
-                        "builtin",
-                        "external",
-                        "internal",
-                        [
-                            "parent",
-                            "sibling"
-                        ]
-                    ],
-                    "newlines-between": "always",
-                    "alphabetize": {
-                        "order": "asc",
-                        "caseInsensitive": true
-                    }
-                }
-            ]
-        }
+        "@eslint/js": "^10.0.1",
+        "eslint": "^10.10.0",
+        "globals": "^17.12.0",
+        "jsdom": "^30.1.0",
+        "vitest": "^5.0.1"
     }
 }

package/vite.config.js

@@ -1,8 +0,0 @@
-import { defineConfig } from 'vite';
-
-export default defineConfig({
-    test: {
-        globals: true,
-        environment: 'jsdom',
-    }
-});