address stdin content review comments

Author: Samsen879

src/cli/actions/defaultAction.ts

@@ -8,7 +8,7 @@ import {
   type RepomixOutputStyle,
   repomixConfigCliSchema,
 } from '../../config/configSchema.js';
-import { readContentFromStdin, readFilePathsFromStdin } from '../../core/file/fileStdin.js';
+import { readFilePathsFromStdin } from '../../core/file/fileStdin.js';
 import { type PackResult, pack } from '../../core/packager.js';
 import { generateDefaultSkillName } from '../../core/skill/skillUtils.js';
 import { RepomixError, rethrowValidationErrorIfSchemaError } from '../../shared/errorHandle.js';
@@ -61,10 +61,6 @@ export const runDefaultAction = async (
 ): Promise<DefaultActionRunnerResult> => {
   logger.trace('Loaded CLI options:', cliOptions);
 
-  if (cliOptions.stdinContent === true) {
-    cliOptions.stdinContent = await readContentFromStdin();
-  }
-
   // Build the merged config (migration + file config + CLI options)
   const config = await buildMergedConfig(cwd, cliOptions);
 

src/cli/cliRun.ts

@@ -304,6 +304,11 @@ export const runCli = async (directories: string[], cwd: string, options: CliOpt
     throw new RepomixError('--stdin cannot be used with --stdin-content. Both options consume standard input.');
   }
 
+  if (options.stdinContent === true) {
+    const { readContentFromStdin } = await import('../core/file/fileStdin.js');
+    options.stdinContent = await readContentFromStdin();
+  }
+
   // Set log level based on verbose and quiet flags
   if (options.quiet) {
     logger.setLogLevel(repomixLogLevels.SILENT);

src/core/output/outputGenerate.ts

@@ -177,7 +177,7 @@ const generateParsableJsonOutput = async (renderContext: RenderContext): Promise
     ...(renderContext.headerText && {
       userProvidedHeader: renderContext.headerText,
     }),
-    ...(renderContext.stdinContent && {
+    ...(renderContext.stdinContent !== undefined && {
       stdinContent: renderContext.stdinContent,
     }),
     ...(renderContext.directoryStructureEnabled && {

tests/cli/cliRun.test.ts

@@ -7,6 +7,7 @@ import * as remoteAction from '../../src/cli/actions/remoteAction.js';
 import * as versionAction from '../../src/cli/actions/versionAction.js';
 import { run, runCli } from '../../src/cli/cliRun.js';
 import type { CliOptions } from '../../src/cli/types.js';
+import * as fileStdin from '../../src/core/file/fileStdin.js';
 import * as gitRemoteHandle from '../../src/core/git/gitRemoteHandle.js';
 import type { PackResult } from '../../src/core/packager.js';
 import { logger, type RepomixLogLevel, repomixLogLevels } from '../../src/shared/logger.js';
@@ -226,6 +227,39 @@ describe('cliRun', () => {
       expect(defaultAction.runDefaultAction).not.toHaveBeenCalled();
     });
 
+    test('should resolve stdin content before executing default action', async () => {
+      vi.spyOn(fileStdin, 'readContentFromStdin').mockResolvedValue('npm run build failed');
+
+      await runCli(['.'], process.cwd(), {
+        stdinContent: true,
+      });
+
+      expect(defaultAction.runDefaultAction).toHaveBeenCalledWith(
+        ['.'],
+        process.cwd(),
+        expect.objectContaining({
+          stdinContent: 'npm run build failed',
+        }),
+      );
+    });
+
+    test('should resolve stdin content before executing remote action', async () => {
+      vi.spyOn(fileStdin, 'readContentFromStdin').mockResolvedValue('remote build failed');
+
+      await runCli(['.'], process.cwd(), {
+        remote: 'yamadashy/repomix',
+        stdinContent: true,
+      });
+
+      expect(remoteAction.runRemoteAction).toHaveBeenCalledWith(
+        'yamadashy/repomix',
+        expect.objectContaining({
+          stdinContent: 'remote build failed',
+        }),
+      );
+      expect(defaultAction.runDefaultAction).not.toHaveBeenCalled();
+    });
+
     test('should auto-detect HTTPS URL and execute remote action', async () => {
       await runCli(['https://github.com/user/repo'], process.cwd(), {});
 

tests/core/output/outputStyles/jsonStyle.test.ts

@@ -124,6 +124,23 @@ describe('JSON Output Style', () => {
     expect(parsed.stdinContent).toBe('npm run build failed');
   });
 
+  test('should preserve empty stdinContent when provided', async () => {
+    const config = createMockConfig({
+      output: {
+        ...createMockConfig().output,
+        stdinContent: '',
+      },
+    });
+    const processedFiles = createMockProcessedFiles();
+    const allFilePaths = processedFiles.map((f) => f.path);
+
+    const result = await generateOutput(['/test'], config, processedFiles, allFilePaths);
+    const parsed = JSON.parse(result);
+
+    expect(Object.hasOwn(parsed, 'stdinContent')).toBe(true);
+    expect(parsed.stdinContent).toBe('');
+  });
+
   test('should include files when enabled', async () => {
     const config = createMockConfig({
       output: {