@hypercerts-org/sdk-core 0.4.0-beta.0 → 0.5.0-beta.0

package/src/repository/OrganizationOperationsImpl.ts

@@ -109,10 +109,18 @@ export class OrganizationOperationsImpl implements OrganizationOperations {
    * ```
    */
   async create(params: { name: string; description?: string; handle?: string }): Promise<OrganizationInfo> {
+    const userDid = this.session.did || this.session.sub;
+    if (!userDid) {
+      throw new NetworkError("No authenticated user found");
+    }
+
     const response = await this.session.fetchHandler(`${this.serverUrl}/xrpc/com.sds.organization.create`, {
       method: "POST",
       headers: { "Content-Type": "application/json" },
-      body: JSON.stringify(params),
+      body: JSON.stringify({
+        ...params,
+        creatorDid: userDid,
+      }),
     });
 
     if (!response.ok) {
@@ -126,8 +134,15 @@ export class OrganizationOperationsImpl implements OrganizationOperations {
       name: data.name,
       description: data.description,
       createdAt: data.createdAt || new Date().toISOString(),
-      accessType: "owner",
-      permissions: { read: true, create: true, update: true, delete: true, admin: true, owner: true },
+      accessType: data.accessType || "owner",
+      permissions: data.permissions || {
+        read: true,
+        create: true,
+        update: true,
+        delete: true,
+        admin: true,
+        owner: true,
+      },
     };
   }
 
@@ -155,8 +170,8 @@ export class OrganizationOperationsImpl implements OrganizationOperations {
    */
   async get(did: string): Promise<OrganizationInfo | null> {
     try {
-      const orgs = await this.list();
-      return orgs.find((o) => o.did === did) ?? null;
+      const { organizations } = await this.list();
+      return organizations.find((o) => o.did === did) ?? null;
     } catch {
       return null;
     }
@@ -165,7 +180,10 @@ export class OrganizationOperationsImpl implements OrganizationOperations {
   /**
    * Lists organizations the current user has access to.
    *
-   * @returns Promise resolving to array of organization info
+   * @param params - Optional pagination parameters
+   * @param params.limit - Maximum number of results (1-100, default 50)
+   * @param params.cursor - Pagination cursor from previous response
+   * @returns Promise resolving to organizations and optional cursor
    * @throws {@link NetworkError} if the list operation fails
    *
    * @remarks
@@ -177,21 +195,25 @@ export class OrganizationOperationsImpl implements OrganizationOperations {
    *
    * @example
    * ```typescript
-   * const orgs = await repo.organizations.list();
+   * // Get first page
+   * const page1 = await repo.organizations.list({ limit: 20 });
+   * console.log(`Found ${page1.organizations.length} organizations`);
    *
-   * // Filter by access type
-   * const owned = orgs.filter(o => o.accessType === "owner");
-   * const collaborated = orgs.filter(o => o.accessType === "collaborator");
+   * // Get next page if available
+   * if (page1.cursor) {
+   *   const page2 = await repo.organizations.list({ limit: 20, cursor: page1.cursor });
+   * }
    *
-   * console.log(`You own ${owned.length} organizations`);
-   * console.log(`You collaborate on ${collaborated.length} organizations`);
+   * // Filter by access type
+   * const owned = page1.organizations.filter(o => o.accessType === "owner");
+   * const shared = page1.organizations.filter(o => o.accessType === "shared");
    * ```
    *
    * @example Display organization details
    * ```typescript
-   * const orgs = await repo.organizations.list();
+   * const { organizations } = await repo.organizations.list();
    *
-   * for (const org of orgs) {
+   * for (const org of organizations) {
    *   console.log(`${org.name} (@${org.handle})`);
    *   console.log(`  DID: ${org.did}`);
    *   console.log(`  Access: ${org.accessType}`);
@@ -201,9 +223,29 @@ export class OrganizationOperationsImpl implements OrganizationOperations {
    * }
    * ```
    */
-  async list(): Promise<OrganizationInfo[]> {
+  async list(params?: { limit?: number; cursor?: string }): Promise<{
+    organizations: OrganizationInfo[];
+    cursor?: string;
+  }> {
+    const userDid = this.session.did || this.session.sub;
+    if (!userDid) {
+      throw new NetworkError("No authenticated user found");
+    }
+
+    const queryParams = new URLSearchParams({
+      userDid,
+    });
+
+    if (params?.limit !== undefined) {
+      queryParams.set("limit", params.limit.toString());
+    }
+
+    if (params?.cursor) {
+      queryParams.set("cursor", params.cursor);
+    }
+
     const response = await this.session.fetchHandler(
-      `${this.serverUrl}/xrpc/com.sds.organization.list?userDid=${encodeURIComponent(this.session.did || this.session.sub)}`,
+      `${this.serverUrl}/xrpc/com.sds.organization.list?${queryParams.toString()}`,
       { method: "GET" },
     );
 
@@ -212,23 +254,29 @@ export class OrganizationOperationsImpl implements OrganizationOperations {
     }
 
     const data = await response.json();
-    return (data.repositories || []).map(
+    const organizations = (data.organizations || []).map(
       (r: {
         did: string;
         handle: string;
         name: string;
         description?: string;
-        accessType: "owner" | "collaborator";
+        createdAt?: string;
+        accessType: "owner" | "shared" | "none";
         permissions: CollaboratorPermissions;
       }) => ({
         did: r.did,
         handle: r.handle,
         name: r.name,
         description: r.description,
-        createdAt: new Date().toISOString(), // SDS may not return this
+        createdAt: r.createdAt || new Date().toISOString(),
         accessType: r.accessType,
         permissions: r.permissions,
       }),
     );
+
+    return {
+      organizations,
+      cursor: data.cursor,
+    };
   }
 }

package/src/repository/interfaces.ts

@@ -809,9 +809,15 @@ export interface CollaboratorOperations {
   /**
    * Lists all collaborators on the repository.
    *
-   * @returns Promise resolving to array of access grants
+   * @param params - Optional pagination parameters
+   * @param params.limit - Maximum number of results (1-100, default 50)
+   * @param params.cursor - Pagination cursor from previous response
+   * @returns Promise resolving to collaborators and optional cursor
    */
-  list(): Promise<RepositoryAccessGrant[]>;
+  list(params?: { limit?: number; cursor?: string }): Promise<{
+    collaborators: RepositoryAccessGrant[];
+    cursor?: string;
+  }>;
 
   /**
    * Checks if a user has any access to the repository.
@@ -891,7 +897,13 @@ export interface OrganizationOperations {
   /**
    * Lists organizations the current user has access to.
    *
-   * @returns Promise resolving to array of organization info
+   * @param params - Optional pagination parameters
+   * @param params.limit - Maximum number of results (1-100, default 50)
+   * @param params.cursor - Pagination cursor from previous response
+   * @returns Promise resolving to organizations and optional cursor
    */
-  list(): Promise<OrganizationInfo[]>;
+  list(params?: { limit?: number; cursor?: string }): Promise<{
+    organizations: OrganizationInfo[];
+    cursor?: string;
+  }>;
 }

package/src/repository/types.ts

@@ -85,7 +85,7 @@ export interface OrganizationInfo {
   name: string;
   description?: string;
   createdAt: string;
-  accessType: "owner" | "collaborator";
+  accessType: "owner" | "shared" | "none";
   permissions: CollaboratorPermissions;
   collaboratorCount?: number;
   profile?: {

package/tests/repository/BlobOperationsImpl.test.ts

@@ -28,11 +28,14 @@ describe("BlobOperationsImpl", () => {
   describe("upload", () => {
     it("should upload a blob successfully", async () => {
       const mockBlob = new Blob(["test content"], { type: "text/plain" });
+      const mockCID = {
+        toString: () => "bafyrei123",
+      };
       mockAgent.com.atproto.repo.uploadBlob.mockResolvedValue({
         success: true,
         data: {
           blob: {
-            ref: { $link: "bafyrei123" },
+            ref: mockCID,
             mimeType: "text/plain",
             size: 12,
           },
@@ -61,10 +64,9 @@ describe("BlobOperationsImpl", () => {
 
       await blobOps.upload(mockBlob);
 
-      expect(mockAgent.com.atproto.repo.uploadBlob).toHaveBeenCalledWith(
-        expect.any(Uint8Array),
-        { encoding: "image/png" },
-      );
+      expect(mockAgent.com.atproto.repo.uploadBlob).toHaveBeenCalledWith(expect.any(Uint8Array), {
+        encoding: "image/png",
+      });
     });
 
     it("should default to application/octet-stream for blobs without type", async () => {
@@ -82,10 +84,9 @@ describe("BlobOperationsImpl", () => {
 
       await blobOps.upload(mockBlob);
 
-      expect(mockAgent.com.atproto.repo.uploadBlob).toHaveBeenCalledWith(
-        expect.any(Uint8Array),
-        { encoding: "application/octet-stream" },
-      );
+      expect(mockAgent.com.atproto.repo.uploadBlob).toHaveBeenCalledWith(expect.any(Uint8Array), {
+        encoding: "application/octet-stream",
+      });
     });
 
     it("should throw NetworkError when API returns success: false", async () => {

package/tests/repository/CollaboratorOperationsImpl.test.ts

@@ -131,13 +131,13 @@ describe("CollaboratorOperationsImpl", () => {
           collaborators: [
             {
               userDid: "did:plc:user1",
-              permissions: { read: true, create: true, update: true, delete: false, admin: false, owner: false },
+              permissions: ["read", "create", "update"],
               grantedBy: "did:plc:owner",
               grantedAt: "2024-01-01T00:00:00Z",
             },
             {
               userDid: "did:plc:user2",
-              permissions: { read: true, create: false, update: false, delete: false, admin: false, owner: false },
+              permissions: ["read"],
               grantedBy: "did:plc:owner",
               grantedAt: "2024-01-02T00:00:00Z",
             },
@@ -147,10 +147,10 @@ describe("CollaboratorOperationsImpl", () => {
 
       const result = await collaboratorOps.list();
 
-      expect(result).toHaveLength(2);
-      expect(result[0].userDid).toBe("did:plc:user1");
-      expect(result[0].role).toBe("editor");
-      expect(result[1].role).toBe("viewer");
+      expect(result.collaborators).toHaveLength(2);
+      expect(result.collaborators[0].userDid).toBe("did:plc:user1");
+      expect(result.collaborators[0].role).toBe("editor");
+      expect(result.collaborators[1].role).toBe("viewer");
     });
 
     it("should handle empty collaborators list", async () => {
@@ -161,7 +161,7 @@ describe("CollaboratorOperationsImpl", () => {
 
       const result = await collaboratorOps.list();
 
-      expect(result).toHaveLength(0);
+      expect(result.collaborators).toHaveLength(0);
     });
 
     it("should correctly map permissions to roles", async () => {
@@ -171,13 +171,13 @@ describe("CollaboratorOperationsImpl", () => {
           collaborators: [
             {
               userDid: "did:plc:owner",
-              permissions: { read: true, create: true, update: true, delete: true, admin: true, owner: true },
+              permissions: ["read", "create", "update", "delete", "admin", "owner"],
               grantedBy: "did:plc:system",
               grantedAt: "2024-01-01T00:00:00Z",
             },
             {
               userDid: "did:plc:admin",
-              permissions: { read: true, create: true, update: true, delete: true, admin: true, owner: false },
+              permissions: ["read", "create", "update", "delete", "admin"],
               grantedBy: "did:plc:owner",
               grantedAt: "2024-01-01T00:00:00Z",
             },
@@ -187,8 +187,8 @@ describe("CollaboratorOperationsImpl", () => {
 
       const result = await collaboratorOps.list();
 
-      expect(result[0].role).toBe("owner");
-      expect(result[1].role).toBe("admin");
+      expect(result.collaborators[0].role).toBe("owner");
+      expect(result.collaborators[1].role).toBe("admin");
     });
 
     it("should throw NetworkError on failure", async () => {
@@ -209,7 +209,7 @@ describe("CollaboratorOperationsImpl", () => {
           collaborators: [
             {
               userDid: "did:plc:activeuser",
-              permissions: { read: true, create: false, update: false, delete: false, admin: false, owner: false },
+              permissions: ["read"],
               grantedBy: "did:plc:owner",
               grantedAt: "2024-01-01T00:00:00Z",
             },
@@ -240,7 +240,7 @@ describe("CollaboratorOperationsImpl", () => {
           collaborators: [
             {
               userDid: "did:plc:revokeduser",
-              permissions: { read: true, create: false, update: false, delete: false, admin: false, owner: false },
+              permissions: ["read"],
               grantedBy: "did:plc:owner",
               grantedAt: "2024-01-01T00:00:00Z",
               revokedAt: "2024-02-01T00:00:00Z",
@@ -271,7 +271,7 @@ describe("CollaboratorOperationsImpl", () => {
           collaborators: [
             {
               userDid: "did:plc:editor",
-              permissions: { read: true, create: true, update: true, delete: false, admin: false, owner: false },
+              permissions: ["read", "create", "update"],
               grantedBy: "did:plc:owner",
               grantedAt: "2024-01-01T00:00:00Z",
             },
@@ -302,7 +302,7 @@ describe("CollaboratorOperationsImpl", () => {
           collaborators: [
             {
               userDid: "did:plc:revoked",
-              permissions: { read: true, create: true, update: true, delete: false, admin: false, owner: false },
+              permissions: ["read", "create", "update"],
               grantedBy: "did:plc:owner",
               grantedAt: "2024-01-01T00:00:00Z",
               revokedAt: "2024-02-01T00:00:00Z",

package/tests/repository/OrganizationOperationsImpl.test.ts

@@ -74,9 +74,7 @@ describe("OrganizationOperationsImpl", () => {
         statusText: "Conflict",
       });
 
-      await expect(
-        orgOps.create({ name: "Test Org" }),
-      ).rejects.toThrow(NetworkError);
+      await expect(orgOps.create({ name: "Test Org" })).rejects.toThrow(NetworkError);
     });
   });
 
@@ -85,7 +83,7 @@ describe("OrganizationOperationsImpl", () => {
       mockSession.fetchHandler.mockResolvedValue({
         ok: true,
         json: async () => ({
-          repositories: [
+          organizations: [
             {
               did: "did:plc:org1",
               handle: "org1.example.com",
@@ -98,7 +96,7 @@ describe("OrganizationOperationsImpl", () => {
               did: "did:plc:org2",
               handle: "org2.example.com",
               name: "Organization 2",
-              accessType: "collaborator",
+              accessType: "shared",
               permissions: { read: true, create: true, update: true, delete: false, admin: false, owner: false },
             },
           ],
@@ -116,7 +114,7 @@ describe("OrganizationOperationsImpl", () => {
       mockSession.fetchHandler.mockResolvedValue({
         ok: true,
         json: async () => ({
-          repositories: [],
+          organizations: [],
         }),
       });
 
@@ -139,7 +137,7 @@ describe("OrganizationOperationsImpl", () => {
       mockSession.fetchHandler.mockResolvedValue({
         ok: true,
         json: async () => ({
-          repositories: [
+          organizations: [
             {
               did: "did:plc:org1",
               handle: "org1.example.com",
@@ -152,7 +150,7 @@ describe("OrganizationOperationsImpl", () => {
               handle: "org2.example.com",
               name: "Organization 2",
               description: "Second org",
-              accessType: "collaborator",
+              accessType: "shared",
               permissions: { read: true, create: true, update: false, delete: false, admin: false, owner: false },
             },
           ],
@@ -161,28 +159,28 @@ describe("OrganizationOperationsImpl", () => {
 
       const result = await orgOps.list();
 
-      expect(result).toHaveLength(2);
-      expect(result[0].did).toBe("did:plc:org1");
-      expect(result[0].accessType).toBe("owner");
-      expect(result[1].did).toBe("did:plc:org2");
-      expect(result[1].accessType).toBe("collaborator");
+      expect(result.organizations).toHaveLength(2);
+      expect(result.organizations[0].did).toBe("did:plc:org1");
+      expect(result.organizations[0].accessType).toBe("owner");
+      expect(result.organizations[1].did).toBe("did:plc:org2");
+      expect(result.organizations[1].accessType).toBe("shared");
     });
 
     it("should handle empty repositories list", async () => {
       mockSession.fetchHandler.mockResolvedValue({
         ok: true,
-        json: async () => ({ repositories: [] }),
+        json: async () => ({ organizations: [] }),
       });
 
       const result = await orgOps.list();
 
-      expect(result).toHaveLength(0);
+      expect(result.organizations).toHaveLength(0);
     });
 
     it("should use session DID in query", async () => {
       mockSession.fetchHandler.mockResolvedValue({
         ok: true,
-        json: async () => ({ repositories: [] }),
+        json: async () => ({ organizations: [] }),
       });
 
       await orgOps.list();
@@ -197,7 +195,7 @@ describe("OrganizationOperationsImpl", () => {
       mockSession.did = undefined;
       mockSession.fetchHandler.mockResolvedValue({
         ok: true,
-        json: async () => ({ repositories: [] }),
+        json: async () => ({ organizations: [] }),
       });
 
       await orgOps.list();
@@ -221,7 +219,7 @@ describe("OrganizationOperationsImpl", () => {
       mockSession.fetchHandler.mockResolvedValue({
         ok: true,
         json: async () => ({
-          repositories: [
+          organizations: [
             {
               did: "did:plc:org",
               handle: "org.example.com",
@@ -236,7 +234,7 @@ describe("OrganizationOperationsImpl", () => {
 
       const result = await orgOps.list();
 
-      expect(result[0].createdAt).toBeDefined();
+      expect(result.organizations[0].createdAt).toBeDefined();
     });
   });
 });