feedback

Author: brendan-kellam

CHANGELOG.md

@@ -18,7 +18,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added commit graph generation to improve performance for commit traversal operations. [#791](https://github.com/sourcebot-dev/sourcebot/pull/791)
 - Made the code search `lang:` filter case insensitive. [#795](https://github.com/sourcebot-dev/sourcebot/pull/795)
 - Changed the `/api/source` endpoint from a POST request to a GET request. Repo, path, and ref are specified as query params. [#795](https://github.com/sourcebot-dev/sourcebot/pull/795)
-- Changed the `/api/commits` endpoint to be a POST request to a GET request. [#795](https://github.com/sourcebot-dev/sourcebot/pull/795)
+- Changed the `/api/commits` endpoint from a POST request to a GET request. [#795](https://github.com/sourcebot-dev/sourcebot/pull/795)
 - Renamed `webUrl` to `externalWebUrl` for various apis. Moving forward, `webUrl` will be used for URLs that point to Sourcebot, and `externalWebUrl` will be used for URLs that point to external code hosts. [#795](https://github.com/sourcebot-dev/sourcebot/pull/795)
 - Renamed various fields on the `/api/source` endpoint response body. [#795](https://github.com/sourcebot-dev/sourcebot/pull/795)
 

packages/mcp/src/index.ts

@@ -45,7 +45,7 @@ server.tool(
             .optional(),
         filterByFilepaths: z
             .array(z.string())
-            .describe(`Scope the search to the provided filepaths. Interpretted as a regex.`)
+            .describe(`Scope the search to the provided filepaths.`)
             .optional(),
         caseSensitive: z
             .boolean()

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

@@ -51,11 +51,11 @@ export const GET = async (request: NextRequest): Promise<Response> => {
         totalCount,
         extraParams: {
             repo: searchParams.repo,
-            ...(searchParams.query && { query: searchParams.query }),
-            ...(searchParams.since && { since: searchParams.since }),
-            ...(searchParams.until && { until: searchParams.until }),
-            ...(searchParams.author && { author: searchParams.author }),
-            ...(searchParams.ref && { ref: searchParams.ref }),
+            ...(searchParams.query ? { query: searchParams.query } : {}),
+            ...(searchParams.since ? { since: searchParams.since } : {}),
+            ...(searchParams.until ? { until: searchParams.until } : {}),
+            ...(searchParams.author ? { author: searchParams.author } : {}),
+            ...(searchParams.ref ? { ref: searchParams.ref } : {}),
         },
     });
     if (linkHeader) headers.set('Link', linkHeader);

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

@@ -78,7 +78,11 @@ export const GET = async (request: NextRequest) => {
         page,
         perPage,
         totalCount,
-        extraParams: { sort, direction },
+        extraParams: {
+            sort,
+            direction,
+            ...(query ? { query } : {}),
+        },
     });
     if (linkHeader) headers.set('Link', linkHeader);
 

packages/web/src/features/search/gitApi.ts

@@ -76,7 +76,6 @@ export const listCommits = async ({
 
         try {
             const sharedOptions: Record<string, string | number | null> = {
-                [ref]: null,
                 ...(gitSince ? { '--since': gitSince } : {}),
                 ...(gitUntil ? { '--until': gitUntil } : {}),
                 ...(author ? {
@@ -91,6 +90,7 @@ export const listCommits = async ({
 
             // First, get the commits
             const log = await git.log({
+                [ref]: null,
                 maxCount,
                 ...(skip > 0 ? { '--skip': skip } : {}),
                 ...sharedOptions,