@pierre/storage 0.4.2 → 0.6.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@pierre/storage",
-	"version": "0.4.2",
+	"version": "0.6.0",
 	"description": "Pierre Git Storage SDK",
 	"license": "MIT",
 	"type": "module",

package/src/fetch.ts

@@ -45,7 +45,17 @@ export class ApiFetcher {
 		if (typeof path === 'string') {
 			return `${this.getBaseUrl()}/${path}`;
 		} else if (path.params) {
-			const paramStr = new URLSearchParams(path.params).toString();
+			const searchParams = new URLSearchParams();
+			for (const [key, value] of Object.entries(path.params)) {
+				if (Array.isArray(value)) {
+					for (const v of value) {
+						searchParams.append(key, v);
+					}
+				} else {
+					searchParams.append(key, value);
+				}
+			}
+			const paramStr = searchParams.toString();
 			return `${this.getBaseUrl()}/${path.path}${paramStr ? `?${paramStr}` : ''}`;
 		} else {
 			return `${this.getBaseUrl()}/${path.path}`;

package/src/index.ts

@@ -363,6 +363,7 @@ class RepoImpl implements Repo {
 
 	constructor(
 		public readonly id: string,
+		public readonly defaultBranch: string,
 		private readonly options: GitStorageOptions,
 		private readonly generateJWT: (
 			repoId: string,
@@ -506,7 +507,7 @@ class RepoImpl implements Repo {
 			ttl,
 		});
 
-		const params: Record<string, string> = {
+		const params: Record<string, string | string[]> = {
 			branch: options.branch,
 		};
 
@@ -519,6 +520,9 @@ class RepoImpl implements Repo {
 		if (typeof options.ephemeralBase === 'boolean') {
 			params.ephemeral_base = String(options.ephemeralBase);
 		}
+		if (options.paths && options.paths.length > 0) {
+			params.path = options.paths;
+		}
 
 		const response = await this.api.get({ path: 'repos/branches/diff', params }, jwt);
 
@@ -533,13 +537,16 @@ class RepoImpl implements Repo {
 			ttl,
 		});
 
-		const params: Record<string, string> = {
+		const params: Record<string, string | string[]> = {
 			sha: options.sha,
 		};
 
 		if (options.baseSha) {
 			params.baseSha = options.baseSha;
 		}
+		if (options.paths && options.paths.length > 0) {
+			params.path = options.paths;
+		}
 
 		const response = await this.api.get({ path: 'repos/diff', params }, jwt);
 
@@ -886,8 +893,8 @@ export class GitStorage {
 				}
 			: null;
 
-		// Default defaultBranch to 'main' if not provided
-		const defaultBranch = options?.defaultBranch ?? 'main';
+		// Match backend priority: baseRepo.defaultBranch > options.defaultBranch > 'main'
+		const defaultBranch = options?.baseRepo?.defaultBranch ?? options?.defaultBranch ?? 'main';
 
 		const createRepoPath =
 			baseRepoOptions || defaultBranch
@@ -906,7 +913,7 @@ export class GitStorage {
 			throw new Error('Repository already exists');
 		}
 
-		return new RepoImpl(repoId, this.options, this.generateJWT.bind(this));
+		return new RepoImpl(repoId, defaultBranch, this.options, this.generateJWT.bind(this));
 	}
 
 	/**
@@ -925,9 +932,9 @@ export class GitStorage {
 		if (resp.status === 404) {
 			return null;
 		}
-		// On 200, we could validate response, but RepoImpl only needs the repo URL/id
-		// const body = await resp.json(); // not required for now
-		return new RepoImpl(options.id, this.options, this.generateJWT.bind(this));
+		const body = (await resp.json()) as { default_branch?: string };
+		const defaultBranch = body.default_branch ?? 'main';
+		return new RepoImpl(options.id, defaultBranch, this.options, this.generateJWT.bind(this));
 	}
 
 	/**

package/src/types.ts

@@ -37,6 +37,7 @@ export interface GetRemoteURLOptions {
 
 export interface Repo {
 	id: string;
+	defaultBranch: string;
 	getRemoteURL(options?: GetRemoteURLOptions): Promise<string>;
 	getEphemeralRemoteURL(options?: GetRemoteURLOptions): Promise<string>;
 
@@ -58,7 +59,7 @@ export type ValidMethod = 'GET' | 'POST' | 'PUT' | 'DELETE';
 type SimplePath = string;
 type ComplexPath = {
 	path: string;
-	params?: Record<string, string>;
+	params?: Record<string, string | string[]>;
 	body?: Record<string, any>;
 };
 export type ValidPath = SimplePath | ComplexPath;
@@ -197,6 +198,8 @@ export interface GetBranchDiffOptions extends GitStorageInvocationOptions {
 	base?: string;
 	ephemeral?: boolean;
 	ephemeralBase?: boolean;
+	/** Optional paths to filter the diff to specific files */
+	paths?: string[];
 }
 
 export type GetBranchDiffResponse = GetBranchDiffResponseRaw;
@@ -213,6 +216,8 @@ export interface GetBranchDiffResult {
 export interface GetCommitDiffOptions extends GitStorageInvocationOptions {
 	sha: string;
 	baseSha?: string;
+	/** Optional paths to filter the diff to specific files */
+	paths?: string[];
 }
 
 export type GetCommitDiffResponse = GetCommitDiffResponseRaw;