mirror of
https://github.com/google-gemini/gemini-cli.git
synced 2026-08-03 21:51:11 -07:00
feat(tracker): simplify tracker storage path and flatten directory structure
This commit is contained in:
@@ -61,3 +61,5 @@ gemini-debug.log
|
||||
.gemini-clipboard/
|
||||
.eslintcache
|
||||
evals/logs/
|
||||
|
||||
.tracker/
|
||||
|
||||
@@ -1913,7 +1913,7 @@ export class Config {
|
||||
getTrackerService(): TrackerService {
|
||||
if (!this.trackerService) {
|
||||
this.trackerService = new TrackerService(
|
||||
this.storage.getProjectTempTrackerDir(this.sessionId),
|
||||
this.storage.getProjectTempTrackerDir(),
|
||||
);
|
||||
}
|
||||
return this.trackerService;
|
||||
|
||||
@@ -245,13 +245,8 @@ export class Storage {
|
||||
return path.join(this.getProjectTempDir(), 'plans');
|
||||
}
|
||||
|
||||
getProjectTempTrackerDir(sessionId: string): string {
|
||||
return path.join(
|
||||
this.getProjectTempDir(),
|
||||
sessionId,
|
||||
'.tracker',
|
||||
sessionId,
|
||||
);
|
||||
getProjectTempTrackerDir(): string {
|
||||
return path.join(this.getProjectTempDir(), 'tracker');
|
||||
}
|
||||
|
||||
getExtensionsDir(): string {
|
||||
|
||||
@@ -28,7 +28,7 @@ describe('TrackerService', () => {
|
||||
|
||||
it('should initialize the tracker directory', async () => {
|
||||
await service.ensureInitialized();
|
||||
const tasksDir = path.join(testTrackerDir, 'tasks');
|
||||
const tasksDir = testTrackerDir;
|
||||
const stats = await fs.stat(tasksDir);
|
||||
expect(stats.isDirectory()).toBe(true);
|
||||
});
|
||||
|
||||
@@ -13,7 +13,7 @@ export class TrackerService {
|
||||
private readonly tasksDir: string;
|
||||
|
||||
constructor(readonly trackerDir: string) {
|
||||
this.tasksDir = path.join(this.trackerDir, 'tasks');
|
||||
this.tasksDir = trackerDir;
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -47,7 +47,7 @@ describe('Tracker Tools Integration', () => {
|
||||
|
||||
expect(result.llmContent).toContain('Task tracker initialized');
|
||||
const trackerDir = config.getTrackerService().trackerDir;
|
||||
const tasksDir = path.join(trackerDir, 'tasks');
|
||||
const tasksDir = trackerDir;
|
||||
const stats = await fs.stat(tasksDir);
|
||||
expect(stats.isDirectory()).toBe(true);
|
||||
// Verify it is NOT in the tempDir root (which was the old behavior)
|
||||
|
||||
@@ -69,7 +69,7 @@ class TrackerInitInvocation extends BaseTrackerInvocation<
|
||||
override async execute(_signal: AbortSignal): Promise<ToolResult> {
|
||||
await this.service.ensureInitialized();
|
||||
return {
|
||||
llmContent: `Task tracker initialized successfully. Storage is ready at ${this.service.trackerDir}/tasks/`,
|
||||
llmContent: `Task tracker initialized successfully. Storage is ready at ${this.service.trackerDir}`,
|
||||
returnDisplay: 'Tracker initialized.',
|
||||
};
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user