Address CodeRabbit suggestions and apply other minor fixes

Author: DVC2

.github/copilot/tasktracker-commands.md

@@ -23,7 +23,7 @@ This document helps GitHub Copilot understand TaskTracker commands and functiona
 ## Task Fields
 
 Update these fields with the `update` command:
-- status: The task status (e.g., todo, in_progress, done, blocked)
+- status: The task status. Valid statuses are: todo, in-progress, done, blocked.
 - category: The type of task (e.g., feature, bugfix, refactor, docs, test, chore)
 - title: The task title
 - description: The task description

.gitignore

@@ -109,11 +109,6 @@ Thumbs.db
 logs
 *.log
 
-# Runtime data
-pids
-*.pid
-*.seed
-
 # Temporary files
 tmp/
 temp/

README.md

@@ -2,7 +2,7 @@
 
 [![npm version](https://badge.fury.io/js/tasktracker-cli.svg)](https://badge.fury.io/js/tasktracker-cli)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
-[![Node.js CI](https://github.com/tasktracker-cli/tasktracker/workflows/Node.js%20CI/badge.svg)](https://github.com/tasktracker-cli/tasktracker/actions)
+[![Node.js CI](https://github.com/DVC2/task_tracker/actions/workflows/test.yml/badge.svg)](https://github.com/DVC2/task_tracker/actions/workflows/test.yml)
 
 **Your development memory across AI sessions.**
 
@@ -223,13 +223,6 @@ We welcome contributions! Please see our [Contributing Guide](CONTRIBUTING.md) f
 3. Make your changes with tests
 4. Submit a pull request
 
-## 📊 **Project Stats**
-
-- 🧪 **28 tests** - All passing
-- 🎯 **Zero lint issues** - Clean codebase
-- 📦 **Lightweight** - Minimal dependencies
-- 🚀 **Fast** - Optimized for developer workflow
-
 ## 🐛 **Issues & Support**
 
 - **Bug Reports**: [GitHub Issues](https://github.com/tasktracker-cli/tasktracker/issues)

bin/tt.bash

@@ -12,7 +12,7 @@ _tasktracker_completions()
   local commands="init add quick list view update delete help context link unlink files-for-task ai ls status attach detach"
 
   # Task statuses for filtering
-  local statuses="todo in_progress done blocked" // Assuming 'review' was removed/optional
+  local statuses="todo in_progress done blocked"
 
   # Task categories (Example - might need update based on actual usage)
   local categories="feature bug docs test refactor chore"

eslint_output.txt

@@ -8,6 +8,7 @@
 const fs = require('fs');
 const path = require('path');
 const { output } = require('../core/formatting');
+const { getCurrentSession, saveJournalEntry, loadJournalEntries } = require('../utils/journal-utils');
 
 /**
  * Initialize paths required by the journal command
@@ -321,47 +322,6 @@ function displayEntries(entries, options) {
       });
 }
 
-/**
- * Save a journal entry to storage
- */
-function saveJournalEntry(entry) {
-  const journalDir = path.join(process.cwd(), '.tasktracker', 'journal');
-  if (!fs.existsSync(journalDir)) {
-    fs.mkdirSync(journalDir, { recursive: true });
-  }
-
-  const journalFile = path.join(journalDir, 'entries.json');
-  let entries = [];
-  
-  if (fs.existsSync(journalFile)) {
-    try {
-      entries = JSON.parse(fs.readFileSync(journalFile, 'utf8'));
-    } catch (e) {
-      entries = [];
-    }
-  }
-
-  entries.push(entry);
-  fs.writeFileSync(journalFile, JSON.stringify(entries, null, 2));
-}
-
-/**
- * Load all journal entries
- */
-function loadJournalEntries() {
-  const journalFile = path.join(process.cwd(), '.tasktracker', 'journal', 'entries.json');
-  
-  if (!fs.existsSync(journalFile)) {
-    return [];
-  }
-
-  try {
-    return JSON.parse(fs.readFileSync(journalFile, 'utf8'));
-  } catch (e) {
-    return [];
-  }
-}
-
 /**
  * Build context summary for AI assistants
  */
@@ -441,15 +401,6 @@ function buildContextSummary(entries, options) {
   return context;
 }
 
-/**
- * Get current session identifier
- */
-function getCurrentSession() {
-  // Simple session based on day + hour
-  const now = new Date();
-  return `${now.getFullYear()}-${now.getMonth() + 1}-${now.getDate()}-${now.getHours()}`;
-}
-
 /**
  * Get emoji for entry type
  */
@@ -469,8 +420,9 @@ function getTypeEmoji(type) {
 module.exports = {
   initPaths,
   addEntry,
+  searchEntries,
   generateContext,
   showEntries,
-  searchEntries,
-  exportEntries
+  exportEntries,
+  // Potentially other exported functions if they exist
 }; 

lib/commands/journal.js

@@ -8,6 +8,7 @@
 const fs = require('fs');
 const path = require('path');
 const { output } = require('../core/formatting');
+const { loadJournalEntries, saveJournalEntry, getCurrentSession } = require('../utils/journal-utils');
 
 /**
  * Initialize paths required by the PRD command
@@ -452,9 +453,11 @@ function buildPRDContext(prd) {
  * Generate initial journal entries from PRD
  */
 function generateJournalFromPRD(prd) {
+  const baseTimestamp = Date.now(); // Capture base timestamp
+
   // Add PRD context entry
   const contextEntry = {
-    id: Date.now(),
+    id: baseTimestamp, // Use base
     timestamp: new Date().toISOString(),
     type: 'context',
     content: `Project initialized: ${prd.title}. Goals: ${prd.goals.length}, Features: ${prd.features.length}, Requirements: ${prd.requirements.length}`,
@@ -464,57 +467,29 @@ function generateJournalFromPRD(prd) {
   };
 
   // Add goal entries
+  const goalEntries = [];
   prd.goals.forEach((goal, i) => {
     const goalEntry = {
-      id: Date.now() + i + 1,
+      id: baseTimestamp + 1000 + i, // Offset significantly to avoid collision with contextEntry and other goal entries
       timestamp: new Date().toISOString(),
       type: 'idea',
       content: `Project Goal: ${goal}`,
       tags: ['prd', 'goal'],
       files: [],
       session: getCurrentSession()
     };
-    saveJournalEntry(goalEntry);
+    goalEntries.push(goalEntry); // Collect goal entries before saving
   });
 
+  // Save entries - goals first, then context. Order can be debated but this is fine.
+  goalEntries.forEach(entry => saveJournalEntry(entry));
   saveJournalEntry(contextEntry);
 }
 
-/**
- * Get current session identifier
- */
-function getCurrentSession() {
-  const now = new Date();
-  return `${now.getFullYear()}-${now.getMonth() + 1}-${now.getDate()}-${now.getHours()}`;
-}
-
-/**
- * Save a journal entry (helper function)
- */
-function saveJournalEntry(entry) {
-  const journalDir = path.join(process.cwd(), '.tasktracker', 'journal');
-  if (!fs.existsSync(journalDir)) {
-    fs.mkdirSync(journalDir, { recursive: true });
-  }
-
-  const journalFile = path.join(journalDir, 'entries.json');
-  let entries = [];
-  
-  if (fs.existsSync(journalFile)) {
-    try {
-      entries = JSON.parse(fs.readFileSync(journalFile, 'utf8'));
-    } catch (e) {
-      entries = [];
-    }
-  }
-
-  entries.push(entry);
-  fs.writeFileSync(journalFile, JSON.stringify(entries, null, 2));
-}
-
 module.exports = {
   initPaths,
   parsePRD,
   showPRD,
-  generatePRDContext
+  generatePRDContext,
+  loadPRD
 };