Compare commits
8 Commits
feat/ms22-
...
chore/ms21
| Author | SHA1 | Date | |
|---|---|---|---|
| f4c1c9d816 | |||
| 6521f655a8 | |||
| 0e74b03d9c | |||
| a925f91062 | |||
| 7106512fa9 | |||
| 1df20f0e13 | |||
| 8dab20c022 | |||
| 7073057e8d |
@@ -0,0 +1,2 @@
|
|||||||
|
-- AlterTable
|
||||||
|
ALTER TABLE "tasks" ADD COLUMN "assigned_agent" TEXT;
|
||||||
@@ -379,6 +379,7 @@ model Task {
|
|||||||
creatorId String @map("creator_id") @db.Uuid
|
creatorId String @map("creator_id") @db.Uuid
|
||||||
projectId String? @map("project_id") @db.Uuid
|
projectId String? @map("project_id") @db.Uuid
|
||||||
parentId String? @map("parent_id") @db.Uuid
|
parentId String? @map("parent_id") @db.Uuid
|
||||||
|
assignedAgent String? @map("assigned_agent")
|
||||||
domainId String? @map("domain_id") @db.Uuid
|
domainId String? @map("domain_id") @db.Uuid
|
||||||
sortOrder Int @default(0) @map("sort_order")
|
sortOrder Int @default(0) @map("sort_order")
|
||||||
metadata Json @default("{}")
|
metadata Json @default("{}")
|
||||||
|
|||||||
198
apps/api/src/agent-memory/agent-memory.integration.spec.ts
Normal file
198
apps/api/src/agent-memory/agent-memory.integration.spec.ts
Normal file
@@ -0,0 +1,198 @@
|
|||||||
|
import { beforeAll, beforeEach, describe, expect, it, afterAll } from "vitest";
|
||||||
|
import { randomUUID as uuid } from "crypto";
|
||||||
|
import { Test, TestingModule } from "@nestjs/testing";
|
||||||
|
import { NotFoundException } from "@nestjs/common";
|
||||||
|
import { PrismaClient } from "@prisma/client";
|
||||||
|
import { AgentMemoryService } from "./agent-memory.service";
|
||||||
|
import { PrismaService } from "../prisma/prisma.service";
|
||||||
|
|
||||||
|
const shouldRunDbIntegrationTests =
|
||||||
|
process.env.RUN_DB_TESTS === "true" && Boolean(process.env.DATABASE_URL);
|
||||||
|
const describeFn = shouldRunDbIntegrationTests ? describe : describe.skip;
|
||||||
|
|
||||||
|
async function createWorkspace(
|
||||||
|
prisma: PrismaClient,
|
||||||
|
label: string
|
||||||
|
): Promise<{ workspaceId: string; ownerId: string }> {
|
||||||
|
const workspace = await prisma.workspace.create({
|
||||||
|
data: {
|
||||||
|
name: `${label} ${Date.now()}`,
|
||||||
|
owner: {
|
||||||
|
create: {
|
||||||
|
email: `${label.toLowerCase().replace(/\s+/g, "-")}-${Date.now()}@example.com`,
|
||||||
|
name: `${label} Owner`,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
return {
|
||||||
|
workspaceId: workspace.id,
|
||||||
|
ownerId: workspace.ownerId,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
describeFn("AgentMemoryService Integration", () => {
|
||||||
|
let moduleRef: TestingModule;
|
||||||
|
let prisma: PrismaClient;
|
||||||
|
let service: AgentMemoryService;
|
||||||
|
let setupComplete = false;
|
||||||
|
|
||||||
|
let workspaceAId: string;
|
||||||
|
let workspaceAOwnerId: string;
|
||||||
|
let workspaceBId: string;
|
||||||
|
let workspaceBOwnerId: string;
|
||||||
|
|
||||||
|
beforeAll(async () => {
|
||||||
|
prisma = new PrismaClient();
|
||||||
|
await prisma.$connect();
|
||||||
|
|
||||||
|
const workspaceA = await createWorkspace(prisma, "Agent Memory Integration A");
|
||||||
|
workspaceAId = workspaceA.workspaceId;
|
||||||
|
workspaceAOwnerId = workspaceA.ownerId;
|
||||||
|
|
||||||
|
const workspaceB = await createWorkspace(prisma, "Agent Memory Integration B");
|
||||||
|
workspaceBId = workspaceB.workspaceId;
|
||||||
|
workspaceBOwnerId = workspaceB.ownerId;
|
||||||
|
|
||||||
|
moduleRef = await Test.createTestingModule({
|
||||||
|
providers: [
|
||||||
|
AgentMemoryService,
|
||||||
|
{
|
||||||
|
provide: PrismaService,
|
||||||
|
useValue: prisma,
|
||||||
|
},
|
||||||
|
],
|
||||||
|
}).compile();
|
||||||
|
|
||||||
|
service = moduleRef.get<AgentMemoryService>(AgentMemoryService);
|
||||||
|
setupComplete = true;
|
||||||
|
});
|
||||||
|
|
||||||
|
beforeEach(async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
await prisma.agentMemory.deleteMany({
|
||||||
|
where: {
|
||||||
|
workspaceId: {
|
||||||
|
in: [workspaceAId, workspaceBId],
|
||||||
|
},
|
||||||
|
},
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
afterAll(async () => {
|
||||||
|
if (!prisma) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const workspaceIds = [workspaceAId, workspaceBId].filter(
|
||||||
|
(id): id is string => typeof id === "string"
|
||||||
|
);
|
||||||
|
const ownerIds = [workspaceAOwnerId, workspaceBOwnerId].filter(
|
||||||
|
(id): id is string => typeof id === "string"
|
||||||
|
);
|
||||||
|
|
||||||
|
if (workspaceIds.length > 0) {
|
||||||
|
await prisma.agentMemory.deleteMany({
|
||||||
|
where: {
|
||||||
|
workspaceId: {
|
||||||
|
in: workspaceIds,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
});
|
||||||
|
await prisma.workspace.deleteMany({ where: { id: { in: workspaceIds } } });
|
||||||
|
}
|
||||||
|
|
||||||
|
if (ownerIds.length > 0) {
|
||||||
|
await prisma.user.deleteMany({ where: { id: { in: ownerIds } } });
|
||||||
|
}
|
||||||
|
|
||||||
|
if (moduleRef) {
|
||||||
|
await moduleRef.close();
|
||||||
|
}
|
||||||
|
await prisma.$disconnect();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("upserts and lists memory entries", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const agentId = `agent-${uuid()}`;
|
||||||
|
|
||||||
|
const entry = await service.upsert(workspaceAId, agentId, "session-context", {
|
||||||
|
value: { intent: "create-tests", depth: "integration" },
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(entry.workspaceId).toBe(workspaceAId);
|
||||||
|
expect(entry.agentId).toBe(agentId);
|
||||||
|
expect(entry.key).toBe("session-context");
|
||||||
|
|
||||||
|
const listed = await service.findAll(workspaceAId, agentId);
|
||||||
|
|
||||||
|
expect(listed).toHaveLength(1);
|
||||||
|
expect(listed[0]?.id).toBe(entry.id);
|
||||||
|
expect(listed[0]?.value).toMatchObject({ intent: "create-tests" });
|
||||||
|
});
|
||||||
|
|
||||||
|
it("updates existing key via upsert without creating duplicates", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const agentId = `agent-${uuid()}`;
|
||||||
|
|
||||||
|
const first = await service.upsert(workspaceAId, agentId, "preferences", {
|
||||||
|
value: { model: "fast" },
|
||||||
|
});
|
||||||
|
|
||||||
|
const second = await service.upsert(workspaceAId, agentId, "preferences", {
|
||||||
|
value: { model: "accurate" },
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(second.id).toBe(first.id);
|
||||||
|
expect(second.value).toMatchObject({ model: "accurate" });
|
||||||
|
|
||||||
|
const rowCount = await prisma.agentMemory.count({
|
||||||
|
where: {
|
||||||
|
workspaceId: workspaceAId,
|
||||||
|
agentId,
|
||||||
|
key: "preferences",
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(rowCount).toBe(1);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("lists keys in sorted order and isolates by workspace", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const agentId = `agent-${uuid()}`;
|
||||||
|
|
||||||
|
await service.upsert(workspaceAId, agentId, "beta", { value: { v: 2 } });
|
||||||
|
await service.upsert(workspaceAId, agentId, "alpha", { value: { v: 1 } });
|
||||||
|
await service.upsert(workspaceBId, agentId, "alpha", { value: { v: 99 } });
|
||||||
|
|
||||||
|
const workspaceAEntries = await service.findAll(workspaceAId, agentId);
|
||||||
|
const workspaceBEntries = await service.findAll(workspaceBId, agentId);
|
||||||
|
|
||||||
|
expect(workspaceAEntries.map((row) => row.key)).toEqual(["alpha", "beta"]);
|
||||||
|
expect(workspaceBEntries).toHaveLength(1);
|
||||||
|
expect(workspaceBEntries[0]?.value).toMatchObject({ v: 99 });
|
||||||
|
});
|
||||||
|
|
||||||
|
it("throws NotFoundException when requesting unknown key", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
await expect(service.findOne(workspaceAId, `agent-${uuid()}`, "missing")).rejects.toThrow(
|
||||||
|
NotFoundException
|
||||||
|
);
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -0,0 +1,239 @@
|
|||||||
|
import { beforeAll, beforeEach, describe, expect, it, afterAll, vi } from "vitest";
|
||||||
|
import { randomUUID as uuid } from "crypto";
|
||||||
|
import { Test, TestingModule } from "@nestjs/testing";
|
||||||
|
import { ConflictException } from "@nestjs/common";
|
||||||
|
import { PrismaClient, Prisma } from "@prisma/client";
|
||||||
|
import { EMBEDDING_DIMENSION } from "@mosaic/shared";
|
||||||
|
import { ConversationArchiveService } from "./conversation-archive.service";
|
||||||
|
import { PrismaService } from "../prisma/prisma.service";
|
||||||
|
import { EmbeddingService } from "../knowledge/services/embedding.service";
|
||||||
|
|
||||||
|
const shouldRunDbIntegrationTests =
|
||||||
|
process.env.RUN_DB_TESTS === "true" && Boolean(process.env.DATABASE_URL);
|
||||||
|
const describeFn = shouldRunDbIntegrationTests ? describe : describe.skip;
|
||||||
|
|
||||||
|
function vector(value: number): number[] {
|
||||||
|
return Array.from({ length: EMBEDDING_DIMENSION }, () => value);
|
||||||
|
}
|
||||||
|
|
||||||
|
function toVectorLiteral(input: number[]): string {
|
||||||
|
return `[${input.join(",")}]`;
|
||||||
|
}
|
||||||
|
|
||||||
|
describeFn("ConversationArchiveService Integration", () => {
|
||||||
|
let moduleRef: TestingModule;
|
||||||
|
let prisma: PrismaClient;
|
||||||
|
let service: ConversationArchiveService;
|
||||||
|
let workspaceId: string;
|
||||||
|
let ownerId: string;
|
||||||
|
let setupComplete = false;
|
||||||
|
|
||||||
|
const embeddingServiceMock = {
|
||||||
|
isConfigured: vi.fn(),
|
||||||
|
generateEmbedding: vi.fn(),
|
||||||
|
};
|
||||||
|
|
||||||
|
beforeAll(async () => {
|
||||||
|
prisma = new PrismaClient();
|
||||||
|
await prisma.$connect();
|
||||||
|
|
||||||
|
const workspace = await prisma.workspace.create({
|
||||||
|
data: {
|
||||||
|
name: `Conversation Archive Integration ${Date.now()}`,
|
||||||
|
owner: {
|
||||||
|
create: {
|
||||||
|
email: `conversation-archive-integration-${Date.now()}@example.com`,
|
||||||
|
name: "Conversation Archive Integration Owner",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
workspaceId = workspace.id;
|
||||||
|
ownerId = workspace.ownerId;
|
||||||
|
|
||||||
|
moduleRef = await Test.createTestingModule({
|
||||||
|
providers: [
|
||||||
|
ConversationArchiveService,
|
||||||
|
{
|
||||||
|
provide: PrismaService,
|
||||||
|
useValue: prisma,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
provide: EmbeddingService,
|
||||||
|
useValue: embeddingServiceMock,
|
||||||
|
},
|
||||||
|
],
|
||||||
|
}).compile();
|
||||||
|
|
||||||
|
service = moduleRef.get<ConversationArchiveService>(ConversationArchiveService);
|
||||||
|
setupComplete = true;
|
||||||
|
});
|
||||||
|
|
||||||
|
beforeEach(async () => {
|
||||||
|
vi.clearAllMocks();
|
||||||
|
embeddingServiceMock.isConfigured.mockReturnValue(false);
|
||||||
|
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
await prisma.conversationArchive.deleteMany({ where: { workspaceId } });
|
||||||
|
});
|
||||||
|
|
||||||
|
afterAll(async () => {
|
||||||
|
if (!prisma) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (workspaceId) {
|
||||||
|
await prisma.conversationArchive.deleteMany({ where: { workspaceId } });
|
||||||
|
await prisma.workspace.deleteMany({ where: { id: workspaceId } });
|
||||||
|
}
|
||||||
|
if (ownerId) {
|
||||||
|
await prisma.user.deleteMany({ where: { id: ownerId } });
|
||||||
|
}
|
||||||
|
|
||||||
|
if (moduleRef) {
|
||||||
|
await moduleRef.close();
|
||||||
|
}
|
||||||
|
await prisma.$disconnect();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("ingests a conversation log", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const sessionId = `session-${uuid()}`;
|
||||||
|
|
||||||
|
const result = await service.ingest(workspaceId, {
|
||||||
|
sessionId,
|
||||||
|
agentId: "agent-conversation-ingest",
|
||||||
|
messages: [
|
||||||
|
{ role: "user", content: "Can you summarize deployment issues?" },
|
||||||
|
{ role: "assistant", content: "Yes, three retries timed out in staging." },
|
||||||
|
],
|
||||||
|
summary: "Deployment retry failures discussed",
|
||||||
|
startedAt: "2026-02-28T21:00:00.000Z",
|
||||||
|
endedAt: "2026-02-28T21:05:00.000Z",
|
||||||
|
metadata: { source: "integration-test" },
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(result.id).toBeDefined();
|
||||||
|
|
||||||
|
const stored = await prisma.conversationArchive.findUnique({
|
||||||
|
where: {
|
||||||
|
id: result.id,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(stored).toBeTruthy();
|
||||||
|
expect(stored?.workspaceId).toBe(workspaceId);
|
||||||
|
expect(stored?.sessionId).toBe(sessionId);
|
||||||
|
expect(stored?.messageCount).toBe(2);
|
||||||
|
expect(stored?.summary).toBe("Deployment retry failures discussed");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("rejects duplicate session ingest per workspace", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const sessionId = `session-${uuid()}`;
|
||||||
|
const dto = {
|
||||||
|
sessionId,
|
||||||
|
agentId: "agent-conversation-duplicate",
|
||||||
|
messages: [{ role: "user", content: "hello" }],
|
||||||
|
summary: "simple conversation",
|
||||||
|
startedAt: "2026-02-28T22:00:00.000Z",
|
||||||
|
};
|
||||||
|
|
||||||
|
await service.ingest(workspaceId, dto);
|
||||||
|
|
||||||
|
await expect(service.ingest(workspaceId, dto)).rejects.toThrow(ConflictException);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("rejects semantic search when embeddings are disabled", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
embeddingServiceMock.isConfigured.mockReturnValue(false);
|
||||||
|
|
||||||
|
await expect(
|
||||||
|
service.search(workspaceId, {
|
||||||
|
query: "deployment retries",
|
||||||
|
})
|
||||||
|
).rejects.toThrow(ConflictException);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("searches archived conversations by vector similarity", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const near = vector(0.02);
|
||||||
|
const far = vector(0.85);
|
||||||
|
|
||||||
|
const matching = await prisma.conversationArchive.create({
|
||||||
|
data: {
|
||||||
|
workspaceId,
|
||||||
|
sessionId: `session-search-${uuid()}`,
|
||||||
|
agentId: "agent-conversation-search-a",
|
||||||
|
messages: [
|
||||||
|
{ role: "user", content: "What caused deployment retries?" },
|
||||||
|
{ role: "assistant", content: "A connection pool timeout." },
|
||||||
|
] as unknown as Prisma.InputJsonValue,
|
||||||
|
messageCount: 2,
|
||||||
|
summary: "Deployment retries caused by connection pool timeout",
|
||||||
|
startedAt: new Date("2026-02-28T23:00:00.000Z"),
|
||||||
|
metadata: { channel: "cli" } as Prisma.InputJsonValue,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
const nonMatching = await prisma.conversationArchive.create({
|
||||||
|
data: {
|
||||||
|
workspaceId,
|
||||||
|
sessionId: `session-search-${uuid()}`,
|
||||||
|
agentId: "agent-conversation-search-b",
|
||||||
|
messages: [
|
||||||
|
{ role: "user", content: "How is billing configured?" },
|
||||||
|
] as unknown as Prisma.InputJsonValue,
|
||||||
|
messageCount: 1,
|
||||||
|
summary: "Billing and quotas conversation",
|
||||||
|
startedAt: new Date("2026-02-28T23:10:00.000Z"),
|
||||||
|
metadata: { channel: "cli" } as Prisma.InputJsonValue,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
await prisma.$executeRaw`
|
||||||
|
UPDATE conversation_archives
|
||||||
|
SET embedding = ${toVectorLiteral(near)}::vector(${EMBEDDING_DIMENSION})
|
||||||
|
WHERE id = ${matching.id}::uuid
|
||||||
|
`;
|
||||||
|
await prisma.$executeRaw`
|
||||||
|
UPDATE conversation_archives
|
||||||
|
SET embedding = ${toVectorLiteral(far)}::vector(${EMBEDDING_DIMENSION})
|
||||||
|
WHERE id = ${nonMatching.id}::uuid
|
||||||
|
`;
|
||||||
|
|
||||||
|
embeddingServiceMock.isConfigured.mockReturnValue(true);
|
||||||
|
embeddingServiceMock.generateEmbedding.mockResolvedValue(near);
|
||||||
|
|
||||||
|
const result = await service.search(workspaceId, {
|
||||||
|
query: "deployment retries timeout",
|
||||||
|
agentId: "agent-conversation-search-a",
|
||||||
|
similarityThreshold: 0,
|
||||||
|
limit: 10,
|
||||||
|
});
|
||||||
|
|
||||||
|
const rows = result.data as Array<{ id: string; agent_id: string; similarity: number }>;
|
||||||
|
|
||||||
|
expect(result.pagination.total).toBe(1);
|
||||||
|
expect(rows).toHaveLength(1);
|
||||||
|
expect(rows[0]?.id).toBe(matching.id);
|
||||||
|
expect(rows[0]?.agent_id).toBe("agent-conversation-search-a");
|
||||||
|
expect(rows[0]?.similarity).toBeGreaterThan(0);
|
||||||
|
});
|
||||||
|
});
|
||||||
226
apps/api/src/findings/findings.integration.spec.ts
Normal file
226
apps/api/src/findings/findings.integration.spec.ts
Normal file
@@ -0,0 +1,226 @@
|
|||||||
|
import { beforeAll, beforeEach, describe, expect, it, afterAll, vi } from "vitest";
|
||||||
|
import { randomUUID as uuid } from "crypto";
|
||||||
|
import { Test, TestingModule } from "@nestjs/testing";
|
||||||
|
import { BadRequestException, NotFoundException } from "@nestjs/common";
|
||||||
|
import { PrismaClient, Prisma } from "@prisma/client";
|
||||||
|
import { FindingsService } from "./findings.service";
|
||||||
|
import { PrismaService } from "../prisma/prisma.service";
|
||||||
|
import { EmbeddingService } from "../knowledge/services/embedding.service";
|
||||||
|
|
||||||
|
const shouldRunDbIntegrationTests =
|
||||||
|
process.env.RUN_DB_TESTS === "true" && Boolean(process.env.DATABASE_URL);
|
||||||
|
const describeFn = shouldRunDbIntegrationTests ? describe : describe.skip;
|
||||||
|
|
||||||
|
const EMBEDDING_DIMENSION = 1536;
|
||||||
|
|
||||||
|
function vector(value: number): number[] {
|
||||||
|
return Array.from({ length: EMBEDDING_DIMENSION }, () => value);
|
||||||
|
}
|
||||||
|
|
||||||
|
function toVectorLiteral(input: number[]): string {
|
||||||
|
return `[${input.join(",")}]`;
|
||||||
|
}
|
||||||
|
|
||||||
|
describeFn("FindingsService Integration", () => {
|
||||||
|
let moduleRef: TestingModule;
|
||||||
|
let prisma: PrismaClient;
|
||||||
|
let service: FindingsService;
|
||||||
|
let workspaceId: string;
|
||||||
|
let ownerId: string;
|
||||||
|
let setupComplete = false;
|
||||||
|
|
||||||
|
const embeddingServiceMock = {
|
||||||
|
isConfigured: vi.fn(),
|
||||||
|
generateEmbedding: vi.fn(),
|
||||||
|
};
|
||||||
|
|
||||||
|
beforeAll(async () => {
|
||||||
|
prisma = new PrismaClient();
|
||||||
|
await prisma.$connect();
|
||||||
|
|
||||||
|
const workspace = await prisma.workspace.create({
|
||||||
|
data: {
|
||||||
|
name: `Findings Integration ${Date.now()}`,
|
||||||
|
owner: {
|
||||||
|
create: {
|
||||||
|
email: `findings-integration-${Date.now()}@example.com`,
|
||||||
|
name: "Findings Integration Owner",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
workspaceId = workspace.id;
|
||||||
|
ownerId = workspace.ownerId;
|
||||||
|
|
||||||
|
moduleRef = await Test.createTestingModule({
|
||||||
|
providers: [
|
||||||
|
FindingsService,
|
||||||
|
{
|
||||||
|
provide: PrismaService,
|
||||||
|
useValue: prisma,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
provide: EmbeddingService,
|
||||||
|
useValue: embeddingServiceMock,
|
||||||
|
},
|
||||||
|
],
|
||||||
|
}).compile();
|
||||||
|
|
||||||
|
service = moduleRef.get<FindingsService>(FindingsService);
|
||||||
|
setupComplete = true;
|
||||||
|
});
|
||||||
|
|
||||||
|
beforeEach(() => {
|
||||||
|
vi.clearAllMocks();
|
||||||
|
embeddingServiceMock.isConfigured.mockReturnValue(false);
|
||||||
|
});
|
||||||
|
|
||||||
|
afterAll(async () => {
|
||||||
|
if (!prisma) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (workspaceId) {
|
||||||
|
await prisma.finding.deleteMany({ where: { workspaceId } });
|
||||||
|
await prisma.workspace.deleteMany({ where: { id: workspaceId } });
|
||||||
|
}
|
||||||
|
if (ownerId) {
|
||||||
|
await prisma.user.deleteMany({ where: { id: ownerId } });
|
||||||
|
}
|
||||||
|
|
||||||
|
if (moduleRef) {
|
||||||
|
await moduleRef.close();
|
||||||
|
}
|
||||||
|
await prisma.$disconnect();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("creates, lists, fetches, and deletes findings", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const created = await service.create(workspaceId, {
|
||||||
|
agentId: "agent-findings-crud",
|
||||||
|
type: "security",
|
||||||
|
title: "Unescaped SQL fragment",
|
||||||
|
data: { severity: "high" },
|
||||||
|
summary: "Potential injection risk in dynamic query path.",
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(created.id).toBeDefined();
|
||||||
|
expect(created.workspaceId).toBe(workspaceId);
|
||||||
|
expect(created.taskId).toBeNull();
|
||||||
|
|
||||||
|
const listed = await service.findAll(workspaceId, {
|
||||||
|
page: 1,
|
||||||
|
limit: 10,
|
||||||
|
agentId: "agent-findings-crud",
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(listed.meta.total).toBeGreaterThanOrEqual(1);
|
||||||
|
expect(listed.data.some((row) => row.id === created.id)).toBe(true);
|
||||||
|
|
||||||
|
const found = await service.findOne(created.id, workspaceId);
|
||||||
|
expect(found.id).toBe(created.id);
|
||||||
|
expect(found.title).toBe("Unescaped SQL fragment");
|
||||||
|
|
||||||
|
await expect(service.findOne(created.id, uuid())).rejects.toThrow(NotFoundException);
|
||||||
|
|
||||||
|
await expect(service.remove(created.id, workspaceId)).resolves.toEqual({
|
||||||
|
message: "Finding deleted successfully",
|
||||||
|
});
|
||||||
|
|
||||||
|
await expect(service.findOne(created.id, workspaceId)).rejects.toThrow(NotFoundException);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("rejects create when taskId does not exist in workspace", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
await expect(
|
||||||
|
service.create(workspaceId, {
|
||||||
|
taskId: uuid(),
|
||||||
|
agentId: "agent-findings-missing-task",
|
||||||
|
type: "bug",
|
||||||
|
title: "Invalid task id",
|
||||||
|
data: { source: "integration-test" },
|
||||||
|
summary: "Should fail when task relation is not found.",
|
||||||
|
})
|
||||||
|
).rejects.toThrow(NotFoundException);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("rejects vector search when embeddings are disabled", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
embeddingServiceMock.isConfigured.mockReturnValue(false);
|
||||||
|
|
||||||
|
await expect(
|
||||||
|
service.search(workspaceId, {
|
||||||
|
query: "security issue",
|
||||||
|
})
|
||||||
|
).rejects.toThrow(BadRequestException);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("searches findings by vector similarity with filters", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const near = vector(0.01);
|
||||||
|
const far = vector(0.9);
|
||||||
|
|
||||||
|
const matchedFinding = await prisma.finding.create({
|
||||||
|
data: {
|
||||||
|
workspaceId,
|
||||||
|
agentId: "agent-findings-search-a",
|
||||||
|
type: "incident",
|
||||||
|
title: "Authentication bypass",
|
||||||
|
data: { score: 0.9 } as Prisma.InputJsonValue,
|
||||||
|
summary: "Bypass risk found in login checks.",
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
const otherFinding = await prisma.finding.create({
|
||||||
|
data: {
|
||||||
|
workspaceId,
|
||||||
|
agentId: "agent-findings-search-b",
|
||||||
|
type: "incident",
|
||||||
|
title: "Retry timeout",
|
||||||
|
data: { score: 0.2 } as Prisma.InputJsonValue,
|
||||||
|
summary: "Timeout issue in downstream retries.",
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
await prisma.$executeRaw`
|
||||||
|
UPDATE findings
|
||||||
|
SET embedding = ${toVectorLiteral(near)}::vector(1536)
|
||||||
|
WHERE id = ${matchedFinding.id}::uuid
|
||||||
|
`;
|
||||||
|
await prisma.$executeRaw`
|
||||||
|
UPDATE findings
|
||||||
|
SET embedding = ${toVectorLiteral(far)}::vector(1536)
|
||||||
|
WHERE id = ${otherFinding.id}::uuid
|
||||||
|
`;
|
||||||
|
|
||||||
|
embeddingServiceMock.isConfigured.mockReturnValue(true);
|
||||||
|
embeddingServiceMock.generateEmbedding.mockResolvedValue(near);
|
||||||
|
|
||||||
|
const result = await service.search(workspaceId, {
|
||||||
|
query: "authentication bypass risk",
|
||||||
|
agentId: "agent-findings-search-a",
|
||||||
|
limit: 10,
|
||||||
|
similarityThreshold: 0,
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(result.query).toBe("authentication bypass risk");
|
||||||
|
expect(result.meta.total).toBe(1);
|
||||||
|
expect(result.data).toHaveLength(1);
|
||||||
|
expect(result.data[0]?.id).toBe(matchedFinding.id);
|
||||||
|
expect(result.data[0]?.agentId).toBe("agent-findings-search-a");
|
||||||
|
expect(result.data.find((row) => row.id === otherFinding.id)).toBeUndefined();
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -50,6 +50,12 @@ export class CreateTaskDto {
|
|||||||
@IsUUID("4", { message: "parentId must be a valid UUID" })
|
@IsUUID("4", { message: "parentId must be a valid UUID" })
|
||||||
parentId?: string;
|
parentId?: string;
|
||||||
|
|
||||||
|
@IsOptional()
|
||||||
|
@IsString({ message: "assignedAgent must be a string" })
|
||||||
|
@MinLength(1, { message: "assignedAgent must not be empty" })
|
||||||
|
@MaxLength(255, { message: "assignedAgent must not exceed 255 characters" })
|
||||||
|
assignedAgent?: string;
|
||||||
|
|
||||||
@IsOptional()
|
@IsOptional()
|
||||||
@IsInt({ message: "sortOrder must be an integer" })
|
@IsInt({ message: "sortOrder must be an integer" })
|
||||||
@Min(0, { message: "sortOrder must be at least 0" })
|
@Min(0, { message: "sortOrder must be at least 0" })
|
||||||
|
|||||||
@@ -52,6 +52,12 @@ export class UpdateTaskDto {
|
|||||||
@IsUUID("4", { message: "parentId must be a valid UUID" })
|
@IsUUID("4", { message: "parentId must be a valid UUID" })
|
||||||
parentId?: string | null;
|
parentId?: string | null;
|
||||||
|
|
||||||
|
@IsOptional()
|
||||||
|
@IsString({ message: "assignedAgent must be a string" })
|
||||||
|
@MinLength(1, { message: "assignedAgent must not be empty" })
|
||||||
|
@MaxLength(255, { message: "assignedAgent must not exceed 255 characters" })
|
||||||
|
assignedAgent?: string | null;
|
||||||
|
|
||||||
@IsOptional()
|
@IsOptional()
|
||||||
@IsInt({ message: "sortOrder must be an integer" })
|
@IsInt({ message: "sortOrder must be an integer" })
|
||||||
@Min(0, { message: "sortOrder must be at least 0" })
|
@Min(0, { message: "sortOrder must be at least 0" })
|
||||||
|
|||||||
162
apps/api/src/tasks/tasks.assigned-agent.integration.spec.ts
Normal file
162
apps/api/src/tasks/tasks.assigned-agent.integration.spec.ts
Normal file
@@ -0,0 +1,162 @@
|
|||||||
|
import { beforeAll, beforeEach, describe, expect, it, afterAll, vi } from "vitest";
|
||||||
|
import { randomUUID as uuid } from "crypto";
|
||||||
|
import { Test, TestingModule } from "@nestjs/testing";
|
||||||
|
import { PrismaClient } from "@prisma/client";
|
||||||
|
import { TasksService } from "./tasks.service";
|
||||||
|
import { PrismaService } from "../prisma/prisma.service";
|
||||||
|
import { ActivityService } from "../activity/activity.service";
|
||||||
|
|
||||||
|
const shouldRunDbIntegrationTests =
|
||||||
|
process.env.RUN_DB_TESTS === "true" && Boolean(process.env.DATABASE_URL);
|
||||||
|
const describeFn = shouldRunDbIntegrationTests ? describe : describe.skip;
|
||||||
|
|
||||||
|
describeFn("TasksService assignedAgent Integration", () => {
|
||||||
|
let moduleRef: TestingModule;
|
||||||
|
let prisma: PrismaClient;
|
||||||
|
let service: TasksService;
|
||||||
|
let workspaceId: string;
|
||||||
|
let ownerId: string;
|
||||||
|
let setupComplete = false;
|
||||||
|
|
||||||
|
const activityServiceMock = {
|
||||||
|
logTaskCreated: vi.fn().mockResolvedValue(undefined),
|
||||||
|
logTaskUpdated: vi.fn().mockResolvedValue(undefined),
|
||||||
|
logTaskDeleted: vi.fn().mockResolvedValue(undefined),
|
||||||
|
logTaskCompleted: vi.fn().mockResolvedValue(undefined),
|
||||||
|
logTaskAssigned: vi.fn().mockResolvedValue(undefined),
|
||||||
|
};
|
||||||
|
|
||||||
|
beforeAll(async () => {
|
||||||
|
prisma = new PrismaClient();
|
||||||
|
await prisma.$connect();
|
||||||
|
|
||||||
|
const workspace = await prisma.workspace.create({
|
||||||
|
data: {
|
||||||
|
name: `Tasks Assigned Agent Integration ${Date.now()}`,
|
||||||
|
owner: {
|
||||||
|
create: {
|
||||||
|
email: `tasks-assigned-agent-integration-${Date.now()}@example.com`,
|
||||||
|
name: "Tasks Assigned Agent Integration Owner",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
workspaceId = workspace.id;
|
||||||
|
ownerId = workspace.ownerId;
|
||||||
|
|
||||||
|
moduleRef = await Test.createTestingModule({
|
||||||
|
providers: [
|
||||||
|
TasksService,
|
||||||
|
{
|
||||||
|
provide: PrismaService,
|
||||||
|
useValue: prisma,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
provide: ActivityService,
|
||||||
|
useValue: activityServiceMock,
|
||||||
|
},
|
||||||
|
],
|
||||||
|
}).compile();
|
||||||
|
|
||||||
|
service = moduleRef.get<TasksService>(TasksService);
|
||||||
|
setupComplete = true;
|
||||||
|
});
|
||||||
|
|
||||||
|
beforeEach(async () => {
|
||||||
|
vi.clearAllMocks();
|
||||||
|
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
await prisma.task.deleteMany({ where: { workspaceId } });
|
||||||
|
});
|
||||||
|
|
||||||
|
afterAll(async () => {
|
||||||
|
if (!prisma) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (workspaceId) {
|
||||||
|
await prisma.task.deleteMany({ where: { workspaceId } });
|
||||||
|
await prisma.workspace.deleteMany({ where: { id: workspaceId } });
|
||||||
|
}
|
||||||
|
if (ownerId) {
|
||||||
|
await prisma.user.deleteMany({ where: { id: ownerId } });
|
||||||
|
}
|
||||||
|
|
||||||
|
if (moduleRef) {
|
||||||
|
await moduleRef.close();
|
||||||
|
}
|
||||||
|
await prisma.$disconnect();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("persists assignedAgent on create", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const task = await service.create(workspaceId, ownerId, {
|
||||||
|
title: `Assigned agent create ${uuid()}`,
|
||||||
|
assignedAgent: "fleet-worker-1",
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(task.assignedAgent).toBe("fleet-worker-1");
|
||||||
|
|
||||||
|
const stored = await prisma.task.findUnique({
|
||||||
|
where: {
|
||||||
|
id: task.id,
|
||||||
|
},
|
||||||
|
select: {
|
||||||
|
id: true,
|
||||||
|
assignedAgent: true,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(stored).toMatchObject({
|
||||||
|
id: task.id,
|
||||||
|
assignedAgent: "fleet-worker-1",
|
||||||
|
});
|
||||||
|
|
||||||
|
const listed = await service.findAll({ workspaceId, page: 1, limit: 10 }, ownerId);
|
||||||
|
const listedTask = listed.data.find((row) => row.id === task.id);
|
||||||
|
|
||||||
|
expect(listedTask?.assignedAgent).toBe("fleet-worker-1");
|
||||||
|
});
|
||||||
|
|
||||||
|
it("updates and clears assignedAgent", async () => {
|
||||||
|
if (!setupComplete) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const created = await service.create(workspaceId, ownerId, {
|
||||||
|
title: `Assigned agent update ${uuid()}`,
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(created.assignedAgent).toBeNull();
|
||||||
|
|
||||||
|
const updated = await service.update(created.id, workspaceId, ownerId, {
|
||||||
|
assignedAgent: "fleet-worker-2",
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(updated.assignedAgent).toBe("fleet-worker-2");
|
||||||
|
|
||||||
|
const cleared = await service.update(created.id, workspaceId, ownerId, {
|
||||||
|
assignedAgent: null,
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(cleared.assignedAgent).toBeNull();
|
||||||
|
|
||||||
|
const stored = await prisma.task.findUnique({
|
||||||
|
where: {
|
||||||
|
id: created.id,
|
||||||
|
},
|
||||||
|
select: {
|
||||||
|
assignedAgent: true,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(stored?.assignedAgent).toBeNull();
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -48,6 +48,7 @@ describe("TasksService", () => {
|
|||||||
creatorId: mockUserId,
|
creatorId: mockUserId,
|
||||||
projectId: null,
|
projectId: null,
|
||||||
parentId: null,
|
parentId: null,
|
||||||
|
assignedAgent: null,
|
||||||
sortOrder: 0,
|
sortOrder: 0,
|
||||||
metadata: {},
|
metadata: {},
|
||||||
createdAt: new Date(),
|
createdAt: new Date(),
|
||||||
@@ -158,6 +159,28 @@ describe("TasksService", () => {
|
|||||||
})
|
})
|
||||||
);
|
);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it("should include assignedAgent when provided", async () => {
|
||||||
|
const createDto = {
|
||||||
|
title: "Agent-owned Task",
|
||||||
|
assignedAgent: "fleet-worker-1",
|
||||||
|
};
|
||||||
|
|
||||||
|
mockPrismaService.task.create.mockResolvedValue({
|
||||||
|
...mockTask,
|
||||||
|
assignedAgent: createDto.assignedAgent,
|
||||||
|
});
|
||||||
|
|
||||||
|
await service.create(mockWorkspaceId, mockUserId, createDto);
|
||||||
|
|
||||||
|
expect(prisma.task.create).toHaveBeenCalledWith(
|
||||||
|
expect.objectContaining({
|
||||||
|
data: expect.objectContaining({
|
||||||
|
assignedAgent: createDto.assignedAgent,
|
||||||
|
}),
|
||||||
|
})
|
||||||
|
);
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe("findAll", () => {
|
describe("findAll", () => {
|
||||||
@@ -469,6 +492,26 @@ describe("TasksService", () => {
|
|||||||
service.update(mockTaskId, mockWorkspaceId, mockUserId, { title: "Test" })
|
service.update(mockTaskId, mockWorkspaceId, mockUserId, { title: "Test" })
|
||||||
).rejects.toThrow(NotFoundException);
|
).rejects.toThrow(NotFoundException);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it("should update assignedAgent when provided", async () => {
|
||||||
|
const updateDto = { assignedAgent: "fleet-worker-2" };
|
||||||
|
|
||||||
|
mockPrismaService.task.findUnique.mockResolvedValue(mockTask);
|
||||||
|
mockPrismaService.task.update.mockResolvedValue({
|
||||||
|
...mockTask,
|
||||||
|
assignedAgent: updateDto.assignedAgent,
|
||||||
|
});
|
||||||
|
|
||||||
|
await service.update(mockTaskId, mockWorkspaceId, mockUserId, updateDto);
|
||||||
|
|
||||||
|
expect(prisma.task.update).toHaveBeenCalledWith(
|
||||||
|
expect.objectContaining({
|
||||||
|
data: expect.objectContaining({
|
||||||
|
assignedAgent: updateDto.assignedAgent,
|
||||||
|
}),
|
||||||
|
})
|
||||||
|
);
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe("remove", () => {
|
describe("remove", () => {
|
||||||
|
|||||||
@@ -67,6 +67,9 @@ export class TasksService {
|
|||||||
metadata: createTaskDto.metadata
|
metadata: createTaskDto.metadata
|
||||||
? (createTaskDto.metadata as unknown as Prisma.InputJsonValue)
|
? (createTaskDto.metadata as unknown as Prisma.InputJsonValue)
|
||||||
: {},
|
: {},
|
||||||
|
...(createTaskDto.assignedAgent !== undefined && {
|
||||||
|
assignedAgent: createTaskDto.assignedAgent,
|
||||||
|
}),
|
||||||
...(assigneeConnection && { assignee: assigneeConnection }),
|
...(assigneeConnection && { assignee: assigneeConnection }),
|
||||||
...(projectConnection && { project: projectConnection }),
|
...(projectConnection && { project: projectConnection }),
|
||||||
...(parentConnection && { parent: parentConnection }),
|
...(parentConnection && { parent: parentConnection }),
|
||||||
@@ -291,6 +294,9 @@ export class TasksService {
|
|||||||
if (updateTaskDto.parentId !== undefined && updateTaskDto.parentId !== null) {
|
if (updateTaskDto.parentId !== undefined && updateTaskDto.parentId !== null) {
|
||||||
data.parent = { connect: { id: updateTaskDto.parentId } };
|
data.parent = { connect: { id: updateTaskDto.parentId } };
|
||||||
}
|
}
|
||||||
|
if (updateTaskDto.assignedAgent !== undefined) {
|
||||||
|
data.assignedAgent = updateTaskDto.assignedAgent;
|
||||||
|
}
|
||||||
|
|
||||||
// Handle completedAt based on status changes
|
// Handle completedAt based on status changes
|
||||||
if (updateTaskDto.status) {
|
if (updateTaskDto.status) {
|
||||||
|
|||||||
@@ -230,6 +230,7 @@ const categories: CategoryConfig[] = [
|
|||||||
title: "Teams",
|
title: "Teams",
|
||||||
description: "Create and manage teams within your active workspace.",
|
description: "Create and manage teams within your active workspace.",
|
||||||
href: "/settings/teams",
|
href: "/settings/teams",
|
||||||
|
adminOnly: true,
|
||||||
accent: "var(--ms-blue-400)",
|
accent: "var(--ms-blue-400)",
|
||||||
iconBg: "rgba(47, 128, 255, 0.12)",
|
iconBg: "rgba(47, 128, 255, 0.12)",
|
||||||
icon: (
|
icon: (
|
||||||
|
|||||||
@@ -1,9 +1,12 @@
|
|||||||
import type { ReactElement, ReactNode } from "react";
|
import type { ReactElement, ReactNode } from "react";
|
||||||
import type { TeamRecord } from "@/lib/api/teams";
|
import type { TeamRecord } from "@/lib/api/teams";
|
||||||
|
|
||||||
import { render, screen } from "@testing-library/react";
|
import { WorkspaceMemberRole } from "@mosaic/shared";
|
||||||
|
import { render, screen, waitFor } from "@testing-library/react";
|
||||||
|
import userEvent from "@testing-library/user-event";
|
||||||
import { beforeEach, describe, expect, it, vi } from "vitest";
|
import { beforeEach, describe, expect, it, vi } from "vitest";
|
||||||
import { fetchTeams } from "@/lib/api/teams";
|
import { createTeam, deleteTeam, fetchTeams, updateTeam } from "@/lib/api/teams";
|
||||||
|
import { fetchUserWorkspaces } from "@/lib/api/workspaces";
|
||||||
|
|
||||||
import TeamsSettingsPage from "./page";
|
import TeamsSettingsPage from "./page";
|
||||||
|
|
||||||
@@ -22,9 +25,19 @@ vi.mock("next/link", () => ({
|
|||||||
vi.mock("@/lib/api/teams", () => ({
|
vi.mock("@/lib/api/teams", () => ({
|
||||||
fetchTeams: vi.fn(),
|
fetchTeams: vi.fn(),
|
||||||
createTeam: vi.fn(),
|
createTeam: vi.fn(),
|
||||||
|
updateTeam: vi.fn(),
|
||||||
|
deleteTeam: vi.fn(),
|
||||||
|
}));
|
||||||
|
|
||||||
|
vi.mock("@/lib/api/workspaces", () => ({
|
||||||
|
fetchUserWorkspaces: vi.fn(),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const fetchTeamsMock = vi.mocked(fetchTeams);
|
const fetchTeamsMock = vi.mocked(fetchTeams);
|
||||||
|
const createTeamMock = vi.mocked(createTeam);
|
||||||
|
const updateTeamMock = vi.mocked(updateTeam);
|
||||||
|
const deleteTeamMock = vi.mocked(deleteTeam);
|
||||||
|
const fetchUserWorkspacesMock = vi.mocked(fetchUserWorkspaces);
|
||||||
|
|
||||||
const baseTeam: TeamRecord = {
|
const baseTeam: TeamRecord = {
|
||||||
id: "team-1",
|
id: "team-1",
|
||||||
@@ -42,6 +55,33 @@ const baseTeam: TeamRecord = {
|
|||||||
describe("TeamsSettingsPage", () => {
|
describe("TeamsSettingsPage", () => {
|
||||||
beforeEach(() => {
|
beforeEach(() => {
|
||||||
vi.clearAllMocks();
|
vi.clearAllMocks();
|
||||||
|
fetchTeamsMock.mockResolvedValue([]);
|
||||||
|
fetchUserWorkspacesMock.mockResolvedValue([
|
||||||
|
{
|
||||||
|
id: "workspace-1",
|
||||||
|
name: "Personal Workspace",
|
||||||
|
ownerId: "owner-1",
|
||||||
|
role: WorkspaceMemberRole.OWNER,
|
||||||
|
createdAt: "2026-01-01T00:00:00.000Z",
|
||||||
|
},
|
||||||
|
]);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("shows access denied to non-admin users", async () => {
|
||||||
|
fetchUserWorkspacesMock.mockResolvedValueOnce([
|
||||||
|
{
|
||||||
|
id: "workspace-1",
|
||||||
|
name: "Personal Workspace",
|
||||||
|
ownerId: "owner-1",
|
||||||
|
role: WorkspaceMemberRole.MEMBER,
|
||||||
|
createdAt: "2026-01-01T00:00:00.000Z",
|
||||||
|
},
|
||||||
|
]);
|
||||||
|
|
||||||
|
render(<TeamsSettingsPage />);
|
||||||
|
|
||||||
|
expect(await screen.findByText("Access Denied")).toBeInTheDocument();
|
||||||
|
expect(fetchTeamsMock).not.toHaveBeenCalled();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("loads and renders teams from the API", async () => {
|
it("loads and renders teams from the API", async () => {
|
||||||
@@ -49,9 +89,7 @@ describe("TeamsSettingsPage", () => {
|
|||||||
|
|
||||||
render(<TeamsSettingsPage />);
|
render(<TeamsSettingsPage />);
|
||||||
|
|
||||||
expect(screen.getByText("Loading teams...")).toBeInTheDocument();
|
expect(await screen.findByText("Team Directory")).toBeInTheDocument();
|
||||||
|
|
||||||
expect(await screen.findByText("Your Teams (1)")).toBeInTheDocument();
|
|
||||||
expect(screen.getByText("Platform Team")).toBeInTheDocument();
|
expect(screen.getByText("Platform Team")).toBeInTheDocument();
|
||||||
expect(fetchTeamsMock).toHaveBeenCalledTimes(1);
|
expect(fetchTeamsMock).toHaveBeenCalledTimes(1);
|
||||||
});
|
});
|
||||||
@@ -61,8 +99,8 @@ describe("TeamsSettingsPage", () => {
|
|||||||
|
|
||||||
render(<TeamsSettingsPage />);
|
render(<TeamsSettingsPage />);
|
||||||
|
|
||||||
expect(await screen.findByText("Your Teams (0)")).toBeInTheDocument();
|
expect(await screen.findByText("No Teams Yet")).toBeInTheDocument();
|
||||||
expect(screen.getByText("No teams yet")).toBeInTheDocument();
|
expect(screen.getByText("Create the first team to get started.")).toBeInTheDocument();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("shows error state and does not show empty state", async () => {
|
it("shows error state and does not show empty state", async () => {
|
||||||
@@ -71,6 +109,82 @@ describe("TeamsSettingsPage", () => {
|
|||||||
render(<TeamsSettingsPage />);
|
render(<TeamsSettingsPage />);
|
||||||
|
|
||||||
expect(await screen.findByText("Unable to load teams")).toBeInTheDocument();
|
expect(await screen.findByText("Unable to load teams")).toBeInTheDocument();
|
||||||
expect(screen.queryByText("No teams yet")).not.toBeInTheDocument();
|
});
|
||||||
|
|
||||||
|
it("creates a team from the create dialog", async () => {
|
||||||
|
const user = userEvent.setup();
|
||||||
|
fetchTeamsMock.mockResolvedValue([baseTeam]);
|
||||||
|
createTeamMock.mockResolvedValue({
|
||||||
|
...baseTeam,
|
||||||
|
id: "team-2",
|
||||||
|
name: "Design Team",
|
||||||
|
description: "Owns design quality",
|
||||||
|
});
|
||||||
|
|
||||||
|
render(<TeamsSettingsPage />);
|
||||||
|
|
||||||
|
expect(await screen.findByText("Platform Team")).toBeInTheDocument();
|
||||||
|
|
||||||
|
const triggerButton = screen.getByRole("button", { name: "Create Team" });
|
||||||
|
await user.click(triggerButton);
|
||||||
|
|
||||||
|
await user.type(screen.getByLabelText("Name"), "Design Team");
|
||||||
|
await user.type(screen.getByLabelText("Description"), "Owns design quality");
|
||||||
|
|
||||||
|
const submitButton = screen.getAllByRole("button", { name: "Create Team" })[1];
|
||||||
|
if (!submitButton) {
|
||||||
|
throw new Error("Expected create-team submit button to be rendered");
|
||||||
|
}
|
||||||
|
await user.click(submitButton);
|
||||||
|
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(createTeamMock).toHaveBeenCalledWith({
|
||||||
|
name: "Design Team",
|
||||||
|
description: "Owns design quality",
|
||||||
|
});
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it("opens team details and updates name", async () => {
|
||||||
|
const user = userEvent.setup();
|
||||||
|
fetchTeamsMock.mockResolvedValue([baseTeam]);
|
||||||
|
updateTeamMock.mockResolvedValue({
|
||||||
|
...baseTeam,
|
||||||
|
name: "Platform Engineering",
|
||||||
|
});
|
||||||
|
|
||||||
|
render(<TeamsSettingsPage />);
|
||||||
|
|
||||||
|
expect(await screen.findByText("Platform Team")).toBeInTheDocument();
|
||||||
|
|
||||||
|
await user.click(screen.getByText("Platform Team"));
|
||||||
|
|
||||||
|
const nameInput = await screen.findByLabelText("Name");
|
||||||
|
await user.clear(nameInput);
|
||||||
|
await user.type(nameInput, "Platform Engineering");
|
||||||
|
await user.click(screen.getByRole("button", { name: "Save Changes" }));
|
||||||
|
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(updateTeamMock).toHaveBeenCalledWith("team-1", {
|
||||||
|
name: "Platform Engineering",
|
||||||
|
});
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it("deletes a team from the confirmation dialog", async () => {
|
||||||
|
const user = userEvent.setup();
|
||||||
|
fetchTeamsMock.mockResolvedValue([baseTeam]);
|
||||||
|
deleteTeamMock.mockResolvedValue();
|
||||||
|
|
||||||
|
render(<TeamsSettingsPage />);
|
||||||
|
|
||||||
|
expect(await screen.findByText("Platform Team")).toBeInTheDocument();
|
||||||
|
|
||||||
|
await user.click(screen.getByRole("button", { name: "Delete" }));
|
||||||
|
await user.click(screen.getByRole("button", { name: "Delete Team" }));
|
||||||
|
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(deleteTeamMock).toHaveBeenCalledWith("team-1");
|
||||||
|
});
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -1,244 +1,582 @@
|
|||||||
"use client";
|
"use client";
|
||||||
|
|
||||||
import type { ReactElement, SyntheticEvent } from "react";
|
import {
|
||||||
|
useCallback,
|
||||||
import { useCallback, useEffect, useState } from "react";
|
useEffect,
|
||||||
|
useState,
|
||||||
|
type ChangeEvent,
|
||||||
|
type KeyboardEvent,
|
||||||
|
type ReactElement,
|
||||||
|
type SyntheticEvent,
|
||||||
|
} from "react";
|
||||||
import Link from "next/link";
|
import Link from "next/link";
|
||||||
import { createTeam, fetchTeams, type CreateTeamDto, type TeamRecord } from "@/lib/api/teams";
|
import { Plus, Trash2, Users } from "lucide-react";
|
||||||
|
import { WorkspaceMemberRole } from "@mosaic/shared";
|
||||||
|
import { SettingsAccessDenied } from "@/components/settings/SettingsAccessDenied";
|
||||||
|
import { Badge } from "@/components/ui/badge";
|
||||||
|
import { Button } from "@/components/ui/button";
|
||||||
|
import { Card, CardContent, CardDescription, CardHeader, CardTitle } from "@/components/ui/card";
|
||||||
|
import {
|
||||||
|
Dialog,
|
||||||
|
DialogContent,
|
||||||
|
DialogDescription,
|
||||||
|
DialogFooter,
|
||||||
|
DialogHeader,
|
||||||
|
DialogTitle,
|
||||||
|
DialogTrigger,
|
||||||
|
} from "@/components/ui/dialog";
|
||||||
|
import { Input } from "@/components/ui/input";
|
||||||
|
import { Label } from "@/components/ui/label";
|
||||||
|
import { Textarea } from "@/components/ui/textarea";
|
||||||
|
import {
|
||||||
|
AlertDialog,
|
||||||
|
AlertDialogAction,
|
||||||
|
AlertDialogCancel,
|
||||||
|
AlertDialogContent,
|
||||||
|
AlertDialogDescription,
|
||||||
|
AlertDialogFooter,
|
||||||
|
AlertDialogHeader,
|
||||||
|
AlertDialogTitle,
|
||||||
|
} from "@/components/ui/alert-dialog";
|
||||||
|
import {
|
||||||
|
createTeam,
|
||||||
|
deleteTeam,
|
||||||
|
fetchTeams,
|
||||||
|
updateTeam,
|
||||||
|
type CreateTeamDto,
|
||||||
|
type TeamRecord,
|
||||||
|
type UpdateTeamDto,
|
||||||
|
} from "@/lib/api/teams";
|
||||||
|
import { fetchUserWorkspaces } from "@/lib/api/workspaces";
|
||||||
|
|
||||||
function getErrorMessage(error: unknown, fallback: string): string {
|
const INITIAL_CREATE_FORM = {
|
||||||
if (error instanceof Error) {
|
name: "",
|
||||||
return error.message;
|
description: "",
|
||||||
}
|
};
|
||||||
|
|
||||||
return fallback;
|
const INITIAL_DETAIL_FORM = {
|
||||||
|
name: "",
|
||||||
|
description: "",
|
||||||
|
};
|
||||||
|
|
||||||
|
interface DetailInitialState {
|
||||||
|
name: string;
|
||||||
|
description: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
function toMemberLabel(count: number): string {
|
||||||
|
return `${String(count)} member${count === 1 ? "" : "s"}`;
|
||||||
}
|
}
|
||||||
|
|
||||||
export default function TeamsSettingsPage(): ReactElement {
|
export default function TeamsSettingsPage(): ReactElement {
|
||||||
const [teams, setTeams] = useState<TeamRecord[]>([]);
|
const [teams, setTeams] = useState<TeamRecord[]>([]);
|
||||||
const [isLoading, setIsLoading] = useState(true);
|
const [isLoading, setIsLoading] = useState<boolean>(true);
|
||||||
const [loadError, setLoadError] = useState<string | null>(null);
|
const [isRefreshing, setIsRefreshing] = useState<boolean>(false);
|
||||||
const [isCreateDialogOpen, setIsCreateDialogOpen] = useState(false);
|
const [error, setError] = useState<string | null>(null);
|
||||||
const [isCreating, setIsCreating] = useState(false);
|
const [isAdmin, setIsAdmin] = useState<boolean | null>(null);
|
||||||
const [newTeamName, setNewTeamName] = useState("");
|
|
||||||
const [newTeamDescription, setNewTeamDescription] = useState("");
|
const [isCreateOpen, setIsCreateOpen] = useState<boolean>(false);
|
||||||
|
const [createForm, setCreateForm] = useState(INITIAL_CREATE_FORM);
|
||||||
const [createError, setCreateError] = useState<string | null>(null);
|
const [createError, setCreateError] = useState<string | null>(null);
|
||||||
|
const [isCreating, setIsCreating] = useState<boolean>(false);
|
||||||
|
|
||||||
const loadTeams = useCallback(async (): Promise<void> => {
|
const [detailTarget, setDetailTarget] = useState<TeamRecord | null>(null);
|
||||||
setIsLoading(true);
|
const [detailForm, setDetailForm] = useState(INITIAL_DETAIL_FORM);
|
||||||
|
const [detailInitial, setDetailInitial] = useState<DetailInitialState | null>(null);
|
||||||
|
const [detailError, setDetailError] = useState<string | null>(null);
|
||||||
|
const [isSavingDetails, setIsSavingDetails] = useState<boolean>(false);
|
||||||
|
|
||||||
|
const [deleteTarget, setDeleteTarget] = useState<TeamRecord | null>(null);
|
||||||
|
const [isDeleting, setIsDeleting] = useState<boolean>(false);
|
||||||
|
|
||||||
|
const loadTeams = useCallback(async (showLoadingState: boolean): Promise<void> => {
|
||||||
try {
|
try {
|
||||||
|
if (showLoadingState) {
|
||||||
|
setIsLoading(true);
|
||||||
|
} else {
|
||||||
|
setIsRefreshing(true);
|
||||||
|
}
|
||||||
|
|
||||||
const data = await fetchTeams();
|
const data = await fetchTeams();
|
||||||
setTeams(data);
|
setTeams(data);
|
||||||
setLoadError(null);
|
setError(null);
|
||||||
} catch (error) {
|
} catch (err: unknown) {
|
||||||
setLoadError(getErrorMessage(error, "Failed to load teams"));
|
setError(err instanceof Error ? err.message : "Failed to load teams");
|
||||||
} finally {
|
} finally {
|
||||||
setIsLoading(false);
|
setIsLoading(false);
|
||||||
|
setIsRefreshing(false);
|
||||||
}
|
}
|
||||||
}, []);
|
}, []);
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
void loadTeams();
|
fetchUserWorkspaces()
|
||||||
}, [loadTeams]);
|
.then((workspaces) => {
|
||||||
|
const adminRoles: WorkspaceMemberRole[] = [
|
||||||
|
WorkspaceMemberRole.OWNER,
|
||||||
|
WorkspaceMemberRole.ADMIN,
|
||||||
|
];
|
||||||
|
|
||||||
const handleCreateTeam = async (e: SyntheticEvent<HTMLFormElement>): Promise<void> => {
|
setIsAdmin(workspaces.some((workspace) => adminRoles.includes(workspace.role)));
|
||||||
e.preventDefault();
|
})
|
||||||
|
.catch(() => {
|
||||||
|
setIsAdmin(true); // fail open
|
||||||
|
});
|
||||||
|
}, []);
|
||||||
|
|
||||||
const teamName = newTeamName.trim();
|
useEffect(() => {
|
||||||
if (!teamName) return;
|
if (isAdmin !== true) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
setIsCreating(true);
|
void loadTeams(true);
|
||||||
|
}, [isAdmin, loadTeams]);
|
||||||
|
|
||||||
|
function resetCreateForm(): void {
|
||||||
|
setCreateForm(INITIAL_CREATE_FORM);
|
||||||
|
setCreateError(null);
|
||||||
|
}
|
||||||
|
|
||||||
|
function openTeamDetails(team: TeamRecord): void {
|
||||||
|
const nextDetailForm = {
|
||||||
|
name: team.name,
|
||||||
|
description: team.description ?? "",
|
||||||
|
};
|
||||||
|
|
||||||
|
setDetailTarget(team);
|
||||||
|
setDetailForm(nextDetailForm);
|
||||||
|
setDetailInitial({
|
||||||
|
name: nextDetailForm.name,
|
||||||
|
description: nextDetailForm.description,
|
||||||
|
});
|
||||||
|
setDetailError(null);
|
||||||
|
}
|
||||||
|
|
||||||
|
function resetTeamDetails(): void {
|
||||||
|
setDetailTarget(null);
|
||||||
|
setDetailForm(INITIAL_DETAIL_FORM);
|
||||||
|
setDetailInitial(null);
|
||||||
|
setDetailError(null);
|
||||||
|
}
|
||||||
|
|
||||||
|
function handleTeamRowKeyDown(event: KeyboardEvent<HTMLDivElement>, team: TeamRecord): void {
|
||||||
|
if (event.key === "Enter" || event.key === " ") {
|
||||||
|
event.preventDefault();
|
||||||
|
openTeamDetails(team);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
async function handleCreateSubmit(event: SyntheticEvent): Promise<void> {
|
||||||
|
event.preventDefault();
|
||||||
setCreateError(null);
|
setCreateError(null);
|
||||||
|
|
||||||
try {
|
const name = createForm.name.trim();
|
||||||
const description = newTeamDescription.trim();
|
if (!name) {
|
||||||
const dto: CreateTeamDto = { name: teamName };
|
setCreateError("Name is required.");
|
||||||
if (description.length > 0) {
|
return;
|
||||||
dto.description = description;
|
}
|
||||||
}
|
|
||||||
|
|
||||||
|
const description = createForm.description.trim();
|
||||||
|
const dto: CreateTeamDto = { name };
|
||||||
|
if (description) {
|
||||||
|
dto.description = description;
|
||||||
|
}
|
||||||
|
|
||||||
|
try {
|
||||||
|
setIsCreating(true);
|
||||||
await createTeam(dto);
|
await createTeam(dto);
|
||||||
setNewTeamName("");
|
setIsCreateOpen(false);
|
||||||
setNewTeamDescription("");
|
resetCreateForm();
|
||||||
setIsCreateDialogOpen(false);
|
await loadTeams(false);
|
||||||
await loadTeams();
|
} catch (err: unknown) {
|
||||||
} catch (error) {
|
setCreateError(err instanceof Error ? err.message : "Failed to create team");
|
||||||
setCreateError(getErrorMessage(error, "Failed to create team"));
|
|
||||||
} finally {
|
} finally {
|
||||||
setIsCreating(false);
|
setIsCreating(false);
|
||||||
}
|
}
|
||||||
};
|
}
|
||||||
|
|
||||||
|
async function handleDetailSubmit(event: SyntheticEvent): Promise<void> {
|
||||||
|
event.preventDefault();
|
||||||
|
|
||||||
|
if (detailTarget === null || detailInitial === null) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const name = detailForm.name.trim();
|
||||||
|
if (!name) {
|
||||||
|
setDetailError("Name is required.");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const nextDescription = detailForm.description.trim();
|
||||||
|
const normalizedNextDescription = nextDescription.length > 0 ? nextDescription : null;
|
||||||
|
const normalizedInitialDescription =
|
||||||
|
detailInitial.description.trim().length > 0 ? detailInitial.description.trim() : null;
|
||||||
|
|
||||||
|
const dto: UpdateTeamDto = {};
|
||||||
|
|
||||||
|
if (name !== detailInitial.name) {
|
||||||
|
dto.name = name;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (normalizedNextDescription !== normalizedInitialDescription) {
|
||||||
|
dto.description = normalizedNextDescription;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (Object.keys(dto).length === 0) {
|
||||||
|
resetTeamDetails();
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
try {
|
||||||
|
setIsSavingDetails(true);
|
||||||
|
setDetailError(null);
|
||||||
|
await updateTeam(detailTarget.id, dto);
|
||||||
|
resetTeamDetails();
|
||||||
|
await loadTeams(false);
|
||||||
|
} catch (err: unknown) {
|
||||||
|
setDetailError(err instanceof Error ? err.message : "Failed to update team");
|
||||||
|
} finally {
|
||||||
|
setIsSavingDetails(false);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
async function confirmDelete(): Promise<void> {
|
||||||
|
if (!deleteTarget) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
try {
|
||||||
|
setIsDeleting(true);
|
||||||
|
await deleteTeam(deleteTarget.id);
|
||||||
|
setDeleteTarget(null);
|
||||||
|
await loadTeams(false);
|
||||||
|
setError(null);
|
||||||
|
} catch (err: unknown) {
|
||||||
|
setError(err instanceof Error ? err.message : "Failed to delete team");
|
||||||
|
} finally {
|
||||||
|
setIsDeleting(false);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (isAdmin === null) {
|
||||||
|
return (
|
||||||
|
<Card className="max-w-2xl mx-auto mt-8">
|
||||||
|
<CardContent className="py-12 text-center text-muted-foreground">
|
||||||
|
Checking permissions...
|
||||||
|
</CardContent>
|
||||||
|
</Card>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!isAdmin) {
|
||||||
|
return <SettingsAccessDenied message="You need Admin or Owner role to manage teams." />;
|
||||||
|
}
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<main className="container mx-auto px-4 py-8 max-w-5xl">
|
<div className="max-w-6xl mx-auto p-6 space-y-6">
|
||||||
<div className="mb-8">
|
<div className="flex items-start justify-between gap-4">
|
||||||
<div className="flex items-center justify-between mb-2">
|
<div>
|
||||||
<h1 className="text-3xl font-bold text-gray-900">Teams</h1>
|
<div className="flex items-center gap-3">
|
||||||
<Link href="/settings" className="text-sm text-blue-600 hover:text-blue-700">
|
<h1 className="text-3xl font-bold">Teams</h1>
|
||||||
{"<-"} Back to Settings
|
<Badge variant="outline">{teams.length} total</Badge>
|
||||||
</Link>
|
|
||||||
</div>
|
|
||||||
<p className="text-gray-600">Manage teams in your active workspace</p>
|
|
||||||
</div>
|
|
||||||
|
|
||||||
<div className="bg-white rounded-lg shadow-sm border border-gray-200 p-6 mb-6">
|
|
||||||
<div className="flex items-center justify-between gap-4">
|
|
||||||
<div>
|
|
||||||
<h2 className="text-lg font-semibold text-gray-900">Create New Team</h2>
|
|
||||||
<p className="text-sm text-gray-600 mt-1">
|
|
||||||
Add a team to organize members and permissions.
|
|
||||||
</p>
|
|
||||||
</div>
|
</div>
|
||||||
<button
|
<p className="text-muted-foreground mt-1">Create and manage workspace teams</p>
|
||||||
type="button"
|
</div>
|
||||||
|
|
||||||
|
<div className="flex items-center gap-2">
|
||||||
|
<Button
|
||||||
|
variant="outline"
|
||||||
onClick={() => {
|
onClick={() => {
|
||||||
setCreateError(null);
|
void loadTeams(false);
|
||||||
setIsCreateDialogOpen(true);
|
|
||||||
}}
|
}}
|
||||||
className="px-6 py-2 bg-blue-600 text-white rounded-lg hover:bg-blue-700 font-medium"
|
disabled={isLoading || isRefreshing}
|
||||||
>
|
>
|
||||||
Create Team
|
{isRefreshing ? "Refreshing..." : "Refresh"}
|
||||||
</button>
|
</Button>
|
||||||
|
|
||||||
|
<Dialog
|
||||||
|
open={isCreateOpen}
|
||||||
|
onOpenChange={(open) => {
|
||||||
|
if (!open && !isCreating) {
|
||||||
|
resetCreateForm();
|
||||||
|
}
|
||||||
|
setIsCreateOpen(open);
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
<DialogTrigger asChild>
|
||||||
|
<Button>
|
||||||
|
<Plus className="h-4 w-4 mr-2" />
|
||||||
|
Create Team
|
||||||
|
</Button>
|
||||||
|
</DialogTrigger>
|
||||||
|
<DialogContent>
|
||||||
|
<DialogHeader>
|
||||||
|
<DialogTitle>Create Team</DialogTitle>
|
||||||
|
<DialogDescription>
|
||||||
|
Create a team in the active workspace to organize members and permissions.
|
||||||
|
</DialogDescription>
|
||||||
|
</DialogHeader>
|
||||||
|
|
||||||
|
<form
|
||||||
|
onSubmit={(event) => {
|
||||||
|
void handleCreateSubmit(event);
|
||||||
|
}}
|
||||||
|
className="space-y-4"
|
||||||
|
>
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="create-name">Name</Label>
|
||||||
|
<Input
|
||||||
|
id="create-name"
|
||||||
|
value={createForm.name}
|
||||||
|
onChange={(event: ChangeEvent<HTMLInputElement>) => {
|
||||||
|
setCreateForm((prev) => ({ ...prev, name: event.target.value }));
|
||||||
|
}}
|
||||||
|
placeholder="Platform Team"
|
||||||
|
maxLength={100}
|
||||||
|
required
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="create-description">Description</Label>
|
||||||
|
<Textarea
|
||||||
|
id="create-description"
|
||||||
|
value={createForm.description}
|
||||||
|
onChange={(event: ChangeEvent<HTMLTextAreaElement>) => {
|
||||||
|
setCreateForm((prev) => ({ ...prev, description: event.target.value }));
|
||||||
|
}}
|
||||||
|
placeholder="Owns platform services and infrastructure"
|
||||||
|
maxLength={500}
|
||||||
|
rows={4}
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{createError ? (
|
||||||
|
<p className="text-sm text-destructive" role="alert">
|
||||||
|
{createError}
|
||||||
|
</p>
|
||||||
|
) : null}
|
||||||
|
|
||||||
|
<DialogFooter>
|
||||||
|
<Button
|
||||||
|
type="button"
|
||||||
|
variant="outline"
|
||||||
|
onClick={() => {
|
||||||
|
if (!isCreating) {
|
||||||
|
setIsCreateOpen(false);
|
||||||
|
resetCreateForm();
|
||||||
|
}
|
||||||
|
}}
|
||||||
|
disabled={isCreating}
|
||||||
|
>
|
||||||
|
Cancel
|
||||||
|
</Button>
|
||||||
|
<Button type="submit" disabled={isCreating}>
|
||||||
|
{isCreating ? "Creating..." : "Create Team"}
|
||||||
|
</Button>
|
||||||
|
</DialogFooter>
|
||||||
|
</form>
|
||||||
|
</DialogContent>
|
||||||
|
</Dialog>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
{isCreateDialogOpen && (
|
<div>
|
||||||
<div
|
<Link href="/settings" className="text-sm text-blue-600 hover:text-blue-700">
|
||||||
className="fixed inset-0 z-50 flex items-center justify-center bg-black/40 px-4"
|
← Back to Settings
|
||||||
role="dialog"
|
</Link>
|
||||||
>
|
</div>
|
||||||
<div className="w-full max-w-lg rounded-lg border border-gray-200 bg-white p-6 shadow-xl">
|
|
||||||
<h3 className="text-lg font-semibold text-gray-900">Create New Team</h3>
|
{error ? (
|
||||||
<p className="mt-1 text-sm text-gray-600">
|
<Card>
|
||||||
Enter a team name and optional description.
|
<CardContent className="py-4">
|
||||||
|
<p className="text-sm text-destructive" role="alert">
|
||||||
|
{error}
|
||||||
</p>
|
</p>
|
||||||
|
</CardContent>
|
||||||
|
</Card>
|
||||||
|
) : null}
|
||||||
|
|
||||||
<form onSubmit={handleCreateTeam} className="mt-4 space-y-4">
|
{isLoading ? (
|
||||||
<div>
|
<Card>
|
||||||
<label htmlFor="team-name" className="mb-1 block text-sm font-medium text-gray-700">
|
<CardContent className="py-12 text-center text-muted-foreground">
|
||||||
Team Name
|
Loading teams...
|
||||||
</label>
|
</CardContent>
|
||||||
<input
|
</Card>
|
||||||
id="team-name"
|
) : teams.length === 0 ? (
|
||||||
type="text"
|
<Card>
|
||||||
value={newTeamName}
|
<CardHeader>
|
||||||
onChange={(e) => {
|
<CardTitle>No Teams Yet</CardTitle>
|
||||||
setNewTeamName(e.target.value);
|
<CardDescription>Create the first team to get started.</CardDescription>
|
||||||
}}
|
</CardHeader>
|
||||||
placeholder="Enter team name..."
|
</Card>
|
||||||
disabled={isCreating}
|
) : (
|
||||||
className="w-full rounded-lg border border-gray-300 px-4 py-2 focus:border-transparent focus:ring-2 focus:ring-blue-500 disabled:bg-gray-100"
|
<Card>
|
||||||
autoFocus
|
<CardHeader>
|
||||||
/>
|
<CardTitle>Team Directory</CardTitle>
|
||||||
</div>
|
<CardDescription>
|
||||||
|
Click a team to view details or edit name and description.
|
||||||
|
</CardDescription>
|
||||||
|
</CardHeader>
|
||||||
|
<CardContent className="space-y-3">
|
||||||
|
{teams.map((team) => {
|
||||||
|
const memberCount = team._count?.members ?? 0;
|
||||||
|
const description = team.description?.trim();
|
||||||
|
|
||||||
<div>
|
return (
|
||||||
<label
|
<div
|
||||||
htmlFor="team-description"
|
key={team.id}
|
||||||
className="mb-1 block text-sm font-medium text-gray-700"
|
className="rounded-md border p-4 flex flex-col gap-3 md:flex-row md:items-center md:justify-between cursor-pointer hover:bg-muted/30"
|
||||||
>
|
role="button"
|
||||||
Description (optional)
|
tabIndex={0}
|
||||||
</label>
|
|
||||||
<textarea
|
|
||||||
id="team-description"
|
|
||||||
value={newTeamDescription}
|
|
||||||
onChange={(e) => {
|
|
||||||
setNewTeamDescription(e.target.value);
|
|
||||||
}}
|
|
||||||
placeholder="Describe this team's purpose..."
|
|
||||||
disabled={isCreating}
|
|
||||||
rows={3}
|
|
||||||
className="w-full rounded-lg border border-gray-300 px-4 py-2 focus:border-transparent focus:ring-2 focus:ring-blue-500 disabled:bg-gray-100"
|
|
||||||
/>
|
|
||||||
</div>
|
|
||||||
|
|
||||||
{createError !== null && (
|
|
||||||
<div className="rounded-md border border-red-200 bg-red-50 px-3 py-2 text-sm text-red-700">
|
|
||||||
{createError}
|
|
||||||
</div>
|
|
||||||
)}
|
|
||||||
|
|
||||||
<div className="flex justify-end gap-3">
|
|
||||||
<button
|
|
||||||
type="button"
|
|
||||||
onClick={() => {
|
onClick={() => {
|
||||||
if (!isCreating) {
|
openTeamDetails(team);
|
||||||
setIsCreateDialogOpen(false);
|
}}
|
||||||
}
|
onKeyDown={(event) => {
|
||||||
|
handleTeamRowKeyDown(event, team);
|
||||||
}}
|
}}
|
||||||
disabled={isCreating}
|
|
||||||
className="px-4 py-2 rounded-lg border border-gray-300 text-gray-700 hover:bg-gray-50 disabled:cursor-not-allowed"
|
|
||||||
>
|
>
|
||||||
Cancel
|
<div className="space-y-1 min-w-0">
|
||||||
</button>
|
<p className="font-semibold truncate">{team.name}</p>
|
||||||
<button
|
<p className="text-sm text-muted-foreground truncate">
|
||||||
type="submit"
|
{description && description.length > 0 ? description : "No description"}
|
||||||
disabled={isCreating || !newTeamName.trim()}
|
</p>
|
||||||
className="px-5 py-2 rounded-lg bg-blue-600 text-white font-medium hover:bg-blue-700 disabled:cursor-not-allowed disabled:opacity-50"
|
</div>
|
||||||
>
|
|
||||||
{isCreating ? "Creating..." : "Create Team"}
|
<div className="flex items-center gap-2 flex-wrap md:justify-end">
|
||||||
</button>
|
<Badge variant="outline">
|
||||||
</div>
|
<Users className="h-3.5 w-3.5 mr-1" />
|
||||||
</form>
|
{toMemberLabel(memberCount)}
|
||||||
</div>
|
</Badge>
|
||||||
</div>
|
<Badge variant="secondary">
|
||||||
|
Created {new Date(team.createdAt).toLocaleDateString()}
|
||||||
|
</Badge>
|
||||||
|
<Button
|
||||||
|
variant="destructive"
|
||||||
|
size="sm"
|
||||||
|
onClick={(event) => {
|
||||||
|
event.stopPropagation();
|
||||||
|
setDeleteTarget(team);
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
<Trash2 className="h-4 w-4 mr-2" />
|
||||||
|
Delete
|
||||||
|
</Button>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
})}
|
||||||
|
</CardContent>
|
||||||
|
</Card>
|
||||||
)}
|
)}
|
||||||
|
|
||||||
<div className="space-y-4">
|
<Dialog
|
||||||
<h2 className="text-xl font-semibold text-gray-900">
|
open={detailTarget !== null}
|
||||||
Your Teams ({isLoading ? "..." : teams.length})
|
onOpenChange={(open) => {
|
||||||
</h2>
|
if (!open && !isSavingDetails) {
|
||||||
{loadError !== null ? (
|
resetTeamDetails();
|
||||||
<div className="rounded-md border border-red-200 bg-red-50 px-4 py-3 text-red-700">
|
}
|
||||||
{loadError}
|
}}
|
||||||
</div>
|
>
|
||||||
) : isLoading ? (
|
<DialogContent>
|
||||||
<div className="bg-white rounded-lg shadow-sm border border-gray-200 p-12 text-center text-gray-600">
|
<DialogHeader>
|
||||||
Loading teams...
|
<DialogTitle>Team Details</DialogTitle>
|
||||||
</div>
|
<DialogDescription>
|
||||||
) : teams.length === 0 ? (
|
Edit team details for {detailTarget?.name ?? "selected team"}.
|
||||||
<div className="bg-white rounded-lg shadow-sm border border-gray-200 p-12 text-center">
|
</DialogDescription>
|
||||||
<svg
|
</DialogHeader>
|
||||||
className="mx-auto h-12 w-12 text-gray-400 mb-4"
|
|
||||||
fill="none"
|
<form
|
||||||
stroke="currentColor"
|
onSubmit={(event) => {
|
||||||
viewBox="0 0 24 24"
|
void handleDetailSubmit(event);
|
||||||
>
|
}}
|
||||||
<path
|
className="space-y-4"
|
||||||
strokeLinecap="round"
|
>
|
||||||
strokeLinejoin="round"
|
<div className="space-y-2">
|
||||||
strokeWidth={2}
|
<Label htmlFor="detail-name">Name</Label>
|
||||||
d="M17 20h5V8H2v12h5m10 0v-4a3 3 0 10-6 0v4m6 0H7"
|
<Input
|
||||||
|
id="detail-name"
|
||||||
|
value={detailForm.name}
|
||||||
|
onChange={(event: ChangeEvent<HTMLInputElement>) => {
|
||||||
|
setDetailForm((prev) => ({ ...prev, name: event.target.value }));
|
||||||
|
}}
|
||||||
|
placeholder="Team name"
|
||||||
|
maxLength={100}
|
||||||
|
disabled={isSavingDetails}
|
||||||
|
required
|
||||||
/>
|
/>
|
||||||
</svg>
|
</div>
|
||||||
<h3 className="text-lg font-medium text-gray-900 mb-2">No teams yet</h3>
|
|
||||||
<p className="text-gray-600">Create your first team to get started</p>
|
<div className="space-y-2">
|
||||||
</div>
|
<Label htmlFor="detail-description">Description</Label>
|
||||||
) : (
|
<Textarea
|
||||||
<div className="grid grid-cols-1 md:grid-cols-2 gap-4">
|
id="detail-description"
|
||||||
{teams.map((team) => (
|
value={detailForm.description}
|
||||||
<article
|
onChange={(event: ChangeEvent<HTMLTextAreaElement>) => {
|
||||||
key={team.id}
|
setDetailForm((prev) => ({ ...prev, description: event.target.value }));
|
||||||
className="rounded-lg border border-gray-200 bg-white p-5 shadow-sm"
|
}}
|
||||||
data-testid="team-card"
|
placeholder="Describe this team"
|
||||||
|
maxLength={500}
|
||||||
|
rows={4}
|
||||||
|
disabled={isSavingDetails}
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{detailError !== null ? (
|
||||||
|
<p className="text-sm text-destructive" role="alert">
|
||||||
|
{detailError}
|
||||||
|
</p>
|
||||||
|
) : null}
|
||||||
|
|
||||||
|
<DialogFooter>
|
||||||
|
<Button
|
||||||
|
type="button"
|
||||||
|
variant="outline"
|
||||||
|
onClick={() => {
|
||||||
|
if (!isSavingDetails) {
|
||||||
|
resetTeamDetails();
|
||||||
|
}
|
||||||
|
}}
|
||||||
|
disabled={isSavingDetails}
|
||||||
>
|
>
|
||||||
<h3 className="text-lg font-semibold text-gray-900">{team.name}</h3>
|
Cancel
|
||||||
{team.description ? (
|
</Button>
|
||||||
<p className="mt-1 text-sm text-gray-600">{team.description}</p>
|
<Button type="submit" disabled={isSavingDetails}>
|
||||||
) : (
|
{isSavingDetails ? "Saving..." : "Save Changes"}
|
||||||
<p className="mt-1 text-sm text-gray-400 italic">No description</p>
|
</Button>
|
||||||
)}
|
</DialogFooter>
|
||||||
<div className="mt-4 flex items-center gap-3 text-xs text-gray-500">
|
</form>
|
||||||
<span>{team._count?.members ?? 0} members</span>
|
</DialogContent>
|
||||||
<span>|</span>
|
</Dialog>
|
||||||
<span>Created {new Date(team.createdAt).toLocaleDateString()}</span>
|
|
||||||
</div>
|
<AlertDialog
|
||||||
</article>
|
open={deleteTarget !== null}
|
||||||
))}
|
onOpenChange={(open) => {
|
||||||
</div>
|
if (!open && !isDeleting) {
|
||||||
)}
|
setDeleteTarget(null);
|
||||||
</div>
|
}
|
||||||
</main>
|
}}
|
||||||
|
>
|
||||||
|
<AlertDialogContent>
|
||||||
|
<AlertDialogHeader>
|
||||||
|
<AlertDialogTitle>Delete Team</AlertDialogTitle>
|
||||||
|
<AlertDialogDescription>
|
||||||
|
Delete {deleteTarget?.name}? Team members will be removed from this team assignment.
|
||||||
|
</AlertDialogDescription>
|
||||||
|
</AlertDialogHeader>
|
||||||
|
<AlertDialogFooter>
|
||||||
|
<AlertDialogCancel disabled={isDeleting}>Cancel</AlertDialogCancel>
|
||||||
|
<AlertDialogAction
|
||||||
|
disabled={isDeleting}
|
||||||
|
onClick={() => {
|
||||||
|
void confirmDelete();
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
{isDeleting ? "Deleting..." : "Delete Team"}
|
||||||
|
</AlertDialogAction>
|
||||||
|
</AlertDialogFooter>
|
||||||
|
</AlertDialogContent>
|
||||||
|
</AlertDialog>
|
||||||
|
</div>
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|||||||
177
apps/web/src/app/(authenticated)/settings/users/page.test.tsx
Normal file
177
apps/web/src/app/(authenticated)/settings/users/page.test.tsx
Normal file
@@ -0,0 +1,177 @@
|
|||||||
|
import type { ReactElement, ReactNode } from "react";
|
||||||
|
|
||||||
|
import { WorkspaceMemberRole } from "@mosaic/shared";
|
||||||
|
import { render, screen, waitFor } from "@testing-library/react";
|
||||||
|
import userEvent from "@testing-library/user-event";
|
||||||
|
import { beforeEach, describe, expect, it, vi } from "vitest";
|
||||||
|
|
||||||
|
import {
|
||||||
|
deactivateUser,
|
||||||
|
fetchAdminUsers,
|
||||||
|
inviteUser,
|
||||||
|
updateUser,
|
||||||
|
type AdminUsersResponse,
|
||||||
|
} from "@/lib/api/admin";
|
||||||
|
import { fetchUserWorkspaces, updateWorkspaceMemberRole } from "@/lib/api/workspaces";
|
||||||
|
import UsersSettingsPage from "./page";
|
||||||
|
|
||||||
|
vi.mock("next/link", () => ({
|
||||||
|
default: function LinkMock({
|
||||||
|
children,
|
||||||
|
href,
|
||||||
|
}: {
|
||||||
|
children: ReactNode;
|
||||||
|
href: string;
|
||||||
|
}): ReactElement {
|
||||||
|
return <a href={href}>{children}</a>;
|
||||||
|
},
|
||||||
|
}));
|
||||||
|
|
||||||
|
vi.mock("@/lib/api/admin", () => ({
|
||||||
|
fetchAdminUsers: vi.fn(),
|
||||||
|
inviteUser: vi.fn(),
|
||||||
|
updateUser: vi.fn(),
|
||||||
|
deactivateUser: vi.fn(),
|
||||||
|
}));
|
||||||
|
|
||||||
|
vi.mock("@/lib/api/workspaces", () => ({
|
||||||
|
fetchUserWorkspaces: vi.fn(),
|
||||||
|
updateWorkspaceMemberRole: vi.fn(),
|
||||||
|
}));
|
||||||
|
|
||||||
|
const fetchAdminUsersMock = vi.mocked(fetchAdminUsers);
|
||||||
|
const inviteUserMock = vi.mocked(inviteUser);
|
||||||
|
const updateUserMock = vi.mocked(updateUser);
|
||||||
|
const deactivateUserMock = vi.mocked(deactivateUser);
|
||||||
|
const fetchUserWorkspacesMock = vi.mocked(fetchUserWorkspaces);
|
||||||
|
const updateWorkspaceMemberRoleMock = vi.mocked(updateWorkspaceMemberRole);
|
||||||
|
|
||||||
|
const adminUsersResponse: AdminUsersResponse = {
|
||||||
|
data: [
|
||||||
|
{
|
||||||
|
id: "user-1",
|
||||||
|
name: "Alice",
|
||||||
|
email: "alice@example.com",
|
||||||
|
emailVerified: true,
|
||||||
|
image: null,
|
||||||
|
createdAt: "2026-01-01T00:00:00.000Z",
|
||||||
|
deactivatedAt: null,
|
||||||
|
isLocalAuth: false,
|
||||||
|
invitedAt: null,
|
||||||
|
invitedBy: null,
|
||||||
|
workspaceMemberships: [
|
||||||
|
{
|
||||||
|
workspaceId: "workspace-1",
|
||||||
|
workspaceName: "Personal Workspace",
|
||||||
|
role: WorkspaceMemberRole.ADMIN,
|
||||||
|
joinedAt: "2026-01-01T00:00:00.000Z",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
},
|
||||||
|
],
|
||||||
|
meta: {
|
||||||
|
total: 1,
|
||||||
|
page: 1,
|
||||||
|
limit: 50,
|
||||||
|
totalPages: 1,
|
||||||
|
},
|
||||||
|
};
|
||||||
|
|
||||||
|
describe("UsersSettingsPage", () => {
|
||||||
|
beforeEach(() => {
|
||||||
|
vi.clearAllMocks();
|
||||||
|
|
||||||
|
fetchAdminUsersMock.mockResolvedValue(adminUsersResponse);
|
||||||
|
fetchUserWorkspacesMock.mockResolvedValue([
|
||||||
|
{
|
||||||
|
id: "workspace-1",
|
||||||
|
name: "Personal Workspace",
|
||||||
|
ownerId: "owner-1",
|
||||||
|
role: WorkspaceMemberRole.OWNER,
|
||||||
|
createdAt: "2026-01-01T00:00:00.000Z",
|
||||||
|
},
|
||||||
|
]);
|
||||||
|
inviteUserMock.mockResolvedValue({
|
||||||
|
userId: "user-2",
|
||||||
|
invitationToken: "token-1",
|
||||||
|
email: "new@example.com",
|
||||||
|
invitedAt: "2026-01-02T00:00:00.000Z",
|
||||||
|
});
|
||||||
|
const firstUser = adminUsersResponse.data[0];
|
||||||
|
if (!firstUser) {
|
||||||
|
throw new Error("Expected at least one admin user in test fixtures");
|
||||||
|
}
|
||||||
|
|
||||||
|
updateUserMock.mockResolvedValue(firstUser);
|
||||||
|
deactivateUserMock.mockResolvedValue(firstUser);
|
||||||
|
updateWorkspaceMemberRoleMock.mockResolvedValue({
|
||||||
|
workspaceId: "workspace-1",
|
||||||
|
userId: "user-1",
|
||||||
|
role: WorkspaceMemberRole.ADMIN,
|
||||||
|
joinedAt: "2026-01-01T00:00:00.000Z",
|
||||||
|
user: {
|
||||||
|
id: "user-1",
|
||||||
|
email: "alice@example.com",
|
||||||
|
name: "Alice",
|
||||||
|
image: null,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it("shows access denied to non-admin users", async () => {
|
||||||
|
fetchUserWorkspacesMock.mockResolvedValueOnce([
|
||||||
|
{
|
||||||
|
id: "workspace-1",
|
||||||
|
name: "Personal Workspace",
|
||||||
|
ownerId: "owner-1",
|
||||||
|
role: WorkspaceMemberRole.MEMBER,
|
||||||
|
createdAt: "2026-01-01T00:00:00.000Z",
|
||||||
|
},
|
||||||
|
]);
|
||||||
|
|
||||||
|
render(<UsersSettingsPage />);
|
||||||
|
|
||||||
|
expect(await screen.findByText("Access Denied")).toBeInTheDocument();
|
||||||
|
expect(fetchAdminUsersMock).not.toHaveBeenCalled();
|
||||||
|
});
|
||||||
|
|
||||||
|
it("invites a user with email and role from the dialog", async () => {
|
||||||
|
const user = userEvent.setup();
|
||||||
|
render(<UsersSettingsPage />);
|
||||||
|
|
||||||
|
expect(await screen.findByText("User Directory")).toBeInTheDocument();
|
||||||
|
|
||||||
|
await user.click(screen.getByRole("button", { name: "Invite User" }));
|
||||||
|
await user.type(screen.getByLabelText("Email"), "new@example.com");
|
||||||
|
await user.click(screen.getByRole("button", { name: "Send Invite" }));
|
||||||
|
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(inviteUserMock).toHaveBeenCalledWith({
|
||||||
|
email: "new@example.com",
|
||||||
|
role: WorkspaceMemberRole.MEMBER,
|
||||||
|
workspaceId: "workspace-1",
|
||||||
|
});
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it("opens user detail dialog from row click and saves edited profile fields", async () => {
|
||||||
|
const user = userEvent.setup();
|
||||||
|
render(<UsersSettingsPage />);
|
||||||
|
|
||||||
|
expect(await screen.findByText("alice@example.com")).toBeInTheDocument();
|
||||||
|
|
||||||
|
await user.click(screen.getByText("Alice"));
|
||||||
|
|
||||||
|
const nameInput = await screen.findByLabelText("Name");
|
||||||
|
await user.clear(nameInput);
|
||||||
|
await user.type(nameInput, "Alice Updated");
|
||||||
|
|
||||||
|
await user.click(screen.getByRole("button", { name: "Save Changes" }));
|
||||||
|
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(updateUserMock).toHaveBeenCalledWith("user-1", { name: "Alice Updated" });
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(updateWorkspaceMemberRoleMock).not.toHaveBeenCalled();
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -5,12 +5,14 @@ import {
|
|||||||
useEffect,
|
useEffect,
|
||||||
useState,
|
useState,
|
||||||
type ChangeEvent,
|
type ChangeEvent,
|
||||||
|
type KeyboardEvent,
|
||||||
type ReactElement,
|
type ReactElement,
|
||||||
type SyntheticEvent,
|
type SyntheticEvent,
|
||||||
} from "react";
|
} from "react";
|
||||||
import Link from "next/link";
|
import Link from "next/link";
|
||||||
import { Pencil, UserPlus, UserX } from "lucide-react";
|
import { UserPlus, UserX } from "lucide-react";
|
||||||
import { WorkspaceMemberRole } from "@mosaic/shared";
|
import { WorkspaceMemberRole } from "@mosaic/shared";
|
||||||
|
import { isValidEmail } from "@/components/workspace/validation";
|
||||||
import { Badge } from "@/components/ui/badge";
|
import { Badge } from "@/components/ui/badge";
|
||||||
import { Button } from "@/components/ui/button";
|
import { Button } from "@/components/ui/button";
|
||||||
import { Card, CardContent, CardDescription, CardHeader, CardTitle } from "@/components/ui/card";
|
import { Card, CardContent, CardDescription, CardHeader, CardTitle } from "@/components/ui/card";
|
||||||
@@ -42,7 +44,6 @@ import {
|
|||||||
AlertDialogHeader,
|
AlertDialogHeader,
|
||||||
AlertDialogTitle,
|
AlertDialogTitle,
|
||||||
} from "@/components/ui/alert-dialog";
|
} from "@/components/ui/alert-dialog";
|
||||||
import { fetchUserWorkspaces } from "@/lib/api/workspaces";
|
|
||||||
import {
|
import {
|
||||||
deactivateUser,
|
deactivateUser,
|
||||||
fetchAdminUsers,
|
fetchAdminUsers,
|
||||||
@@ -50,9 +51,12 @@ import {
|
|||||||
updateUser,
|
updateUser,
|
||||||
type AdminUser,
|
type AdminUser,
|
||||||
type AdminUsersResponse,
|
type AdminUsersResponse,
|
||||||
|
type AdminWorkspaceMembership,
|
||||||
type InviteUserDto,
|
type InviteUserDto,
|
||||||
type UpdateUserDto,
|
type UpdateUserDto,
|
||||||
} from "@/lib/api/admin";
|
} from "@/lib/api/admin";
|
||||||
|
import { fetchUserWorkspaces, updateWorkspaceMemberRole } from "@/lib/api/workspaces";
|
||||||
|
import { SettingsAccessDenied } from "@/components/settings/SettingsAccessDenied";
|
||||||
|
|
||||||
const ROLE_PRIORITY: Record<WorkspaceMemberRole, number> = {
|
const ROLE_PRIORITY: Record<WorkspaceMemberRole, number> = {
|
||||||
[WorkspaceMemberRole.OWNER]: 4,
|
[WorkspaceMemberRole.OWNER]: 4,
|
||||||
@@ -63,27 +67,40 @@ const ROLE_PRIORITY: Record<WorkspaceMemberRole, number> = {
|
|||||||
|
|
||||||
const INITIAL_INVITE_FORM = {
|
const INITIAL_INVITE_FORM = {
|
||||||
email: "",
|
email: "",
|
||||||
name: "",
|
|
||||||
workspaceId: "",
|
|
||||||
role: WorkspaceMemberRole.MEMBER,
|
role: WorkspaceMemberRole.MEMBER,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const INITIAL_DETAIL_FORM = {
|
||||||
|
name: "",
|
||||||
|
email: "",
|
||||||
|
role: WorkspaceMemberRole.MEMBER,
|
||||||
|
workspaceId: null as string | null,
|
||||||
|
workspaceName: null as string | null,
|
||||||
|
};
|
||||||
|
|
||||||
|
interface DetailInitialState {
|
||||||
|
name: string;
|
||||||
|
email: string;
|
||||||
|
role: WorkspaceMemberRole;
|
||||||
|
workspaceId: string | null;
|
||||||
|
}
|
||||||
|
|
||||||
function toRoleLabel(role: WorkspaceMemberRole): string {
|
function toRoleLabel(role: WorkspaceMemberRole): string {
|
||||||
return `${role.charAt(0)}${role.slice(1).toLowerCase()}`;
|
return `${role.charAt(0)}${role.slice(1).toLowerCase()}`;
|
||||||
}
|
}
|
||||||
|
|
||||||
function getPrimaryRole(user: AdminUser): WorkspaceMemberRole | null {
|
function getPrimaryMembership(user: AdminUser): AdminWorkspaceMembership | null {
|
||||||
const [firstMembership, ...restMemberships] = user.workspaceMemberships;
|
const [firstMembership, ...restMemberships] = user.workspaceMemberships;
|
||||||
if (!firstMembership) {
|
if (!firstMembership) {
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
||||||
return restMemberships.reduce((highest, membership) => {
|
return restMemberships.reduce((highest, membership) => {
|
||||||
if (ROLE_PRIORITY[membership.role] > ROLE_PRIORITY[highest]) {
|
if (ROLE_PRIORITY[membership.role] > ROLE_PRIORITY[highest.role]) {
|
||||||
return membership.role;
|
return membership;
|
||||||
}
|
}
|
||||||
return highest;
|
return highest;
|
||||||
}, firstMembership.role);
|
}, firstMembership);
|
||||||
}
|
}
|
||||||
|
|
||||||
export default function UsersSettingsPage(): ReactElement {
|
export default function UsersSettingsPage(): ReactElement {
|
||||||
@@ -93,21 +110,23 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
const [isRefreshing, setIsRefreshing] = useState<boolean>(false);
|
const [isRefreshing, setIsRefreshing] = useState<boolean>(false);
|
||||||
const [error, setError] = useState<string | null>(null);
|
const [error, setError] = useState<string | null>(null);
|
||||||
|
|
||||||
|
const [defaultWorkspaceId, setDefaultWorkspaceId] = useState<string | null>(null);
|
||||||
|
const [isAdmin, setIsAdmin] = useState<boolean | null>(null);
|
||||||
|
|
||||||
const [isInviteOpen, setIsInviteOpen] = useState<boolean>(false);
|
const [isInviteOpen, setIsInviteOpen] = useState<boolean>(false);
|
||||||
const [inviteForm, setInviteForm] = useState(INITIAL_INVITE_FORM);
|
const [inviteForm, setInviteForm] = useState(INITIAL_INVITE_FORM);
|
||||||
const [inviteError, setInviteError] = useState<string | null>(null);
|
const [inviteError, setInviteError] = useState<string | null>(null);
|
||||||
const [isInviting, setIsInviting] = useState<boolean>(false);
|
const [isInviting, setIsInviting] = useState<boolean>(false);
|
||||||
|
|
||||||
|
const [detailTarget, setDetailTarget] = useState<AdminUser | null>(null);
|
||||||
|
const [detailForm, setDetailForm] = useState(INITIAL_DETAIL_FORM);
|
||||||
|
const [detailInitial, setDetailInitial] = useState<DetailInitialState | null>(null);
|
||||||
|
const [detailError, setDetailError] = useState<string | null>(null);
|
||||||
|
const [isSavingDetails, setIsSavingDetails] = useState<boolean>(false);
|
||||||
|
|
||||||
const [deactivateTarget, setDeactivateTarget] = useState<AdminUser | null>(null);
|
const [deactivateTarget, setDeactivateTarget] = useState<AdminUser | null>(null);
|
||||||
const [isDeactivating, setIsDeactivating] = useState<boolean>(false);
|
const [isDeactivating, setIsDeactivating] = useState<boolean>(false);
|
||||||
|
|
||||||
const [editTarget, setEditTarget] = useState<AdminUser | null>(null);
|
|
||||||
const [editName, setEditName] = useState<string>("");
|
|
||||||
const [editError, setEditError] = useState<string | null>(null);
|
|
||||||
const [isEditing, setIsEditing] = useState<boolean>(false);
|
|
||||||
|
|
||||||
const [isAdmin, setIsAdmin] = useState<boolean | null>(null);
|
|
||||||
|
|
||||||
const loadUsers = useCallback(async (showLoadingState: boolean): Promise<void> => {
|
const loadUsers = useCallback(async (showLoadingState: boolean): Promise<void> => {
|
||||||
try {
|
try {
|
||||||
if (showLoadingState) {
|
if (showLoadingState) {
|
||||||
@@ -128,10 +147,6 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
}
|
}
|
||||||
}, []);
|
}, []);
|
||||||
|
|
||||||
useEffect(() => {
|
|
||||||
void loadUsers(true);
|
|
||||||
}, [loadUsers]);
|
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
fetchUserWorkspaces()
|
fetchUserWorkspaces()
|
||||||
.then((workspaces) => {
|
.then((workspaces) => {
|
||||||
@@ -139,27 +154,67 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
WorkspaceMemberRole.OWNER,
|
WorkspaceMemberRole.OWNER,
|
||||||
WorkspaceMemberRole.ADMIN,
|
WorkspaceMemberRole.ADMIN,
|
||||||
];
|
];
|
||||||
setIsAdmin(workspaces.some((ws) => adminRoles.includes(ws.role)));
|
|
||||||
|
setDefaultWorkspaceId(workspaces[0]?.id ?? null);
|
||||||
|
setIsAdmin(workspaces.some((workspace) => adminRoles.includes(workspace.role)));
|
||||||
})
|
})
|
||||||
.catch(() => {
|
.catch(() => {
|
||||||
|
setDefaultWorkspaceId(null);
|
||||||
setIsAdmin(true); // fail open
|
setIsAdmin(true); // fail open
|
||||||
});
|
});
|
||||||
}, []);
|
}, []);
|
||||||
|
|
||||||
|
useEffect(() => {
|
||||||
|
if (isAdmin !== true) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
void loadUsers(true);
|
||||||
|
}, [isAdmin, loadUsers]);
|
||||||
|
|
||||||
function resetInviteForm(): void {
|
function resetInviteForm(): void {
|
||||||
setInviteForm(INITIAL_INVITE_FORM);
|
setInviteForm(INITIAL_INVITE_FORM);
|
||||||
setInviteError(null);
|
setInviteError(null);
|
||||||
}
|
}
|
||||||
|
|
||||||
function handleInviteOpenChange(open: boolean): void {
|
function openUserDetails(user: AdminUser): void {
|
||||||
if (!open && !isInviting) {
|
const primaryMembership = getPrimaryMembership(user);
|
||||||
resetInviteForm();
|
|
||||||
}
|
const nextDetailForm = {
|
||||||
setIsInviteOpen(open);
|
name: user.name,
|
||||||
|
email: user.email,
|
||||||
|
role: primaryMembership?.role ?? WorkspaceMemberRole.MEMBER,
|
||||||
|
workspaceId: primaryMembership?.workspaceId ?? null,
|
||||||
|
workspaceName: primaryMembership?.workspaceName ?? null,
|
||||||
|
};
|
||||||
|
|
||||||
|
setDetailTarget(user);
|
||||||
|
setDetailForm(nextDetailForm);
|
||||||
|
setDetailInitial({
|
||||||
|
name: nextDetailForm.name,
|
||||||
|
email: nextDetailForm.email,
|
||||||
|
role: nextDetailForm.role,
|
||||||
|
workspaceId: nextDetailForm.workspaceId,
|
||||||
|
});
|
||||||
|
setDetailError(null);
|
||||||
}
|
}
|
||||||
|
|
||||||
async function handleInviteSubmit(e: SyntheticEvent): Promise<void> {
|
function resetUserDetails(): void {
|
||||||
e.preventDefault();
|
setDetailTarget(null);
|
||||||
|
setDetailForm(INITIAL_DETAIL_FORM);
|
||||||
|
setDetailInitial(null);
|
||||||
|
setDetailError(null);
|
||||||
|
}
|
||||||
|
|
||||||
|
function handleUserRowKeyDown(event: KeyboardEvent<HTMLDivElement>, user: AdminUser): void {
|
||||||
|
if (event.key === "Enter" || event.key === " ") {
|
||||||
|
event.preventDefault();
|
||||||
|
openUserDetails(user);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
async function handleInviteSubmit(event: SyntheticEvent): Promise<void> {
|
||||||
|
event.preventDefault();
|
||||||
setInviteError(null);
|
setInviteError(null);
|
||||||
|
|
||||||
const email = inviteForm.email.trim();
|
const email = inviteForm.email.trim();
|
||||||
@@ -168,17 +223,18 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
const dto: InviteUserDto = { email };
|
if (!isValidEmail(email)) {
|
||||||
|
setInviteError("Please enter a valid email address.");
|
||||||
const name = inviteForm.name.trim();
|
return;
|
||||||
if (name) {
|
|
||||||
dto.name = name;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
const workspaceId = inviteForm.workspaceId.trim();
|
const dto: InviteUserDto = {
|
||||||
if (workspaceId) {
|
email,
|
||||||
dto.workspaceId = workspaceId;
|
role: inviteForm.role,
|
||||||
dto.role = inviteForm.role;
|
};
|
||||||
|
|
||||||
|
if (defaultWorkspaceId) {
|
||||||
|
dto.workspaceId = defaultWorkspaceId;
|
||||||
}
|
}
|
||||||
|
|
||||||
try {
|
try {
|
||||||
@@ -194,6 +250,75 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
async function handleDetailSubmit(event: SyntheticEvent): Promise<void> {
|
||||||
|
event.preventDefault();
|
||||||
|
|
||||||
|
if (detailTarget === null || detailInitial === null) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const name = detailForm.name.trim();
|
||||||
|
const email = detailForm.email.trim();
|
||||||
|
|
||||||
|
if (!name) {
|
||||||
|
setDetailError("Name is required.");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!email) {
|
||||||
|
setDetailError("Email is required.");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!isValidEmail(email)) {
|
||||||
|
setDetailError("Please enter a valid email address.");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const didUpdateUser = name !== detailInitial.name || email !== detailInitial.email;
|
||||||
|
const didUpdateRole =
|
||||||
|
detailForm.workspaceId !== null &&
|
||||||
|
detailForm.workspaceId === detailInitial.workspaceId &&
|
||||||
|
detailForm.role !== detailInitial.role;
|
||||||
|
|
||||||
|
if (!didUpdateUser && !didUpdateRole) {
|
||||||
|
resetUserDetails();
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
try {
|
||||||
|
setIsSavingDetails(true);
|
||||||
|
setDetailError(null);
|
||||||
|
|
||||||
|
if (didUpdateUser) {
|
||||||
|
const dto: UpdateUserDto = {};
|
||||||
|
|
||||||
|
if (name !== detailInitial.name) {
|
||||||
|
dto.name = name;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (email !== detailInitial.email) {
|
||||||
|
dto.email = email;
|
||||||
|
}
|
||||||
|
|
||||||
|
await updateUser(detailTarget.id, dto);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (didUpdateRole && detailForm.workspaceId !== null) {
|
||||||
|
await updateWorkspaceMemberRole(detailForm.workspaceId, detailTarget.id, {
|
||||||
|
role: detailForm.role,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
resetUserDetails();
|
||||||
|
await loadUsers(false);
|
||||||
|
} catch (err: unknown) {
|
||||||
|
setDetailError(err instanceof Error ? err.message : "Failed to update user");
|
||||||
|
} finally {
|
||||||
|
setIsSavingDetails(false);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
async function confirmDeactivate(): Promise<void> {
|
async function confirmDeactivate(): Promise<void> {
|
||||||
if (!deactivateTarget) {
|
if (!deactivateTarget) {
|
||||||
return;
|
return;
|
||||||
@@ -212,32 +337,18 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
async function handleEditSubmit(): Promise<void> {
|
if (isAdmin === null) {
|
||||||
if (editTarget === null) return;
|
return (
|
||||||
setIsEditing(true);
|
<Card className="max-w-2xl mx-auto mt-8">
|
||||||
setEditError(null);
|
<CardContent className="py-12 text-center text-muted-foreground">
|
||||||
try {
|
Checking permissions...
|
||||||
const dto: UpdateUserDto = {};
|
</CardContent>
|
||||||
if (editName.trim()) dto.name = editName.trim();
|
</Card>
|
||||||
await updateUser(editTarget.id, dto);
|
);
|
||||||
setEditTarget(null);
|
|
||||||
await loadUsers(false);
|
|
||||||
} catch (err: unknown) {
|
|
||||||
setEditError(err instanceof Error ? err.message : "Failed to update user");
|
|
||||||
} finally {
|
|
||||||
setIsEditing(false);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (isAdmin === false) {
|
if (!isAdmin) {
|
||||||
return (
|
return <SettingsAccessDenied message="You need Admin or Owner role to manage users." />;
|
||||||
<div className="p-8 max-w-2xl">
|
|
||||||
<div className="rounded-lg border border-red-200 bg-red-50 p-6 text-center">
|
|
||||||
<p className="text-lg font-semibold text-red-700">Access Denied</p>
|
|
||||||
<p className="mt-2 text-sm text-red-600">You need Admin or Owner role to manage users.</p>
|
|
||||||
</div>
|
|
||||||
</div>
|
|
||||||
);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return (
|
return (
|
||||||
@@ -262,7 +373,15 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
{isRefreshing ? "Refreshing..." : "Refresh"}
|
{isRefreshing ? "Refreshing..." : "Refresh"}
|
||||||
</Button>
|
</Button>
|
||||||
|
|
||||||
<Dialog open={isInviteOpen} onOpenChange={handleInviteOpenChange}>
|
<Dialog
|
||||||
|
open={isInviteOpen}
|
||||||
|
onOpenChange={(open) => {
|
||||||
|
if (!open && !isInviting) {
|
||||||
|
resetInviteForm();
|
||||||
|
}
|
||||||
|
setIsInviteOpen(open);
|
||||||
|
}}
|
||||||
|
>
|
||||||
<DialogTrigger asChild>
|
<DialogTrigger asChild>
|
||||||
<Button>
|
<Button>
|
||||||
<UserPlus className="h-4 w-4 mr-2" />
|
<UserPlus className="h-4 w-4 mr-2" />
|
||||||
@@ -273,13 +392,13 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
<DialogHeader>
|
<DialogHeader>
|
||||||
<DialogTitle>Invite User</DialogTitle>
|
<DialogTitle>Invite User</DialogTitle>
|
||||||
<DialogDescription>
|
<DialogDescription>
|
||||||
Create an invited account and optionally assign workspace access.
|
Invite a new user and assign their role for your default workspace.
|
||||||
</DialogDescription>
|
</DialogDescription>
|
||||||
</DialogHeader>
|
</DialogHeader>
|
||||||
|
|
||||||
<form
|
<form
|
||||||
onSubmit={(e) => {
|
onSubmit={(event) => {
|
||||||
void handleInviteSubmit(e);
|
void handleInviteSubmit(event);
|
||||||
}}
|
}}
|
||||||
className="space-y-4"
|
className="space-y-4"
|
||||||
>
|
>
|
||||||
@@ -289,8 +408,8 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
id="invite-email"
|
id="invite-email"
|
||||||
type="email"
|
type="email"
|
||||||
value={inviteForm.email}
|
value={inviteForm.email}
|
||||||
onChange={(e: ChangeEvent<HTMLInputElement>) => {
|
onChange={(event: ChangeEvent<HTMLInputElement>) => {
|
||||||
setInviteForm((prev) => ({ ...prev, email: e.target.value }));
|
setInviteForm((prev) => ({ ...prev, email: event.target.value }));
|
||||||
}}
|
}}
|
||||||
placeholder="user@example.com"
|
placeholder="user@example.com"
|
||||||
maxLength={255}
|
maxLength={255}
|
||||||
@@ -298,33 +417,6 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
/>
|
/>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div className="space-y-2">
|
|
||||||
<Label htmlFor="invite-name">Name (optional)</Label>
|
|
||||||
<Input
|
|
||||||
id="invite-name"
|
|
||||||
type="text"
|
|
||||||
value={inviteForm.name}
|
|
||||||
onChange={(e: ChangeEvent<HTMLInputElement>) => {
|
|
||||||
setInviteForm((prev) => ({ ...prev, name: e.target.value }));
|
|
||||||
}}
|
|
||||||
placeholder="Jane Doe"
|
|
||||||
maxLength={255}
|
|
||||||
/>
|
|
||||||
</div>
|
|
||||||
|
|
||||||
<div className="space-y-2">
|
|
||||||
<Label htmlFor="invite-workspace-id">Workspace ID (optional)</Label>
|
|
||||||
<Input
|
|
||||||
id="invite-workspace-id"
|
|
||||||
type="text"
|
|
||||||
value={inviteForm.workspaceId}
|
|
||||||
onChange={(e: ChangeEvent<HTMLInputElement>) => {
|
|
||||||
setInviteForm((prev) => ({ ...prev, workspaceId: e.target.value }));
|
|
||||||
}}
|
|
||||||
placeholder="UUID workspace id"
|
|
||||||
/>
|
|
||||||
</div>
|
|
||||||
|
|
||||||
<div className="space-y-2">
|
<div className="space-y-2">
|
||||||
<Label htmlFor="invite-role">Role</Label>
|
<Label htmlFor="invite-role">Role</Label>
|
||||||
<Select
|
<Select
|
||||||
@@ -344,9 +436,13 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
))}
|
))}
|
||||||
</SelectContent>
|
</SelectContent>
|
||||||
</Select>
|
</Select>
|
||||||
<p className="text-xs text-muted-foreground">
|
{defaultWorkspaceId ? (
|
||||||
Role is only applied when workspace ID is provided.
|
<p className="text-xs text-muted-foreground">Role will be applied on invite.</p>
|
||||||
</p>
|
) : (
|
||||||
|
<p className="text-xs text-muted-foreground">
|
||||||
|
No default workspace found. User will be invited without workspace assignment.
|
||||||
|
</p>
|
||||||
|
)}
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
{inviteError ? (
|
{inviteError ? (
|
||||||
@@ -360,7 +456,10 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
type="button"
|
type="button"
|
||||||
variant="outline"
|
variant="outline"
|
||||||
onClick={() => {
|
onClick={() => {
|
||||||
handleInviteOpenChange(false);
|
if (!isInviting) {
|
||||||
|
setIsInviteOpen(false);
|
||||||
|
resetInviteForm();
|
||||||
|
}
|
||||||
}}
|
}}
|
||||||
disabled={isInviting}
|
disabled={isInviting}
|
||||||
>
|
>
|
||||||
@@ -409,17 +508,25 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
<Card>
|
<Card>
|
||||||
<CardHeader>
|
<CardHeader>
|
||||||
<CardTitle>User Directory</CardTitle>
|
<CardTitle>User Directory</CardTitle>
|
||||||
<CardDescription>Name, email, role, and account status.</CardDescription>
|
<CardDescription>Click a user to view details or edit profile fields.</CardDescription>
|
||||||
</CardHeader>
|
</CardHeader>
|
||||||
<CardContent className="space-y-3">
|
<CardContent className="space-y-3">
|
||||||
{users.map((user) => {
|
{users.map((user) => {
|
||||||
const primaryRole = getPrimaryRole(user);
|
const primaryMembership = getPrimaryMembership(user);
|
||||||
const isActive = user.deactivatedAt === null;
|
const isActive = user.deactivatedAt === null;
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div
|
<div
|
||||||
key={user.id}
|
key={user.id}
|
||||||
className="rounded-md border p-4 flex flex-col gap-3 md:flex-row md:items-center md:justify-between"
|
className="rounded-md border p-4 flex flex-col gap-3 md:flex-row md:items-center md:justify-between cursor-pointer hover:bg-muted/30"
|
||||||
|
role="button"
|
||||||
|
tabIndex={0}
|
||||||
|
onClick={() => {
|
||||||
|
openUserDetails(user);
|
||||||
|
}}
|
||||||
|
onKeyDown={(event) => {
|
||||||
|
handleUserRowKeyDown(event, user);
|
||||||
|
}}
|
||||||
>
|
>
|
||||||
<div className="space-y-1 min-w-0">
|
<div className="space-y-1 min-w-0">
|
||||||
<p className="font-semibold truncate">{user.name || "Unnamed User"}</p>
|
<p className="font-semibold truncate">{user.name || "Unnamed User"}</p>
|
||||||
@@ -428,28 +535,17 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
|
|
||||||
<div className="flex items-center gap-2 flex-wrap md:justify-end">
|
<div className="flex items-center gap-2 flex-wrap md:justify-end">
|
||||||
<Badge variant="outline">
|
<Badge variant="outline">
|
||||||
{primaryRole ? toRoleLabel(primaryRole) : "No role"}
|
{primaryMembership ? toRoleLabel(primaryMembership.role) : "No role"}
|
||||||
</Badge>
|
</Badge>
|
||||||
<Badge variant={isActive ? "secondary" : "destructive"}>
|
<Badge variant={isActive ? "secondary" : "destructive"}>
|
||||||
{isActive ? "Active" : "Inactive"}
|
{isActive ? "Active" : "Inactive"}
|
||||||
</Badge>
|
</Badge>
|
||||||
<Button
|
|
||||||
variant="outline"
|
|
||||||
size="sm"
|
|
||||||
onClick={() => {
|
|
||||||
setEditTarget(user);
|
|
||||||
setEditName(user.name);
|
|
||||||
setEditError(null);
|
|
||||||
}}
|
|
||||||
>
|
|
||||||
<Pencil className="h-4 w-4 mr-2" />
|
|
||||||
Edit Role
|
|
||||||
</Button>
|
|
||||||
{isActive ? (
|
{isActive ? (
|
||||||
<Button
|
<Button
|
||||||
variant="destructive"
|
variant="destructive"
|
||||||
size="sm"
|
size="sm"
|
||||||
onClick={() => {
|
onClick={(event) => {
|
||||||
|
event.stopPropagation();
|
||||||
setDeactivateTarget(user);
|
setDeactivateTarget(user);
|
||||||
}}
|
}}
|
||||||
>
|
>
|
||||||
@@ -465,6 +561,117 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
</Card>
|
</Card>
|
||||||
)}
|
)}
|
||||||
|
|
||||||
|
<Dialog
|
||||||
|
open={detailTarget !== null}
|
||||||
|
onOpenChange={(open) => {
|
||||||
|
if (!open && !isSavingDetails) {
|
||||||
|
resetUserDetails();
|
||||||
|
}
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
<DialogContent>
|
||||||
|
<DialogHeader>
|
||||||
|
<DialogTitle>User Details</DialogTitle>
|
||||||
|
<DialogDescription>
|
||||||
|
Edit profile details for {detailTarget?.email ?? "selected user"}.
|
||||||
|
</DialogDescription>
|
||||||
|
</DialogHeader>
|
||||||
|
|
||||||
|
<form
|
||||||
|
onSubmit={(event) => {
|
||||||
|
void handleDetailSubmit(event);
|
||||||
|
}}
|
||||||
|
className="space-y-4"
|
||||||
|
>
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="detail-name">Name</Label>
|
||||||
|
<Input
|
||||||
|
id="detail-name"
|
||||||
|
value={detailForm.name}
|
||||||
|
onChange={(event: ChangeEvent<HTMLInputElement>) => {
|
||||||
|
setDetailForm((prev) => ({ ...prev, name: event.target.value }));
|
||||||
|
}}
|
||||||
|
placeholder="Full name"
|
||||||
|
maxLength={255}
|
||||||
|
disabled={isSavingDetails}
|
||||||
|
required
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="detail-email">Email</Label>
|
||||||
|
<Input
|
||||||
|
id="detail-email"
|
||||||
|
type="email"
|
||||||
|
value={detailForm.email}
|
||||||
|
onChange={(event: ChangeEvent<HTMLInputElement>) => {
|
||||||
|
setDetailForm((prev) => ({ ...prev, email: event.target.value }));
|
||||||
|
}}
|
||||||
|
placeholder="user@example.com"
|
||||||
|
maxLength={255}
|
||||||
|
disabled={isSavingDetails}
|
||||||
|
required
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="detail-role">Role</Label>
|
||||||
|
<Select
|
||||||
|
value={detailForm.role}
|
||||||
|
disabled={detailForm.workspaceId === null || isSavingDetails}
|
||||||
|
onValueChange={(value) => {
|
||||||
|
setDetailForm((prev) => ({ ...prev, role: value as WorkspaceMemberRole }));
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
<SelectTrigger id="detail-role">
|
||||||
|
<SelectValue placeholder="Select role" />
|
||||||
|
</SelectTrigger>
|
||||||
|
<SelectContent>
|
||||||
|
{Object.values(WorkspaceMemberRole).map((role) => (
|
||||||
|
<SelectItem key={role} value={role}>
|
||||||
|
{toRoleLabel(role)}
|
||||||
|
</SelectItem>
|
||||||
|
))}
|
||||||
|
</SelectContent>
|
||||||
|
</Select>
|
||||||
|
{detailForm.workspaceName ? (
|
||||||
|
<p className="text-xs text-muted-foreground">
|
||||||
|
Role updates apply to: {detailForm.workspaceName}
|
||||||
|
</p>
|
||||||
|
) : (
|
||||||
|
<p className="text-xs text-muted-foreground">
|
||||||
|
This user has no workspace membership. Role cannot be updated.
|
||||||
|
</p>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{detailError !== null ? (
|
||||||
|
<p className="text-sm text-destructive" role="alert">
|
||||||
|
{detailError}
|
||||||
|
</p>
|
||||||
|
) : null}
|
||||||
|
|
||||||
|
<DialogFooter>
|
||||||
|
<Button
|
||||||
|
type="button"
|
||||||
|
variant="outline"
|
||||||
|
onClick={() => {
|
||||||
|
if (!isSavingDetails) {
|
||||||
|
resetUserDetails();
|
||||||
|
}
|
||||||
|
}}
|
||||||
|
disabled={isSavingDetails}
|
||||||
|
>
|
||||||
|
Cancel
|
||||||
|
</Button>
|
||||||
|
<Button type="submit" disabled={isSavingDetails}>
|
||||||
|
{isSavingDetails ? "Saving..." : "Save Changes"}
|
||||||
|
</Button>
|
||||||
|
</DialogFooter>
|
||||||
|
</form>
|
||||||
|
</DialogContent>
|
||||||
|
</Dialog>
|
||||||
|
|
||||||
<AlertDialog
|
<AlertDialog
|
||||||
open={deactivateTarget !== null}
|
open={deactivateTarget !== null}
|
||||||
onOpenChange={(open) => {
|
onOpenChange={(open) => {
|
||||||
@@ -496,55 +703,4 @@ export default function UsersSettingsPage(): ReactElement {
|
|||||||
</AlertDialog>
|
</AlertDialog>
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
|
|
||||||
<Dialog
|
|
||||||
open={editTarget !== null}
|
|
||||||
onOpenChange={(open) => {
|
|
||||||
if (!open && !isEditing) {
|
|
||||||
setEditTarget(null);
|
|
||||||
setEditError(null);
|
|
||||||
}
|
|
||||||
}}
|
|
||||||
>
|
|
||||||
<DialogContent>
|
|
||||||
<DialogHeader>
|
|
||||||
<DialogTitle>Edit User Role</DialogTitle>
|
|
||||||
<DialogDescription>Change role for {editTarget?.email ?? "user"}.</DialogDescription>
|
|
||||||
</DialogHeader>
|
|
||||||
<div className="space-y-4 py-2">
|
|
||||||
{editError !== null ? <p className="text-sm text-destructive">{editError}</p> : null}
|
|
||||||
<div className="space-y-2">
|
|
||||||
<Label htmlFor="edit-name">Display Name</Label>
|
|
||||||
<Input
|
|
||||||
id="edit-name"
|
|
||||||
value={editName}
|
|
||||||
onChange={(e: ChangeEvent<HTMLInputElement>) => {
|
|
||||||
setEditName(e.target.value);
|
|
||||||
}}
|
|
||||||
placeholder="Full name"
|
|
||||||
disabled={isEditing}
|
|
||||||
/>
|
|
||||||
</div>
|
|
||||||
</div>
|
|
||||||
<DialogFooter>
|
|
||||||
<Button
|
|
||||||
variant="outline"
|
|
||||||
onClick={() => {
|
|
||||||
setEditTarget(null);
|
|
||||||
}}
|
|
||||||
disabled={isEditing}
|
|
||||||
>
|
|
||||||
Cancel
|
|
||||||
</Button>
|
|
||||||
<Button
|
|
||||||
onClick={() => {
|
|
||||||
void handleEditSubmit();
|
|
||||||
}}
|
|
||||||
disabled={isEditing}
|
|
||||||
>
|
|
||||||
{isEditing ? "Saving..." : "Save"}
|
|
||||||
</Button>
|
|
||||||
</DialogFooter>
|
|
||||||
</DialogContent>
|
|
||||||
</Dialog>;
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,12 +1,12 @@
|
|||||||
import type { UserWorkspace } from "@/lib/api/workspaces";
|
import type { UserWorkspace, WorkspaceMemberEntry } from "@/lib/api/workspaces";
|
||||||
import type { ReactElement, ReactNode } from "react";
|
import type { ReactElement, ReactNode } from "react";
|
||||||
|
|
||||||
import { WorkspaceMemberRole } from "@mosaic/shared";
|
import { WorkspaceMemberRole } from "@mosaic/shared";
|
||||||
import { render, screen, waitFor } from "@testing-library/react";
|
import { render, screen, waitFor } from "@testing-library/react";
|
||||||
import userEvent from "@testing-library/user-event";
|
import userEvent from "@testing-library/user-event";
|
||||||
import { beforeEach, describe, expect, it, vi } from "vitest";
|
import { beforeEach, describe, expect, it, vi } from "vitest";
|
||||||
import { createWorkspace, fetchUserWorkspaces } from "@/lib/api/workspaces";
|
|
||||||
|
|
||||||
|
import { createWorkspace, fetchUserWorkspaces, fetchWorkspaceMembers } from "@/lib/api/workspaces";
|
||||||
import WorkspacesPage from "./page";
|
import WorkspacesPage from "./page";
|
||||||
|
|
||||||
vi.mock("next/link", () => ({
|
vi.mock("next/link", () => ({
|
||||||
@@ -21,33 +21,23 @@ vi.mock("next/link", () => ({
|
|||||||
},
|
},
|
||||||
}));
|
}));
|
||||||
|
|
||||||
vi.mock("@/components/workspace/WorkspaceCard", () => ({
|
|
||||||
WorkspaceCard: function WorkspaceCardMock({
|
|
||||||
workspace,
|
|
||||||
userRole,
|
|
||||||
memberCount,
|
|
||||||
}: {
|
|
||||||
workspace: { name: string };
|
|
||||||
userRole: WorkspaceMemberRole;
|
|
||||||
memberCount: number;
|
|
||||||
}): ReactElement {
|
|
||||||
return (
|
|
||||||
<div data-testid="workspace-card">
|
|
||||||
{workspace.name} | {userRole} | {String(memberCount)}
|
|
||||||
</div>
|
|
||||||
);
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
vi.mock("@/lib/api/workspaces", () => ({
|
vi.mock("@/lib/api/workspaces", () => ({
|
||||||
fetchUserWorkspaces: vi.fn(),
|
fetchUserWorkspaces: vi.fn(),
|
||||||
createWorkspace: vi.fn(),
|
createWorkspace: vi.fn(),
|
||||||
|
fetchWorkspaceMembers: vi.fn(),
|
||||||
|
addWorkspaceMember: vi.fn(),
|
||||||
|
removeWorkspaceMember: vi.fn(),
|
||||||
|
}));
|
||||||
|
|
||||||
|
vi.mock("@/lib/api/admin", () => ({
|
||||||
|
fetchAdminUsers: vi.fn(),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const fetchUserWorkspacesMock = vi.mocked(fetchUserWorkspaces);
|
const fetchUserWorkspacesMock = vi.mocked(fetchUserWorkspaces);
|
||||||
const createWorkspaceMock = vi.mocked(createWorkspace);
|
const createWorkspaceMock = vi.mocked(createWorkspace);
|
||||||
|
const fetchWorkspaceMembersMock = vi.mocked(fetchWorkspaceMembers);
|
||||||
|
|
||||||
const baseWorkspace: UserWorkspace = {
|
const workspaceA: UserWorkspace = {
|
||||||
id: "workspace-1",
|
id: "workspace-1",
|
||||||
name: "Personal Workspace",
|
name: "Personal Workspace",
|
||||||
ownerId: "owner-1",
|
ownerId: "owner-1",
|
||||||
@@ -55,45 +45,93 @@ const baseWorkspace: UserWorkspace = {
|
|||||||
createdAt: "2026-01-01T00:00:00.000Z",
|
createdAt: "2026-01-01T00:00:00.000Z",
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const workspaceB: UserWorkspace = {
|
||||||
|
id: "workspace-2",
|
||||||
|
name: "Client Workspace",
|
||||||
|
ownerId: "owner-2",
|
||||||
|
role: WorkspaceMemberRole.ADMIN,
|
||||||
|
createdAt: "2026-01-02T00:00:00.000Z",
|
||||||
|
};
|
||||||
|
|
||||||
|
const membersA: WorkspaceMemberEntry[] = [
|
||||||
|
{
|
||||||
|
workspaceId: "workspace-1",
|
||||||
|
userId: "user-a",
|
||||||
|
role: WorkspaceMemberRole.OWNER,
|
||||||
|
joinedAt: "2026-01-03T00:00:00.000Z",
|
||||||
|
user: {
|
||||||
|
id: "user-a",
|
||||||
|
email: "alice@example.com",
|
||||||
|
name: "Alice",
|
||||||
|
image: null,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
];
|
||||||
|
|
||||||
|
const membersB: WorkspaceMemberEntry[] = [
|
||||||
|
{
|
||||||
|
workspaceId: "workspace-2",
|
||||||
|
userId: "user-b",
|
||||||
|
role: WorkspaceMemberRole.MEMBER,
|
||||||
|
joinedAt: "2026-01-04T00:00:00.000Z",
|
||||||
|
user: {
|
||||||
|
id: "user-b",
|
||||||
|
email: "bob@example.com",
|
||||||
|
name: "Bob",
|
||||||
|
image: null,
|
||||||
|
},
|
||||||
|
},
|
||||||
|
];
|
||||||
|
|
||||||
describe("WorkspacesPage", () => {
|
describe("WorkspacesPage", () => {
|
||||||
beforeEach(() => {
|
beforeEach(() => {
|
||||||
vi.clearAllMocks();
|
vi.clearAllMocks();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("loads and renders user workspaces from the API", async () => {
|
it("loads workspaces and fetches members for the first workspace", async () => {
|
||||||
fetchUserWorkspacesMock.mockResolvedValue([baseWorkspace]);
|
fetchUserWorkspacesMock.mockResolvedValue([workspaceA, workspaceB]);
|
||||||
|
fetchWorkspaceMembersMock.mockResolvedValue(membersA);
|
||||||
|
|
||||||
render(<WorkspacesPage />);
|
render(<WorkspacesPage />);
|
||||||
|
|
||||||
expect(screen.getByText("Loading workspaces...")).toBeInTheDocument();
|
expect(await screen.findByText("Your Workspaces (2)")).toBeInTheDocument();
|
||||||
|
expect(await screen.findByText("Personal Workspace Members")).toBeInTheDocument();
|
||||||
|
|
||||||
expect(await screen.findByText("Your Workspaces (1)")).toBeInTheDocument();
|
await waitFor(() => {
|
||||||
expect(screen.getByTestId("workspace-card")).toHaveTextContent("Personal Workspace");
|
expect(fetchWorkspaceMembersMock).toHaveBeenCalledWith("workspace-1");
|
||||||
expect(fetchUserWorkspacesMock).toHaveBeenCalledTimes(1);
|
});
|
||||||
|
|
||||||
|
expect(screen.getByText("alice@example.com")).toBeInTheDocument();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("shows fetch errors in the UI", async () => {
|
it("switches selected workspace and reloads member list", async () => {
|
||||||
fetchUserWorkspacesMock.mockRejectedValue(new Error("Unable to load workspaces"));
|
fetchUserWorkspacesMock.mockResolvedValue([workspaceA, workspaceB]);
|
||||||
|
fetchWorkspaceMembersMock.mockResolvedValueOnce(membersA).mockResolvedValueOnce(membersB);
|
||||||
|
|
||||||
|
const user = userEvent.setup();
|
||||||
render(<WorkspacesPage />);
|
render(<WorkspacesPage />);
|
||||||
|
|
||||||
expect(await screen.findByText("Unable to load workspaces")).toBeInTheDocument();
|
expect(await screen.findByText("Personal Workspace Members")).toBeInTheDocument();
|
||||||
|
|
||||||
|
await user.click(screen.getByRole("button", { name: /client workspace/i }));
|
||||||
|
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(fetchWorkspaceMembersMock).toHaveBeenLastCalledWith("workspace-2");
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(await screen.findByText("Client Workspace Members")).toBeInTheDocument();
|
||||||
|
expect(screen.getByText("bob@example.com")).toBeInTheDocument();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("creates a workspace and refreshes the list", async () => {
|
it("creates a workspace and refreshes the list", async () => {
|
||||||
fetchUserWorkspacesMock.mockResolvedValueOnce([baseWorkspace]).mockResolvedValueOnce([
|
fetchUserWorkspacesMock
|
||||||
baseWorkspace,
|
.mockResolvedValueOnce([workspaceA])
|
||||||
{
|
.mockResolvedValueOnce([workspaceA, workspaceB]);
|
||||||
...baseWorkspace,
|
fetchWorkspaceMembersMock.mockResolvedValue(membersA);
|
||||||
id: "workspace-2",
|
|
||||||
name: "New Workspace",
|
|
||||||
role: WorkspaceMemberRole.MEMBER,
|
|
||||||
},
|
|
||||||
]);
|
|
||||||
createWorkspaceMock.mockResolvedValue({
|
createWorkspaceMock.mockResolvedValue({
|
||||||
id: "workspace-2",
|
id: "workspace-2",
|
||||||
name: "New Workspace",
|
name: "Client Workspace",
|
||||||
ownerId: "owner-1",
|
ownerId: "owner-2",
|
||||||
settings: {},
|
settings: {},
|
||||||
createdAt: "2026-01-02T00:00:00.000Z",
|
createdAt: "2026-01-02T00:00:00.000Z",
|
||||||
updatedAt: "2026-01-02T00:00:00.000Z",
|
updatedAt: "2026-01-02T00:00:00.000Z",
|
||||||
@@ -105,31 +143,17 @@ describe("WorkspacesPage", () => {
|
|||||||
|
|
||||||
expect(await screen.findByText("Your Workspaces (1)")).toBeInTheDocument();
|
expect(await screen.findByText("Your Workspaces (1)")).toBeInTheDocument();
|
||||||
|
|
||||||
await user.type(screen.getByPlaceholderText("Enter workspace name..."), "New Workspace");
|
await user.type(screen.getByPlaceholderText("Enter workspace name..."), "Client Workspace");
|
||||||
await user.click(screen.getByRole("button", { name: "Create Workspace" }));
|
await user.click(screen.getByRole("button", { name: "Create Workspace" }));
|
||||||
|
|
||||||
await waitFor(() => {
|
await waitFor(() => {
|
||||||
expect(createWorkspaceMock).toHaveBeenCalledWith({ name: "New Workspace" });
|
expect(createWorkspaceMock).toHaveBeenCalledWith({ name: "Client Workspace" });
|
||||||
});
|
});
|
||||||
|
|
||||||
await waitFor(() => {
|
await waitFor(() => {
|
||||||
expect(fetchUserWorkspacesMock).toHaveBeenCalledTimes(2);
|
expect(fetchUserWorkspacesMock).toHaveBeenCalledTimes(2);
|
||||||
});
|
});
|
||||||
|
|
||||||
expect(await screen.findByText("Your Workspaces (2)")).toBeInTheDocument();
|
expect(await screen.findByText("Your Workspaces (2)")).toBeInTheDocument();
|
||||||
});
|
});
|
||||||
|
|
||||||
it("shows create errors in the UI", async () => {
|
|
||||||
fetchUserWorkspacesMock.mockResolvedValue([baseWorkspace]);
|
|
||||||
createWorkspaceMock.mockRejectedValue(new Error("Workspace creation failed"));
|
|
||||||
|
|
||||||
const user = userEvent.setup();
|
|
||||||
render(<WorkspacesPage />);
|
|
||||||
|
|
||||||
expect(await screen.findByText("Your Workspaces (1)")).toBeInTheDocument();
|
|
||||||
|
|
||||||
await user.type(screen.getByPlaceholderText("Enter workspace name..."), "Bad Workspace");
|
|
||||||
await user.click(screen.getByRole("button", { name: "Create Workspace" }));
|
|
||||||
|
|
||||||
expect(await screen.findByText("Workspace creation failed")).toBeInTheDocument();
|
|
||||||
});
|
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -2,10 +2,51 @@
|
|||||||
|
|
||||||
import type { ReactElement, SyntheticEvent } from "react";
|
import type { ReactElement, SyntheticEvent } from "react";
|
||||||
|
|
||||||
import { useCallback, useEffect, useState } from "react";
|
import { useCallback, useEffect, useMemo, useState } from "react";
|
||||||
import { WorkspaceCard } from "@/components/workspace/WorkspaceCard";
|
|
||||||
import { createWorkspace, fetchUserWorkspaces, type UserWorkspace } from "@/lib/api/workspaces";
|
|
||||||
import Link from "next/link";
|
import Link from "next/link";
|
||||||
|
import { UserPlus, UserX } from "lucide-react";
|
||||||
|
import { WorkspaceMemberRole } from "@mosaic/shared";
|
||||||
|
import { Badge } from "@/components/ui/badge";
|
||||||
|
import { Button } from "@/components/ui/button";
|
||||||
|
import { Card, CardContent, CardDescription, CardHeader, CardTitle } from "@/components/ui/card";
|
||||||
|
import {
|
||||||
|
Dialog,
|
||||||
|
DialogContent,
|
||||||
|
DialogDescription,
|
||||||
|
DialogFooter,
|
||||||
|
DialogHeader,
|
||||||
|
DialogTitle,
|
||||||
|
DialogTrigger,
|
||||||
|
} from "@/components/ui/dialog";
|
||||||
|
import {
|
||||||
|
AlertDialog,
|
||||||
|
AlertDialogAction,
|
||||||
|
AlertDialogCancel,
|
||||||
|
AlertDialogContent,
|
||||||
|
AlertDialogDescription,
|
||||||
|
AlertDialogFooter,
|
||||||
|
AlertDialogHeader,
|
||||||
|
AlertDialogTitle,
|
||||||
|
} from "@/components/ui/alert-dialog";
|
||||||
|
import { Input } from "@/components/ui/input";
|
||||||
|
import { Label } from "@/components/ui/label";
|
||||||
|
import {
|
||||||
|
Select,
|
||||||
|
SelectContent,
|
||||||
|
SelectItem,
|
||||||
|
SelectTrigger,
|
||||||
|
SelectValue,
|
||||||
|
} from "@/components/ui/select";
|
||||||
|
import {
|
||||||
|
addWorkspaceMember,
|
||||||
|
createWorkspace,
|
||||||
|
fetchUserWorkspaces,
|
||||||
|
fetchWorkspaceMembers,
|
||||||
|
removeWorkspaceMember,
|
||||||
|
type UserWorkspace,
|
||||||
|
type WorkspaceMemberEntry,
|
||||||
|
} from "@/lib/api/workspaces";
|
||||||
|
import { fetchAdminUsers, type AdminUser } from "@/lib/api/admin";
|
||||||
|
|
||||||
function getErrorMessage(error: unknown, fallback: string): string {
|
function getErrorMessage(error: unknown, fallback: string): string {
|
||||||
if (error instanceof Error) {
|
if (error instanceof Error) {
|
||||||
@@ -15,18 +56,53 @@ function getErrorMessage(error: unknown, fallback: string): string {
|
|||||||
return fallback;
|
return fallback;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
function toRoleLabel(role: WorkspaceMemberRole): string {
|
||||||
* Workspaces Page
|
return `${role.charAt(0)}${role.slice(1).toLowerCase()}`;
|
||||||
* Fetches and creates workspaces through the real API.
|
}
|
||||||
*/
|
|
||||||
|
interface RemoveMemberTarget {
|
||||||
|
userId: string;
|
||||||
|
email: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
const ROLE_BADGE_CLASS: Record<WorkspaceMemberRole, string> = {
|
||||||
|
[WorkspaceMemberRole.OWNER]: "border-purple-200 bg-purple-50 text-purple-700",
|
||||||
|
[WorkspaceMemberRole.ADMIN]: "border-blue-200 bg-blue-50 text-blue-700",
|
||||||
|
[WorkspaceMemberRole.MEMBER]: "border-green-200 bg-green-50 text-green-700",
|
||||||
|
[WorkspaceMemberRole.GUEST]: "border-gray-200 bg-gray-50 text-gray-700",
|
||||||
|
};
|
||||||
|
|
||||||
export default function WorkspacesPage(): ReactElement {
|
export default function WorkspacesPage(): ReactElement {
|
||||||
const [workspaces, setWorkspaces] = useState<UserWorkspace[]>([]);
|
const [workspaces, setWorkspaces] = useState<UserWorkspace[]>([]);
|
||||||
|
const [selectedWorkspaceId, setSelectedWorkspaceId] = useState<string | null>(null);
|
||||||
|
|
||||||
const [isLoading, setIsLoading] = useState(true);
|
const [isLoading, setIsLoading] = useState(true);
|
||||||
const [loadError, setLoadError] = useState<string | null>(null);
|
const [loadError, setLoadError] = useState<string | null>(null);
|
||||||
|
|
||||||
const [isCreating, setIsCreating] = useState(false);
|
const [isCreating, setIsCreating] = useState(false);
|
||||||
const [newWorkspaceName, setNewWorkspaceName] = useState("");
|
const [newWorkspaceName, setNewWorkspaceName] = useState("");
|
||||||
const [createError, setCreateError] = useState<string | null>(null);
|
const [createError, setCreateError] = useState<string | null>(null);
|
||||||
|
|
||||||
|
const [members, setMembers] = useState<WorkspaceMemberEntry[]>([]);
|
||||||
|
const [isMembersLoading, setIsMembersLoading] = useState(false);
|
||||||
|
const [membersError, setMembersError] = useState<string | null>(null);
|
||||||
|
|
||||||
|
const [isAddMemberOpen, setIsAddMemberOpen] = useState(false);
|
||||||
|
const [isAddingMember, setIsAddingMember] = useState(false);
|
||||||
|
const [addMemberError, setAddMemberError] = useState<string | null>(null);
|
||||||
|
const [memberUserId, setMemberUserId] = useState<string>("");
|
||||||
|
const [memberRole, setMemberRole] = useState<WorkspaceMemberRole>(WorkspaceMemberRole.MEMBER);
|
||||||
|
const [availableUsers, setAvailableUsers] = useState<AdminUser[]>([]);
|
||||||
|
const [isLoadingUsers, setIsLoadingUsers] = useState(false);
|
||||||
|
|
||||||
|
const [removeTarget, setRemoveTarget] = useState<RemoveMemberTarget | null>(null);
|
||||||
|
const [isRemovingMember, setIsRemovingMember] = useState(false);
|
||||||
|
|
||||||
|
const selectedWorkspace = useMemo(
|
||||||
|
() => workspaces.find((workspace) => workspace.id === selectedWorkspaceId) ?? null,
|
||||||
|
[selectedWorkspaceId, workspaces]
|
||||||
|
);
|
||||||
|
|
||||||
const loadWorkspaces = useCallback(async (): Promise<void> => {
|
const loadWorkspaces = useCallback(async (): Promise<void> => {
|
||||||
setIsLoading(true);
|
setIsLoading(true);
|
||||||
|
|
||||||
@@ -34,31 +110,57 @@ export default function WorkspacesPage(): ReactElement {
|
|||||||
const data = await fetchUserWorkspaces();
|
const data = await fetchUserWorkspaces();
|
||||||
setWorkspaces(data);
|
setWorkspaces(data);
|
||||||
setLoadError(null);
|
setLoadError(null);
|
||||||
|
setSelectedWorkspaceId((current) => {
|
||||||
|
if (current && data.some((workspace) => workspace.id === current)) {
|
||||||
|
return current;
|
||||||
|
}
|
||||||
|
|
||||||
|
return data[0]?.id ?? null;
|
||||||
|
});
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
setLoadError(getErrorMessage(error, "Failed to load workspaces"));
|
setLoadError(getErrorMessage(error, "Failed to load workspaces"));
|
||||||
|
setSelectedWorkspaceId(null);
|
||||||
} finally {
|
} finally {
|
||||||
setIsLoading(false);
|
setIsLoading(false);
|
||||||
}
|
}
|
||||||
}, []);
|
}, []);
|
||||||
|
|
||||||
|
const loadMembers = useCallback(async (workspaceId: string): Promise<void> => {
|
||||||
|
setIsMembersLoading(true);
|
||||||
|
setMembersError(null);
|
||||||
|
|
||||||
|
try {
|
||||||
|
const data = await fetchWorkspaceMembers(workspaceId);
|
||||||
|
setMembers(data);
|
||||||
|
} catch (error) {
|
||||||
|
setMembersError(getErrorMessage(error, "Failed to load workspace members"));
|
||||||
|
setMembers([]);
|
||||||
|
} finally {
|
||||||
|
setIsMembersLoading(false);
|
||||||
|
}
|
||||||
|
}, []);
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
void loadWorkspaces();
|
void loadWorkspaces();
|
||||||
}, [loadWorkspaces]);
|
}, [loadWorkspaces]);
|
||||||
|
|
||||||
const workspacesWithRoles = workspaces.map((workspace) => ({
|
useEffect(() => {
|
||||||
...workspace,
|
if (!selectedWorkspaceId) {
|
||||||
settings: {},
|
setMembers([]);
|
||||||
createdAt: new Date(workspace.createdAt),
|
setMembersError(null);
|
||||||
updatedAt: new Date(workspace.createdAt),
|
return;
|
||||||
userRole: workspace.role,
|
}
|
||||||
memberCount: 1,
|
|
||||||
}));
|
|
||||||
|
|
||||||
const handleCreateWorkspace = async (e: SyntheticEvent<HTMLFormElement>): Promise<void> => {
|
void loadMembers(selectedWorkspaceId);
|
||||||
e.preventDefault();
|
}, [loadMembers, selectedWorkspaceId]);
|
||||||
|
|
||||||
|
const handleCreateWorkspace = async (event: SyntheticEvent<HTMLFormElement>): Promise<void> => {
|
||||||
|
event.preventDefault();
|
||||||
|
|
||||||
const workspaceName = newWorkspaceName.trim();
|
const workspaceName = newWorkspaceName.trim();
|
||||||
if (!workspaceName) return;
|
if (!workspaceName) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
setIsCreating(true);
|
setIsCreating(true);
|
||||||
setCreateError(null);
|
setCreateError(null);
|
||||||
@@ -74,91 +176,394 @@ export default function WorkspacesPage(): ReactElement {
|
|||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const eligibleUsers = useMemo(() => {
|
||||||
|
const memberIds = new Set(members.map((member) => member.userId));
|
||||||
|
return availableUsers.filter((user) => !memberIds.has(user.id));
|
||||||
|
}, [availableUsers, members]);
|
||||||
|
|
||||||
|
const loadAvailableUsers = useCallback(async (): Promise<void> => {
|
||||||
|
setIsLoadingUsers(true);
|
||||||
|
|
||||||
|
try {
|
||||||
|
const response = await fetchAdminUsers(1, 200);
|
||||||
|
const activeUsers = response.data.filter((user) => user.deactivatedAt === null);
|
||||||
|
setAvailableUsers(activeUsers);
|
||||||
|
|
||||||
|
if (memberUserId && activeUsers.some((user) => user.id === memberUserId)) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const memberIds = new Set(members.map((member) => member.userId));
|
||||||
|
const firstEligible = activeUsers.find((user) => !memberIds.has(user.id));
|
||||||
|
setMemberUserId(firstEligible?.id ?? "");
|
||||||
|
} catch (error) {
|
||||||
|
setAddMemberError(getErrorMessage(error, "Failed to load users for member assignment"));
|
||||||
|
setAvailableUsers([]);
|
||||||
|
setMemberUserId("");
|
||||||
|
} finally {
|
||||||
|
setIsLoadingUsers(false);
|
||||||
|
}
|
||||||
|
}, [memberUserId, members]);
|
||||||
|
|
||||||
|
const openAddMemberDialog = async (): Promise<void> => {
|
||||||
|
setAddMemberError(null);
|
||||||
|
setMemberRole(WorkspaceMemberRole.MEMBER);
|
||||||
|
setIsAddMemberOpen(true);
|
||||||
|
await loadAvailableUsers();
|
||||||
|
};
|
||||||
|
|
||||||
|
const handleAddMember = async (event: SyntheticEvent<HTMLFormElement>): Promise<void> => {
|
||||||
|
event.preventDefault();
|
||||||
|
|
||||||
|
if (!selectedWorkspaceId) {
|
||||||
|
setAddMemberError("Select a workspace before adding members.");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!memberUserId) {
|
||||||
|
setAddMemberError("Select a user to add.");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
setIsAddingMember(true);
|
||||||
|
setAddMemberError(null);
|
||||||
|
|
||||||
|
try {
|
||||||
|
await addWorkspaceMember(selectedWorkspaceId, {
|
||||||
|
userId: memberUserId,
|
||||||
|
role: memberRole,
|
||||||
|
});
|
||||||
|
setIsAddMemberOpen(false);
|
||||||
|
await loadMembers(selectedWorkspaceId);
|
||||||
|
} catch (error) {
|
||||||
|
setAddMemberError(getErrorMessage(error, "Failed to add member"));
|
||||||
|
} finally {
|
||||||
|
setIsAddingMember(false);
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
const handleRemoveMember = async (): Promise<void> => {
|
||||||
|
if (!selectedWorkspaceId || !removeTarget) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
setIsRemovingMember(true);
|
||||||
|
|
||||||
|
try {
|
||||||
|
await removeWorkspaceMember(selectedWorkspaceId, removeTarget.userId);
|
||||||
|
setRemoveTarget(null);
|
||||||
|
await loadMembers(selectedWorkspaceId);
|
||||||
|
} catch (error) {
|
||||||
|
setMembersError(getErrorMessage(error, "Failed to remove member"));
|
||||||
|
} finally {
|
||||||
|
setIsRemovingMember(false);
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<main className="container mx-auto px-4 py-8 max-w-5xl">
|
<main className="max-w-6xl mx-auto p-6 space-y-6">
|
||||||
<div className="mb-8">
|
<div className="flex items-start justify-between gap-4">
|
||||||
<div className="flex items-center justify-between mb-2">
|
<div>
|
||||||
<h1 className="text-3xl font-bold text-gray-900">Workspaces</h1>
|
<h1 className="text-3xl font-bold">Workspaces</h1>
|
||||||
<Link href="/settings" className="text-sm text-blue-600 hover:text-blue-700">
|
<p className="text-muted-foreground mt-1">Manage workspaces and workspace members</p>
|
||||||
← Back to Settings
|
|
||||||
</Link>
|
|
||||||
</div>
|
</div>
|
||||||
<p className="text-gray-600">Manage your workspaces and collaborate with your team</p>
|
<Link href="/settings" className="text-sm text-blue-600 hover:text-blue-700">
|
||||||
|
← Back to Settings
|
||||||
|
</Link>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
{/* Create New Workspace */}
|
<Card>
|
||||||
<div className="bg-white rounded-lg shadow-sm border border-gray-200 p-6 mb-6">
|
<CardHeader>
|
||||||
<h2 className="text-lg font-semibold text-gray-900 mb-4">Create New Workspace</h2>
|
<CardTitle>Create New Workspace</CardTitle>
|
||||||
<form onSubmit={handleCreateWorkspace} className="flex gap-3">
|
<CardDescription>Create a workspace for a new team or project.</CardDescription>
|
||||||
<input
|
</CardHeader>
|
||||||
type="text"
|
<CardContent>
|
||||||
value={newWorkspaceName}
|
<form onSubmit={handleCreateWorkspace} className="flex gap-3">
|
||||||
onChange={(e) => {
|
<Input
|
||||||
setNewWorkspaceName(e.target.value);
|
type="text"
|
||||||
}}
|
value={newWorkspaceName}
|
||||||
placeholder="Enter workspace name..."
|
onChange={(event) => {
|
||||||
disabled={isCreating}
|
setNewWorkspaceName(event.target.value);
|
||||||
className="flex-1 px-4 py-2 border border-gray-300 rounded-lg focus:ring-2 focus:ring-blue-500 focus:border-transparent disabled:bg-gray-100"
|
}}
|
||||||
/>
|
placeholder="Enter workspace name..."
|
||||||
<button
|
disabled={isCreating}
|
||||||
type="submit"
|
/>
|
||||||
disabled={isCreating || !newWorkspaceName.trim()}
|
<Button type="submit" disabled={isCreating || !newWorkspaceName.trim()}>
|
||||||
className="px-6 py-2 bg-blue-600 text-white rounded-lg hover:bg-blue-700 disabled:opacity-50 disabled:cursor-not-allowed font-medium"
|
{isCreating ? "Creating..." : "Create Workspace"}
|
||||||
>
|
</Button>
|
||||||
{isCreating ? "Creating..." : "Create Workspace"}
|
</form>
|
||||||
</button>
|
{createError !== null ? (
|
||||||
</form>
|
<p className="mt-3 text-sm text-destructive" role="alert">
|
||||||
{createError !== null && (
|
{createError}
|
||||||
<div className="mt-3 rounded-md border border-red-200 bg-red-50 px-3 py-2 text-sm text-red-700">
|
</p>
|
||||||
{createError}
|
) : null}
|
||||||
</div>
|
</CardContent>
|
||||||
)}
|
</Card>
|
||||||
|
|
||||||
|
{loadError !== null ? (
|
||||||
|
<Card>
|
||||||
|
<CardContent className="py-4">
|
||||||
|
<p className="text-sm text-destructive" role="alert">
|
||||||
|
{loadError}
|
||||||
|
</p>
|
||||||
|
</CardContent>
|
||||||
|
</Card>
|
||||||
|
) : null}
|
||||||
|
|
||||||
|
<div className="grid grid-cols-1 lg:grid-cols-5 gap-6">
|
||||||
|
<Card className="lg:col-span-2">
|
||||||
|
<CardHeader>
|
||||||
|
<CardTitle>Your Workspaces ({isLoading ? "..." : workspaces.length})</CardTitle>
|
||||||
|
<CardDescription>Click a workspace to manage its members.</CardDescription>
|
||||||
|
</CardHeader>
|
||||||
|
<CardContent className="space-y-3">
|
||||||
|
{isLoading ? (
|
||||||
|
<p className="text-sm text-muted-foreground">Loading workspaces...</p>
|
||||||
|
) : workspaces.length === 0 ? (
|
||||||
|
<p className="text-sm text-muted-foreground">
|
||||||
|
No workspaces yet. Create one to begin.
|
||||||
|
</p>
|
||||||
|
) : (
|
||||||
|
workspaces.map((workspace) => {
|
||||||
|
const isSelected = selectedWorkspaceId === workspace.id;
|
||||||
|
|
||||||
|
return (
|
||||||
|
<button
|
||||||
|
key={workspace.id}
|
||||||
|
type="button"
|
||||||
|
onClick={() => {
|
||||||
|
setSelectedWorkspaceId(workspace.id);
|
||||||
|
}}
|
||||||
|
className={`w-full rounded-lg border p-4 text-left transition-colors ${
|
||||||
|
isSelected ? "border-primary bg-muted/40" : "border-border hover:bg-muted/20"
|
||||||
|
}`}
|
||||||
|
>
|
||||||
|
<div className="flex items-start justify-between gap-3">
|
||||||
|
<div className="min-w-0">
|
||||||
|
<p className="font-semibold truncate">{workspace.name}</p>
|
||||||
|
<p className="text-xs text-muted-foreground mt-1">
|
||||||
|
Created {new Date(workspace.createdAt).toLocaleDateString()}
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
<Badge variant="outline">{toRoleLabel(workspace.role)}</Badge>
|
||||||
|
</div>
|
||||||
|
</button>
|
||||||
|
);
|
||||||
|
})
|
||||||
|
)}
|
||||||
|
</CardContent>
|
||||||
|
</Card>
|
||||||
|
|
||||||
|
<Card className="lg:col-span-3">
|
||||||
|
<CardHeader>
|
||||||
|
<div className="flex items-start justify-between gap-3">
|
||||||
|
<div>
|
||||||
|
<CardTitle>
|
||||||
|
{selectedWorkspace ? `${selectedWorkspace.name} Members` : "Workspace Members"}
|
||||||
|
</CardTitle>
|
||||||
|
<CardDescription>
|
||||||
|
{selectedWorkspace
|
||||||
|
? "Manage member roles and access for this workspace."
|
||||||
|
: "Select a workspace to view its members."}
|
||||||
|
</CardDescription>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<Dialog
|
||||||
|
open={isAddMemberOpen}
|
||||||
|
onOpenChange={(open) => {
|
||||||
|
if (!open && !isAddingMember) {
|
||||||
|
setIsAddMemberOpen(false);
|
||||||
|
setAddMemberError(null);
|
||||||
|
}
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
<DialogTrigger asChild>
|
||||||
|
<Button
|
||||||
|
onClick={() => {
|
||||||
|
void openAddMemberDialog();
|
||||||
|
}}
|
||||||
|
disabled={!selectedWorkspace}
|
||||||
|
>
|
||||||
|
<UserPlus className="h-4 w-4 mr-2" />
|
||||||
|
Add Member
|
||||||
|
</Button>
|
||||||
|
</DialogTrigger>
|
||||||
|
<DialogContent>
|
||||||
|
<DialogHeader>
|
||||||
|
<DialogTitle>Add Workspace Member</DialogTitle>
|
||||||
|
<DialogDescription>
|
||||||
|
Add an existing user to {selectedWorkspace?.name ?? "this workspace"}.
|
||||||
|
</DialogDescription>
|
||||||
|
</DialogHeader>
|
||||||
|
|
||||||
|
<form
|
||||||
|
onSubmit={(event) => {
|
||||||
|
void handleAddMember(event);
|
||||||
|
}}
|
||||||
|
className="space-y-4"
|
||||||
|
>
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="member-user">User</Label>
|
||||||
|
<Select
|
||||||
|
value={memberUserId}
|
||||||
|
onValueChange={(value) => {
|
||||||
|
setMemberUserId(value);
|
||||||
|
}}
|
||||||
|
disabled={isLoadingUsers || eligibleUsers.length === 0 || isAddingMember}
|
||||||
|
>
|
||||||
|
<SelectTrigger id="member-user">
|
||||||
|
<SelectValue
|
||||||
|
placeholder={
|
||||||
|
isLoadingUsers
|
||||||
|
? "Loading users..."
|
||||||
|
: eligibleUsers.length === 0
|
||||||
|
? "No eligible users"
|
||||||
|
: "Select a user"
|
||||||
|
}
|
||||||
|
/>
|
||||||
|
</SelectTrigger>
|
||||||
|
<SelectContent>
|
||||||
|
{eligibleUsers.map((user) => (
|
||||||
|
<SelectItem key={user.id} value={user.id}>
|
||||||
|
{user.name} ({user.email})
|
||||||
|
</SelectItem>
|
||||||
|
))}
|
||||||
|
</SelectContent>
|
||||||
|
</Select>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="member-role">Role</Label>
|
||||||
|
<Select
|
||||||
|
value={memberRole}
|
||||||
|
onValueChange={(value) => {
|
||||||
|
setMemberRole(value as WorkspaceMemberRole);
|
||||||
|
}}
|
||||||
|
disabled={isAddingMember}
|
||||||
|
>
|
||||||
|
<SelectTrigger id="member-role">
|
||||||
|
<SelectValue placeholder="Select role" />
|
||||||
|
</SelectTrigger>
|
||||||
|
<SelectContent>
|
||||||
|
{Object.values(WorkspaceMemberRole)
|
||||||
|
.filter((role) => role !== WorkspaceMemberRole.OWNER)
|
||||||
|
.map((role) => (
|
||||||
|
<SelectItem key={role} value={role}>
|
||||||
|
{toRoleLabel(role)}
|
||||||
|
</SelectItem>
|
||||||
|
))}
|
||||||
|
</SelectContent>
|
||||||
|
</Select>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{addMemberError !== null ? (
|
||||||
|
<p className="text-sm text-destructive" role="alert">
|
||||||
|
{addMemberError}
|
||||||
|
</p>
|
||||||
|
) : null}
|
||||||
|
|
||||||
|
<DialogFooter>
|
||||||
|
<Button
|
||||||
|
type="button"
|
||||||
|
variant="outline"
|
||||||
|
onClick={() => {
|
||||||
|
if (!isAddingMember) {
|
||||||
|
setIsAddMemberOpen(false);
|
||||||
|
setAddMemberError(null);
|
||||||
|
}
|
||||||
|
}}
|
||||||
|
disabled={isAddingMember}
|
||||||
|
>
|
||||||
|
Cancel
|
||||||
|
</Button>
|
||||||
|
<Button type="submit" disabled={isAddingMember || !memberUserId}>
|
||||||
|
{isAddingMember ? "Adding..." : "Add Member"}
|
||||||
|
</Button>
|
||||||
|
</DialogFooter>
|
||||||
|
</form>
|
||||||
|
</DialogContent>
|
||||||
|
</Dialog>
|
||||||
|
</div>
|
||||||
|
</CardHeader>
|
||||||
|
|
||||||
|
<CardContent className="space-y-3">
|
||||||
|
{selectedWorkspace === null ? (
|
||||||
|
<p className="text-sm text-muted-foreground">Select a workspace to view members.</p>
|
||||||
|
) : membersError !== null ? (
|
||||||
|
<p className="text-sm text-destructive" role="alert">
|
||||||
|
{membersError}
|
||||||
|
</p>
|
||||||
|
) : isMembersLoading ? (
|
||||||
|
<p className="text-sm text-muted-foreground">Loading members...</p>
|
||||||
|
) : members.length === 0 ? (
|
||||||
|
<p className="text-sm text-muted-foreground">No members found for this workspace.</p>
|
||||||
|
) : (
|
||||||
|
members.map((member) => (
|
||||||
|
<div
|
||||||
|
key={member.userId}
|
||||||
|
className="rounded-md border p-4 flex flex-col gap-3 md:flex-row md:items-center md:justify-between"
|
||||||
|
>
|
||||||
|
<div className="space-y-1 min-w-0">
|
||||||
|
<p className="font-semibold truncate">{member.user.name ?? "Unnamed User"}</p>
|
||||||
|
<p className="text-sm text-muted-foreground truncate">{member.user.email}</p>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="flex items-center gap-2 md:justify-end">
|
||||||
|
<Badge variant="outline" className={ROLE_BADGE_CLASS[member.role]}>
|
||||||
|
{toRoleLabel(member.role)}
|
||||||
|
</Badge>
|
||||||
|
|
||||||
|
<Button
|
||||||
|
type="button"
|
||||||
|
variant="destructive"
|
||||||
|
size="sm"
|
||||||
|
onClick={() => {
|
||||||
|
setRemoveTarget({
|
||||||
|
userId: member.userId,
|
||||||
|
email: member.user.email,
|
||||||
|
});
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
<UserX className="h-4 w-4 mr-2" />
|
||||||
|
Remove
|
||||||
|
</Button>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
))
|
||||||
|
)}
|
||||||
|
</CardContent>
|
||||||
|
</Card>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
{/* Workspace List */}
|
<AlertDialog
|
||||||
<div className="space-y-4">
|
open={removeTarget !== null}
|
||||||
<h2 className="text-xl font-semibold text-gray-900">
|
onOpenChange={(open) => {
|
||||||
Your Workspaces ({isLoading ? "..." : workspacesWithRoles.length})
|
if (!open && !isRemovingMember) {
|
||||||
</h2>
|
setRemoveTarget(null);
|
||||||
{loadError !== null ? (
|
}
|
||||||
<div className="rounded-md border border-red-200 bg-red-50 px-4 py-3 text-red-700">
|
}}
|
||||||
{loadError}
|
>
|
||||||
</div>
|
<AlertDialogContent>
|
||||||
) : isLoading ? (
|
<AlertDialogHeader>
|
||||||
<div className="bg-white rounded-lg shadow-sm border border-gray-200 p-12 text-center text-gray-600">
|
<AlertDialogTitle>Remove Workspace Member</AlertDialogTitle>
|
||||||
Loading workspaces...
|
<AlertDialogDescription>
|
||||||
</div>
|
Remove {removeTarget?.email} from {selectedWorkspace?.name}? They will lose access to
|
||||||
) : workspacesWithRoles.length === 0 ? (
|
this workspace.
|
||||||
<div className="bg-white rounded-lg shadow-sm border border-gray-200 p-12 text-center">
|
</AlertDialogDescription>
|
||||||
<svg
|
</AlertDialogHeader>
|
||||||
className="mx-auto h-12 w-12 text-gray-400 mb-4"
|
<AlertDialogFooter>
|
||||||
fill="none"
|
<AlertDialogCancel disabled={isRemovingMember}>Cancel</AlertDialogCancel>
|
||||||
stroke="currentColor"
|
<AlertDialogAction
|
||||||
viewBox="0 0 24 24"
|
disabled={isRemovingMember}
|
||||||
|
onClick={() => {
|
||||||
|
void handleRemoveMember();
|
||||||
|
}}
|
||||||
>
|
>
|
||||||
<path
|
{isRemovingMember ? "Removing..." : "Remove"}
|
||||||
strokeLinecap="round"
|
</AlertDialogAction>
|
||||||
strokeLinejoin="round"
|
</AlertDialogFooter>
|
||||||
strokeWidth={2}
|
</AlertDialogContent>
|
||||||
d="M19 21V5a2 2 0 00-2-2H7a2 2 0 00-2 2v16m14 0h2m-2 0h-5m-9 0H3m2 0h5M9 7h1m-1 4h1m4-4h1m-1 4h1m-5 10v-5a1 1 0 011-1h2a1 1 0 011 1v5m-4 0h4"
|
</AlertDialog>
|
||||||
/>
|
|
||||||
</svg>
|
|
||||||
<h3 className="text-lg font-medium text-gray-900 mb-2">No workspaces yet</h3>
|
|
||||||
<p className="text-gray-600">Create your first workspace to get started</p>
|
|
||||||
</div>
|
|
||||||
) : (
|
|
||||||
<div className="grid grid-cols-1 md:grid-cols-2 gap-4">
|
|
||||||
{workspacesWithRoles.map((workspace) => (
|
|
||||||
<WorkspaceCard
|
|
||||||
key={workspace.id}
|
|
||||||
workspace={workspace}
|
|
||||||
userRole={workspace.userRole}
|
|
||||||
memberCount={workspace.memberCount}
|
|
||||||
/>
|
|
||||||
))}
|
|
||||||
</div>
|
|
||||||
)}
|
|
||||||
</div>
|
|
||||||
</main>
|
</main>
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|||||||
16
apps/web/src/components/settings/SettingsAccessDenied.tsx
Normal file
16
apps/web/src/components/settings/SettingsAccessDenied.tsx
Normal file
@@ -0,0 +1,16 @@
|
|||||||
|
import type { ReactElement } from "react";
|
||||||
|
|
||||||
|
interface SettingsAccessDeniedProps {
|
||||||
|
message: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function SettingsAccessDenied({ message }: SettingsAccessDeniedProps): ReactElement {
|
||||||
|
return (
|
||||||
|
<div className="p-8 max-w-2xl">
|
||||||
|
<div className="rounded-lg border border-red-200 bg-red-50 p-6 text-center">
|
||||||
|
<p className="text-lg font-semibold text-red-700">Access Denied</p>
|
||||||
|
<p className="mt-2 text-sm text-red-600">{message}</p>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
@@ -53,6 +53,7 @@ export interface InvitationResponse {
|
|||||||
|
|
||||||
export interface UpdateUserDto {
|
export interface UpdateUserDto {
|
||||||
name?: string;
|
name?: string;
|
||||||
|
email?: string;
|
||||||
deactivatedAt?: string | null;
|
deactivatedAt?: string | null;
|
||||||
emailVerified?: boolean;
|
emailVerified?: boolean;
|
||||||
preferences?: Record<string, unknown>;
|
preferences?: Record<string, unknown>;
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
import { describe, it, expect, vi, beforeEach } from "vitest";
|
import { describe, it, expect, vi, beforeEach } from "vitest";
|
||||||
import * as client from "./client";
|
import * as client from "./client";
|
||||||
import { fetchTeams, createTeam, fetchTeamMembers } from "./teams";
|
import { fetchTeams, createTeam, fetchTeamMembers, updateTeam, deleteTeam } from "./teams";
|
||||||
|
|
||||||
vi.mock("./client");
|
vi.mock("./client");
|
||||||
|
|
||||||
@@ -44,6 +44,18 @@ describe("createTeam", (): void => {
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe("updateTeam", (): void => {
|
||||||
|
it("patches team endpoint", async (): Promise<void> => {
|
||||||
|
vi.mocked(client.apiPatch).mockResolvedValueOnce({ id: "t1", name: "Platform" } as never);
|
||||||
|
await updateTeam("t1", { name: "Platform" });
|
||||||
|
expect(client.apiPatch).toHaveBeenCalledWith(
|
||||||
|
"/api/workspaces/ws-1/teams/t1",
|
||||||
|
expect.objectContaining({ name: "Platform" }),
|
||||||
|
"ws-1"
|
||||||
|
);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
describe("fetchTeamMembers", (): void => {
|
describe("fetchTeamMembers", (): void => {
|
||||||
it("calls members endpoint for team", async (): Promise<void> => {
|
it("calls members endpoint for team", async (): Promise<void> => {
|
||||||
vi.mocked(client.apiGet).mockResolvedValueOnce([] as never);
|
vi.mocked(client.apiGet).mockResolvedValueOnce([] as never);
|
||||||
@@ -51,3 +63,11 @@ describe("fetchTeamMembers", (): void => {
|
|||||||
expect(client.apiGet).toHaveBeenCalledWith("/api/workspaces/ws-1/teams/t-1/members", "ws-1");
|
expect(client.apiGet).toHaveBeenCalledWith("/api/workspaces/ws-1/teams/t-1/members", "ws-1");
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe("deleteTeam", (): void => {
|
||||||
|
it("deletes team endpoint", async (): Promise<void> => {
|
||||||
|
vi.mocked(client.apiDelete).mockResolvedValueOnce(undefined as never);
|
||||||
|
await deleteTeam("t1");
|
||||||
|
expect(client.apiDelete).toHaveBeenCalledWith("/api/workspaces/ws-1/teams/t1", "ws-1");
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|||||||
@@ -4,7 +4,7 @@
|
|||||||
*/
|
*/
|
||||||
|
|
||||||
import type { TeamMemberRole } from "@mosaic/shared";
|
import type { TeamMemberRole } from "@mosaic/shared";
|
||||||
import { apiDelete, apiGet, apiPost } from "./client";
|
import { apiDelete, apiGet, apiPatch, apiPost } from "./client";
|
||||||
|
|
||||||
const WORKSPACE_STORAGE_KEY = "mosaic-workspace-id";
|
const WORKSPACE_STORAGE_KEY = "mosaic-workspace-id";
|
||||||
|
|
||||||
@@ -55,6 +55,11 @@ export interface CreateTeamDto {
|
|||||||
description?: string;
|
description?: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
export interface UpdateTeamDto {
|
||||||
|
name?: string;
|
||||||
|
description?: string | null;
|
||||||
|
}
|
||||||
|
|
||||||
export interface AddTeamMemberDto {
|
export interface AddTeamMemberDto {
|
||||||
userId: string;
|
userId: string;
|
||||||
role?: TeamMemberRole;
|
role?: TeamMemberRole;
|
||||||
@@ -80,6 +85,22 @@ export async function createTeam(dto: CreateTeamDto, workspaceId?: string): Prom
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Update a team in the active workspace.
|
||||||
|
*/
|
||||||
|
export async function updateTeam(
|
||||||
|
teamId: string,
|
||||||
|
dto: UpdateTeamDto,
|
||||||
|
workspaceId?: string
|
||||||
|
): Promise<TeamRecord> {
|
||||||
|
const resolvedWorkspaceId = resolveWorkspaceId(workspaceId);
|
||||||
|
return apiPatch<TeamRecord>(
|
||||||
|
`/api/workspaces/${resolvedWorkspaceId}/teams/${teamId}`,
|
||||||
|
dto,
|
||||||
|
resolvedWorkspaceId
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Fetch team members for a team in the active workspace.
|
* Fetch team members for a team in the active workspace.
|
||||||
* The current backend route shape is workspace-scoped team membership.
|
* The current backend route shape is workspace-scoped team membership.
|
||||||
|
|||||||
@@ -1,4 +1,4 @@
|
|||||||
FROM quay.io/openbao/openbao:2.5.0
|
FROM quay.io/openbao/openbao:2.5.1
|
||||||
|
|
||||||
LABEL maintainer="Mosaic Stack <dev@mosaic.local>"
|
LABEL maintainer="Mosaic Stack <dev@mosaic.local>"
|
||||||
LABEL description="OpenBao secrets management for Mosaic Stack"
|
LABEL description="OpenBao secrets management for Mosaic Stack"
|
||||||
|
|||||||
@@ -25,14 +25,14 @@
|
|||||||
| MS21-MIG-003 | not-started | phase-3 | Run migration on production database | #568 | api | — | MS21-MIG-001,MS21-TEST-003 | MS21-VER-001 | — | — | — | 5K | — | Needs deploy coordination; not automatable |
|
| MS21-MIG-003 | not-started | phase-3 | Run migration on production database | #568 | api | — | MS21-MIG-001,MS21-TEST-003 | MS21-VER-001 | — | — | — | 5K | — | Needs deploy coordination; not automatable |
|
||||||
| MS21-MIG-004 | done | phase-3 | Import API endpoints (6/6 tests) | #568 | api | feat/ms21-import-api | MS21-DB-001 | — | codex | 2026-02-28 | 2026-02-28 | 20K | 24K | PR #567 merged, CI green. Review: 0 blockers, 4 should-fix, 1 medium sec (no audit log). |
|
| MS21-MIG-004 | done | phase-3 | Import API endpoints (6/6 tests) | #568 | api | feat/ms21-import-api | MS21-DB-001 | — | codex | 2026-02-28 | 2026-02-28 | 20K | 24K | PR #567 merged, CI green. Review: 0 blockers, 4 should-fix, 1 medium sec (no audit log). |
|
||||||
| MS21-UI-001 | done | phase-4 | Settings/users page | #569 | web | feat/ms21-ui-users | MS21-API-001,MS21-API-002 | — | codex | 2026-02-28 | 2026-02-28 | 20K | ~30K | PR #573 merged. Review: 0 blockers, 4 should-fix → MS21-UI-001-QA |
|
| MS21-UI-001 | done | phase-4 | Settings/users page | #569 | web | feat/ms21-ui-users | MS21-API-001,MS21-API-002 | — | codex | 2026-02-28 | 2026-02-28 | 20K | ~30K | PR #573 merged. Review: 0 blockers, 4 should-fix → MS21-UI-001-QA |
|
||||||
| MS21-UI-001-QA | not-started | phase-4 | QA: fix 4 review findings (pagination, error state, self-deactivate guard, tests) | #569 | web | fix/ms21-ui-001-qa | MS21-UI-001 | — | — | — | — | 15K | — | 0 blockers; merged per framework. Should-fix: pagination cap, error/empty collision, self-deactivate guard, no tests. |
|
| MS21-UI-001-QA | in-progress | phase-4 | QA: fix 4 review findings (pagination, error state, self-deactivate guard, tests) | #569 | web | fix/ms21-ui-001-qa | MS21-UI-001 | — | — | — | — | 15K | — | 0 blockers; merged per framework. Should-fix: pagination cap, error/empty collision, self-deactivate guard, no tests. |
|
||||||
| MS21-UI-002 | not-started | phase-4 | User detail/edit and invite dialogs | #569 | web | feat/ms21-ui-users | MS21-UI-001 | — | — | — | — | 15K | — | |
|
| MS21-UI-002 | done | phase-4 | User detail/edit and invite dialogs | #569 | web | feat/ms21-ui-users | MS21-UI-001 | — | — | — | — | 15K | — | |
|
||||||
| MS21-UI-003 | done | phase-4 | Settings/workspaces page (wire to real API) | #569 | web | feat/ms21-ui-workspaces | MS21-API-003 | — | codex | 2026-02-28 | 2026-02-28 | 15K | ~25K | PR #574 merged. Review: 0 critical, 1 low (raw errors in UI) |
|
| MS21-UI-003 | done | phase-4 | Settings/workspaces page (wire to real API) | #569 | web | feat/ms21-ui-workspaces | MS21-API-003 | — | codex | 2026-02-28 | 2026-02-28 | 15K | ~25K | PR #574 merged. Review: 0 critical, 1 low (raw errors in UI) |
|
||||||
| MS21-UI-004 | not-started | phase-4 | Workspace member management UI | #569 | web | feat/ms21-ui-workspaces | MS21-UI-003,MS21-API-003 | — | — | — | — | 15K | — | Components exist |
|
| MS21-UI-004 | done | phase-4 | Workspace member management UI | #569 | web | feat/ms21-ui-workspaces | MS21-UI-003,MS21-API-003 | — | — | — | — | 15K | — | Components exist |
|
||||||
| MS21-UI-005 | not-started | phase-4 | Settings/teams page | #569 | web | feat/ms21-ui-teams | MS21-API-004 | — | — | — | — | 15K | — | |
|
| MS21-UI-005 | done | phase-4 | Settings/teams page | #569 | web | feat/ms21-ui-teams | MS21-API-004 | — | — | — | — | 15K | — | |
|
||||||
| MS21-TEST-004 | not-started | phase-4 | Frontend component tests | #569 | web | test/ms21-ui | MS21-UI-001,MS21-UI-002,MS21-UI-003,MS21-UI-004,MS21-UI-005 | — | — | — | — | 20K | — | |
|
| MS21-TEST-004 | in-progress | phase-4 | Frontend component tests | #569 | web | test/ms21-ui | MS21-UI-001,MS21-UI-002,MS21-UI-003,MS21-UI-004,MS21-UI-005 | — | — | — | — | 20K | — | |
|
||||||
| MS21-RBAC-001 | not-started | phase-5 | Sidebar navigation role gating | #570 | web | feat/ms21-rbac | MS21-UI-001 | — | — | — | — | 10K | — | |
|
| MS21-RBAC-001 | done | phase-5 | Sidebar navigation role gating | #570 | web | feat/ms21-rbac | MS21-UI-001 | — | — | — | — | 10K | — | |
|
||||||
| MS21-RBAC-002 | not-started | phase-5 | Settings page access restriction | #570 | web | feat/ms21-rbac | MS21-RBAC-001 | — | — | — | — | 8K | — | |
|
| MS21-RBAC-002 | done | phase-5 | Settings page access restriction | #570 | web | feat/ms21-rbac | MS21-RBAC-001 | — | — | — | — | 8K | — | |
|
||||||
| MS21-RBAC-003 | done | phase-5 | Action button permission gating | #570 | web | feat/ms21-rbac | MS21-RBAC-001 | — | — | — | — | 8K | — | |
|
| MS21-RBAC-003 | done | phase-5 | Action button permission gating | #570 | web | feat/ms21-rbac | MS21-RBAC-001 | — | — | — | — | 8K | — | |
|
||||||
| MS21-RBAC-004 | done | phase-5 | User profile role display | #570 | web | feat/ms21-rbac | MS21-RBAC-001 | — | — | — | — | 5K | — | |
|
| MS21-RBAC-004 | done | phase-5 | User profile role display | #570 | web | feat/ms21-rbac | MS21-RBAC-001 | — | — | — | — | 5K | — | |
|
||||||
| MS21-VER-001 | done | phase-6 | Full quality gate pass | #571 | stack | — | MS21-TEST-004,MS21-RBAC-004,MS21-MIG-003 | MS21-VER-002 | — | — | — | 5K | — | |
|
| MS21-VER-001 | done | phase-6 | Full quality gate pass | #571 | stack | — | MS21-TEST-004,MS21-RBAC-004,MS21-MIG-003 | MS21-VER-002 | — | — | — | 5K | — | |
|
||||||
@@ -52,3 +52,22 @@
|
|||||||
| **Total** | **31** | **15** | **~371K** | **~175K** |
|
| **Total** | **31** | **15** | **~371K** | **~175K** |
|
||||||
|
|
||||||
Remaining estimate: ~143K tokens (Codex budget).
|
Remaining estimate: ~143K tokens (Codex budget).
|
||||||
|
|
||||||
|
## MS22 — Fleet Evolution (Phase 0: Knowledge Layer)
|
||||||
|
|
||||||
|
| id | status | milestone | description | issue | repo | branch | depends_on | blocks | agent | started_at | completed_at | estimate | used | notes |
|
||||||
|
| --------------- | ----------- | ------------ | ------------------------------------------------------------ | -------- | ----- | ------------------------------ | --------------------------------------------------------- | ------------- | ------------ | ---------- | ------------ | -------- | ---- | --------------------------------------------- |
|
||||||
|
| MS22-PLAN-001 | done | p0-knowledge | PRD + mission bootstrap + TASKS.md | TASKS:P0 | stack | feat/ms22-knowledge-schema | — | MS22-DB-001 | orchestrator | 2026-02-28 | 2026-02-28 | 10K | 8K | PRD-MS22.md, mission fleet-evolution-20260228 |
|
||||||
|
| MS22-DB-001 | done | p0-knowledge | Findings module (pgvector, CRUD, similarity search) | TASKS:P0 | api | feat/ms22-findings | MS22-PLAN-001 | — | codex | 2026-02-28 | 2026-02-28 | 20K | ~22K | PR #585 merged, CI green |
|
||||||
|
| MS22-API-001 | done | p0-knowledge | Findings API endpoints | TASKS:P0 | api | feat/ms22-findings | MS22-DB-001 | — | codex | 2026-02-28 | 2026-02-28 | — | — | Combined with DB-001 |
|
||||||
|
| MS22-DB-002 | done | p0-knowledge | AgentMemory module (key/value store, upsert) | TASKS:P0 | api | feat/ms22-agent-memory | MS22-DB-001 | — | codex | 2026-02-28 | 2026-02-28 | 15K | ~16K | PR #586 merged, CI green |
|
||||||
|
| MS22-API-002 | done | p0-knowledge | AgentMemory API endpoints | TASKS:P0 | api | feat/ms22-agent-memory | MS22-DB-002 | — | codex | 2026-02-28 | 2026-02-28 | — | — | Combined with DB-002 |
|
||||||
|
| MS22-DB-004 | done | p0-knowledge | ConversationArchive module (pgvector, ingest, search) | TASKS:P0 | api | feat/ms22-conversation-archive | MS22-DB-001 | — | codex | 2026-02-28 | 2026-02-28 | 20K | ~18K | PR #587 merged, CI green |
|
||||||
|
| MS22-API-004 | done | p0-knowledge | ConversationArchive API endpoints | TASKS:P0 | api | feat/ms22-conversation-archive | MS22-DB-004 | — | codex | 2026-02-28 | 2026-02-28 | — | — | Combined with DB-004 |
|
||||||
|
| MS22-API-005 | done | p0-knowledge | EmbeddingService (reuse existing KnowledgeModule) | TASKS:P0 | api | — | — | — | orchestrator | 2026-02-28 | 2026-02-28 | 0 | 0 | Already existed; no work needed |
|
||||||
|
| MS22-DB-003 | not-started | p0-knowledge | Task model: add assigned_agent field + migration | TASKS:P0 | api | feat/ms22-task-agent | MS22-DB-001 | MS22-API-003 | — | — | — | 8K | — | Small schema + migration only |
|
||||||
|
| MS22-API-003 | not-started | p0-knowledge | Task API: expose assigned_agent in CRUD | TASKS:P0 | api | feat/ms22-task-agent | MS22-DB-003 | MS22-TEST-001 | — | — | — | 8K | — | Extend existing TaskModule |
|
||||||
|
| MS22-TEST-001 | not-started | p0-knowledge | Integration tests: Findings + AgentMemory + ConvArchive | TASKS:P0 | api | test/ms22-integration | MS22-API-001,MS22-API-002,MS22-API-004 | MS22-VER-P0 | — | — | — | 20K | — | E2E with live postgres |
|
||||||
|
| MS22-SKILL-001 | not-started | p0-knowledge | OpenClaw mosaic skill (agents read/write findings/memory) | TASKS:P0 | stack | feat/ms22-openclaw-skill | MS22-API-001,MS22-API-002 | MS22-VER-P0 | — | — | — | 15K | — | Skill in ~/.agents/skills/mosaic/ |
|
||||||
|
| MS22-INGEST-001 | done | p0-knowledge | Session log ingestion pipeline (OpenClaw logs → ConvArchive) | TASKS:P0 | stack | feat/ms22-ingest | MS22-API-004 | MS22-VER-P0 | — | — | — | 20K | — | Script to batch-ingest existing logs |
|
||||||
|
| MS22-VER-P0 | not-started | p0-knowledge | Phase 0 verification: all modules deployed + smoke tested | TASKS:P0 | stack | — | MS22-TEST-001,MS22-SKILL-001,MS22-INGEST-001,MS22-API-003 | — | — | — | — | 5K | — | |
|
||||||
|
|||||||
@@ -35,7 +35,8 @@
|
|||||||
"docker:ps": "docker compose ps",
|
"docker:ps": "docker compose ps",
|
||||||
"docker:build": "docker compose build",
|
"docker:build": "docker compose build",
|
||||||
"docker:restart": "docker compose restart",
|
"docker:restart": "docker compose restart",
|
||||||
"prepare": "husky || true"
|
"prepare": "husky || true",
|
||||||
|
"ingest:sessions": "tsx scripts/ingest-openclaw-sessions.ts"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"@typescript-eslint/eslint-plugin": "^8.26.0",
|
"@typescript-eslint/eslint-plugin": "^8.26.0",
|
||||||
|
|||||||
621
scripts/ingest-openclaw-sessions.ts
Normal file
621
scripts/ingest-openclaw-sessions.ts
Normal file
@@ -0,0 +1,621 @@
|
|||||||
|
import { createReadStream, constants as fsConstants } from "node:fs";
|
||||||
|
import { access, readdir, stat } from "node:fs/promises";
|
||||||
|
import { homedir } from "node:os";
|
||||||
|
import * as path from "node:path";
|
||||||
|
import * as process from "node:process";
|
||||||
|
import { createInterface } from "node:readline";
|
||||||
|
|
||||||
|
const DEFAULT_ENDPOINT = "https://mosaic-api.woltje.com/conversation-archive/ingest";
|
||||||
|
|
||||||
|
type IngestRole = "user" | "assistant";
|
||||||
|
|
||||||
|
interface IngestMessage {
|
||||||
|
role: IngestRole;
|
||||||
|
content: string;
|
||||||
|
timestamp?: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
interface IngestPayload {
|
||||||
|
sessionId: string;
|
||||||
|
workspaceId: string;
|
||||||
|
title: string;
|
||||||
|
messages: IngestMessage[];
|
||||||
|
agentId?: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
interface CliOptions {
|
||||||
|
workspaceId: string;
|
||||||
|
agentId?: string;
|
||||||
|
since?: Date;
|
||||||
|
sessionsDir?: string;
|
||||||
|
endpoint: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
interface ParsedSession {
|
||||||
|
sessionId: string;
|
||||||
|
title: string;
|
||||||
|
messages: IngestMessage[];
|
||||||
|
startedAt?: string;
|
||||||
|
endedAt?: string;
|
||||||
|
parseErrors: number;
|
||||||
|
inferredAgentId?: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
interface SendResult {
|
||||||
|
ok: boolean;
|
||||||
|
status: number;
|
||||||
|
body: string;
|
||||||
|
}
|
||||||
|
|
||||||
|
interface IngestSummary {
|
||||||
|
discovered: number;
|
||||||
|
processed: number;
|
||||||
|
ingested: number;
|
||||||
|
skippedSince: number;
|
||||||
|
skippedEmpty: number;
|
||||||
|
skippedDuplicate: number;
|
||||||
|
failed: number;
|
||||||
|
}
|
||||||
|
|
||||||
|
function printUsage(): void {
|
||||||
|
console.log(
|
||||||
|
[
|
||||||
|
"Usage:",
|
||||||
|
" pnpm ingest:sessions --workspace-id <id> [--agent-id <id>] [--since <ISO date>] [--sessions-dir <path>] [--endpoint <url>]",
|
||||||
|
"",
|
||||||
|
"Required:",
|
||||||
|
" --workspace-id Target Mosaic workspace ID",
|
||||||
|
"",
|
||||||
|
"Optional:",
|
||||||
|
" --agent-id Agent ID to include in each ingest payload",
|
||||||
|
" --since Skip sessions before this date/time (ISO8601 or YYYY-MM-DD)",
|
||||||
|
" --sessions-dir Override session directory path",
|
||||||
|
` --endpoint Ingest endpoint (default: ${DEFAULT_ENDPOINT})`,
|
||||||
|
].join("\n")
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
function expandHomePath(inputPath: string): string {
|
||||||
|
if (inputPath === "~") {
|
||||||
|
return homedir();
|
||||||
|
}
|
||||||
|
if (inputPath.startsWith("~/")) {
|
||||||
|
return path.join(homedir(), inputPath.slice(2));
|
||||||
|
}
|
||||||
|
return inputPath;
|
||||||
|
}
|
||||||
|
|
||||||
|
function parseSinceDate(rawDate: string): Date {
|
||||||
|
const parsed = new Date(rawDate);
|
||||||
|
if (Number.isNaN(parsed.getTime())) {
|
||||||
|
throw new Error(`Invalid --since date: "${rawDate}". Use ISO8601 or YYYY-MM-DD.`);
|
||||||
|
}
|
||||||
|
return parsed;
|
||||||
|
}
|
||||||
|
|
||||||
|
function parseCliArgs(args: string[]): CliOptions {
|
||||||
|
let workspaceId: string | null = null;
|
||||||
|
let agentId: string | undefined;
|
||||||
|
let since: Date | undefined;
|
||||||
|
let sessionsDir: string | undefined;
|
||||||
|
let endpoint = DEFAULT_ENDPOINT;
|
||||||
|
|
||||||
|
for (let index = 0; index < args.length; index += 1) {
|
||||||
|
const arg = args[index];
|
||||||
|
|
||||||
|
if (arg === "--help" || arg === "-h") {
|
||||||
|
printUsage();
|
||||||
|
process.exit(0);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (arg.startsWith("--workspace-id=")) {
|
||||||
|
workspaceId = arg.slice("--workspace-id=".length);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (arg === "--workspace-id") {
|
||||||
|
const value = args[index + 1];
|
||||||
|
if (!value) {
|
||||||
|
throw new Error("Missing value for --workspace-id");
|
||||||
|
}
|
||||||
|
workspaceId = value;
|
||||||
|
index += 1;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (arg.startsWith("--agent-id=")) {
|
||||||
|
agentId = arg.slice("--agent-id=".length);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (arg === "--agent-id") {
|
||||||
|
const value = args[index + 1];
|
||||||
|
if (!value) {
|
||||||
|
throw new Error("Missing value for --agent-id");
|
||||||
|
}
|
||||||
|
agentId = value;
|
||||||
|
index += 1;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (arg.startsWith("--since=")) {
|
||||||
|
since = parseSinceDate(arg.slice("--since=".length));
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (arg === "--since") {
|
||||||
|
const value = args[index + 1];
|
||||||
|
if (!value) {
|
||||||
|
throw new Error("Missing value for --since");
|
||||||
|
}
|
||||||
|
since = parseSinceDate(value);
|
||||||
|
index += 1;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (arg.startsWith("--sessions-dir=")) {
|
||||||
|
sessionsDir = arg.slice("--sessions-dir=".length);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (arg === "--sessions-dir") {
|
||||||
|
const value = args[index + 1];
|
||||||
|
if (!value) {
|
||||||
|
throw new Error("Missing value for --sessions-dir");
|
||||||
|
}
|
||||||
|
sessionsDir = value;
|
||||||
|
index += 1;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (arg.startsWith("--endpoint=")) {
|
||||||
|
endpoint = arg.slice("--endpoint=".length);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (arg === "--endpoint") {
|
||||||
|
const value = args[index + 1];
|
||||||
|
if (!value) {
|
||||||
|
throw new Error("Missing value for --endpoint");
|
||||||
|
}
|
||||||
|
endpoint = value;
|
||||||
|
index += 1;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
throw new Error(`Unknown flag: ${arg}`);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!workspaceId || workspaceId.trim().length === 0) {
|
||||||
|
throw new Error("--workspace-id is required");
|
||||||
|
}
|
||||||
|
|
||||||
|
return {
|
||||||
|
workspaceId: workspaceId.trim(),
|
||||||
|
agentId: agentId?.trim(),
|
||||||
|
since,
|
||||||
|
sessionsDir: sessionsDir ? path.resolve(expandHomePath(sessionsDir)) : undefined,
|
||||||
|
endpoint,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
function isRecord(value: unknown): value is Record<string, unknown> {
|
||||||
|
return typeof value === "object" && value !== null;
|
||||||
|
}
|
||||||
|
|
||||||
|
function asString(value: unknown): string | null {
|
||||||
|
return typeof value === "string" ? value : null;
|
||||||
|
}
|
||||||
|
|
||||||
|
function normalizeIsoTimestamp(value: unknown): string | null {
|
||||||
|
if (typeof value === "string") {
|
||||||
|
const parsed = new Date(value);
|
||||||
|
if (!Number.isNaN(parsed.getTime())) {
|
||||||
|
return parsed.toISOString();
|
||||||
|
}
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (typeof value === "number" && Number.isFinite(value)) {
|
||||||
|
const millis = value >= 1_000_000_000_000 ? value : value * 1000;
|
||||||
|
const parsed = new Date(millis);
|
||||||
|
if (!Number.isNaN(parsed.getTime())) {
|
||||||
|
return parsed.toISOString();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
function truncate(value: string, maxLength: number): string {
|
||||||
|
if (value.length <= maxLength) {
|
||||||
|
return value;
|
||||||
|
}
|
||||||
|
return `${value.slice(0, maxLength - 3)}...`;
|
||||||
|
}
|
||||||
|
|
||||||
|
function deriveTitle(content: string, fallbackSessionId: string): string {
|
||||||
|
const firstLine = content
|
||||||
|
.split(/\r?\n/u)
|
||||||
|
.map((line) => line.trim())
|
||||||
|
.find((line) => line.length > 0);
|
||||||
|
|
||||||
|
if (!firstLine) {
|
||||||
|
return `OpenClaw session ${fallbackSessionId}`;
|
||||||
|
}
|
||||||
|
|
||||||
|
const normalized = firstLine.replace(/\s+/gu, " ").trim();
|
||||||
|
return truncate(normalized, 140);
|
||||||
|
}
|
||||||
|
|
||||||
|
function extractTextContent(content: unknown): string {
|
||||||
|
if (typeof content === "string") {
|
||||||
|
return content.trim();
|
||||||
|
}
|
||||||
|
|
||||||
|
if (Array.isArray(content)) {
|
||||||
|
const parts: string[] = [];
|
||||||
|
for (const item of content) {
|
||||||
|
if (typeof item === "string") {
|
||||||
|
const trimmed = item.trim();
|
||||||
|
if (trimmed.length > 0) {
|
||||||
|
parts.push(trimmed);
|
||||||
|
}
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
if (!isRecord(item)) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
const itemType = asString(item.type);
|
||||||
|
if (itemType !== null && itemType !== "text") {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
const textValue = asString(item.text);
|
||||||
|
if (textValue && textValue.trim().length > 0) {
|
||||||
|
parts.push(textValue.trim());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return parts.join("\n\n").trim();
|
||||||
|
}
|
||||||
|
|
||||||
|
if (isRecord(content)) {
|
||||||
|
const textValue = asString(content.text);
|
||||||
|
if (textValue) {
|
||||||
|
return textValue.trim();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return "";
|
||||||
|
}
|
||||||
|
|
||||||
|
function inferAgentIdFromPath(filePath: string): string | null {
|
||||||
|
const pathParts = filePath.split(path.sep);
|
||||||
|
const agentsIndex = pathParts.lastIndexOf("agents");
|
||||||
|
if (agentsIndex < 0) {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
const candidate = pathParts[agentsIndex + 1];
|
||||||
|
return candidate && candidate.trim().length > 0 ? candidate : null;
|
||||||
|
}
|
||||||
|
|
||||||
|
async function parseSessionFile(filePath: string): Promise<ParsedSession> {
|
||||||
|
const fallbackSessionId = path.basename(filePath, path.extname(filePath));
|
||||||
|
const inferredAgentId = inferAgentIdFromPath(filePath) ?? undefined;
|
||||||
|
|
||||||
|
let sessionId = fallbackSessionId;
|
||||||
|
let title: string | null = null;
|
||||||
|
let startedAt: string | undefined;
|
||||||
|
let endedAt: string | undefined;
|
||||||
|
let parseErrors = 0;
|
||||||
|
const messages: IngestMessage[] = [];
|
||||||
|
|
||||||
|
const readStream = createReadStream(filePath, { encoding: "utf8" });
|
||||||
|
const lineReader = createInterface({
|
||||||
|
input: readStream,
|
||||||
|
crlfDelay: Number.POSITIVE_INFINITY,
|
||||||
|
});
|
||||||
|
|
||||||
|
for await (const rawLine of lineReader) {
|
||||||
|
const line = rawLine.trim();
|
||||||
|
if (line.length === 0) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
let parsedLine: unknown;
|
||||||
|
try {
|
||||||
|
parsedLine = JSON.parse(line) as unknown;
|
||||||
|
} catch {
|
||||||
|
parseErrors += 1;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!isRecord(parsedLine)) {
|
||||||
|
parseErrors += 1;
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
const eventType = asString(parsedLine.type);
|
||||||
|
if (eventType === "session") {
|
||||||
|
const rawSessionId = asString(parsedLine.id);
|
||||||
|
if (rawSessionId && rawSessionId.trim().length > 0) {
|
||||||
|
sessionId = rawSessionId;
|
||||||
|
}
|
||||||
|
|
||||||
|
const sessionTimestamp = normalizeIsoTimestamp(parsedLine.timestamp);
|
||||||
|
if (sessionTimestamp) {
|
||||||
|
startedAt ??= sessionTimestamp;
|
||||||
|
}
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (eventType !== "message") {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
const messageRecord = parsedLine.message;
|
||||||
|
if (!isRecord(messageRecord)) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
const role = asString(messageRecord.role);
|
||||||
|
if (role !== "user" && role !== "assistant") {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
const content = extractTextContent(messageRecord.content);
|
||||||
|
if (content.length === 0) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
const timestamp =
|
||||||
|
normalizeIsoTimestamp(messageRecord.timestamp) ?? normalizeIsoTimestamp(parsedLine.timestamp);
|
||||||
|
|
||||||
|
const message: IngestMessage = {
|
||||||
|
role,
|
||||||
|
content,
|
||||||
|
timestamp: timestamp ?? undefined,
|
||||||
|
};
|
||||||
|
messages.push(message);
|
||||||
|
|
||||||
|
if (!title && role === "user") {
|
||||||
|
title = deriveTitle(content, sessionId);
|
||||||
|
}
|
||||||
|
if (!startedAt && timestamp) {
|
||||||
|
startedAt = timestamp;
|
||||||
|
}
|
||||||
|
if (timestamp) {
|
||||||
|
endedAt = timestamp;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return {
|
||||||
|
sessionId,
|
||||||
|
title: title ?? `OpenClaw session ${sessionId}`,
|
||||||
|
messages,
|
||||||
|
startedAt,
|
||||||
|
endedAt,
|
||||||
|
parseErrors,
|
||||||
|
inferredAgentId,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
async function pathExists(candidatePath: string): Promise<boolean> {
|
||||||
|
try {
|
||||||
|
await access(candidatePath, fsConstants.F_OK);
|
||||||
|
return true;
|
||||||
|
} catch {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
async function discoverSessionDirectories(overrideDir?: string): Promise<string[]> {
|
||||||
|
if (overrideDir) {
|
||||||
|
if (!(await pathExists(overrideDir))) {
|
||||||
|
throw new Error(`Provided --sessions-dir does not exist: ${overrideDir}`);
|
||||||
|
}
|
||||||
|
return [overrideDir];
|
||||||
|
}
|
||||||
|
|
||||||
|
const defaultDir = path.join(homedir(), ".openclaw", "sessions");
|
||||||
|
if (await pathExists(defaultDir)) {
|
||||||
|
return [defaultDir];
|
||||||
|
}
|
||||||
|
|
||||||
|
const agentsRoot = path.join(homedir(), ".openclaw", "agents");
|
||||||
|
if (!(await pathExists(agentsRoot))) {
|
||||||
|
return [];
|
||||||
|
}
|
||||||
|
|
||||||
|
const agentEntries = await readdir(agentsRoot, { withFileTypes: true });
|
||||||
|
const directories: string[] = [];
|
||||||
|
for (const entry of agentEntries) {
|
||||||
|
if (!entry.isDirectory()) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
const sessionsDir = path.join(agentsRoot, entry.name, "sessions");
|
||||||
|
if (await pathExists(sessionsDir)) {
|
||||||
|
directories.push(sessionsDir);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return directories.sort((left, right) => left.localeCompare(right));
|
||||||
|
}
|
||||||
|
|
||||||
|
async function discoverSessionFiles(overrideDir?: string): Promise<string[]> {
|
||||||
|
const directories = await discoverSessionDirectories(overrideDir);
|
||||||
|
const files: string[] = [];
|
||||||
|
|
||||||
|
for (const directory of directories) {
|
||||||
|
const entries = await readdir(directory, { withFileTypes: true });
|
||||||
|
for (const entry of entries) {
|
||||||
|
if (!entry.isFile() || !entry.name.endsWith(".jsonl")) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
files.push(path.join(directory, entry.name));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return files.sort((left, right) => left.localeCompare(right));
|
||||||
|
}
|
||||||
|
|
||||||
|
async function resolveSessionTimestamp(session: ParsedSession, filePath: string): Promise<Date> {
|
||||||
|
const sessionTimestamp = session.startedAt ?? session.endedAt;
|
||||||
|
if (sessionTimestamp) {
|
||||||
|
const parsed = new Date(sessionTimestamp);
|
||||||
|
if (!Number.isNaN(parsed.getTime())) {
|
||||||
|
return parsed;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
const fileStat = await stat(filePath);
|
||||||
|
return fileStat.mtime;
|
||||||
|
}
|
||||||
|
|
||||||
|
function buildPayload(
|
||||||
|
options: CliOptions,
|
||||||
|
session: ParsedSession,
|
||||||
|
fallbackAgentId: string | undefined
|
||||||
|
): IngestPayload {
|
||||||
|
const payload: IngestPayload = {
|
||||||
|
sessionId: session.sessionId,
|
||||||
|
workspaceId: options.workspaceId,
|
||||||
|
title: session.title,
|
||||||
|
messages: session.messages,
|
||||||
|
};
|
||||||
|
|
||||||
|
const selectedAgentId = options.agentId ?? fallbackAgentId;
|
||||||
|
if (selectedAgentId && selectedAgentId.trim().length > 0) {
|
||||||
|
payload.agentId = selectedAgentId.trim();
|
||||||
|
}
|
||||||
|
|
||||||
|
return payload;
|
||||||
|
}
|
||||||
|
|
||||||
|
async function sendIngestRequest(
|
||||||
|
endpoint: string,
|
||||||
|
token: string,
|
||||||
|
payload: IngestPayload
|
||||||
|
): Promise<SendResult> {
|
||||||
|
const response = await fetch(endpoint, {
|
||||||
|
method: "POST",
|
||||||
|
headers: {
|
||||||
|
Authorization: `Bearer ${token}`,
|
||||||
|
"Content-Type": "application/json",
|
||||||
|
},
|
||||||
|
body: JSON.stringify(payload),
|
||||||
|
});
|
||||||
|
|
||||||
|
const body = await response.text();
|
||||||
|
return {
|
||||||
|
ok: response.ok,
|
||||||
|
status: response.status,
|
||||||
|
body,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
function summarizeFailureBody(body: string): string {
|
||||||
|
const compact = body.replace(/\s+/gu, " ").trim();
|
||||||
|
if (compact.length === 0) {
|
||||||
|
return "(empty response body)";
|
||||||
|
}
|
||||||
|
return truncate(compact, 220);
|
||||||
|
}
|
||||||
|
|
||||||
|
async function main(): Promise<void> {
|
||||||
|
const options = parseCliArgs(process.argv.slice(2));
|
||||||
|
const token = process.env.MOSAIC_API_TOKEN;
|
||||||
|
if (!token || token.trim().length === 0) {
|
||||||
|
throw new Error("MOSAIC_API_TOKEN environment variable is required.");
|
||||||
|
}
|
||||||
|
|
||||||
|
const sessionFiles = await discoverSessionFiles(options.sessionsDir);
|
||||||
|
if (sessionFiles.length === 0) {
|
||||||
|
console.log("No OpenClaw session files found.");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
console.log(`Discovered ${sessionFiles.length} session file(s).`);
|
||||||
|
if (options.since) {
|
||||||
|
console.log(`Applying --since filter at ${options.since.toISOString()}.`);
|
||||||
|
}
|
||||||
|
|
||||||
|
const summary: IngestSummary = {
|
||||||
|
discovered: sessionFiles.length,
|
||||||
|
processed: 0,
|
||||||
|
ingested: 0,
|
||||||
|
skippedSince: 0,
|
||||||
|
skippedEmpty: 0,
|
||||||
|
skippedDuplicate: 0,
|
||||||
|
failed: 0,
|
||||||
|
};
|
||||||
|
|
||||||
|
for (const [index, filePath] of sessionFiles.entries()) {
|
||||||
|
const position = `${index + 1}/${sessionFiles.length}`;
|
||||||
|
const parsedSession = await parseSessionFile(filePath);
|
||||||
|
summary.processed += 1;
|
||||||
|
|
||||||
|
if (parsedSession.messages.length === 0) {
|
||||||
|
summary.skippedEmpty += 1;
|
||||||
|
console.log(
|
||||||
|
`[${position}] Skipped ${parsedSession.sessionId}: no user/assistant text messages.`
|
||||||
|
);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
const sessionDate = await resolveSessionTimestamp(parsedSession, filePath);
|
||||||
|
if (options.since && sessionDate.getTime() < options.since.getTime()) {
|
||||||
|
summary.skippedSince += 1;
|
||||||
|
console.log(
|
||||||
|
`[${position}] Skipped ${parsedSession.sessionId}: session is before --since (${sessionDate.toISOString()}).`
|
||||||
|
);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
const payload = buildPayload(options, parsedSession, parsedSession.inferredAgentId);
|
||||||
|
let result: SendResult;
|
||||||
|
|
||||||
|
try {
|
||||||
|
result = await sendIngestRequest(options.endpoint, token, payload);
|
||||||
|
} catch (error) {
|
||||||
|
summary.failed += 1;
|
||||||
|
const message = error instanceof Error ? error.message : String(error);
|
||||||
|
console.error(`[${position}] Failed ${parsedSession.sessionId}: request error: ${message}`);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (result.ok) {
|
||||||
|
summary.ingested += 1;
|
||||||
|
const note =
|
||||||
|
parsedSession.parseErrors > 0 ? ` (parse warnings: ${parsedSession.parseErrors})` : "";
|
||||||
|
console.log(
|
||||||
|
`[${position}] Ingested ${parsedSession.sessionId} (${parsedSession.messages.length} messages)${note}.`
|
||||||
|
);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (result.status === 409) {
|
||||||
|
summary.skippedDuplicate += 1;
|
||||||
|
console.log(`[${position}] Skipped ${parsedSession.sessionId}: already exists (409).`);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
summary.failed += 1;
|
||||||
|
console.error(
|
||||||
|
`[${position}] Failed ${parsedSession.sessionId}: HTTP ${result.status} ${summarizeFailureBody(result.body)}`
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
console.log("\nIngestion summary:");
|
||||||
|
console.log(` Discovered: ${summary.discovered}`);
|
||||||
|
console.log(` Processed: ${summary.processed}`);
|
||||||
|
console.log(` Ingested: ${summary.ingested}`);
|
||||||
|
console.log(` Skipped (--since): ${summary.skippedSince}`);
|
||||||
|
console.log(` Skipped (empty): ${summary.skippedEmpty}`);
|
||||||
|
console.log(` Skipped (duplicate): ${summary.skippedDuplicate}`);
|
||||||
|
console.log(` Failed: ${summary.failed}`);
|
||||||
|
|
||||||
|
if (summary.failed > 0) {
|
||||||
|
process.exit(1);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
main().catch((error: unknown) => {
|
||||||
|
const message = error instanceof Error ? error.message : String(error);
|
||||||
|
console.error(`Fatal error: ${message}`);
|
||||||
|
process.exit(1);
|
||||||
|
});
|
||||||
Reference in New Issue
Block a user