streamby-core 0.43.0 → 0.43.1

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 0.43.1
+
+### Fixed
+
+- **Inviting a member to a SQL project, or asking to join one, answered 500**: both write who
+  invited the member (`invitedBy`), a column `streamby.project_members` didn't have. It's added
+  at startup to existing tables, and is in the table made new.
+- **Accepting or rejecting an invitation to a SQL project failed too**: they changed the project's
+  `members` as a MongoDB project has them, a field a SQL project doesn't have (its members are
+  rows of `project_members`); they change the member's row now. A SQL project's members are read
+  with `invitedBy`, so whoever invited them is told the answer.
+
 ## 0.43.0
 
 ### Added

package/dist/adapters/database/connectionManager.js

@@ -123,6 +123,7 @@ const createTables = async (pool) => {
         archived     BOOLEAN NOT NULL DEFAULT FALSE,
         "archivedBy" VARCHAR(255),
         "archivedAt" TIMESTAMP WITH TIME ZONE,
+        "invitedBy"  VARCHAR(255),
         "createdAt"  TIMESTAMP WITH TIME ZONE DEFAULT NOW(),
         "updatedAt"  TIMESTAMP WITH TIME ZONE DEFAULT NOW(),
         UNIQUE ("projectId", "userId")
@@ -130,6 +131,8 @@ const createTables = async (pool) => {
     `);
         await pool.query(`
       ALTER TABLE streamby.project_members ADD COLUMN IF NOT EXISTS status VARCHAR(50) NOT NULL DEFAULT 'active';
+      -- Who invited a pending member (or the member, asking to join): told when it's answered.
+      ALTER TABLE streamby.project_members ADD COLUMN IF NOT EXISTS "invitedBy" VARCHAR(255);
     `);
         console.log('✅ "project_members" table ensured to exist.');
         // Migrate projectId FK column from UUID to TEXT if table predates this schema version

package/dist/middleware/routes/member.js

@@ -157,9 +157,19 @@ function memberRouter(config) {
             if (member.status !== 'pending') {
                 return res.status(400).json({ message: 'Invitation is no longer pending' });
             }
-            const updatedMembers = project.members.map((m) => m.userId?.toString() === targetUserId ? { ...m, status: 'active' } : m);
-            await Project.update({ _id: projectId }, { members: updatedMembers });
-            await (0, notification_1.createNotification)(member.invitedBy, 'member_accepted', `${member.username} aceptó la invitación al proyecto "${project.name}"`, { projectId, userId: targetUserId });
+            if (project.dbType === 'sql') {
+                // A SQL project's members are rows of project_members, not a field of the project.
+                const connection = getSqlConnection();
+                if (!connection) {
+                    return res.status(503).json({ message: 'SQL connection not available' });
+                }
+                await sql_1.sqlAdapter.update(connection, 'project_members', { projectId, userId: targetUserId }, { status: 'active' }, 'streamby');
+            }
+            else {
+                const updatedMembers = project.members.map((m) => m.userId?.toString() === targetUserId ? { ...m, status: 'active' } : m);
+                await Project.update({ _id: projectId }, { members: updatedMembers });
+            }
+            await (0, notification_1.createNotification)(member.invitedBy, 'member_accepted', `${member.username ?? auth.username} aceptó la invitación al proyecto "${project.name}"`, { projectId, userId: targetUserId });
             res.status(200).json({ message: 'Invitation accepted' });
         }
         catch (err) {
@@ -184,8 +194,17 @@ function memberRouter(config) {
             if (member.status !== 'pending') {
                 return res.status(400).json({ message: 'Invitation is no longer pending' });
             }
-            await Project.update({ _id: projectId }, { $pull: { members: { userId: targetUserId } } });
-            await (0, notification_1.createNotification)(member.invitedBy, 'member_rejected', `${member.username} rechazó la invitación al proyecto "${project.name}"`, { projectId, userId: targetUserId });
+            if (project.dbType === 'sql') {
+                const connection = getSqlConnection();
+                if (!connection) {
+                    return res.status(503).json({ message: 'SQL connection not available' });
+                }
+                await sql_1.sqlAdapter.delete(connection, 'project_members', { projectId, userId: targetUserId }, 'streamby');
+            }
+            else {
+                await Project.update({ _id: projectId }, { $pull: { members: { userId: targetUserId } } });
+            }
+            await (0, notification_1.createNotification)(member.invitedBy, 'member_rejected', `${member.username ?? auth.username} rechazó la invitación al proyecto "${project.name}"`, { projectId, userId: targetUserId });
             res.status(200).json({ message: 'Invitation rejected' });
         }
         catch (err) {

package/dist/models/Model.js

@@ -48,27 +48,6 @@ class Model {
                     const result = await connection.query(`SELECT * FROM "${schema}"."projects" WHERE "dbConnections" @> $1::jsonb OR "storageConnections" @> $1::jsonb OR "integrationConnections" @> $1::jsonb`, [has]);
                     sqlResults = result.rows;
                 }
-                else if (this.tableName === 'projects' && processedFilter.members && processedFilter.members.$elemMatch) {
-                    // Handle members filter for SQL projects
-                    const userId = processedFilter.members.$elemMatch.userId;
-                    const schema = this.schema || 'streamby'; // Default to 'streamby' if schema not provided
-                    const query = `
-            SELECT p.*, pm."userId" as memberUserId, pm.role as memberRole, pm.status as memberStatus, pm.archived as memberArchived
-            FROM "${schema}"."projects" p
-            JOIN "${schema}"."project_members" pm ON p.id = pm."projectId"
-            WHERE pm."userId" = $1
-          `;
-                    const result = await connection.query(query, [userId]);
-                    sqlResults = result.rows.map(row => ({
-                        ...row,
-                        members: [{
-                                userId: row.memberUserId,
-                                role: row.memberRole,
-                                status: row.memberStatus,
-                                archived: row.memberArchived,
-                            }]
-                    }));
-                }
                 else {
                     // General SQL find
                     sqlResults = await sql_1.sqlAdapter.find(connection, this.tableName, processedFilter, this.schema);
@@ -76,13 +55,14 @@ class Model {
                 for (const item of sqlResults) {
                     const transformedItem = this.transformResult(item);
                     if (this.tableName === 'projects' && !transformedItem.members) {
-                        // For projects, if members not already populated by join, fetch from project_members table
+                        // A SQL project's members are rows of project_members, read with it
                         const projectMembers = await sql_1.sqlAdapter.find(connection, 'project_members', { projectId: transformedItem.id }, this.schema);
                         transformedItem.members = projectMembers.map((member) => ({
                             userId: member.userId,
                             role: member.role,
                             status: member.status,
                             archived: member.archived,
+                            ...(member.invitedBy ? { invitedBy: member.invitedBy } : {}),
                         }));
                     }
                     allResults.push(transformedItem);
@@ -138,6 +118,7 @@ class Model {
                             role: member.role,
                             status: member.status,
                             archived: member.archived,
+                            ...(member.invitedBy ? { invitedBy: member.invitedBy } : {}),
                         }));
                     }
                     // Track which connection found this record so update() routes correctly

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "streamby-core",
-  "version": "0.43.0",
+  "version": "0.43.1",
   "description": "StreamBy middleware framework for media storage management",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",