hyouji 0.0.5 → 0.0.6

package/README.md

@@ -16,7 +16,8 @@ https://levelup.gitconnected.com/create-github-labels-from-terminal-158d4868fab
 
 ![hyouji_terminal](./hyouji.png)
 
-https://user-images.githubusercontent.com/474225/130368605-b5c6410f-53f6-4ef0-b321-8950edeebf7d.mov
+https://github.com/user-attachments/assets/739f185a-1bd0-411b-8947-dd4600c452c8
+
 
 ### Labels API
 

package/dist/index.js

@@ -280,17 +280,34 @@ const createLabels = async (configs2) => {
   log$2("Created all labels");
   log$2(chalk.bgBlueBright(extraGuideText));
 };
-const deleteLabel = (configs2, labelNames) => {
-  labelNames.forEach(async (labelName) => {
-    await configs2.octokit.request(
-      "DELETE /repos/{owner}/{repo}/labels/{name}",
-      {
-        owner: configs2.owner,
-        repo: configs2.repo,
-        name: labelName
+const deleteLabel = async (configs2, labelNames) => {
+  for (const labelName of labelNames) {
+    try {
+      const resp = await configs2.octokit.request(
+        "DELETE /repos/{owner}/{repo}/labels/{name}",
+        {
+          owner: configs2.owner,
+          repo: configs2.repo,
+          name: labelName
+        }
+      );
+      if (resp.status === 204) {
+        log$2(chalk.green(`${resp.status}: Deleted ${labelName}`));
+      } else {
+        log$2(chalk.yellow(`${resp.status}: Something wrong with ${labelName}`));
       }
-    );
-  });
+    } catch (error) {
+      if (error && typeof error === "object" && "status" in error && error.status === 404) {
+        log$2(chalk.red(`404: Label "${labelName}" not found`));
+      } else {
+        log$2(
+          chalk.red(
+            `Error deleting label "${labelName}": ${error instanceof Error ? error.message : "Unknown error"}`
+          )
+        );
+      }
+    }
+  }
 };
 const getLabels = async (configs2) => {
   const resp = await configs2.octokit.request(
@@ -310,18 +327,39 @@ const getLabels = async (configs2) => {
 };
 const deleteLabels = async (configs2) => {
   const names = await getLabels(configs2);
-  names.forEach(async (name) => {
-    await configs2.octokit.request(
-      "DELETE /repos/{owner}/{repo}/labels/{name}",
-      {
-        owner: configs2.owner,
-        repo: configs2.repo,
-        name
+  if (names.length === 0) {
+    log$2(chalk.yellow("No labels found to delete"));
+    return;
+  }
+  log$2(chalk.blue(`Deleting ${names.length} labels...`));
+  for (const name of names) {
+    try {
+      const resp = await configs2.octokit.request(
+        "DELETE /repos/{owner}/{repo}/labels/{name}",
+        {
+          owner: configs2.owner,
+          repo: configs2.repo,
+          name
+        }
+      );
+      if (resp.status === 204) {
+        log$2(chalk.green(`${resp.status}: Deleted ${name}`));
+      } else {
+        log$2(chalk.yellow(`${resp.status}: Something wrong with ${name}`));
       }
-    );
-  });
-  log$2("");
-  names.forEach((label) => log$2(chalk.bgGreen(`deleted ${label}`)));
+    } catch (error) {
+      if (error && typeof error === "object" && "status" in error && error.status === 404) {
+        log$2(chalk.red(`404: Label "${name}" not found`));
+      } else {
+        log$2(
+          chalk.red(
+            `Error deleting label "${name}": ${error instanceof Error ? error.message : "Unknown error"}`
+          )
+        );
+      }
+    }
+  }
+  log$2(chalk.blue("Finished deleting labels"));
   log$2(chalk.bgBlueBright(extraGuideText));
 };
 const _CryptoUtils = class _CryptoUtils {
@@ -1247,9 +1285,9 @@ const displaySettings = async () => {
   log(chalk.cyan("========================\n"));
 };
 let configs;
-const main = async () => {
+const initializeConfigs = async () => {
   let hasValidConfig = false;
-  if (firstStart && configManager.configExists()) {
+  if (configManager.configExists()) {
     try {
       const existingConfig = await configManager.loadValidatedConfig();
       if (existingConfig && existingConfig.config && !existingConfig.shouldPromptForCredentials) {
@@ -1268,58 +1306,66 @@ const main = async () => {
           `Please go to ${linkToPersonalToken} and generate a personal token!`
         )
       );
-      return;
+      return null;
     }
   }
-  if (firstStart) {
+  try {
+    const asciiText = await getAsciiText();
+    if (asciiText != null) {
+      log(asciiText);
+    }
+  } catch (error) {
+    console.warn("Failed to display ASCII art, continuing...");
+    console.error("Error:", error);
+  }
+  if (hasValidConfig) {
     try {
-      const asciiText = await getAsciiText();
-      if (asciiText != null) {
-        log(asciiText);
+      const existingConfig = await configManager.loadValidatedConfig();
+      if (existingConfig && existingConfig.config) {
+        const repoResponse = await prompts([
+          {
+            type: "text",
+            name: "repo",
+            message: "Please type your target repo name"
+          }
+        ]);
+        const config = {
+          octokit: new Octokit({ auth: existingConfig.config.token }),
+          owner: existingConfig.config.owner,
+          repo: repoResponse.repo,
+          fromSavedConfig: true
+        };
+        log(chalk.green(`Using saved configuration for ${config.owner}`));
+        return config;
+      } else {
+        return await setupConfigs();
       }
     } catch (error) {
-      console.warn("Failed to display ASCII art, continuing...");
       console.error("Error:", error);
+      return await setupConfigs();
     }
-    if (hasValidConfig) {
-      try {
-        const existingConfig = await configManager.loadValidatedConfig();
-        if (existingConfig && existingConfig.config) {
-          const repoResponse = await prompts([
-            {
-              type: "text",
-              name: "repo",
-              message: "Please type your target repo name"
-            }
-          ]);
-          configs = {
-            octokit: new Octokit({ auth: existingConfig.config.token }),
-            owner: existingConfig.config.owner,
-            repo: repoResponse.repo,
-            fromSavedConfig: true
-          };
-          log(chalk.green(`Using saved configuration for ${configs.owner}`));
-        } else {
-          configs = await setupConfigs();
-        }
-      } catch (error) {
-        console.error("Error:", error);
-        configs = await setupConfigs();
-      }
-    } else {
-      try {
-        configs = await setupConfigs();
-        if (configs.fromSavedConfig) {
-          log(chalk.green(`Using saved configuration for ${configs.owner}`));
-        }
-      } catch (error) {
-        log(
-          chalk.red(
-            `Configuration error: ${error instanceof Error ? error.message : "Unknown error"}`
-          )
-        );
-        return;
+  } else {
+    try {
+      const config = await setupConfigs();
+      if (config.fromSavedConfig) {
+        log(chalk.green(`Using saved configuration for ${config.owner}`));
       }
+      return config;
+    } catch (error) {
+      log(
+        chalk.red(
+          `Configuration error: ${error instanceof Error ? error.message : "Unknown error"}`
+        )
+      );
+      return null;
+    }
+  }
+};
+const main = async () => {
+  if (firstStart) {
+    configs = await initializeConfigs();
+    if (!configs) {
+      return;
     }
   }
   let selectedIndex = await selectAction();
@@ -1329,23 +1375,23 @@ const main = async () => {
   switch (selectedIndex) {
     case 0: {
       const newLabel2 = await getNewLabel();
-      createLabel(configs, newLabel2);
+      await createLabel(configs, newLabel2);
       firstStart = firstStart && false;
       break;
     }
     case 1: {
-      createLabels(configs);
+      await createLabels(configs);
       firstStart = firstStart && false;
       break;
     }
     case 2: {
       const targetLabel = await getTargetLabel();
-      deleteLabel(configs, targetLabel);
+      await deleteLabel(configs, targetLabel);
       firstStart = firstStart && false;
       break;
     }
     case 3: {
-      deleteLabels(configs);
+      await deleteLabels(configs);
       firstStart = firstStart && false;
       break;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hyouji",
-  "version": "0.0.5",
+  "version": "0.0.6",
   "description": "Hyouji (表示) — A command-line tool for organizing and displaying GitHub labels with clarity and harmony.",
   "main": "dist/index.js",
   "bin": {
@@ -35,6 +35,11 @@
     "test:coverage": "vitest run --coverage",
     "test:lint": "eslint src --ext .ts",
     "test:prettier": "prettier \"src/**/*.ts\" --list-different",
+    "test:error-handling": "node tests/scripts/error-handling/run-all.cjs",
+    "test:config": "node tests/scripts/config/run-all.cjs",
+    "test:integration": "node tests/scripts/run-integration.cjs",
+    "test:verification": "node tests/scripts/verification/run-all.cjs",
+    "test:all-custom": "npm run test:error-handling && npm run test:config && npm run test:integration && npm run test:verification",
     "check-cli": "run-s test diff-integration-tests check-integration-tests",
     "check-integration-tests": "run-s check-integration-test:*",
     "diff-integration-tests": "mkdir -p diff && rm -rf diff/test && cp -r test diff/test && rm -rf diff/test/test-*/.git && cd diff && git init --quiet && git add -A && git commit --quiet --no-verify --allow-empty -m 'WIP' && echo '\\n\\nCommitted most recent integration test output in the \"diff\" directory. Review the changes with \"cd diff && git diff HEAD\" or your preferred git diff viewer.'",