feat: Add summary option (#2549)

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: Jason3S

.github/workflows/test-action.yml

@@ -127,6 +127,7 @@ jobs:
           config: ./fixtures/cspell.json
           strict: false
           inline: none
+          summary: true
       - name: Show Results
         uses: streetsidesoftware/actions/public/summary@v1
         with:

.vscode/settings.json

@@ -1,9 +1,10 @@
 {
     "search.exclude": {
+        ".git": true,
         "**/.yarn": true,
         "**/.pnp.*": true
     },
     "typescript.enablePromptUseWorkspaceTsdk": true,
     "git.ignoreLimitWarning": true,
-    "editor.formatOnSave": false
+    "editor.formatOnSave": true
 }

action-src/src/ActionParams.ts

@@ -72,6 +72,11 @@ export interface ActionParams {
      * default: use default reporting.
      */
     report: ReportChoices;
+
+    /**
+     * Include a summary of the results in the output.
+     */
+    summary: TrueFalse;
 }
 
 const defaultActionParams: ActionParams = {
@@ -87,6 +92,7 @@ const defaultActionParams: ActionParams = {
     use_cspell_files: 'false',
     suggestions: 'false',
     report: undefined,
+    summary: 'false',
 };
 
 type ValidationFunction = (params: ActionParamsInput) => string | undefined;
@@ -153,6 +159,7 @@ export function validateActionParams(
         validateTrueFalse('suggestions'),
         validateOptions('check_dot_files', ['true', 'false', 'explicit']),
         validateOptions('report', ['all', 'simple', 'typos', 'flagged'], true),
+        validateTrueFalse('summary'),
     ];
     const success = validations
         .map((fn) => fn(params))

action-src/src/__snapshots__/action.test.ts.snap

@@ -92,17 +92,21 @@ function outputResult(runResult: RunResult) {
     setOutput('number_of_files_checked', result.number_of_files_checked);
     setOutput('number_of_issues', result.number_of_issues);
     setOutput('number_of_files_with_issues', result.files_with_issues.length);
-    setOutput('files_with_issues', normalizeFiles(result.files_with_issues));
+    setOutput('files_with_issues', result.files_with_issues);
+    setOutput('number_of_files_skipped', result.number_of_files_skipped);
+    setOutput('number_of_files_cached', result.number_of_files_cached);
     setOutput('result', result);
 }
 
 function normalizeResult(result: RunResult) {
-    const { issues: number_of_issues, files: number_of_files_checked, filesWithIssues } = result;
+    const { issues: number_of_issues, files, filesWithIssues, skippedFiles = 0, cachedFiles = 0 } = result;
     return {
         success: !number_of_issues && !result.errors,
         errors: result.errors,
         number_of_issues,
-        number_of_files_checked,
+        number_of_files_checked: files - skippedFiles,
+        number_of_files_skipped: skippedFiles,
+        number_of_files_cached: cachedFiles,
         files_with_issues: normalizeFiles(filesWithIssues).slice(0, 1000),
     };
 }

action-src/src/action.ts

@@ -7,13 +7,13 @@ import { getDefaultLogger, type Logger } from './logger.js';
 const { gatherFileGlobsFromContext } = __testing__;
 
 vi.mock('./logger.js', async (importActual) => {
-    const f = () => {};
     const logger: Logger = {
-        error: vi.fn(f),
-        debug: vi.fn(f),
-        info: vi.fn(f),
-        warning: vi.fn(f),
-        issueCommand: vi.fn(f),
+        error: vi.fn(),
+        debug: vi.fn(),
+        info: vi.fn(),
+        warning: vi.fn(),
+        issueCommand: vi.fn(),
+        summary: vi.fn(),
     };
 
     return {

action-src/src/checkSpelling.test.ts

@@ -97,6 +97,7 @@ async function checkSpelling(
     const reporterOptions = {
         verbose: params.verbose === 'true',
         treatFlaggedWordsAsErrors: params.treat_flagged_words_as_errors === 'true',
+        summary: params.summary === 'true',
     };
 
     const collector = new CSpellReporterForGithubAction(params.inline, reporterOptions);

action-src/src/checkSpelling.ts

@@ -5,7 +5,6 @@ import { tf } from './utils.js';
 
 export function getActionParams(): ActionParamsInput {
     const params: ActionParamsInput = {
-        // github_token: getInput('github_token', { required: true }),
         files: getInput('files'),
         incremental_files_only: tf(getInput('incremental_files_only')),
         config: getInput('config'),
@@ -18,6 +17,7 @@ export function getActionParams(): ActionParamsInput {
         use_cspell_files: tf(getInput('use_cspell_files')),
         suggestions: tf(getInput('suggestions')),
         report: getInput('report').toLowerCase(),
+        summary: tf(getInput('summary') || 'false'),
     };
     return applyDefaults(params);
 }

action-src/src/getActionParams.ts

@@ -1,9 +1,40 @@
-import { describe, expect, test } from 'vitest';
+import { summary as coreSummary } from '@actions/core';
+import { describe, expect, test, vi } from 'vitest';
 
-import {} from './logger.js';
+import { createLogger } from './logger.js';
+
+vi.mock('@actions/core', async (_importActual) => {
+    return {
+        debug: vi.fn(),
+        info: vi.fn(),
+        warning: vi.fn(),
+        error: vi.fn(),
+        issueCommand: vi.fn(),
+        summary: {
+            addRaw: vi.fn(),
+            write: vi.fn(),
+        },
+    };
+});
 
 describe('Validate Logger', () => {
-    test('placeholder', () => {
-        expect(true).toBe(true);
+    test('createLogger', () => {
+        const logger = createLogger({ debug: vi.fn() });
+        expect(logger.debug).toBeDefined();
+        expect(logger.info).toBeDefined();
+        expect(logger.warning).toBeDefined();
+        expect(logger.error).toBeDefined();
+        expect(logger.issueCommand).toBeDefined();
+        expect(logger.summary).toBeDefined();
+
+        logger.debug('test debug');
+        expect(logger.debug).toHaveBeenCalledWith('test debug');
+    });
+
+    test('summary', () => {
+        const logger = createLogger();
+        logger.summary('test summary');
+        expect(coreSummary.addRaw).toHaveBeenCalledWith('test summary');
+        expect(coreSummary.write).toHaveBeenCalled();
     });
 });

action-src/src/logger.test.ts

@@ -1,4 +1,4 @@
-import { debug, error, info, warning } from '@actions/core';
+import { debug, error, info, summary as coreSummary, warning } from '@actions/core';
 import { issueCommand } from '@actions/core/lib/command.js';
 
 export type IssueCommandFn = typeof issueCommand;
@@ -14,10 +14,16 @@ export interface Logger {
     warning: LogErrorFn;
     error: LogErrorFn;
     issueCommand: IssueCommandFn;
+    summary: LogFn;
+}
+
+function summary(message: string) {
+    coreSummary.addRaw(message);
+    coreSummary.write();
 }
 
 export function createLogger(logger?: Partial<Logger>): Logger {
-    return { debug, info, warning, error, issueCommand, ...logger };
+    return { debug, info, warning, error, issueCommand, summary, ...logger };
 }
 
 export function getDefaultLogger(): Logger {