agent-bot/takt
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: reportDirに.takt/reports/プレフィックスを含め、ルート直下へのレポート作成を修正 (#37)

Browse Source
This commit is contained in:
nrslib 2026-01-30 21:32:09 +09:00
parent 1e1f2c0c58
commit b520ede80d
2 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/__tests__/engine-report.test.ts
View File

@ -14,6 +14,8 @@ import type { WorkflowStep, ReportObjectConfig, ReportConfig } from '../models/t
/** /**
* Extracted emitStepReports logic for unit testing. * Extracted emitStepReports logic for unit testing.
* Mirrors engine.ts emitStepReports + emitIfReportExists. * Mirrors engine.ts emitStepReports + emitIfReportExists.
*
* reportDir already includes the `.takt/reports/` prefix (set by engine constructor).
*/ */
function emitStepReports( function emitStepReports(
emitter: EventEmitter, emitter: EventEmitter,
@ -22,7 +24,7 @@ function emitStepReports(
projectCwd: string, projectCwd: string,
): void { ): void {
if (!step.report || !reportDir) return; if (!step.report || !reportDir) return;
const baseDir = join(projectCwd, '.takt', 'reports', reportDir); const baseDir = join(projectCwd, reportDir);
if (typeof step.report === 'string') { if (typeof step.report === 'string') {
emitIfReportExists(emitter, step, baseDir, step.report); emitIfReportExists(emitter, step, baseDir, step.report);
@ -62,11 +64,12 @@ function createStep(overrides: Partial<WorkflowStep> = {}): WorkflowStep {
describe('emitStepReports', () => { describe('emitStepReports', () => {
let tmpDir: string; let tmpDir: string;
let reportBaseDir: string; let reportBaseDir: string;
const reportDirName = 'test-report-dir'; // reportDir now includes .takt/reports/ prefix (matches engine constructor behavior)
const reportDirName = '.takt/reports/test-report-dir';
beforeEach(() => { beforeEach(() => {
tmpDir = join(tmpdir(), `takt-report-test-${Date.now()}`); tmpDir = join(tmpdir(), `takt-report-test-${Date.now()}`);
reportBaseDir = join(tmpDir, '.takt', 'reports', reportDirName); reportBaseDir = join(tmpDir, reportDirName);
mkdirSync(reportBaseDir, { recursive: true }); mkdirSync(reportBaseDir, { recursive: true });
}); });

6
src/workflow/engine.ts
View File

@ -64,7 +64,7 @@ export class WorkflowEngine extends EventEmitter {
this.options = options; this.options = options;
this.language = options.language; this.language = options.language;
this.loopDetector = new LoopDetector(config.loopDetection); this.loopDetector = new LoopDetector(config.loopDetection);
this.reportDir = generateReportDir(task); this.reportDir = `.takt/reports/${generateReportDir(task)}`;
this.ensureReportDirExists(); this.ensureReportDirExists();
this.validateConfig(); this.validateConfig();
this.state = createInitialState(config, options); this.state = createInitialState(config, options);
@ -78,7 +78,7 @@ export class WorkflowEngine extends EventEmitter {
/** Ensure report directory exists (always in project root, not clone) */ /** Ensure report directory exists (always in project root, not clone) */
private ensureReportDirExists(): void { private ensureReportDirExists(): void {
const reportDirPath = join(this.projectCwd, '.takt', 'reports', this.reportDir); const reportDirPath = join(this.projectCwd, this.reportDir);
if (!existsSync(reportDirPath)) { if (!existsSync(reportDirPath)) {
mkdirSync(reportDirPath, { recursive: true }); mkdirSync(reportDirPath, { recursive: true });
} }
@ -176,7 +176,7 @@ export class WorkflowEngine extends EventEmitter {
*/ */
private emitStepReports(step: WorkflowStep): void { private emitStepReports(step: WorkflowStep): void {
if (!step.report || !this.reportDir) return; if (!step.report || !this.reportDir) return;
const baseDir = join(this.projectCwd, '.takt', 'reports', this.reportDir); const baseDir = join(this.projectCwd, this.reportDir);
if (typeof step.report === 'string') { if (typeof step.report === 'string') {
this.emitIfReportExists(step, baseDir, step.report); this.emitIfReportExists(step, baseDir, step.report);