Merge pull request #2 from codenoobforreal/videoinfo

Videoinfo

.changeset/tame-tigers-heal.md

@@ -0,0 +1,5 @@
+---
+"@codenoobforreal/commonflow": patch
+---
+
+Give the user more information about the output.

README.md

@@ -1,8 +1,12 @@
 # commonflow
 
-CLI to simplify your daily workflow
+CLI to simplify your day!
 
-## usage
+## sub command list
+
+- transcode-videos
+
+### transcode-videos
 
 ```bash
 npx @codenoobforreal/commmonflow transcode-videos /path/to/input/path /path/to/output/path

package.json

@@ -53,7 +53,6 @@
     "eslint-config-prettier": "^9.1.0",
     "globals": "^15.14.0",
     "is-in-ci": "^1.0.0",
-    "playwright": "^1.49.1",
     "prettier": "^3.4.2",
     "tsup": "^8.3.5",
     "typescript": "^5.7.2",

src/index.ts

@@ -10,7 +10,7 @@ const program = new Command();
 program
   .name("commonflow")
   .description(`CLI to simplify your daily workflow`)
-  .version("0.1.1");
+  .version("0.1.2");
 program
   .command("transcode-videos")
   .description(`Transcode all video files in the input path to the output path`)

src/tasks/transcode-video.ts

@@ -68,7 +68,7 @@ export async function runTranscodeVideosSubProgram(args: TranscodeVideoArgs) {
       if ((line as string).includes("Lsize")) {
         const size = lsizeStringCapture(line as string);
         const duration = timeStringCapture(line as string);
-        yield `output stat: ${duration} ${size}`;
+        yield `transcode stat: ${duration} ${size} ${output}`;
       }
     };
     await transcodeVideo(shellCommand, ffmpegInfoTransform);

src/utils.test.ts

@@ -1,5 +1,9 @@
 import { test, expect, describe } from "vitest";
-import { formatFileSize, formatDuration } from "./utils";
+import {
+  formatFileSize,
+  formatDuration,
+  formatFfmpegTimeString,
+} from "./utils";
 
 describe("utils function", () => {
   describe("formatFileSize", () => {
@@ -32,4 +36,14 @@ describe("utils function", () => {
       expect(formatDuration(3700)).toBe(`1h1m40s`);
     });
   });
+  describe("formatFfmpegTimeString", () => {
+    test("discard all meaningless zero", () => {
+      expect(formatFfmpegTimeString("00:00:05.80")).toBe("05.80");
+      expect(formatFfmpegTimeString("00:00:50.80")).toBe("50.80");
+      expect(formatFfmpegTimeString("00:10:05.80")).toBe("10:05.80");
+      expect(formatFfmpegTimeString("00:01:05.80")).toBe("01:05.80");
+      expect(formatFfmpegTimeString("10:10:05.80")).toBe("10:10:05.80");
+      expect(formatFfmpegTimeString("01:10:05.80")).toBe("01:10:05.80");
+    });
+  });
 });

src/utils.ts

@@ -35,6 +35,18 @@ export function formatDuration(duration: number) {
   }
 }
 
+// eg: 00:00:05.80
+export function formatFfmpegTimeString(str: string) {
+  const [h, m] = str.split(":");
+  if (h === "00") {
+    if (m === "00") {
+      return str.slice(6);
+    }
+    return str.slice(3);
+  }
+  return str;
+}
+
 export function now() {
   return new Date().getTime();
 }