Skip to content

Commit f5a3be1

Browse files
authored
Merge pull request #35 from naaa760/fix/pr-creation-404-and-analysis-reliability
fix: resolve repository analysis and PR creation issues
2 parents 7d1824c + b51692c commit f5a3be1

2 files changed

Lines changed: 13 additions & 19 deletions

File tree

src/agents/repository_analysis_agent/models.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ class RepositoryAnalysisRequest(BaseModel):
9595
repository_url: str | None = Field(default=None, description="GitHub repository URL")
9696
repository_full_name: str | None = Field(default=None, description="Full repository name (owner/repo)")
9797
installation_id: int | None = Field(default=None, description="GitHub App installation ID")
98+
user_token: str | None = Field(default=None, description="User token for GitHub operations (optional)")
9899
max_prs: int = Field(default=10, ge=0, le=50, description="Max PRs to sample for analysis")
99100

100101
@field_validator("repository_full_name", mode="before")

src/agents/repository_analysis_agent/nodes.py

Lines changed: 12 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
from __future__ import annotations
1010

1111
import logging
12-
import textwrap
1312
from typing import Any
1413

1514
import yaml
@@ -258,7 +257,7 @@ def _default_recommendations(
258257
severity: medium
259258
event_types:
260259
- pull_request
261-
parameters:
260+
parameters:
262261
source_patterns:
263262
{source_patterns_yaml}
264263
test_patterns:
@@ -294,17 +293,14 @@ def _default_recommendations(
294293

295294
recommendations.append(
296295
RuleRecommendation(
297-
yaml_rule=textwrap.dedent(
298-
"""
299-
description: "Ensure PRs include context"
296+
yaml_rule="""description: "Ensure PRs include context"
300297
enabled: true
301-
severity: low
298+
severity: low
302299
event_types:
303300
- pull_request
304-
parameters:
305-
min_description_length: 50
306-
"""
307-
).strip(),
301+
parameters:
302+
min_description_length: 50
303+
""".strip(),
308304
confidence=desc_confidence,
309305
reasoning=desc_reasoning,
310306
strategy_used="static",
@@ -313,18 +309,15 @@ def _default_recommendations(
313309

314310
# Add a repository-specific rule if we detect specific patterns
315311
if state.repository_features.has_workflows:
316-
workflow_rule = textwrap.dedent(
317-
"""
318-
description: "Protect CI/CD workflows"
312+
workflow_rule = """description: "Protect CI/CD workflows"
319313
enabled: true
320-
severity: high
314+
severity: high
321315
event_types:
322316
- pull_request
323-
parameters:
324-
file_patterns:
325-
- ".github/workflows/**"
326-
"""
327-
).strip()
317+
parameters:
318+
file_patterns:
319+
- ".github/workflows/**"
320+
""".strip()
328321

329322
recommendations.append(
330323
RuleRecommendation(

0 commit comments

Comments
 (0)