fix blacha#6

Author: renhiyama

README.md

@@ -21,7 +21,7 @@ Create a `.hyperfine.json`
 ```json
 [
   { "name": "sleep", "command": "sleep 0.1" },
-  { "name": "node", "command": "node -e 'console.log()'" }
+  { "name": "node", "command": "node -e 'console.log()'", "extraArgs": { "shell": "zsh" } }
 ]
 ```
 

hyperfine.example.json

@@ -1,4 +1,18 @@
 [
-  { "name": "sleep", "command": "sleep 0.1" },
-  { "name": "node", "command": "node -e 'console.log()'" }
-]
+  {
+    "name": "sleep",
+    "command": "sleep 0.1"
+  },
+  {
+    "name": "node",
+    "command": "node -e 'console.log()'"
+  },
+  {
+    "name": "extra arguments test",
+    "command": "for i in {1..10000}; do echo test; done",
+    "extraArgs": {
+      "shell": "zsh",
+      "L": "compiler gcc,clang"
+    }
+  }
+]

src/action.ts

@@ -13,6 +13,7 @@ export type HyperfineConfigFile = HyperfineConfig[];
 export interface HyperfineConfig {
   name: string;
   command: string;
+  extraArgs: any[]
 }
 
 export interface HyperfineResultSuite {
@@ -92,7 +93,7 @@ async function main(): Promise<void> {
 
   for (const suite of config) {
     core.debug(`Starting benchmark: ${suite.name}`);
-    const res = await Hyperfine.run(suite.command);
+    const res = await Hyperfine.run(suite.command, suite.extraArgs);
     const count = res.times?.length ?? 0;
     delete res.times;
     benchmark.results.push({

src/hyperfine/hyperfine.run.ts

@@ -46,11 +46,14 @@ export async function waitForChildProcess(cmd: string): Promise<string> {
   });
 }
 
-export async function runHyperfine(cmd: string): Promise<HyperFineResult> {
+export async function runHyperfine(cmd: string, extraArgs: any): Promise<HyperFineResult> {
+  //extraArgs example: {runs: "3", prepare: 5, shell: "zsh", "N": true}
   const HyperFineCommand = await findHyperfine();
 
   const outputJsonFile = './' + randomBytes(10).toString('hex') + '.json';
-  const hyperfineExecute = [HyperFineCommand, `--export-json ${outputJsonFile}`, `'${cmd}'`]; // TODO escape \'
+  const hyperfineExecute = [HyperFineCommand, `--export-json ${outputJsonFile}`,
+    Object.entries(extraArgs).map((e: any) => `${e[0].length == 1 ? `-${e[0]}` : `--${e[0]}`}${isNaN(e[1]) ? ' ' + JSON.stringify(e[1]) : (typeof e[1] == "boolean") ? "" : ' ' + e[1]}`).join(" "),
+    `${JSON.stringify(cmd)}`];
 
   const buffer = await waitForChildProcess(hyperfineExecute.join(' '));
   console.log(buffer);