add path to /api/commits api

Author: brendan-kellam

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added `find_symbol_definitions`, and `find_symbol_references` tools to the MCP server. [#1014](https://github.com/sourcebot-dev/sourcebot/pull/1014)
 - Added `list_tree` tool to the ask agent. [#1014](https://github.com/sourcebot-dev/sourcebot/pull/1014)
 - Added input & output token breakdown in ask details card. [#1014](https://github.com/sourcebot-dev/sourcebot/pull/1014)
+- Added `path` parameter to the `/api/commits` api to allow filtering commits by paths. [#1014](https://github.com/sourcebot-dev/sourcebot/pull/1014)
 
 ### Fixed
 - Fixed issue where ask responses would sometimes appear in the details panel while generating. [#1014](https://github.com/sourcebot-dev/sourcebot/pull/1014)

packages/web/src/app/api/(server)/commits/route.ts

@@ -13,6 +13,7 @@ const listCommitsQueryParamsSchema = z.object({
     until: z.string().optional(),
     author: z.string().optional(),
     ref: z.string().optional(),
+    path: z.string().optional(),
     page: z.coerce.number().int().positive().default(1),
     perPage: z.coerce.number().int().positive().max(100).default(50),
 });
@@ -57,6 +58,7 @@ export const GET = apiHandler(async (request: NextRequest): Promise<Response> =>
             ...(searchParams.until ? { until: searchParams.until } : {}),
             ...(searchParams.author ? { author: searchParams.author } : {}),
             ...(searchParams.ref ? { ref: searchParams.ref } : {}),
+            ...(searchParams.path ? { path: searchParams.path } : {}),
         },
     });
     if (linkHeader) headers.set('Link', linkHeader);

packages/web/src/features/git/listCommitsApi.ts

@@ -28,6 +28,7 @@ type ListCommitsRequest = {
     until?: string;
     author?: string;
     ref?: string;
+    path?: string;
     maxCount?: number;
     skip?: number;
 }
@@ -46,6 +47,7 @@ export const listCommits = async ({
     until,
     author,
     ref = 'HEAD',
+    path,
     maxCount = 50,
     skip = 0,
 }: ListCommitsRequest): Promise<SearchCommitsResult | ServiceError> => sew(() =>
@@ -93,19 +95,31 @@ export const listCommits = async ({
                 } : {}),
             };
 
+            // Build args array directly to ensure correct ordering:
+            // git log [flags] <ref> [-- <path>]
+            const logArgs: string[] = [`--max-count=${maxCount}`];
+            if (skip > 0) {
+                logArgs.push(`--skip=${skip}`);
+            }
+            for (const [key, value] of Object.entries(sharedOptions)) {
+                logArgs.push(value !== null ? `${key}=${value}` : key);
+            }
+            logArgs.push(ref);
+            if (path) {
+                logArgs.push('--', path);
+            }
+
             // First, get the commits
-            const log = await git.log({
-                [ref]: null,
-                maxCount,
-                ...(skip > 0 ? { '--skip': skip } : {}),
-                ...sharedOptions,
-            });
+            const log = await git.log(logArgs);
 
             // Then, use rev-list to get the total count of commits
             const countArgs = ['rev-list', '--count', ref];
             for (const [key, value] of Object.entries(sharedOptions)) {
                 countArgs.push(value !== null ? `${key}=${value}` : key);
             }
+            if (path) {
+                countArgs.push('--', path);
+            }
 
             const totalCount = parseInt((await git.raw(countArgs)).trim(), 10);
 

packages/web/src/features/tools/listCommits.ts

@@ -4,6 +4,8 @@ import { listCommits, SearchCommitsResult } from "@/features/git";
 import { ToolDefinition } from "./types";
 import { logger } from "./logger";
 import description from "./listCommits.txt";
+import { CodeHostType } from "@sourcebot/db";
+import { getRepoInfoByName } from "@/actions";
 
 const listCommitsShape = {
     repo: z.string().describe("The repository to list commits from"),
@@ -12,11 +14,21 @@ const listCommitsShape = {
     until: z.string().describe("End date for commit range (e.g., 'yesterday', '2024-12-31', 'today')").optional(),
     author: z.string().describe("Filter commits by author name or email (case-insensitive)").optional(),
     ref: z.string().describe("Commit SHA, branch or tag name to list commits of. If not provided, uses the default branch.").optional(),
+    path: z.string().describe("Filter commits to only those that touched this file or directory path (relative to repo root).").optional(),
     page: z.number().int().positive().describe("Page number for pagination (min 1). Default: 1").optional().default(1),
     perPage: z.number().int().positive().max(100).describe("Results per page for pagination (min 1, max 100). Default: 50").optional().default(50),
 };
 
-export type ListCommitsMetadata = SearchCommitsResult;  
+export type ListCommitsRepoInfo = {
+    name: string;
+    displayName: string;
+    codeHostType: CodeHostType;
+};
+
+export type ListCommitsMetadata = SearchCommitsResult & {
+    repo: string;
+    repoInfo: ListCommitsRepoInfo;
+};
 
 export const listCommitsDefinition: ToolDefinition<"list_commits", typeof listCommitsShape, ListCommitsMetadata> = {
     name: "list_commits",
@@ -28,7 +40,7 @@ export const listCommitsDefinition: ToolDefinition<"list_commits", typeof listCo
     execute: async (params, _context) => {
         logger.debug('list_commits', params);
 
-        const { repo, query, since, until, author, ref, page, perPage } = params;
+        const { repo, query, since, until, author, ref, path, page, perPage } = params;
         const skip = (page - 1) * perPage;
 
         const response = await listCommits({
@@ -38,6 +50,7 @@ export const listCommitsDefinition: ToolDefinition<"list_commits", typeof listCo
             until,
             author,
             ref,
+            path,
             maxCount: perPage,
             skip,
         });
@@ -46,9 +59,19 @@ export const listCommitsDefinition: ToolDefinition<"list_commits", typeof listCo
             throw new Error(response.message);
         }
 
+        const repoInfoResult = await getRepoInfoByName(repo);
+        if (isServiceError(repoInfoResult) || !repoInfoResult) {
+            throw new Error(`Repository "${repo}" not found.`);
+        }
+        const repoInfo: ListCommitsRepoInfo = {
+            name: repoInfoResult.name,
+            displayName: repoInfoResult.displayName ?? repoInfoResult.name,
+            codeHostType: repoInfoResult.codeHostType,
+        };
+
         return {
             output: JSON.stringify(response),
-            metadata: response,
+            metadata: { ...response, repo, repoInfo },
         };
     },
 };