@wp-blocks/make-pot 1.6.3 → 1.6.4

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@wp-blocks/make-pot",
-	"version": "1.6.3",
+	"version": "1.6.4",
 	"license": "GPL-3.0-or-later",
 	"homepage": "https://wp-blocks.github.io/make-pot/",
 	"description": "A Node.js script for generating a POT file from source code",
@@ -50,8 +50,10 @@
 		"build": "npx esbuild ./src/**/* ./src/*.ts --format=cjs --minify --outdir=lib --platform=node",
 		"watch": "tsc --watch",
 		"lint": "npx @biomejs/biome check --write src",
+		"type-check": "npx tsc --noEmit",
+		"update-pkg": "npm upgrade -S",
 		"rm": "rmdir /s /q lib",
-		"test:build": "npx esbuild ./src/**/* --format=cjs --sourcemap --outdir=lib --platform=node",
+		"test:build": "npx esbuild ./src/**/* ./src/*.ts --format=cjs --sourcemap --outdir=lib --platform=node",
 		"test": "npm run test:build && node --test",
 		"build:build-ci": "npx esbuild ./src/index.ts --format=cjs --outdir=lib --bundle --external:tree-sitter --external:tree-sitter-typescript --external:tree-sitter-php --external:tree-sitter-javascript --external:@babel/preset-typescript --platform=node",
 		"test:ci": "npm run build:build-ci && npm run test",
@@ -59,26 +61,27 @@
 		"test:coverage": "node --test --experimental-test-coverage"
 	},
 	"dependencies": {
-		"@babel/core": "^7.27.1",
-		"@babel/preset-env": "^7.27.2",
+		"@babel/core": "^7.29.0",
+		"@babel/preset-env": "^7.29.0",
 		"cli-progress": "^3.12.0",
 		"gettext-merger": "^1.2.1",
-		"gettext-parser": "^4.0.4",
-		"glob": "^11.0.3",
+		"gettext-parser": "^4.2.0",
+		"glob": "^11.1.0",
 		"tannin": "^1.2.0",
 		"tree-sitter": "^0.21.1",
 		"tree-sitter-javascript": "^0.23.1",
 		"tree-sitter-php": "^0.23.12",
 		"tree-sitter-typescript": "^0.23.2",
-		"yargs": "^17.7.1"
+		"yargs": "^17.7.2"
 	},
 	"devDependencies": {
-		"@biomejs/biome": "2.2.4",
+		"@biomejs/biome": "2.3.14",
+		"@types/babel__core": "^7.20.5",
 		"@types/cli-progress": "^3.11.6",
 		"@types/gettext-parser": "^4.0.4",
-		"@types/node": "^22.15.17",
-		"@types/yargs": "^17.0.33",
+		"@types/node": "^22.19.11",
+		"@types/yargs": "^17.0.35",
 		"esbuild": "0.25.9",
-		"typescript": "^5.9.2"
+		"typescript": "^5.9.3"
 	}
 }

package/tests/extract-headers.test.js

@@ -1,7 +1,7 @@
 const { describe, it } = require("node:test");
 const { join } = require("node:path");
 const assert = require("node:assert");
-const { extractMainFileData } = require("../lib");
+const { extractMainFileData, getAuthorFromPackage } = require("../lib");
 
 describe("should parse plugin main file", () => {
 	describe("should parse plugin.php", () => {
@@ -52,3 +52,71 @@ describe("should parse theme main file", () => {
 		});
 	});
 });
+
+describe("getAuthorFromPackage", () => {
+	it("extracts author from string with name and email", () => {
+		const pkgJson = {
+			author: "My Name <myname@example.com>",
+		};
+		const author = getAuthorFromPackage(pkgJson);
+		assert.deepStrictEqual(author, {
+			name: "My Name",
+			email: "myname@example.com",
+			website: undefined,
+		});
+	});
+
+	it("extracts author from string with specific user format", () => {
+		const pkgJson = {
+			author: "my name <myname@asdasdasdasd.it>",
+		};
+		const author = getAuthorFromPackage(pkgJson);
+		assert.deepStrictEqual(author, {
+			name: "my name",
+			email: "myname@asdasdasdasd.it",
+			website: undefined,
+		});
+	});
+
+	it("extracts author from string with name, email and url", () => {
+		const pkgJson = {
+			author: "My Name <myname@example.com> (https://example.com)",
+		};
+		const author = getAuthorFromPackage(pkgJson);
+		assert.deepStrictEqual(author, {
+			name: "My Name",
+			email: "myname@example.com",
+			website: "https://example.com",
+		});
+	});
+
+	it("extracts author from object", () => {
+		const pkgJson = {
+			author: {
+				name: "Object Author",
+				email: "obj@example.com",
+				website: "https://obj.example.com"
+			}
+		};
+		const author = getAuthorFromPackage(pkgJson);
+		assert.deepStrictEqual(author, {
+			name: "Object Author",
+			email: "obj@example.com",
+			website: "https://obj.example.com",
+		});
+	});
+
+	it("extracts author from array of strings", () => {
+		const pkgJson = {
+			authors: [
+				"Array Author <array@example.com>"
+			]
+		};
+		const author = getAuthorFromPackage(pkgJson);
+		assert.deepStrictEqual(author, {
+			name: "Array Author",
+			email: "array@example.com",
+			website: undefined
+		});
+	});
+});

package/tests/generate-header.test.js

@@ -0,0 +1,27 @@
+const { describe, it } = require("node:test");
+const assert = require("node:assert");
+const { generateHeader } = require("../lib/extractors/headers");
+const process = require("node:process");
+
+describe("generateHeader", () => {
+    it("should return default headers when silent is true and fields are missing", async () => {
+        const args = {
+            slug: "test-slug",
+            debug: false,
+            domain: "plugin",
+            paths: { cwd: process.cwd(), out: "languages" },
+            options: { silent: true },
+            headers: {
+                version: "0.0.1",
+                author: "AUTHOR",
+                email: "AUTHOR EMAIL"
+            },
+        };
+
+        const headers = await generateHeader(args);
+
+        assert.ok(headers, "Headers should be generated");
+        assert.strictEqual(headers["Project-Id-Version"], "test-slug 0.0.1");
+        assert.strictEqual(headers["Last-Translator"], "AUTHOR <AUTHOR EMAIL>");
+    });
+});

package/tests/tree.test.js

@@ -35,7 +35,7 @@ describe("doTree php", () => {
 				translator: undefined,
 			},
 			msgctxt: undefined,
-			msgid: "You\\'re a silly monkey",
+			msgid: "You're a silly monkey",
 			msgid_plural: undefined,
 			msgstr: [""],
 		});
@@ -214,3 +214,76 @@ describe("doTree php _n, _nx", async () => {
 		assert.strictEqual(r.filter((block) => block.msgctxt === 'context').length, 1);
 	});
 });
+
+describe("doTree php escape sequences", async () => {
+	it("should correctly unescape newlines, tabs, and quotes in double-quoted strings", () => {
+		const content = `<?php
+       // Double quotes with escape sequences
+       __("Line 1\\nLine 2", "text-domain");
+
+       // Double quotes with escaped quotes
+       _e("Hello \\"World\\"", "text-domain");
+
+       // Double quotes with tabs
+       __("Col1\\tCol2", "text-domain");
+       `;
+
+		const filename = "escapes.php";
+		const r = doTree(content, filename).blocks;
+
+		// 1. Verify newline
+		const newlineBlock = r.find(b => b.msgid.includes('Line 1'));
+		// The msgid should contain an actual newline character, not the literal characters '\' and 'n'
+		assert.strictEqual(newlineBlock?.msgid, "Line 1\nLine 2");
+
+		// 2. Verify escaped quotes
+		const quoteBlock = r.find(b => b.msgid.includes('Hello'));
+		assert.strictEqual(quoteBlock?.msgid, 'Hello "World"');
+
+		// 3. Verify tabs
+		const tabBlock = r.find(b => b.msgid.includes('Col1'));
+		assert.strictEqual(tabBlock?.msgid, "Col1\tCol2");
+	});
+});
+
+describe("doTree php single quotes", async () => {
+	it("should treat escape sequences as literals in single-quoted strings", () => {
+		const content = `<?php
+       // Single quotes should NOT interpret \\n as newline
+       __('Line 1\\nLine 2', 'text-domain');
+
+       // Single quotes SHOULD handle escaped single quotes
+       __('It\\'s a sunny day', 'text-domain');
+       `;
+
+		const filename = "single_quotes.php";
+		const r = doTree(content, filename).blocks;
+
+		// 1. Verify literal \\n
+		const literalBlock = r.find(b => b.msgid.includes('Line 1'));
+		// In single quotes, \\n is two characters: backslash and n
+		assert.strictEqual(literalBlock?.msgid, "Line 1\\nLine 2");
+
+		// 2. Verify escaped single quote
+		const quoteBlock = r.find(b => b.msgid.includes('sunny'));
+		assert.strictEqual(quoteBlock?.msgid, "It's a sunny day");
+	});
+});
+
+describe("doTree php variables in strings", async () => {
+	it("should preserve PHP variables inside double-quoted strings", () => {
+		const content = `<?php
+       $name = 'John';
+       // Variable interpolation
+       __("Hello $name, how are you?", "text-domain");
+       `;
+
+		const filename = "variables.php";
+		const r = doTree(content, filename).blocks;
+
+		const varBlock = r.find(b => b.msgid.startsWith('Hello'));
+
+		// We expect the variable name to be preserved in the msgid
+		assert.strictEqual(varBlock?.msgid, "Hello $name, how are you?");
+	});
+});