Address some more Coderabbit suggestions

Author: Gavin Williams

docs/docs/features/agents/review-agent.mdx

@@ -139,7 +139,7 @@ You can also trigger a review manually by commenting `/review` on any PR or MR.
 
 # Agent configs
 
-Agent configs let you customise how the review agent behaves per repository, connection, or your whole org. You can override the model, review command, auto-review behaviour, custom prompt, and context files — all without changing environment variables.
+Agent configs let you customise how the review agent behaves per repository, connection, or your whole org. You can override the model, review command, auto-review behaviour, custom prompt, and context files, all without changing environment variables.
 
 Configs are managed on the **Agents** page in the Sourcebot UI. Each config has a **scope**:
 
@@ -153,20 +153,20 @@ When a PR or MR arrives, Sourcebot selects the most specific matching config. If
 
 Each config can include a custom prompt. Two modes are available:
 
-- **Append** (default) — your instructions are added after the built-in review rules.
-- **Replace** — your instructions entirely replace the built-in rules. Use this when you want full control over what the agent looks for.
+- **Append** (default): your instructions are added after the built-in review rules.
+- **Replace**: your instructions entirely replace the built-in rules. Use this when you want full control over what the agent looks for.
 
 ## Context files
 
-You can configure one or more repository files to be fetched at review time and injected as additional context for the model. This is useful for encoding project-specific conventions that the model should be aware of when reviewing diffs — for example, preferred error handling patterns, style rules, or areas of the codebase that need extra scrutiny.
+You can configure one or more repository files to be fetched at review time and injected as additional context for the model. This is useful for encoding project-specific conventions that the model should be aware of when reviewing diffs (for example, preferred error handling patterns, style rules, or areas of the codebase that need extra scrutiny).
 
 Set **Context files** in the agent config form to a comma or space separated list of paths relative to the repository root (e.g. `AGENTS.md .sourcebot/review.md`). Files that do not exist in the repository are silently ignored. The files are fetched once per PR from the head commit and included in the context for every diff hunk.
 
 # Environment variable reference
 
 | Variable | Default | Description |
 |---|---|---|
-| `REVIEW_AGENT_AUTO_REVIEW_ENABLED` | `false` | Automatically review new and updated PRs/MRs. Can be overridden per agent config. |
-| `REVIEW_AGENT_REVIEW_COMMAND` | `review` | Comment command that triggers a manual review (without the `/`). Can be overridden per agent config. |
-| `REVIEW_AGENT_MODEL` | first configured model | `displayName` of the language model to use for reviews. Can be overridden per agent config. |
+| `REVIEW_AGENT_AUTO_REVIEW_ENABLED` | `false` | Automatically review new and updated PRs/MRs. You can override this in an agent config. |
+| `REVIEW_AGENT_REVIEW_COMMAND` | `review` | Comment command that triggers a manual review (without the `/`). You can override this in an agent config. |
+| `REVIEW_AGENT_MODEL` | first configured model | `displayName` of the language model to use for reviews. You can override this in an agent config. |
 | `REVIEW_AGENT_LOGGING_ENABLED` | unset | Write prompt and response logs to disk for debugging. |

…421203635_add_agent_config/migration.sql …422071731_add_agent_config/migration.sqlpackages/db/prisma/migrations/20260421203635_add_agent_config/migration.sql renamed to packages/db/prisma/migrations/20260422071731_add_agent_config/migration.sql packages/db/prisma/migrations/20260421203635_add_agent_config/migration.sql renamed to packages/db/prisma/migrations/20260422071731_add_agent_config/migration.sql

@@ -47,6 +47,12 @@ CREATE INDEX "AgentConfig_orgId_type_enabled_idx" ON "AgentConfig"("orgId", "typ
 -- CreateIndex
 CREATE UNIQUE INDEX "AgentConfig_orgId_name_key" ON "AgentConfig"("orgId", "name");
 
+-- CreateIndex
+CREATE INDEX "AgentConfigToRepo_repoId_idx" ON "AgentConfigToRepo"("repoId");
+
+-- CreateIndex
+CREATE INDEX "AgentConfigToConnection_connectionId_idx" ON "AgentConfigToConnection"("connectionId");
+
 -- AddForeignKey
 ALTER TABLE "AgentConfig" ADD CONSTRAINT "AgentConfig_orgId_fkey" FOREIGN KEY ("orgId") REFERENCES "Org"("id") ON DELETE CASCADE ON UPDATE CASCADE;
 

packages/db/prisma/schema.prisma

@@ -646,6 +646,7 @@ model AgentConfigToRepo {
   repoId        Int
 
   @@id([agentConfigId, repoId])
+  @@index([repoId])
 }
 
 /// Maps an AgentConfig to specific connections (used when scope = CONNECTION).
@@ -657,4 +658,5 @@ model AgentConfigToConnection {
   connectionId  Int
 
   @@id([agentConfigId, connectionId])
+  @@index([connectionId])
 }

packages/web/src/features/agents/review-agent/app.test.ts

@@ -53,12 +53,14 @@ describe('parseAgentConfigSettings', () => {
             autoReviewEnabled: false,
             reviewCommand: 'check',
             model: 'claude-sonnet-4-6',
+            contextFiles: 'AGENTS.md .sourcebot/review.md',
         });
 
         expect(result).toEqual({
             autoReviewEnabled: false,
             reviewCommand: 'check',
             model: 'claude-sonnet-4-6',
+            contextFiles: 'AGENTS.md .sourcebot/review.md',
         });
     });