From be35ac1df9cc0d2395d15d74355ddb392e90cd69 Mon Sep 17 00:00:00 2001
From: neverland <chenjiahan.jait@bytedance.com>
Date: Tue, 1 Sep 2026 12:51:27 +0800
Subject: [PATCH] feat(fmt): align successful format check output

---
 packages/rstack/src/fmt/cli.ts                 | 7 +++----
 packages/rstack/tests/cli/check.test.ts        | 2 +-
 packages/rstack/tests/cli/fmt/files.test.ts    | 2 +-
 packages/rstack/tests/cli/fmt/patterns.test.ts | 5 +++--
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/packages/rstack/src/fmt/cli.ts b/packages/rstack/src/fmt/cli.ts
index 40518c0f..a2c90160 100644
--- a/packages/rstack/src/fmt/cli.ts
+++ b/packages/rstack/src/fmt/cli.ts
@@ -205,7 +205,7 @@ const logFmtResult = (
     return;
   }
 
-  const time = color.bold(formatDuration(durationMilliseconds));
+  const time = formatDuration(durationMilliseconds);
 
   if (mode === 'write') {
     if (writtenCount === 0 && result.exitCode !== 0) {
@@ -230,9 +230,8 @@ const logFmtResult = (
     logger.error(`Formatting issues found in ${differentFiles}. ${fixHint}`);
     logger.info(`Checked ${processedFiles} in ${time}.`);
   } else if (result.exitCode === 0) {
-    logger.success(
-      `Checked ${formatFileCount(processedFileCount)} in ${time}. No issues found.`,
-    );
+    const files = `${processedFileCount} ${processedFileCount === 1 ? 'file' : 'files'}`;
+    logger.success(`Format check passed in ${time} ${color.dim(`(${files})`)}`);
   }
 };
 
diff --git a/packages/rstack/tests/cli/check.test.ts b/packages/rstack/tests/cli/check.test.ts
index 3d9fce78..f764457f 100644
--- a/packages/rstack/tests/cli/check.test.ts
+++ b/packages/rstack/tests/cli/check.test.ts
@@ -44,7 +44,7 @@ test('runs lint followed by a formatting check', () => {
   const formatted = runCheck();
 
   expect(formatted.status).toBe(0);
-  expect(formatted.stdout).toContain('No issues found.');
+  expect(formatted.stdout).toContain('Format check passed in');
   expect(formatted.stderr).toBe('');
 });
 
diff --git a/packages/rstack/tests/cli/fmt/files.test.ts b/packages/rstack/tests/cli/fmt/files.test.ts
index 47208688..569a95c9 100644
--- a/packages/rstack/tests/cli/fmt/files.test.ts
+++ b/packages/rstack/tests/cli/fmt/files.test.ts
@@ -117,7 +117,7 @@ test('checks formatting without writing files', () => {
 
   expect(formattedResult.status).toBe(0);
   expect(normalizeDuration(formattedResult.stdout)).toBe(
-    'start   Checking formatting...\nsuccess Checked 1 file in <duration>. No issues found.\n',
+    'start   Checking formatting...\nsuccess Format check passed in <duration> (1 file)\n',
   );
   expect(formattedResult.stderr).toBe('');
 });
diff --git a/packages/rstack/tests/cli/fmt/patterns.test.ts b/packages/rstack/tests/cli/fmt/patterns.test.ts
index b87eee16..106baf07 100644
--- a/packages/rstack/tests/cli/fmt/patterns.test.ts
+++ b/packages/rstack/tests/cli/fmt/patterns.test.ts
@@ -33,13 +33,14 @@ test('allows no files to match with --no-error-on-unmatched-pattern', () => {
 
 test('counts only supported files', () => {
   writeProjectFile('index.ts', 'const value = 1;\n');
+  writeProjectFile('other.ts', 'const other = 2;\n');
   writeProjectFile('notes.unknown', 'plain text');
 
-  const result = runFmt(['--check', 'index.ts', 'notes.unknown']);
+  const result = runFmt(['--check', 'index.ts', 'other.ts', 'notes.unknown']);
 
   expect(result.status).toBe(0);
   expect(normalizeDuration(result.stdout)).toBe(
-    'start   Checking formatting...\nsuccess Checked 1 file in <duration>. No issues found.\n',
+    'start   Checking formatting...\nsuccess Format check passed in <duration> (2 files)\n',
   );
   expect(result.stderr).toBe('');
 });