Make it clearer which bit we need to be careful with

Author: benjie

src/commands/uncommit.ts

@@ -40,17 +40,20 @@ export async function _uncommit(parsedSettings: ParsedSettings): Promise<void> {
   // Restore current.sql from migration
   const lastMigrationFilepath = lastMigration.fullPath;
   const contents = await fsp.readFile(lastMigrationFilepath, "utf8");
-  const { headers, body } = parseMigrationText(lastMigrationFilepath, contents);
+  const { headers, body: committedBody } = parseMigrationText(
+    lastMigrationFilepath,
+    contents,
+  );
 
   // Replace included migrations with their `--!include` equivalent
-  const decompiledBody = body.replace(
+  const body = committedBody.replace(
     /^--! Included (?<filename>\S+)$[\s\S]*?^--! EndIncluded \k<filename>$/gm,
     (_, filename) => `--!include ${filename}`,
   );
 
   // Drop Hash, Previous and AllowInvalidHash from headers; then write out
   const { Hash, Previous, AllowInvalidHash, ...otherHeaders } = headers;
-  const completeBody = serializeMigration(decompiledBody, otherHeaders);
+  const completeBody = serializeMigration(body, otherHeaders);
   await writeCurrentMigration(parsedSettings, currentLocation, completeBody);
 
   // Delete the migration from committed and from the DB