cleanup

Author: brendan-kellam

CHANGELOG.md

@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 - Added ask sidebar to homepage. [#721](https://github.com/sourcebot-dev/sourcebot/pull/721)
+- Added endpoint for searching commit history for a git repository. [#625](https://github.com/sourcebot-dev/sourcebot/pull/625)
 
 ## [4.10.17] - 2026-01-23
 
@@ -162,7 +163,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [4.10.0] - 2025-11-24
 
 ### Added
-- Added temporal filtering to search and repository APIs with support for git branch/revision filtering and repository index date filtering (since/until parameters). Supports both ISO 8601 and relative date formats (e.g., "30 days ago", "last week").
 - Added support for streaming code search results. [#623](https://github.com/sourcebot-dev/sourcebot/pull/623)
 - Added buttons to toggle case sensitivity and regex patterns. [#623](https://github.com/sourcebot-dev/sourcebot/pull/623)
 - Added counts to members, requets, and invites tabs in the members settings. [#621](https://github.com/sourcebot-dev/sourcebot/pull/621)

packages/mcp/CHANGELOG.md

@@ -8,20 +8,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 ### Added
-- Added comprehensive relative date support for all temporal parameters (e.g., "30 days ago", "last week", "yesterday")
-- Added `search_commits` tool to search commits by actual commit time with full temporal filtering. Accepts both numeric database IDs (e.g., 123) and string repository names (e.g., "github.com/owner/repo") for the `repoId` parameter, allowing direct use of repository names from `list_repos` output
-- Added `since`/`until` parameters to `search_code` (filters by index time - when Sourcebot indexed the repo)
-- Added `gitRevision` parameter to `search_code`
-- Added `activeAfter`/`activeBefore` parameters to `list_repos` (filters by index time - when Sourcebot indexed the repo)
-- Added date range validation to prevent invalid date ranges (since > until)
-- Added 30-second timeout for git operations to handle large repositories
-- Added enhanced error messages for git operations (timeout, repository not found, invalid git repository, ambiguous arguments)
-- Added clarification that repositories must be cloned on Sourcebot server disk for `search_commits` to work
-- Added comprehensive temporal parameter documentation to README with clear distinction between index time and commit time filtering
-- Added comprehensive unit tests for date parsing utilities (90+ test cases)
-- Added unit tests for git commit search functionality with mocking
-- Added integration tests for temporal parameter validation
-- Added unit tests for repository identifier resolution (both string and number types)
+- Added `search_commits` tool to search a repos commit history. [#625](https://github.com/sourcebot-dev/sourcebot/pull/625)
+- Added `gitRevision` parameter to the `search_code` tool to allow for searching on different branches. [#625](https://github.com/sourcebot-dev/sourcebot/pull/625)
 
 ## [1.0.12] - 2026-01-13
 

packages/mcp/README.md

@@ -166,8 +166,6 @@ For a more detailed guide, checkout [the docs](https://docs.sourcebot.dev/docs/f
 
 Fetches code that matches the provided regex pattern in `query`.
 
-**Temporal Filtering**: Use `since` and `until` to filter by repository index time (when Sourcebot last indexed the repo). This is different from commit time. See `search_commits` for commit-time filtering.
-
 <details>
 <summary>Parameters</summary>
 
@@ -178,9 +176,6 @@ Fetches code that matches the provided regex pattern in `query`.
 | `filterByLanguages`   | no       | Restrict search to specific languages (GitHub linguist format, e.g., Python, JavaScript).                                         |
 | `caseSensitive`       | no       | Case sensitive search (default: false).                                                                                           |
 | `includeCodeSnippets` | no       | Include code snippets in results (default: false).                                                                                |
-| `gitRevision`         | no       | Git revision to search (e.g., 'main', 'develop', 'v1.0.0'). Defaults to HEAD.                                                    |
-| `since`               | no       | Only search repos indexed after this date. Supports ISO 8601 or relative (e.g., "30 days ago").                                   |
-| `until`               | no       | Only search repos indexed before this date. Supports ISO 8601 or relative (e.g., "yesterday").                                    |
 | `maxTokens`           | no       | Max tokens to return (default: env.DEFAULT_MINIMUM_TOKENS).                                                                       |
 </details>
 
@@ -189,18 +184,14 @@ Fetches code that matches the provided regex pattern in `query`.
 
 Lists repositories indexed by Sourcebot with optional filtering and pagination.
 
-**Temporal Filtering**: Use `activeAfter` and `activeBefore` to filter by repository index time (when Sourcebot last indexed the repo). This is the same filtering behavior as `search_code`'s `since`/`until` parameters.
-
 <details>
 <summary>Parameters</summary>
 
-| Name            | Required | Description                                                                                    |
-|:----------------|:---------|:-----------------------------------------------------------------------------------------------|
-| `query`         | no       | Filter repositories by name (case-insensitive).                                                |
-| `pageNumber`    | no       | Page number (1-indexed, default: 1).                                                           |
-| `limit`         | no       | Number of repositories per page (default: 50).                                                 |
-| `activeAfter`   | no       | Only return repos indexed after this date. Supports ISO 8601 or relative (e.g., "30 days ago"). |
-| `activeBefore`  | no       | Only return repos indexed before this date. Supports ISO 8601 or relative (e.g., "yesterday").  |
+| Name         | Required | Description                                                         |
+|:-------------|:---------|:--------------------------------------------------------------------|
+| `query`      | no       | Filter repositories by name (case-insensitive).                     |
+| `pageNumber` | no       | Page number (1-indexed, default: 1).                                |
+| `limit`      | no       | Number of repositories per page (default: 50).                      |
 
 </details>
 
@@ -219,11 +210,7 @@ Fetches the source code for a given file.
 
 ### search_commits
 
-Searches for commits in a specific repository based on actual commit time (NOT index time).
-
-**Requirements**: Repository must be cloned on the Sourcebot server disk. Sourcebot automatically clones repositories during indexing, but the cloning process may not be finished when this query is executed. Use `list_repos` first to get the repository ID.
-
-**Date Formats**: Supports ISO 8601 dates (e.g., "2024-01-01") and relative formats (e.g., "30 days ago", "last week", "yesterday").
+Searches for commits in a specific repository based on actual commit time.
 
 <details>
 <summary>Parameters</summary>
@@ -239,17 +226,6 @@ Searches for commits in a specific repository based on actual commit time (NOT i
 
 </details>
 
-## Date Format Examples
-
-All temporal parameters support:
-- **ISO 8601**: `"2024-01-01"`, `"2024-12-31T23:59:59Z"`
-- **Relative dates**: `"30 days ago"`, `"1 week ago"`, `"last month"`, `"yesterday"`
-
-**Important**: Different tools filter by different time dimensions:
-- `search_code` `since`/`until`: Filters by **index time** (when Sourcebot indexed the repo)
-- `list_repos` `activeAfter`/`activeBefore`: Filters by **index time** (when Sourcebot indexed the repo)
-- `search_commits` `since`/`until`: Filters by **commit time** (actual git commit dates)
-
 
 ## Supported Code Hosts
 Sourcebot supports the following code hosts:

packages/mcp/src/client.ts

@@ -21,9 +21,7 @@ export const search = async (request: SearchRequest): Promise<SearchResponse | S
 }
 
 export const listRepos = async (): Promise<ListRepositoriesResponse | ServiceError> => {
-    const url = new URL(`${env.SOURCEBOT_HOST}/api/repos`);
-
-    const result = await fetch(url.toString(), {
+    const result = await fetch(`${env.SOURCEBOT_HOST}/api/repos`, {
         method: 'GET',
         headers: {
             'Content-Type': 'application/json',

packages/mcp/src/schemas.ts

@@ -203,14 +203,12 @@ export const searchCommitsRequestSchema = z.object({
     maxCount: z.number().int().positive().max(500).optional(),
 });
 
-export const commitSchema = z.object({
+export const searchCommitsResponseSchema = z.array(z.object({
     hash: z.string(),
     date: z.string(),
     message: z.string(),
     refs: z.string(),
     body: z.string(),
     author_name: z.string(),
     author_email: z.string(),
-});
-
-export const searchCommitsResponseSchema = z.array(commitSchema);
+}));

packages/shared/src/utils.ts

@@ -87,10 +87,6 @@ export const getConfigSettings = async (configPath?: string): Promise<ConfigSett
     }
 }
 
-
-// @todo: this is duplicated from the `getRepoPath` function in the
-// backend's `utils.ts` file. Eventually we should move this to a shared
-// package.
 export const getRepoPath = (repo: Repo): { path: string, isReadOnly: boolean } => {
     // If we are dealing with a local repository, then use that as the path.
     // Mark as read-only since we aren't guaranteed to have write access to the local filesystem.

packages/web/src/actions.ts

@@ -312,12 +312,12 @@ export const createApiKey = async (name: string, domain: string): Promise<{ key:
     withAuth((userId) =>
         withOrgMembership(userId, domain, async ({ org, userRole }) => {
             if (env.EXPERIMENT_DISABLE_API_KEY_CREATION_FOR_NON_ADMIN_USERS === 'true' && userRole !== OrgRole.OWNER) {
-                logger.error(`API key creation is disabled for non-admin users. User ${userId} is not an owner.`);
-                return {
-                    statusCode: StatusCodes.FORBIDDEN,
-                    errorCode: ErrorCode.INSUFFICIENT_PERMISSIONS,
-                    message: "API key creation is disabled for non-admin users.",
-                } satisfies ServiceError;
+               logger.error(`API key creation is disabled for non-admin users. User ${userId} is not an owner.`);
+               return {
+                statusCode: StatusCodes.FORBIDDEN,
+                errorCode: ErrorCode.INSUFFICIENT_PERMISSIONS,
+                message: "API key creation is disabled for non-admin users.",
+               } satisfies ServiceError;
             }
 
             const existingApiKey = await prisma.apiKey.findFirst({
@@ -465,7 +465,7 @@ export const getRepos = async ({
     take,
 }: {
     where?: Prisma.RepoWhereInput,
-    take?: number,
+    take?: number
 } = {}) => sew(() =>
     withOptionalAuthV2(async ({ org, prisma }) => {
         const repos = await prisma.repo.findMany({

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

@@ -3,6 +3,7 @@ import { serviceErrorResponse } from "@/lib/serviceError";
 import { isServiceError } from "@/lib/utils";
 import { GetReposResponse } from "@/lib/types";
 
+
 export const GET = async () => {
     const response: GetReposResponse = await getRepos();
     if (isServiceError(response)) {

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

@@ -8,6 +8,7 @@ import { parseQuerySyntaxIntoIR } from './parser';
 import { SearchOptions } from "./types";
 import { createZoektSearchRequest, zoektSearch, zoektStreamSearch } from './zoektSearcher';
 
+
 type QueryStringSearchRequest = {
     queryType: 'string';
     query: string;
@@ -45,7 +46,6 @@ export const search = (request: SearchRequest) => sew(() =>
 
 export const streamSearch = (request: SearchRequest) => sew(() =>
     withOptionalAuthV2(async ({ prisma, user }) => {
-        // Get repos filtered by permissions (if enabled)
         const repoSearchScope = await getAccessibleRepoNamesForUser({ user, prisma });
 
         // If needed, parse the query syntax into the query intermediate representation.