Merge pull request #112 from webdeveric/dev

Refactor skip message format and update text reporter icons

Author: webdeveric

package.json

@@ -32,7 +32,7 @@
   "engines": {
     "node": ">=22.22.2"
   },
-  "packageManager": "pnpm@11.16.0+sha512.b767e9a98fc87aec0f42daefcd0e84941c2cdb45d73c4e1e68860fb2bdfdbe032ab592105479e5e05c237f740c8a5ffdbbb52385797ddc2296745a1bbb883e8d",
+  "packageManager": "pnpm@11.17.0+sha512.cca3cea332ad254bb84145f966d19f4879615210346fc92c79a047f23a0d7b3cca3c3792f0076ba1f1831d277efbcf0a9119b31a9a60eca7fb3d6231f331ef72",
   "scripts": {
     "prevalidate": "pnpm build:dev",
     "validate": "./dist/cli.mjs",

pnpm-lock.yaml

@@ -0,0 +1,15 @@
+export const checkMark = '✓';
+
+export const xMark = '✗';
+
+export const emptyBox = '□';
+
+export const leftArrow = '←';
+
+export const rightArrow = '→';
+
+export const upArrow = '↑';
+
+export const downArrow = '↓';
+
+export const pencil = '✎';

src/lib/reporters/text-icons.ts

@@ -5,6 +5,7 @@ import { assertExhaustive } from '@webdeveric/utils/assertion';
 import { graphemeLength } from '@webdeveric/utils/graphemeLength';
 import { indent } from '@webdeveric/utils/indent';
 
+import { checkMark, emptyBox, xMark } from '@lib/reporters/text-icons.js';
 import { ResultCode, type Result } from '@lib/Result.js';
 import { stdoutWritableStream } from '@utils/stdoutWritableStream.js';
 
@@ -59,11 +60,11 @@ export class TextWritableStream extends WritableStream<Result> {
   getIcon(resultCode: ResultCode): string {
     switch (resultCode) {
       case ResultCode.Success:
-        return styleText(['greenBright'], '🗹');
+        return styleText(['greenBright'], checkMark);
       case ResultCode.Error:
-        return styleText(['redBright'], '🗷');
+        return styleText(['redBright'], xMark);
       case ResultCode.Skip:
-        return styleText(['whiteBright'], '🞏');
+        return styleText(['whiteBright'], emptyBox);
       default:
         assertExhaustive(resultCode, 'Unhandled ResultCode');
     }

src/lib/reporters/web-stream/TextWritableStream.ts

@@ -45,15 +45,15 @@ export function checkImport(entryPoint: EntryPoint): Result {
     return new Result({
       code: ResultCode.Skip,
       entryPoint,
-      message: `Import skipped: ${entryPoint.itemPath.join('.')}`,
+      message: `"${entryPoint.moduleName}" skipped`,
       name: 'import',
     });
   } catch (error) {
     return new Result({
       code: ResultCode.Error,
       entryPoint,
       error: asError(error),
-      message: `${entryPoint.moduleName ?? entryPoint.itemPath.join('.')} cannot be imported`,
+      message: `${entryPoint.moduleName ?? entryPoint.relativePath} cannot be imported`,
       name: 'import',
     });
   }

src/utils/checkImport.ts

@@ -42,7 +42,7 @@ export function checkRequire(entryPoint: EntryPoint): Result {
     return new Result({
       code: ResultCode.Skip,
       entryPoint,
-      message: `Require skipped: ${entryPoint.itemPath.join('.')}`,
+      message: `"${entryPoint.moduleName}" skipped`,
       name: 'require',
     });
   } catch (error) {