fix: Cosmos DB compatibility for tasks, sprints, and projects

- Fixed Task model: use countDocuments instead of sort for taskKey generation
- Fixed Project model: added unique key field, fixed async pre-save hook
- Fixed Sprint model: fixed async pre-save hook (removed next() callback)
- Added Cosmos DB indexes to all models for sort operations
- Updated seed script to use local backend URL with configurable port

Author: DevITJAX

backend/models/Activity.js

@@ -28,4 +28,8 @@ const ActivitySchema = new mongoose.Schema({
     }
 });
 
+// Index for Cosmos DB compatibility
+ActivitySchema.index({ targetType: 1, targetId: 1, createdAt: -1 });
+ActivitySchema.index({ user: 1, createdAt: -1 });
+
 module.exports = mongoose.model('Activity', ActivitySchema);

backend/models/Attachment.js

@@ -47,4 +47,7 @@ AttachmentSchema.virtual('formattedSize').get(function () {
 
 AttachmentSchema.set('toJSON', { virtuals: true });
 
+// Index for Cosmos DB compatibility
+AttachmentSchema.index({ task: 1, createdAt: -1 });
+
 module.exports = mongoose.model('Attachment', AttachmentSchema);

backend/models/Comment.js

@@ -33,6 +33,9 @@ const CommentSchema = new mongoose.Schema({
     }
 });
 
+// Index for Cosmos DB compatibility
+CommentSchema.index({ task: 1, createdAt: -1 });
+
 // Update the updatedAt field on save
 CommentSchema.pre('save', function (next) {
     if (!this.isNew) {

backend/models/Project.js

@@ -6,6 +6,14 @@ const ProjectSchema = new mongoose.Schema({
         required: [true, 'Please add a project name'],
         trim: true
     },
+    key: {
+        type: String,
+        uppercase: true,
+        trim: true,
+        unique: true,
+        sparse: true, // Allow multiple nulls during creation
+        maxlength: [10, 'Project key cannot exceed 10 characters']
+    },
     description: {
         type: String,
         required: [true, 'Please add a description']
@@ -30,4 +38,23 @@ const ProjectSchema = new mongoose.Schema({
     }
 });
 
+// Auto-generate key from project name if not provided
+ProjectSchema.pre('save', async function () {
+    if (this.isNew && !this.key) {
+        // Generate key from first 4 chars of name, uppercase, alphanumeric only
+        let baseKey = this.name.substring(0, 4).toUpperCase().replace(/[^A-Z0-9]/g, '');
+        if (baseKey.length === 0) baseKey = 'PROJ';
+
+        // Ensure uniqueness
+        let key = baseKey;
+        let suffix = 1;
+        while (await this.constructor.findOne({ key })) {
+            key = `${baseKey}${suffix}`;
+            suffix++;
+        }
+        this.key = key;
+    }
+});
+
 module.exports = mongoose.model('Project', ProjectSchema);
+

backend/models/Sprint.js

@@ -48,22 +48,22 @@ const SprintSchema = new mongoose.Schema({
     }
 });
 
+// Indexes for Cosmos DB compatibility - required for sort operations
+SprintSchema.index({ project: 1, startDate: -1 });
+SprintSchema.index({ project: 1, endDate: -1 });
+SprintSchema.index({ project: 1, status: 1 });
+
 // Ensure only one active sprint per project
-SprintSchema.pre('save', async function (next) {
-    try {
-        if (this.status === 'active') {
-            const existingActive = await this.constructor.findOne({
-                project: this.project,
-                status: 'active',
-                _id: { $ne: this._id }
-            });
-            if (existingActive) {
-                return next(new Error('Project already has an active sprint'));
-            }
+SprintSchema.pre('save', async function () {
+    if (this.status === 'active') {
+        const existingActive = await this.constructor.findOne({
+            project: this.project,
+            status: 'active',
+            _id: { $ne: this._id }
+        });
+        if (existingActive) {
+            throw new Error('Project already has an active sprint');
         }
-        next();
-    } catch (error) {
-        next(error);
     }
 });
 

backend/models/Task.js

@@ -90,14 +90,33 @@ const TaskSchema = new mongoose.Schema({
     }
 });
 
+// Index for Cosmos DB compatibility - required for sort operations
+TaskSchema.index({ createdAt: -1 });
+TaskSchema.index({ project: 1, createdAt: -1 });
+TaskSchema.index({ sprint: 1, createdAt: -1 });
+TaskSchema.index({ taskKey: 1 });
+
 // Generate task key before saving (e.g., PROJ-1, PROJ-2)
 TaskSchema.pre('save', async function () {
     if (this.isNew && !this.taskKey) {
         const Project = mongoose.model('Project');
         const project = await Project.findById(this.project);
+        // Use project key if available, otherwise derive from name
+        const prefix = project?.key || project?.name?.substring(0, 4).toUpperCase().replace(/[^A-Z0-9]/g, '') || 'TASK';
+
+        // Count existing tasks for this project and add 1
+        // This avoids Cosmos DB sorting issues
         const count = await this.constructor.countDocuments({ project: this.project });
-        const prefix = project?.name?.substring(0, 4).toUpperCase() || 'TASK';
-        this.taskKey = `${prefix}-${count + 1}`;
+        let nextNum = count + 1;
+
+        // Ensure uniqueness by checking if key exists and incrementing if needed
+        let taskKey = `${prefix}-${nextNum}`;
+        while (await this.constructor.findOne({ taskKey })) {
+            nextNum++;
+            taskKey = `${prefix}-${nextNum}`;
+        }
+
+        this.taskKey = taskKey;
     }
     this.updatedAt = Date.now();
 });

backend/models/TimeLog.js

@@ -30,4 +30,7 @@ const TimeLogSchema = new mongoose.Schema({
     }
 });
 
+// Index for Cosmos DB compatibility
+TimeLogSchema.index({ task: 1, loggedAt: -1 });
+
 module.exports = mongoose.model('TimeLog', TimeLogSchema);

seed-data.js

@@ -1,6 +1,8 @@
 // Seed script for FlowOps - Run with: node seed-data.js
 
-const API_URL = 'https://flowops-backend.azurewebsites.net/api';
+// Change to localhost for local testing, or use Azure backend for production
+const API_URL = process.env.API_URL || 'http://localhost:3001/api';
+// const API_URL = 'https://flowops-backend.azurewebsites.net/api';
 
 async function seedData() {
     console.log('🌱 Seeding FlowOps database...\n');
@@ -148,20 +150,27 @@ async function seedData() {
 
     // 5. Create sample sprints for first project
     console.log('\n5. Creating sample sprints...');
-    
+
     if (createdProjects.length > 0) {
         const project = createdProjects[0];
         const projectId = project._id || project.id;
-        
-        // Get tasks for this project to add to sprints
-        const tasksRes = await fetch(`${API_URL}/projects/${projectId}/tasks`, { headers });
-        const tasksData = await tasksRes.json();
-        const projectTasks = tasksData.data || [];
-        
+
+        // Refresh tasks list for this project to ensure we have all tasks (newly created or existing)
+        console.log('   🔄 Fetching latest tasks for project...');
+        let projectTasks = [];
+        try {
+            const tasksRes = await fetch(`${API_URL}/projects/${projectId}/tasks`, { headers });
+            const tasksData = await tasksRes.json();
+            projectTasks = tasksData.data || [];
+            console.log(`   ✅ Found ${projectTasks.length} tasks available for sprints`);
+        } catch (e) {
+            console.log('   ⚠️ Failed to fetch tasks:', e.message);
+        }
+
         const today = new Date();
         const twoWeeksFromNow = new Date(today.getTime() + 14 * 24 * 60 * 60 * 1000);
         const fourWeeksFromNow = new Date(today.getTime() + 28 * 24 * 60 * 60 * 1000);
-        
+
         const sprints = [
             {
                 name: 'Sprint 1 - Foundation',
@@ -176,7 +185,7 @@ async function seedData() {
                 endDate: fourWeeksFromNow.toISOString()
             }
         ];
-        
+
         const createdSprints = [];
         for (const sprint of sprints) {
             try {
@@ -196,11 +205,11 @@ async function seedData() {
                 console.log(`   ❌ Failed: ${sprint.name} - ${e.message}`);
             }
         }
-        
+
         // Start the first sprint and add tasks to it
         if (createdSprints.length > 0) {
             const firstSprint = createdSprints[0];
-            
+
             // Add some tasks to the sprint
             const taskIdsForSprint = projectTasks.slice(0, 6).map(t => t._id);
             if (taskIdsForSprint.length > 0) {
@@ -215,7 +224,7 @@ async function seedData() {
                     console.log(`   ❌ Failed to add tasks to sprint`);
                 }
             }
-            
+
             // Start the sprint
             try {
                 await fetch(`${API_URL}/sprints/${firstSprint._id}/start`, {