rbxts-transformer-commit-migrations 1.0.1 → 1.1.0

package/bin/init.js

@@ -15,7 +15,7 @@ if (!existsSync(huskyDir)) {
 	process.exit(1);
 }
 
-const setupResolve = `npx resolve-commit-migrations\n`;
+const setupResolve = `npx resolve-commit-migrations | xargs git add\n`;
 
 if (existsSync(hookFile)) {
 	const existing = readFileSync(hookFile, "utf-8");

package/out/cli/resolveMacros.js

@@ -24,16 +24,13 @@ const files = glob.sync("src/**/*.ts");
 const migrations = [];
 const migrationData = {};
 const usedOrders = new Set();
+const changed = [];
 function walk(node, ctx) {
     if (ts.isCallExpression(node) && ts.isIdentifier(node.expression) && MACRO_REGEX.test(node.expression.text)) {
         const entry = findEntryName(node);
         const orderArg = node.arguments[0];
         const order = orderArg ? ts.isNumericLiteral(orderArg) ? parseInt(orderArg.getText()) : null : 0;
-        if (order === null) {
-            console.error("Recieved null order when resolving migration macros.");
-            process.exit(1);
-        }
-        if (usedOrders.has(order)) {
+        if (order !== null && usedOrders.has(order)) {
             console.error("Multiple of the same migration orders found. Every migration has to have a unique order.");
             process.exit(1);
         }
@@ -42,8 +39,10 @@ function walk(node, ctx) {
             process.exit(1);
         }
         if (!orderArg?.getText().includes("__resolved")) {
+            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
             usedOrders.add(order);
-            migrationData[entry] = { timestamp: Date.now(), order };
+            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+            migrationData[entry] = { timestamp: Date.now(), order: order };
             ctx.changed = true;
         }
         migrations.push(entry);
@@ -58,6 +57,7 @@ for (const file of files) {
     const ctx = { changed: false };
     walk(sourceFile, ctx);
     if (ctx.changed) {
+        changed.push(file);
         writeFileSync(file, src.replaceAll(MACRO_CALL_REGEX, `${MACRO}("__resolved")`));
     }
 }
@@ -68,8 +68,10 @@ if (removedMigrations.length > 0) {
     process.exit(1);
 }
 const newMigrations = migrations.filter((v) => !previousMigrations.includes(v));
-if (newMigrations.length === 0)
+if (newMigrations.length === 0) {
+    console.error("No new migrations to resolve.");
     process.exit(0);
+}
 const newLines = newMigrations
     .map((e) => `\t${e}: { timestamp: ${migrationData[e].timestamp}, order: ${migrationData[e].order} },`)
     .join("\n");
@@ -86,4 +88,8 @@ else {
     const updated = lines.join("\n").replace(/\};\s*$/, `${newLines}\n};`);
     writeFileSync(CONFIG, updated);
 }
-console.log(`Successfuly resolved ${newMigrations.length} new migrations. [${newMigrations}]`);
+changed.push(CONFIG);
+console.error(`Successfuly resolved ${newMigrations.length} new migrations. [${newMigrations}]`);
+for (const path of changed) {
+    console.log(path.replace(/\\/g, "/"));
+}

package/out/test.js

@@ -0,0 +1,7 @@
+import { $migrate } from "../index.js";
+function test(b) {
+    return 0;
+}
+export const abc = {
+    hello: test($migrate(0))
+};

package/out/util/loadMigrationEntries.js

@@ -30,7 +30,6 @@ export default function (config) {
                 continue;
             result.push(prop.name.text);
             object[prop.name.text] = reconstructProperties(prop.initializer.properties);
-            console.log(reconstructProperties(prop.initializer.properties));
         }
         return [result, object];
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rbxts-transformer-commit-migrations",
-  "version": "1.0.1",
+  "version": "1.1.0",
   "description": "rbxts transformer used in user data migration builder chains",
   "main": "out/index.js",
   "type": "module",