project-roadmap-tracking 0.1.0 → 0.2.1

package/dist/commands/init.js

@@ -1,5 +1,6 @@
 import { Args, Command, Flags } from '@oclif/core';
 import { mkdir, readdir, writeFile } from 'node:fs/promises';
+import errorHandlerService from '../services/error-handler.service.js';
 import { PRIORITY, STATUS, TASK_TYPE } from '../util/types.js';
 export default class Init extends Command {
     static args = {
@@ -13,6 +14,11 @@ export default class Init extends Command {
         force: Flags.boolean({ char: 'f', description: 'force initialization even if files already exist' }),
         // flag with a value (-n, --name=VALUE)
         name: Flags.string({ char: 'n', description: 'name to print' }),
+        verbose: Flags.boolean({
+            char: 'v',
+            default: false,
+            description: 'show detailed error information including stack traces',
+        }),
         withSampleTasks: Flags.boolean({ description: 'include sample tasks in the initialized roadmap' }),
     };
     buildBlankRoadmap({ description, name, withSampleTasks, }) {
@@ -48,7 +54,12 @@ export default class Init extends Command {
     }
     buildConfig({ description, name, path }) {
         return {
-            $schema: 'https://raw.githubusercontent.com/ZacharyEggert/project-roadmap-tracking/refs/heads/master/schemas/config/v1.json',
+            $schema: 'https://raw.githubusercontent.com/ZacharyEggert/project-roadmap-tracking/refs/heads/master/schemas/config/v1.1.json',
+            cache: {
+                enabled: true,
+                maxSize: 10,
+                watchFiles: true,
+            },
             metadata: {
                 description: `${description}`,
                 name: `${name}`,
@@ -58,54 +69,60 @@ export default class Init extends Command {
     }
     async run() {
         const { args, flags } = await this.parse(Init);
-        const path = args.folder ?? '.';
-        this.log(`creating project roadmap in${path === '.' ? ' current directory' : ': ' + args.folder}`);
-        const name = flags.name ?? 'My Project Roadmap';
-        const description = flags.description ?? 'A project roadmap managed by Project Roadmap Tracking';
-        // create prt.json and prt.config.json files here
-        const config = this.buildConfig({ description, name, path });
-        const roadmap = this.buildBlankRoadmap({ description, name, withSampleTasks: flags.withSampleTasks ?? false });
-        if (path !== '.') {
-            // check if target directory exists
-            // if it does not, create it
-            await readdir(path).catch(async (error) => {
-                if (error) {
-                    this.log(`target directory does not exist, creating: ${path}`);
-                    try {
-                        // create the directory
-                        await mkdir(path, { recursive: true });
-                    }
-                    catch (error) {
-                        this.error(`failed to create target directory: ${error.message}`);
+        try {
+            const path = args.folder ?? '.';
+            this.log(`creating project roadmap in${path === '.' ? ' current directory' : ': ' + args.folder}`);
+            const name = flags.name ?? 'My Project Roadmap';
+            const description = flags.description ?? 'A project roadmap managed by Project Roadmap Tracking';
+            // create prt.json and prt.config.json files here
+            const config = this.buildConfig({ description, name, path });
+            const roadmap = this.buildBlankRoadmap({ description, name, withSampleTasks: flags.withSampleTasks ?? false });
+            if (path !== '.') {
+                // check if target directory exists
+                // if it does not, create it
+                await readdir(path).catch(async (error) => {
+                    if (error) {
+                        this.log(`target directory does not exist, creating: ${path}`);
+                        try {
+                            // create the directory
+                            await mkdir(path, { recursive: true });
+                        }
+                        catch (error) {
+                            this.error(`failed to create target directory: ${error.message}`);
+                        }
                     }
+                });
+            }
+            // check if config already exists in the target directory
+            // if it does, and --force is not set, throw an error
+            // if it does, and --force is set, overwrite the files
+            await readdir('.')
+                .then(async (files) => {
+                if (!flags.force && (files.includes('prt.json') || files.includes('prt.config.json'))) {
+                    this.error('prt.config.json already exist in the current directory. Use --force to overwrite.');
+                }
+                await writeFile(`./.prtrc.json`, JSON.stringify(config, null, 2));
+                this.log('project roadmap config initialized');
+            })
+                .catch((error) => {
+                this.error(`failed to read current directory: ${error.message}`);
+            });
+            // create prt.json and prt.config.json files in specified directory
+            await readdir(path)
+                .then(async (files) => {
+                if (!flags.force && files.includes('prt.json')) {
+                    this.error('prt.json already exists in the target directory. Use --force to overwrite.');
                 }
+                await writeFile(`${path}/prt.json`, JSON.stringify(roadmap, null, 2));
+                this.log('project roadmap initialized');
+            })
+                .catch((error) => {
+                this.error(`failed to read directory: ${error.message}`);
             });
         }
-        // check if config already exists in the target directory
-        // if it does, and --force is not set, throw an error
-        // if it does, and --force is set, overwrite the files
-        await readdir('.')
-            .then(async (files) => {
-            if (!flags.force && (files.includes('prt.json') || files.includes('prt.config.json'))) {
-                this.error('prt.config.json already exist in the current directory. Use --force to overwrite.');
-            }
-            await writeFile(`./.prtrc.json`, JSON.stringify(config, null, 2));
-            this.log('project roadmap config initialized');
-        })
-            .catch((error) => {
-            this.error(`failed to read current directory: ${error.message}`);
-        });
-        // create prt.json and prt.config.json files in specified directory
-        await readdir(path)
-            .then(async (files) => {
-            if (!flags.force && files.includes('prt.json')) {
-                this.error('prt.json already exists in the target directory. Use --force to overwrite.');
-            }
-            await writeFile(`${path}/prt.json`, JSON.stringify(roadmap, null, 2));
-            this.log('project roadmap initialized');
-        })
-            .catch((error) => {
-            this.error(`failed to read directory: ${error.message}`);
-        });
+        catch (error) {
+            const exitCode = errorHandlerService.handleError(error);
+            this.error(errorHandlerService.formatErrorMessage(error, flags.verbose), { exit: exitCode });
+        }
     }
 }

package/dist/commands/list.d.ts

@@ -5,9 +5,12 @@ export default class List extends Command {
     static examples: string[];
     static flags: {
         incomplete: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        'no-repo': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         priority: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         sort: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         status: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        verbose: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
+    private static readonly priorityMap;
     run(): Promise<void>;
 }

package/dist/commands/list.js

@@ -1,7 +1,12 @@
 import { /* Args, */ Command, Flags } from '@oclif/core';
+import { getDefaultConfigRepository } from '../repositories/config.repository.js';
+import { RoadmapRepository } from '../repositories/roadmap.repository.js';
+import displayService from '../services/display.service.js';
+import errorHandlerService from '../services/error-handler.service.js';
+import taskQueryService, { SortOrder } from '../services/task-query.service.js';
 import { readConfigFile } from '../util/read-config.js';
 import { readRoadmapFile } from '../util/read-roadmap.js';
-import { STATUS } from '../util/types.js';
+import { PRIORITY } from '../util/types.js';
 export default class List extends Command {
     static args = {
     // file: Args.string({description: 'file to read'}),
@@ -14,6 +19,10 @@ export default class List extends Command {
         // flag with a value (-n, --name=VALUE)
         // name: Flags.string({char: 'n', description: 'name to print'}),
         incomplete: Flags.boolean({ char: 'i', description: 'filter tasks to show in-progress and not-started only' }),
+        'no-repo': Flags.boolean({
+            default: false,
+            description: 'use legacy direct file I/O instead of repository pattern',
+        }),
         priority: Flags.string({
             char: 'p',
             description: 'filter tasks by priority (high, medium, low)',
@@ -29,73 +38,67 @@ export default class List extends Command {
             description: 'filter tasks by status (completed, in-progress, not-started)',
             options: ['completed', 'in-progress', 'not-started'],
         }),
+        verbose: Flags.boolean({
+            char: 'v',
+            default: false,
+            description: 'show detailed error information including stack traces',
+        }),
+    };
+    static priorityMap = {
+        h: PRIORITY.High,
+        high: PRIORITY.High,
+        l: PRIORITY.Low,
+        low: PRIORITY.Low,
+        m: PRIORITY.Medium,
+        medium: PRIORITY.Medium,
     };
     async run() {
         const { flags } = await this.parse(List);
-        const incompleteOnly = flags.incomplete ?? false;
-        const priority = (flags.priority ?? null);
-        const sortBy = (flags.sort ?? null);
-        const statusFilter = (flags.status ?? null);
-        const priorityFilter = (priority
-            ? priority === 'high' || priority === 'h'
-                ? 'high'
-                : priority === 'medium' || priority === 'm'
-                    ? 'medium'
-                    : priority === 'low' || priority === 'l'
-                        ? 'low'
-                        : null
-            : null);
-        // if statusFilter is set, it overrides incompleteOnly
-        const effectiveStatusFilter = statusFilter
-            ? [statusFilter]
-            : incompleteOnly
-                ? ['in-progress', 'not-started']
-                : ['completed', 'in-progress', 'not-started'];
-        const config = await readConfigFile().catch((error) => {
-            this.error(`failed to read config file: ${error ? error.message : String(error)}`);
-        });
-        const roadmapPath = config.path;
-        const roadmap = await readRoadmapFile(roadmapPath);
-        const tasks = roadmap.tasks
-            .filter((task) => effectiveStatusFilter.includes(task.status))
-            .filter((task) => priorityFilter === null || task.priority === priorityFilter)
-            .sort((a, b) => {
-            if (sortBy === 'dueDate') {
-                const dateA = a.dueDate ? new Date(a.dueDate).getTime() : Infinity;
-                const dateB = b.dueDate ? new Date(b.dueDate).getTime() : Infinity;
-                return dateA - dateB;
+        try {
+            const incompleteOnly = flags.incomplete ?? false;
+            const priority = (flags.priority ?? null);
+            const sortBy = (flags.sort ?? null);
+            const statusFilter = (flags.status ?? null);
+            const priorityFilter = (priority ? List.priorityMap[priority] : null);
+            // if statusFilter is set, it overrides incompleteOnly
+            const effectiveStatusFilter = statusFilter
+                ? [statusFilter]
+                : incompleteOnly
+                    ? ['in-progress', 'not-started']
+                    : ['completed', 'in-progress', 'not-started'];
+            // Use repository pattern by default, unless --no-repo flag is set
+            const config = flags['no-repo'] ? await readConfigFile() : await getDefaultConfigRepository().load();
+            const roadmapPath = config.path;
+            const roadmap = flags['no-repo']
+                ? await readRoadmapFile(roadmapPath)
+                : await RoadmapRepository.fromConfig(config).load(roadmapPath);
+            // Build filter criteria
+            const filterCriteria = {};
+            if (effectiveStatusFilter.length > 0) {
+                filterCriteria.status =
+                    effectiveStatusFilter.length === 3
+                        ? undefined // All statuses, no filter needed
+                        : effectiveStatusFilter;
             }
-            if (sortBy === 'priority') {
-                const priorityOrder = {
-                    high: 1,
-                    low: 3,
-                    medium: 2,
-                };
-                return priorityOrder[a.priority] - priorityOrder[b.priority];
+            if (priorityFilter) {
+                filterCriteria.priority = priorityFilter;
             }
-            if (sortBy === 'createdAt') {
-                const createdA = a.createdAt ? new Date(a.createdAt).getTime() : Infinity;
-                const createdB = b.createdAt ? new Date(b.createdAt).getTime() : Infinity;
-                return createdA - createdB;
+            // Apply filtering and sorting using TaskQueryService
+            const hasFilters = Object.keys(filterCriteria).length > 0;
+            const filtered = hasFilters
+                ? // eslint-disable-next-line unicorn/no-array-callback-reference, unicorn/no-array-method-this-argument
+                    taskQueryService.filter(roadmap.tasks, filterCriteria)
+                : roadmap.tasks;
+            const tasks = sortBy ? taskQueryService.sort(filtered, sortBy, SortOrder.Ascending) : filtered;
+            // Display using DisplayService
+            const lines = displayService.formatTaskList(tasks);
+            for (const line of lines) {
+                console.log(line);
             }
-            return 0;
-        });
-        // Display
-        console.log(`\nTasks (${tasks.length} total):\n`);
-        for (const task of tasks) {
-            const status = task.status === STATUS.Completed ? '✓' : task.status === STATUS.InProgress ? '~' : '○';
-            const tests = task['passes-tests'] ? '✓' : '✗';
-            const prioritySymbol = {
-                high: 'H',
-                low: 'L',
-                medium: 'M',
-            }[task.priority];
-            console.log(`${status} [${prioritySymbol}] [${task.id}] ${task.title}`);
-            console.log(`   Type: ${task.type} | Tests: ${tests} | Deps: ${task['depends-on'].length}`);
-            if (task['depends-on'].length > 0) {
-                console.log(`   Depends on: ${task['depends-on'].join(', ')}`);
-            }
-            console.log();
+        }
+        catch (error) {
+            const exitCode = errorHandlerService.handleError(error);
+            this.error(errorHandlerService.formatErrorMessage(error, flags.verbose), { exit: exitCode });
         }
     }
 }

package/dist/commands/pass-test.d.ts

@@ -5,6 +5,9 @@ export default class PassTest extends Command {
     };
     static description: string;
     static examples: string[];
-    static flags: {};
+    static flags: {
+        'no-repo': import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        verbose: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+    };
     run(): Promise<void>;
 }

package/dist/commands/pass-test.js

@@ -1,4 +1,7 @@
-import { Args, Command } from '@oclif/core';
+import { Args, Command, Flags } from '@oclif/core';
+import { getDefaultConfigRepository } from '../repositories/config.repository.js';
+import { RoadmapRepository } from '../repositories/roadmap.repository.js';
+import errorHandlerService from '../services/error-handler.service.js';
 import { readConfigFile } from '../util/read-config.js';
 import { readRoadmapFile } from '../util/read-roadmap.js';
 import { updateTaskInRoadmap } from '../util/update-task.js';
@@ -10,19 +13,39 @@ export default class PassTest extends Command {
     static description = 'Mark a task as passes-tests';
     static examples = ['<%= config.bin %> <%= command.id %> F-001'];
     static flags = {
-    // flag with no value (-f, --force)
-    // force: Flags.boolean({char: 'f'}),
-    // flag with a value (-n, --name=VALUE)
-    // name: Flags.string({char: 'n', description: 'name to print'}),
+        'no-repo': Flags.boolean({
+            default: false,
+            description: 'use legacy direct file I/O instead of repository pattern',
+        }),
+        // flag with no value (-f, --force)
+        // force: Flags.boolean({char: 'f'}),
+        // flag with a value (-n, --name=VALUE)
+        // name: Flags.string({char: 'n', description: 'name to print'}),
+        verbose: Flags.boolean({
+            char: 'v',
+            default: false,
+            description: 'show detailed error information including stack traces',
+        }),
     };
     async run() {
-        const { args } = await this.parse(PassTest);
-        const config = await readConfigFile();
-        const roadmap = await readRoadmapFile(config.path);
-        const updatedRoadmap = await updateTaskInRoadmap(roadmap, args.taskID, {
-            'passes-tests': true,
-        });
-        await writeRoadmapFile(config.path, updatedRoadmap);
-        this.log(`Task ${args.taskID} marked as passing tests.`);
+        const { args, flags } = await this.parse(PassTest);
+        try {
+            // Use repository pattern by default, unless --no-repo flag is set
+            const config = flags['no-repo'] ? await readConfigFile() : await getDefaultConfigRepository().load();
+            const roadmap = flags['no-repo']
+                ? await readRoadmapFile(config.path)
+                : await RoadmapRepository.fromConfig(config).load(config.path);
+            const updatedRoadmap = await updateTaskInRoadmap(roadmap, args.taskID, {
+                'passes-tests': true,
+            });
+            await (flags['no-repo']
+                ? writeRoadmapFile(config.path, updatedRoadmap)
+                : RoadmapRepository.fromConfig(config).save(config.path, updatedRoadmap));
+            this.log(`Task ${args.taskID} marked as passing tests.`);
+        }
+        catch (error) {
+            const exitCode = errorHandlerService.handleError(error);
+            this.error(errorHandlerService.formatErrorMessage(error, flags.verbose), { exit: exitCode });
+        }
     }
 }

package/dist/commands/show.d.ts

@@ -5,6 +5,9 @@ export default class Show extends Command {
     };
     static description: string;
     static examples: string[];
-    static flags: {};
+    static flags: {
+        'no-repo': import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        verbose: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+    };
     run(): Promise<void>;
 }

package/dist/commands/show.js

@@ -1,4 +1,8 @@
-import { Args, Command /* , Flags */ } from '@oclif/core';
+import { Args, Command, Flags } from '@oclif/core';
+import { getDefaultConfigRepository } from '../repositories/config.repository.js';
+import { RoadmapRepository } from '../repositories/roadmap.repository.js';
+import displayService from '../services/display.service.js';
+import errorHandlerService from '../services/error-handler.service.js';
 import { readConfigFile } from '../util/read-config.js';
 import { readRoadmapFile } from '../util/read-roadmap.js';
 export default class Show extends Command {
@@ -8,66 +12,41 @@ export default class Show extends Command {
     static description = 'show details of a specific task in the project roadmap';
     static examples = ['<%= config.bin %> <%= command.id %> F-001'];
     static flags = {
-    // flag with no value (-f, --force)
-    // force: Flags.boolean({char: 'f'}),
-    // flag with a value (-n, --name=VALUE)
-    // name: Flags.string({char: 'n', description: 'name to print'}),
+        'no-repo': Flags.boolean({
+            default: false,
+            description: 'use legacy direct file I/O instead of repository pattern',
+        }),
+        // flag with no value (-f, --force)
+        // force: Flags.boolean({char: 'f'}),
+        // flag with a value (-n, --name=VALUE)
+        // name: Flags.string({char: 'n', description: 'name to print'}),
+        verbose: Flags.boolean({
+            char: 'v',
+            default: false,
+            description: 'show detailed error information including stack traces',
+        }),
     };
     async run() {
-        const { args } = await this.parse(Show);
-        const config = await readConfigFile().catch((error) => {
-            this.error(`failed to read config file: ${error ? error.message : String(error)}`);
-        });
-        const roadmapPath = config.path;
-        const roadmap = await readRoadmapFile(roadmapPath);
-        const task = roadmap.tasks.find((t) => t.id === args.task);
-        if (!task) {
-            this.error(`task with ID ${args.task} not found in roadmap \n  see list of tasks with: 'prt list'`);
+        const { args, flags } = await this.parse(Show);
+        try {
+            // Use repository pattern by default, unless --no-repo flag is set
+            const config = flags['no-repo'] ? await readConfigFile() : await getDefaultConfigRepository().load();
+            const roadmapPath = config.path;
+            const roadmap = flags['no-repo']
+                ? await readRoadmapFile(roadmapPath)
+                : await RoadmapRepository.fromConfig(config).load(roadmapPath);
+            const task = roadmap.tasks.find((t) => t.id === args.task);
+            if (!task) {
+                this.error(`task with ID ${args.task} not found in roadmap \n  see list of tasks with: 'prt list'`);
+            }
+            const lines = displayService.formatTaskDetails(task);
+            for (const line of lines) {
+                console.log(line);
+            }
+        }
+        catch (error) {
+            const exitCode = errorHandlerService.handleError(error);
+            this.error(errorHandlerService.formatErrorMessage(error, flags.verbose), { exit: exitCode });
         }
-        const completeStatus = task.status === 'completed' ? '✓' : task.status === 'in-progress' ? '~' : '○';
-        const testStatus = task['passes-tests'] ? '✓' : '✗';
-        const priorityLabel = {
-            high: 'High',
-            low: 'Low',
-            medium: 'Medium',
-        }[task.priority];
-        console.log(`\nTask: ${task.id}\n`);
-        console.log(`Title: ${task.title}`);
-        console.log(`Type: ${task.type}`);
-        console.log(`Priority: ${priorityLabel}`);
-        console.log(`Status: ${completeStatus} ${task.status
-            .replaceAll('-', ' ')
-            .split(' ')
-            .map((word) => word.charAt(0).toUpperCase() + word.slice(1))
-            .join(' ')} | ${testStatus} Tests Passing`);
-        console.log(`\nDetails:\n${task.details}`);
-        // Dependencies
-        if (task['depends-on'].length > 0) {
-            console.log(`\nDepends On: ${task['depends-on'].join(', ')}`);
-        }
-        else {
-            console.log(`\nDepends On: None`);
-        }
-        // Blocks (optional)
-        if (task.blocks && task.blocks.length > 0) {
-            console.log(`Blocks: ${task.blocks.join(', ')}`);
-        }
-        // Timestamps
-        console.log(`\nCreated: ${task.createdAt}`);
-        console.log(`Updated: ${task.updatedAt}`);
-        // Optional fields - only display if present
-        if (task.tags && task.tags.length > 0) {
-            console.log(`\nTags: ${task.tags.join(', ')}`);
-        }
-        if (task.effort !== undefined) {
-            console.log(`Effort: ${task.effort}`);
-        }
-        if (task['github-refs'] && task['github-refs'].length > 0) {
-            console.log(`GitHub Refs: ${task['github-refs'].join(', ')}`);
-        }
-        if (task.notes) {
-            console.log(`\nNotes:\n${task.notes}`);
-        }
-        console.log('');
     }
 }

package/dist/commands/update.d.ts

@@ -8,9 +8,12 @@ export default class Update extends Command {
     static flags: {
         'clear-notes': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         deps: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        'no-repo': import("@oclif/core/interfaces").BooleanFlag<boolean>;
         notes: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         status: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         tested: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        type: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        verbose: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
 }