xai-review 0.25.0__py3-none-any.whl → 0.27.0__py3-none-any.whl
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Potentially problematic release.
This version of xai-review might be problematic. Click here for more details.
- ai_review/clients/bitbucket/__init__.py +0 -0
- ai_review/clients/bitbucket/client.py +31 -0
- ai_review/clients/bitbucket/pr/__init__.py +0 -0
- ai_review/clients/bitbucket/pr/client.py +141 -0
- ai_review/clients/bitbucket/pr/schema/__init__.py +0 -0
- ai_review/clients/bitbucket/pr/schema/comments.py +49 -0
- ai_review/clients/bitbucket/pr/schema/files.py +30 -0
- ai_review/clients/bitbucket/pr/schema/pull_request.py +38 -0
- ai_review/clients/bitbucket/pr/types.py +44 -0
- ai_review/clients/bitbucket/tools.py +6 -0
- ai_review/clients/github/pr/client.py +66 -12
- ai_review/clients/github/pr/schema/comments.py +2 -1
- ai_review/clients/github/pr/schema/files.py +2 -1
- ai_review/clients/github/pr/schema/reviews.py +2 -1
- ai_review/clients/github/tools.py +6 -0
- ai_review/clients/gitlab/mr/client.py +35 -6
- ai_review/clients/gitlab/mr/schema/discussions.py +2 -1
- ai_review/clients/gitlab/mr/schema/notes.py +2 -1
- ai_review/clients/gitlab/tools.py +5 -0
- ai_review/libs/config/vcs/base.py +13 -1
- ai_review/libs/config/vcs/bitbucket.py +13 -0
- ai_review/libs/config/vcs/pagination.py +6 -0
- ai_review/libs/constants/vcs_provider.py +1 -0
- ai_review/libs/http/paginate.py +43 -0
- ai_review/services/vcs/bitbucket/__init__.py +0 -0
- ai_review/services/vcs/bitbucket/client.py +185 -0
- ai_review/services/vcs/factory.py +3 -0
- ai_review/tests/fixtures/clients/bitbucket.py +204 -0
- ai_review/tests/suites/clients/bitbucket/__init__.py +0 -0
- ai_review/tests/suites/clients/bitbucket/test_client.py +14 -0
- ai_review/tests/suites/clients/bitbucket/test_tools.py +31 -0
- ai_review/tests/suites/clients/github/test_tools.py +31 -0
- ai_review/tests/suites/clients/gitlab/test_tools.py +26 -0
- ai_review/tests/suites/libs/http/__init__.py +0 -0
- ai_review/tests/suites/libs/http/test_paginate.py +95 -0
- ai_review/tests/suites/services/vcs/bitbucket/__init__.py +0 -0
- ai_review/tests/suites/services/vcs/bitbucket/test_service.py +117 -0
- ai_review/tests/suites/services/vcs/test_factory.py +8 -1
- {xai_review-0.25.0.dist-info → xai_review-0.27.0.dist-info}/METADATA +4 -4
- {xai_review-0.25.0.dist-info → xai_review-0.27.0.dist-info}/RECORD +44 -17
- {xai_review-0.25.0.dist-info → xai_review-0.27.0.dist-info}/WHEEL +0 -0
- {xai_review-0.25.0.dist-info → xai_review-0.27.0.dist-info}/entry_points.txt +0 -0
- {xai_review-0.25.0.dist-info → xai_review-0.27.0.dist-info}/licenses/LICENSE +0 -0
- {xai_review-0.25.0.dist-info → xai_review-0.27.0.dist-info}/top_level.txt +0 -0
|
File without changes
|
|
@@ -0,0 +1,31 @@
|
|
|
1
|
+
from ai_review.clients.bitbucket.pr.client import BitbucketPullRequestsHTTPClient
|
|
2
|
+
from httpx import AsyncClient, AsyncHTTPTransport
|
|
3
|
+
|
|
4
|
+
from ai_review.config import settings
|
|
5
|
+
from ai_review.libs.http.event_hooks.logger import LoggerEventHook
|
|
6
|
+
from ai_review.libs.http.transports.retry import RetryTransport
|
|
7
|
+
from ai_review.libs.logger import get_logger
|
|
8
|
+
|
|
9
|
+
|
|
10
|
+
class BitbucketHTTPClient:
|
|
11
|
+
def __init__(self, client: AsyncClient):
|
|
12
|
+
self.pr = BitbucketPullRequestsHTTPClient(client)
|
|
13
|
+
|
|
14
|
+
|
|
15
|
+
def get_bitbucket_http_client() -> BitbucketHTTPClient:
|
|
16
|
+
logger = get_logger("BITBUCKET_HTTP_CLIENT")
|
|
17
|
+
logger_event_hook = LoggerEventHook(logger=logger)
|
|
18
|
+
retry_transport = RetryTransport(logger=logger, transport=AsyncHTTPTransport())
|
|
19
|
+
|
|
20
|
+
client = AsyncClient(
|
|
21
|
+
timeout=settings.llm.http_client.timeout,
|
|
22
|
+
headers={"Authorization": f"Bearer {settings.vcs.http_client.api_token_value}"},
|
|
23
|
+
base_url=settings.vcs.http_client.api_url_value,
|
|
24
|
+
transport=retry_transport,
|
|
25
|
+
event_hooks={
|
|
26
|
+
"request": [logger_event_hook.request],
|
|
27
|
+
"response": [logger_event_hook.response],
|
|
28
|
+
}
|
|
29
|
+
)
|
|
30
|
+
|
|
31
|
+
return BitbucketHTTPClient(client=client)
|
|
File without changes
|
|
@@ -0,0 +1,141 @@
|
|
|
1
|
+
from httpx import Response, QueryParams
|
|
2
|
+
|
|
3
|
+
from ai_review.clients.bitbucket.pr.schema.comments import (
|
|
4
|
+
BitbucketPRCommentSchema,
|
|
5
|
+
BitbucketGetPRCommentsQuerySchema,
|
|
6
|
+
BitbucketGetPRCommentsResponseSchema,
|
|
7
|
+
BitbucketCreatePRCommentRequestSchema,
|
|
8
|
+
BitbucketCreatePRCommentResponseSchema,
|
|
9
|
+
)
|
|
10
|
+
from ai_review.clients.bitbucket.pr.schema.files import (
|
|
11
|
+
BitbucketPRFileSchema,
|
|
12
|
+
BitbucketGetPRFilesQuerySchema,
|
|
13
|
+
BitbucketGetPRFilesResponseSchema,
|
|
14
|
+
)
|
|
15
|
+
from ai_review.clients.bitbucket.pr.schema.pull_request import BitbucketGetPRResponseSchema
|
|
16
|
+
from ai_review.clients.bitbucket.pr.types import BitbucketPullRequestsHTTPClientProtocol
|
|
17
|
+
from ai_review.clients.bitbucket.tools import bitbucket_has_next_page
|
|
18
|
+
from ai_review.config import settings
|
|
19
|
+
from ai_review.libs.http.client import HTTPClient
|
|
20
|
+
from ai_review.libs.http.handlers import handle_http_error, HTTPClientError
|
|
21
|
+
from ai_review.libs.http.paginate import paginate
|
|
22
|
+
|
|
23
|
+
|
|
24
|
+
class BitbucketPullRequestsHTTPClientError(HTTPClientError):
|
|
25
|
+
pass
|
|
26
|
+
|
|
27
|
+
|
|
28
|
+
class BitbucketPullRequestsHTTPClient(HTTPClient, BitbucketPullRequestsHTTPClientProtocol):
|
|
29
|
+
@handle_http_error(client="BitbucketPullRequestsHTTPClient", exception=BitbucketPullRequestsHTTPClientError)
|
|
30
|
+
async def get_pull_request_api(self, workspace: str, repo_slug: str, pull_request_id: str) -> Response:
|
|
31
|
+
return await self.get(f"/repositories/{workspace}/{repo_slug}/pullrequests/{pull_request_id}")
|
|
32
|
+
|
|
33
|
+
@handle_http_error(client="BitbucketPullRequestsHTTPClient", exception=BitbucketPullRequestsHTTPClientError)
|
|
34
|
+
async def get_diffstat_api(
|
|
35
|
+
self,
|
|
36
|
+
workspace: str,
|
|
37
|
+
repo_slug: str,
|
|
38
|
+
pull_request_id: str,
|
|
39
|
+
query: BitbucketGetPRFilesQuerySchema,
|
|
40
|
+
) -> Response:
|
|
41
|
+
return await self.get(
|
|
42
|
+
f"/repositories/{workspace}/{repo_slug}/pullrequests/{pull_request_id}/diffstat",
|
|
43
|
+
query=QueryParams(**query.model_dump(by_alias=True)),
|
|
44
|
+
)
|
|
45
|
+
|
|
46
|
+
@handle_http_error(client="BitbucketPullRequestsHTTPClient", exception=BitbucketPullRequestsHTTPClientError)
|
|
47
|
+
async def get_comments_api(
|
|
48
|
+
self,
|
|
49
|
+
workspace: str,
|
|
50
|
+
repo_slug: str,
|
|
51
|
+
pull_request_id: str,
|
|
52
|
+
query: BitbucketGetPRCommentsQuerySchema,
|
|
53
|
+
) -> Response:
|
|
54
|
+
return await self.get(
|
|
55
|
+
f"/repositories/{workspace}/{repo_slug}/pullrequests/{pull_request_id}/comments",
|
|
56
|
+
query=QueryParams(**query.model_dump(by_alias=True)),
|
|
57
|
+
)
|
|
58
|
+
|
|
59
|
+
@handle_http_error(client="BitbucketPullRequestsHTTPClient", exception=BitbucketPullRequestsHTTPClientError)
|
|
60
|
+
async def create_comment_api(
|
|
61
|
+
self,
|
|
62
|
+
workspace: str,
|
|
63
|
+
repo_slug: str,
|
|
64
|
+
pull_request_id: str,
|
|
65
|
+
request: BitbucketCreatePRCommentRequestSchema,
|
|
66
|
+
) -> Response:
|
|
67
|
+
return await self.post(
|
|
68
|
+
f"/repositories/{workspace}/{repo_slug}/pullrequests/{pull_request_id}/comments",
|
|
69
|
+
json=request.model_dump(by_alias=True),
|
|
70
|
+
)
|
|
71
|
+
|
|
72
|
+
async def get_pull_request(
|
|
73
|
+
self,
|
|
74
|
+
workspace: str,
|
|
75
|
+
repo_slug: str,
|
|
76
|
+
pull_request_id: str
|
|
77
|
+
) -> BitbucketGetPRResponseSchema:
|
|
78
|
+
resp = await self.get_pull_request_api(workspace, repo_slug, pull_request_id)
|
|
79
|
+
return BitbucketGetPRResponseSchema.model_validate_json(resp.text)
|
|
80
|
+
|
|
81
|
+
async def get_files(
|
|
82
|
+
self,
|
|
83
|
+
workspace: str,
|
|
84
|
+
repo_slug: str,
|
|
85
|
+
pull_request_id: str
|
|
86
|
+
) -> BitbucketGetPRFilesResponseSchema:
|
|
87
|
+
async def fetch_page(page: int) -> Response:
|
|
88
|
+
query = BitbucketGetPRFilesQuerySchema(page=page, page_len=settings.vcs.pagination.per_page)
|
|
89
|
+
return await self.get_diffstat_api(workspace, repo_slug, pull_request_id, query)
|
|
90
|
+
|
|
91
|
+
def extract_items(response: Response) -> list[BitbucketPRFileSchema]:
|
|
92
|
+
result = BitbucketGetPRFilesResponseSchema.model_validate_json(response.text)
|
|
93
|
+
return result.values
|
|
94
|
+
|
|
95
|
+
items = await paginate(
|
|
96
|
+
max_pages=settings.vcs.pagination.max_pages,
|
|
97
|
+
fetch_page=fetch_page,
|
|
98
|
+
extract_items=extract_items,
|
|
99
|
+
has_next_page=bitbucket_has_next_page
|
|
100
|
+
)
|
|
101
|
+
return BitbucketGetPRFilesResponseSchema(
|
|
102
|
+
size=len(items),
|
|
103
|
+
values=items,
|
|
104
|
+
page_len=settings.vcs.pagination.per_page
|
|
105
|
+
)
|
|
106
|
+
|
|
107
|
+
async def get_comments(
|
|
108
|
+
self,
|
|
109
|
+
workspace: str,
|
|
110
|
+
repo_slug: str,
|
|
111
|
+
pull_request_id: str
|
|
112
|
+
) -> BitbucketGetPRCommentsResponseSchema:
|
|
113
|
+
async def fetch_page(page: int) -> Response:
|
|
114
|
+
query = BitbucketGetPRCommentsQuerySchema(page=page, page_len=settings.vcs.pagination.per_page)
|
|
115
|
+
return await self.get_comments_api(workspace, repo_slug, pull_request_id, query)
|
|
116
|
+
|
|
117
|
+
def extract_items(response: Response) -> list[BitbucketPRCommentSchema]:
|
|
118
|
+
result = BitbucketGetPRCommentsResponseSchema.model_validate_json(response.text)
|
|
119
|
+
return result.values
|
|
120
|
+
|
|
121
|
+
items = await paginate(
|
|
122
|
+
max_pages=settings.vcs.pagination.max_pages,
|
|
123
|
+
fetch_page=fetch_page,
|
|
124
|
+
extract_items=extract_items,
|
|
125
|
+
has_next_page=bitbucket_has_next_page
|
|
126
|
+
)
|
|
127
|
+
return BitbucketGetPRCommentsResponseSchema(
|
|
128
|
+
size=len(items),
|
|
129
|
+
values=items,
|
|
130
|
+
page_len=settings.vcs.pagination.per_page
|
|
131
|
+
)
|
|
132
|
+
|
|
133
|
+
async def create_comment(
|
|
134
|
+
self,
|
|
135
|
+
workspace: str,
|
|
136
|
+
repo_slug: str,
|
|
137
|
+
pull_request_id: str,
|
|
138
|
+
request: BitbucketCreatePRCommentRequestSchema
|
|
139
|
+
) -> BitbucketCreatePRCommentResponseSchema:
|
|
140
|
+
response = await self.create_comment_api(workspace, repo_slug, pull_request_id, request)
|
|
141
|
+
return BitbucketCreatePRCommentResponseSchema.model_validate_json(response.text)
|
|
File without changes
|
|
@@ -0,0 +1,49 @@
|
|
|
1
|
+
from pydantic import BaseModel, Field, ConfigDict
|
|
2
|
+
|
|
3
|
+
|
|
4
|
+
class BitbucketCommentContentSchema(BaseModel):
|
|
5
|
+
raw: str
|
|
6
|
+
html: str | None = None
|
|
7
|
+
markup: str | None = None
|
|
8
|
+
|
|
9
|
+
|
|
10
|
+
class BitbucketCommentInlineSchema(BaseModel):
|
|
11
|
+
model_config = ConfigDict(populate_by_name=True)
|
|
12
|
+
|
|
13
|
+
path: str
|
|
14
|
+
to_line: int | None = Field(alias="to", default=None)
|
|
15
|
+
from_line: int | None = Field(alias="from", default=None)
|
|
16
|
+
|
|
17
|
+
|
|
18
|
+
class BitbucketPRCommentSchema(BaseModel):
|
|
19
|
+
id: int
|
|
20
|
+
inline: BitbucketCommentInlineSchema | None = None
|
|
21
|
+
content: BitbucketCommentContentSchema
|
|
22
|
+
|
|
23
|
+
|
|
24
|
+
class BitbucketGetPRCommentsQuerySchema(BaseModel):
|
|
25
|
+
model_config = ConfigDict(populate_by_name=True)
|
|
26
|
+
|
|
27
|
+
page: int = 1
|
|
28
|
+
page_len: int = Field(alias="pagelen", default=100)
|
|
29
|
+
|
|
30
|
+
|
|
31
|
+
class BitbucketGetPRCommentsResponseSchema(BaseModel):
|
|
32
|
+
model_config = ConfigDict(populate_by_name=True)
|
|
33
|
+
|
|
34
|
+
size: int
|
|
35
|
+
page: int | None = None
|
|
36
|
+
next: str | None = None
|
|
37
|
+
values: list[BitbucketPRCommentSchema]
|
|
38
|
+
page_len: int = Field(alias="pagelen")
|
|
39
|
+
|
|
40
|
+
|
|
41
|
+
class BitbucketCreatePRCommentRequestSchema(BaseModel):
|
|
42
|
+
inline: BitbucketCommentInlineSchema | None = None
|
|
43
|
+
content: BitbucketCommentContentSchema
|
|
44
|
+
|
|
45
|
+
|
|
46
|
+
class BitbucketCreatePRCommentResponseSchema(BaseModel):
|
|
47
|
+
id: int
|
|
48
|
+
inline: BitbucketCommentInlineSchema | None = None
|
|
49
|
+
content: BitbucketCommentContentSchema
|
|
@@ -0,0 +1,30 @@
|
|
|
1
|
+
from pydantic import BaseModel, Field, ConfigDict
|
|
2
|
+
|
|
3
|
+
|
|
4
|
+
class BitbucketPRFilePathSchema(BaseModel):
|
|
5
|
+
path: str
|
|
6
|
+
|
|
7
|
+
|
|
8
|
+
class BitbucketPRFileSchema(BaseModel):
|
|
9
|
+
new: BitbucketPRFilePathSchema | None = None
|
|
10
|
+
old: BitbucketPRFilePathSchema | None = None
|
|
11
|
+
status: str
|
|
12
|
+
lines_added: int
|
|
13
|
+
lines_removed: int
|
|
14
|
+
|
|
15
|
+
|
|
16
|
+
class BitbucketGetPRFilesQuerySchema(BaseModel):
|
|
17
|
+
model_config = ConfigDict(populate_by_name=True)
|
|
18
|
+
|
|
19
|
+
page: int = 1
|
|
20
|
+
page_len: int = Field(alias="pagelen", default=100)
|
|
21
|
+
|
|
22
|
+
|
|
23
|
+
class BitbucketGetPRFilesResponseSchema(BaseModel):
|
|
24
|
+
model_config = ConfigDict(populate_by_name=True)
|
|
25
|
+
|
|
26
|
+
size: int
|
|
27
|
+
page: int | None = None
|
|
28
|
+
next: str | None = None
|
|
29
|
+
values: list[BitbucketPRFileSchema]
|
|
30
|
+
page_len: int = Field(alias="pagelen")
|
|
@@ -0,0 +1,38 @@
|
|
|
1
|
+
from pydantic import BaseModel, Field
|
|
2
|
+
|
|
3
|
+
|
|
4
|
+
class BitbucketUserSchema(BaseModel):
|
|
5
|
+
uuid: str
|
|
6
|
+
nickname: str
|
|
7
|
+
display_name: str
|
|
8
|
+
|
|
9
|
+
|
|
10
|
+
class BitbucketBranchSchema(BaseModel):
|
|
11
|
+
name: str
|
|
12
|
+
|
|
13
|
+
|
|
14
|
+
class BitbucketCommitSchema(BaseModel):
|
|
15
|
+
hash: str
|
|
16
|
+
|
|
17
|
+
|
|
18
|
+
class BitbucketRepositorySchema(BaseModel):
|
|
19
|
+
uuid: str
|
|
20
|
+
full_name: str
|
|
21
|
+
|
|
22
|
+
|
|
23
|
+
class BitbucketPRLocationSchema(BaseModel):
|
|
24
|
+
branch: BitbucketBranchSchema
|
|
25
|
+
commit: BitbucketCommitSchema
|
|
26
|
+
repository: BitbucketRepositorySchema
|
|
27
|
+
|
|
28
|
+
|
|
29
|
+
class BitbucketGetPRResponseSchema(BaseModel):
|
|
30
|
+
id: int
|
|
31
|
+
title: str
|
|
32
|
+
description: str | None = None
|
|
33
|
+
state: str
|
|
34
|
+
author: BitbucketUserSchema
|
|
35
|
+
source: BitbucketPRLocationSchema
|
|
36
|
+
destination: BitbucketPRLocationSchema
|
|
37
|
+
reviewers: list[BitbucketUserSchema] = Field(default_factory=list)
|
|
38
|
+
participants: list[BitbucketUserSchema] = Field(default_factory=list)
|
|
@@ -0,0 +1,44 @@
|
|
|
1
|
+
from typing import Protocol
|
|
2
|
+
|
|
3
|
+
from ai_review.clients.bitbucket.pr.schema.comments import (
|
|
4
|
+
BitbucketGetPRCommentsResponseSchema,
|
|
5
|
+
BitbucketCreatePRCommentRequestSchema,
|
|
6
|
+
BitbucketCreatePRCommentResponseSchema,
|
|
7
|
+
)
|
|
8
|
+
from ai_review.clients.bitbucket.pr.schema.files import BitbucketGetPRFilesResponseSchema
|
|
9
|
+
from ai_review.clients.bitbucket.pr.schema.pull_request import BitbucketGetPRResponseSchema
|
|
10
|
+
|
|
11
|
+
|
|
12
|
+
class BitbucketPullRequestsHTTPClientProtocol(Protocol):
|
|
13
|
+
async def get_pull_request(
|
|
14
|
+
self,
|
|
15
|
+
workspace: str,
|
|
16
|
+
repo_slug: str,
|
|
17
|
+
pull_request_id: str
|
|
18
|
+
) -> BitbucketGetPRResponseSchema:
|
|
19
|
+
...
|
|
20
|
+
|
|
21
|
+
async def get_files(
|
|
22
|
+
self,
|
|
23
|
+
workspace: str,
|
|
24
|
+
repo_slug: str,
|
|
25
|
+
pull_request_id: str
|
|
26
|
+
) -> BitbucketGetPRFilesResponseSchema:
|
|
27
|
+
...
|
|
28
|
+
|
|
29
|
+
async def get_comments(
|
|
30
|
+
self,
|
|
31
|
+
workspace: str,
|
|
32
|
+
repo_slug: str,
|
|
33
|
+
pull_request_id: str
|
|
34
|
+
) -> BitbucketGetPRCommentsResponseSchema:
|
|
35
|
+
...
|
|
36
|
+
|
|
37
|
+
async def create_comment(
|
|
38
|
+
self,
|
|
39
|
+
workspace: str,
|
|
40
|
+
repo_slug: str,
|
|
41
|
+
pull_request_id: str,
|
|
42
|
+
request: BitbucketCreatePRCommentRequestSchema,
|
|
43
|
+
) -> BitbucketCreatePRCommentResponseSchema:
|
|
44
|
+
...
|
|
@@ -1,6 +1,7 @@
|
|
|
1
1
|
from httpx import Response, QueryParams
|
|
2
2
|
|
|
3
3
|
from ai_review.clients.github.pr.schema.comments import (
|
|
4
|
+
GitHubPRCommentSchema,
|
|
4
5
|
GitHubGetPRCommentsQuerySchema,
|
|
5
6
|
GitHubGetPRCommentsResponseSchema,
|
|
6
7
|
GitHubCreateIssueCommentRequestSchema,
|
|
@@ -9,17 +10,22 @@ from ai_review.clients.github.pr.schema.comments import (
|
|
|
9
10
|
GitHubCreateReviewCommentResponseSchema
|
|
10
11
|
)
|
|
11
12
|
from ai_review.clients.github.pr.schema.files import (
|
|
13
|
+
GitHubPRFileSchema,
|
|
12
14
|
GitHubGetPRFilesQuerySchema,
|
|
13
15
|
GitHubGetPRFilesResponseSchema
|
|
14
16
|
)
|
|
15
17
|
from ai_review.clients.github.pr.schema.pull_request import GitHubGetPRResponseSchema
|
|
16
18
|
from ai_review.clients.github.pr.schema.reviews import (
|
|
19
|
+
GitHubPRReviewSchema,
|
|
17
20
|
GitHubGetPRReviewsQuerySchema,
|
|
18
21
|
GitHubGetPRReviewsResponseSchema
|
|
19
22
|
)
|
|
20
23
|
from ai_review.clients.github.pr.types import GitHubPullRequestsHTTPClientProtocol
|
|
24
|
+
from ai_review.clients.github.tools import github_has_next_page
|
|
25
|
+
from ai_review.config import settings
|
|
21
26
|
from ai_review.libs.http.client import HTTPClient
|
|
22
27
|
from ai_review.libs.http.handlers import HTTPClientError, handle_http_error
|
|
28
|
+
from ai_review.libs.http.paginate import paginate
|
|
23
29
|
|
|
24
30
|
|
|
25
31
|
class GitHubPullRequestsHTTPClientError(HTTPClientError):
|
|
@@ -114,24 +120,72 @@ class GitHubPullRequestsHTTPClient(HTTPClient, GitHubPullRequestsHTTPClientProto
|
|
|
114
120
|
return GitHubGetPRResponseSchema.model_validate_json(response.text)
|
|
115
121
|
|
|
116
122
|
async def get_files(self, owner: str, repo: str, pull_number: str) -> GitHubGetPRFilesResponseSchema:
|
|
117
|
-
|
|
118
|
-
|
|
119
|
-
|
|
123
|
+
async def fetch_page(page: int) -> Response:
|
|
124
|
+
query = GitHubGetPRFilesQuerySchema(page=page, per_page=settings.vcs.pagination.per_page)
|
|
125
|
+
return await self.get_files_api(owner, repo, pull_number, query)
|
|
126
|
+
|
|
127
|
+
def extract_items(response: Response) -> list[GitHubPRFileSchema]:
|
|
128
|
+
result = GitHubGetPRFilesResponseSchema.model_validate_json(response.text)
|
|
129
|
+
return result.root
|
|
130
|
+
|
|
131
|
+
items = await paginate(
|
|
132
|
+
max_pages=settings.vcs.pagination.max_pages,
|
|
133
|
+
fetch_page=fetch_page,
|
|
134
|
+
extract_items=extract_items,
|
|
135
|
+
has_next_page=github_has_next_page
|
|
136
|
+
)
|
|
137
|
+
return GitHubGetPRFilesResponseSchema(root=items)
|
|
120
138
|
|
|
121
139
|
async def get_issue_comments(self, owner: str, repo: str, issue_number: str) -> GitHubGetPRCommentsResponseSchema:
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
140
|
+
async def fetch_page(page: int) -> Response:
|
|
141
|
+
query = GitHubGetPRCommentsQuerySchema(page=page, per_page=settings.vcs.pagination.per_page)
|
|
142
|
+
return await self.get_issue_comments_api(owner, repo, issue_number, query)
|
|
143
|
+
|
|
144
|
+
def extract_items(response: Response) -> list[GitHubPRCommentSchema]:
|
|
145
|
+
result = GitHubGetPRCommentsResponseSchema.model_validate_json(response.text)
|
|
146
|
+
return result.root
|
|
147
|
+
|
|
148
|
+
items = await paginate(
|
|
149
|
+
max_pages=settings.vcs.pagination.max_pages,
|
|
150
|
+
fetch_page=fetch_page,
|
|
151
|
+
extract_items=extract_items,
|
|
152
|
+
has_next_page=github_has_next_page
|
|
153
|
+
)
|
|
154
|
+
return GitHubGetPRCommentsResponseSchema(root=items)
|
|
125
155
|
|
|
126
156
|
async def get_review_comments(self, owner: str, repo: str, pull_number: str) -> GitHubGetPRCommentsResponseSchema:
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
|
|
157
|
+
async def fetch_page(page: int) -> Response:
|
|
158
|
+
query = GitHubGetPRCommentsQuerySchema(page=page, per_page=settings.vcs.pagination.per_page)
|
|
159
|
+
return await self.get_review_comments_api(owner, repo, pull_number, query)
|
|
160
|
+
|
|
161
|
+
def extract_items(response: Response) -> list[GitHubPRCommentSchema]:
|
|
162
|
+
result = GitHubGetPRCommentsResponseSchema.model_validate_json(response.text)
|
|
163
|
+
return result.root
|
|
164
|
+
|
|
165
|
+
items = await paginate(
|
|
166
|
+
max_pages=settings.vcs.pagination.max_pages,
|
|
167
|
+
fetch_page=fetch_page,
|
|
168
|
+
extract_items=extract_items,
|
|
169
|
+
has_next_page=github_has_next_page
|
|
170
|
+
)
|
|
171
|
+
return GitHubGetPRCommentsResponseSchema(root=items)
|
|
130
172
|
|
|
131
173
|
async def get_reviews(self, owner: str, repo: str, pull_number: str) -> GitHubGetPRReviewsResponseSchema:
|
|
132
|
-
|
|
133
|
-
|
|
134
|
-
|
|
174
|
+
async def fetch_page(page: int) -> Response:
|
|
175
|
+
query = GitHubGetPRReviewsQuerySchema(page=page, per_page=settings.vcs.pagination.per_page)
|
|
176
|
+
return await self.get_reviews_api(owner, repo, pull_number, query)
|
|
177
|
+
|
|
178
|
+
def extract_items(response: Response) -> list[GitHubPRReviewSchema]:
|
|
179
|
+
result = GitHubGetPRReviewsResponseSchema.model_validate_json(response.text)
|
|
180
|
+
return result.root
|
|
181
|
+
|
|
182
|
+
items = await paginate(
|
|
183
|
+
max_pages=settings.vcs.pagination.max_pages,
|
|
184
|
+
fetch_page=fetch_page,
|
|
185
|
+
extract_items=extract_items,
|
|
186
|
+
has_next_page=github_has_next_page
|
|
187
|
+
)
|
|
188
|
+
return GitHubGetPRReviewsResponseSchema(root=items)
|
|
135
189
|
|
|
136
190
|
async def create_review_comment(
|
|
137
191
|
self,
|
|
@@ -2,20 +2,25 @@ from httpx import Response, QueryParams
|
|
|
2
2
|
|
|
3
3
|
from ai_review.clients.gitlab.mr.schema.changes import GitLabGetMRChangesResponseSchema
|
|
4
4
|
from ai_review.clients.gitlab.mr.schema.discussions import (
|
|
5
|
+
GitLabDiscussionSchema,
|
|
5
6
|
GitLabGetMRDiscussionsQuerySchema,
|
|
6
7
|
GitLabGetMRDiscussionsResponseSchema,
|
|
7
8
|
GitLabCreateMRDiscussionRequestSchema,
|
|
8
9
|
GitLabCreateMRDiscussionResponseSchema
|
|
9
10
|
)
|
|
10
11
|
from ai_review.clients.gitlab.mr.schema.notes import (
|
|
12
|
+
GitLabNoteSchema,
|
|
11
13
|
GitLabGetMRNotesQuerySchema,
|
|
12
14
|
GitLabGetMRNotesResponseSchema,
|
|
13
15
|
GitLabCreateMRNoteRequestSchema,
|
|
14
16
|
GitLabCreateMRNoteResponseSchema,
|
|
15
17
|
)
|
|
16
18
|
from ai_review.clients.gitlab.mr.types import GitLabMergeRequestsHTTPClientProtocol
|
|
19
|
+
from ai_review.clients.gitlab.tools import gitlab_has_next_page
|
|
20
|
+
from ai_review.config import settings
|
|
17
21
|
from ai_review.libs.http.client import HTTPClient
|
|
18
22
|
from ai_review.libs.http.handlers import handle_http_error, HTTPClientError
|
|
23
|
+
from ai_review.libs.http.paginate import paginate
|
|
19
24
|
|
|
20
25
|
|
|
21
26
|
class GitLabMergeRequestsHTTPClientError(HTTPClientError):
|
|
@@ -86,18 +91,42 @@ class GitLabMergeRequestsHTTPClient(HTTPClient, GitLabMergeRequestsHTTPClientPro
|
|
|
86
91
|
project_id: str,
|
|
87
92
|
merge_request_id: str
|
|
88
93
|
) -> GitLabGetMRNotesResponseSchema:
|
|
89
|
-
|
|
90
|
-
|
|
91
|
-
|
|
94
|
+
async def fetch_page(page: int) -> Response:
|
|
95
|
+
query = GitLabGetMRNotesQuerySchema(page=page, per_page=settings.vcs.pagination.per_page)
|
|
96
|
+
return await self.get_notes_api(project_id, merge_request_id, query)
|
|
97
|
+
|
|
98
|
+
def extract_items(response: Response) -> list[GitLabNoteSchema]:
|
|
99
|
+
result = GitLabGetMRNotesResponseSchema.model_validate_json(response.text)
|
|
100
|
+
return result.root
|
|
101
|
+
|
|
102
|
+
items = await paginate(
|
|
103
|
+
max_pages=settings.vcs.pagination.max_pages,
|
|
104
|
+
fetch_page=fetch_page,
|
|
105
|
+
extract_items=extract_items,
|
|
106
|
+
has_next_page=gitlab_has_next_page
|
|
107
|
+
)
|
|
108
|
+
return GitLabGetMRNotesResponseSchema(root=items)
|
|
92
109
|
|
|
93
110
|
async def get_discussions(
|
|
94
111
|
self,
|
|
95
112
|
project_id: str,
|
|
96
113
|
merge_request_id: str
|
|
97
114
|
) -> GitLabGetMRDiscussionsResponseSchema:
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
115
|
+
async def fetch_page(page: int) -> Response:
|
|
116
|
+
query = GitLabGetMRDiscussionsQuerySchema(page=page, per_page=settings.vcs.pagination.per_page)
|
|
117
|
+
return await self.get_discussions_api(project_id, merge_request_id, query)
|
|
118
|
+
|
|
119
|
+
def extract_items(response: Response) -> list[GitLabDiscussionSchema]:
|
|
120
|
+
result = GitLabGetMRDiscussionsResponseSchema.model_validate_json(response.text)
|
|
121
|
+
return result.root
|
|
122
|
+
|
|
123
|
+
items = await paginate(
|
|
124
|
+
max_pages=settings.vcs.pagination.max_pages,
|
|
125
|
+
fetch_page=fetch_page,
|
|
126
|
+
extract_items=extract_items,
|
|
127
|
+
has_next_page=gitlab_has_next_page
|
|
128
|
+
)
|
|
129
|
+
return GitLabGetMRDiscussionsResponseSchema(root=items)
|
|
101
130
|
|
|
102
131
|
async def create_note(
|
|
103
132
|
self,
|
|
@@ -18,7 +18,8 @@ class GitLabDiscussionPositionSchema(BaseModel):
|
|
|
18
18
|
|
|
19
19
|
|
|
20
20
|
class GitLabGetMRDiscussionsQuerySchema(BaseModel):
|
|
21
|
-
|
|
21
|
+
page: int = 1
|
|
22
|
+
per_page: int = 100
|
|
22
23
|
|
|
23
24
|
|
|
24
25
|
class GitLabGetMRDiscussionsResponseSchema(RootModel[list[GitLabDiscussionSchema]]):
|
|
@@ -2,13 +2,16 @@ from typing import Annotated, Literal
|
|
|
2
2
|
|
|
3
3
|
from pydantic import BaseModel, Field
|
|
4
4
|
|
|
5
|
+
from ai_review.libs.config.vcs.bitbucket import BitbucketPipelineConfig, BitbucketHTTPClientConfig
|
|
5
6
|
from ai_review.libs.config.vcs.github import GitHubPipelineConfig, GitHubHTTPClientConfig
|
|
6
7
|
from ai_review.libs.config.vcs.gitlab import GitLabPipelineConfig, GitLabHTTPClientConfig
|
|
8
|
+
from ai_review.libs.config.vcs.pagination import VCSPaginationConfig
|
|
7
9
|
from ai_review.libs.constants.vcs_provider import VCSProvider
|
|
8
10
|
|
|
9
11
|
|
|
10
12
|
class VCSConfigBase(BaseModel):
|
|
11
13
|
provider: VCSProvider
|
|
14
|
+
pagination: VCSPaginationConfig = VCSPaginationConfig()
|
|
12
15
|
|
|
13
16
|
|
|
14
17
|
class GitLabVCSConfig(VCSConfigBase):
|
|
@@ -23,4 +26,13 @@ class GitHubVCSConfig(VCSConfigBase):
|
|
|
23
26
|
http_client: GitHubHTTPClientConfig
|
|
24
27
|
|
|
25
28
|
|
|
26
|
-
|
|
29
|
+
class BitbucketVCSConfig(VCSConfigBase):
|
|
30
|
+
provider: Literal[VCSProvider.BITBUCKET]
|
|
31
|
+
pipeline: BitbucketPipelineConfig
|
|
32
|
+
http_client: BitbucketHTTPClientConfig
|
|
33
|
+
|
|
34
|
+
|
|
35
|
+
VCSConfig = Annotated[
|
|
36
|
+
GitLabVCSConfig | GitHubVCSConfig | BitbucketVCSConfig,
|
|
37
|
+
Field(discriminator="provider")
|
|
38
|
+
]
|
|
@@ -0,0 +1,13 @@
|
|
|
1
|
+
from pydantic import BaseModel
|
|
2
|
+
|
|
3
|
+
from ai_review.libs.config.http import HTTPClientWithTokenConfig
|
|
4
|
+
|
|
5
|
+
|
|
6
|
+
class BitbucketPipelineConfig(BaseModel):
|
|
7
|
+
workspace: str
|
|
8
|
+
repo_slug: str
|
|
9
|
+
pull_request_id: str
|
|
10
|
+
|
|
11
|
+
|
|
12
|
+
class BitbucketHTTPClientConfig(HTTPClientWithTokenConfig):
|
|
13
|
+
pass
|