Renamed code review feature to pull request assistant.

Author: srtab

README.md

@@ -151,7 +151,7 @@ DAIV's agent has access to a set of capabilities that make this possible:
 - [x] Add support to Agent Skills.
 - [ ] Add support to custom MCP servers.
 - [x] Add an evaluation system to measure the quality of DAIV's agents.
-- [ ] Add support to automated code review.
+- [ ] Add support to automated pull request review.
 - [ ] Create a frontend to DAIV initial setup and configuration, dashboard with some metrics, a chat interface to interact with DAIV...
 - [ ] Automate the onboarding of new projects into DAIV, by adding a `.daiv.yml` file to the repository.
 

daiv/codebase/clients/github/api/callbacks.py

@@ -128,7 +128,7 @@ def _is_merge_request_review(self) -> bool:
         Accept the webhook if the note is a merge request comment that mentions DAIV.
         """
         return bool(
-            self._repo_config.code_review.enabled
+            self._repo_config.pull_request_assistant.enabled
             and self.issue.is_pull_request()
             and self.issue.state == "open"
             # and not self.issue.draft

daiv/codebase/clients/gitlab/api/callbacks.py

@@ -145,7 +145,7 @@ def _is_merge_request_comment(self) -> bool:
         Accept the webhook if the note is a merge request comment that mentions DAIV.
         """
         return bool(
-            self._repo_config.code_review.enabled
+            self._repo_config.pull_request_assistant.enabled
             and self.object_attributes.type is None  # This is a comment note.
             and self.object_attributes.noteable_type == NoteableType.MERGE_REQUEST
             and self.object_attributes.action in [NoteAction.CREATE, NoteAction.UPDATE]

daiv/codebase/repo_config.py

@@ -35,12 +35,12 @@ class IssueAddressing(BaseModel):
     enabled: bool = Field(default=True, description="Enable issue addressing features.")
 
 
-class CodeReview(BaseModel):
+class PullRequestAssistant(BaseModel):
     """
-    Code review configuration.
+    Pull request assistant configuration.
     """
 
-    enabled: bool = Field(default=True, description="Enable code review features.")
+    enabled: bool = Field(default=True, description="Enable pull request assistant features.")
 
 
 class SlashCommands(BaseModel):
@@ -210,7 +210,11 @@ class RepositoryConfig(BaseModel):
     slash_commands: SlashCommands = Field(
         default_factory=SlashCommands, description="Configure slash command features."
     )
-    code_review: CodeReview = Field(default_factory=CodeReview, description="Configure code review features.")
+    pull_request_assistant: PullRequestAssistant = Field(
+        default_factory=PullRequestAssistant,
+        validation_alias=AliasChoices("pull_request_assistant", "code_review"),
+        description="Configure pull request assistant features.",
+    )
     issue_addressing: IssueAddressing = Field(
         default_factory=IssueAddressing, description="Configure issue addressing features."
     )

docs/customization/repository-config.md

@@ -19,7 +19,7 @@ omit_content_patterns:
 issue_addressing:
   enabled: true
 
-code_review:
+pull_request_assistant:
   enabled: true
 
 slash_commands:
@@ -75,7 +75,7 @@ All features are enabled by default. Disable any you don't need:
 issue_addressing:
   enabled: false
 
-code_review:
+pull_request_assistant:
   enabled: false
 
 slash_commands:
@@ -85,7 +85,7 @@ slash_commands:
 | Section | Option | Default | Description |
 |---------|--------|---------|-------------|
 | `issue_addressing` | `enabled` | `true` | [Issue Addressing](../features/issue-addressing.md) |
-| `code_review` | `enabled` | `true` | [Pull Request Assistant](../features/pull-request-assistant.md) |
+| `pull_request_assistant` | `enabled` | `true` | [Pull Request Assistant](../features/pull-request-assistant.md) |
 | `slash_commands` | `enabled` | `true` | [Slash Commands & Skills](../features/slash-commands.md) |
 
 ## Sandbox

docs/features/pull-request-assistant.md

@@ -75,9 +75,9 @@ DAIV maintains context across multiple interactions on the same pull request. Yo
 
 ## Configuration
 
-Code review is enabled by default. To disable it, add the following to your `.daiv.yml`:
+The pull request assistant is enabled by default. To disable it, add the following to your `.daiv.yml`:
 
 ```yaml
-code_review:
+pull_request_assistant:
   enabled: false
 ```

tests/unit_tests/codebase/test_config.py

@@ -14,7 +14,7 @@ def test_get_config_from_cache(self, mock_cache):
         repo_id = "test_repo"
         cached_config = {
             "default_branch": "main",
-            "code_review": {"enabled": True},
+            "pull_request_assistant": {"enabled": True},
             "issue_addressing": {"enabled": True},
             "slash_commands": {"enabled": True},
         }
@@ -23,7 +23,7 @@ def test_get_config_from_cache(self, mock_cache):
         config = RepositoryConfig.get_config(repo_id)
 
         assert config.default_branch == "main"
-        assert config.code_review.enabled is True
+        assert config.pull_request_assistant.enabled is True
         assert config.issue_addressing.enabled is True
         assert config.slash_commands.enabled is True
         mock_cache.get.assert_called_once_with(f"{CONFIGURATION_CACHE_KEY_PREFIX}{repo_id}")
@@ -35,7 +35,7 @@ def test_get_config_from_repo(self, mock_cache, mock_repo_client):
         mock_repo_client.get_repository.return_value.default_branch = "main"
         mock_repo_client.get_repository_file.return_value = """
         default_branch: main
-        code_review:
+        pull_request_assistant:
           enabled: true
         issue_addressing:
           enabled: true
@@ -46,7 +46,7 @@ def test_get_config_from_repo(self, mock_cache, mock_repo_client):
         config = RepositoryConfig.get_config(repo_id)
 
         assert config.default_branch == "main"
-        assert config.code_review.enabled is True
+        assert config.pull_request_assistant.enabled is True
         assert config.issue_addressing.enabled is True
         assert config.slash_commands.enabled is True
         mock_cache.set.assert_called_once_with(
@@ -63,7 +63,7 @@ def test_get_config_with_default_values(self, mock_cache, mock_repo_client):
         config = RepositoryConfig.get_config(repo_id)
 
         assert config.default_branch == "main"
-        assert config.code_review.enabled is True
+        assert config.pull_request_assistant.enabled is True
         assert config.issue_addressing.enabled is True
         assert config.slash_commands.enabled is True
         mock_cache.set.assert_called_once_with(
@@ -86,7 +86,7 @@ def test_get_config_with_invalid_yaml(self, mock_cache, mock_repo_client):
         config = RepositoryConfig.get_config(repo_id)
 
         assert config.default_branch == "main"
-        assert config.code_review.enabled is True
+        assert config.pull_request_assistant.enabled is True
         assert config.issue_addressing.enabled is True
         assert config.slash_commands.enabled is True
         mock_cache.set.assert_called_once_with(
@@ -105,7 +105,7 @@ def test_get_config_with_partial_yaml(self, mock_cache, mock_repo_client):
         config = RepositoryConfig.get_config(repo_id)
 
         assert config.default_branch == "main"
-        assert config.code_review.enabled is True
+        assert config.pull_request_assistant.enabled is True
         assert config.issue_addressing.enabled is True
         assert config.slash_commands.enabled is True
         mock_cache.set.assert_called_once_with(