From eab97af46c9f5033e7ca5d126dcbddc822acf799 Mon Sep 17 00:00:00 2001 From: Joseph O'Connor Date: Fri, 24 Jan 2025 18:51:56 -0500 Subject: [PATCH] refactor(resolver): Improve formatting of attachment context handling - Enhance code readability by using multi-line string formatting - Consistently use single quotes for string literals - Improve line breaks and indentation in attachment context processing --- openhands/resolver/issue_definitions.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/openhands/resolver/issue_definitions.py b/openhands/resolver/issue_definitions.py index bbec398a738d..b64120840348 100644 --- a/openhands/resolver/issue_definitions.py +++ b/openhands/resolver/issue_definitions.py @@ -184,7 +184,9 @@ def _get_attachments_context(self, issue_body: str) -> str: for file_url, file_name in attachments: content = self._download_attachment(file_url) if content: - attachment_contexts.append(f"Content of attached file '{file_name}':\n{content}") + attachment_contexts.append( + f"Content of attached file '{file_name}':\n{content}" + ) return '\n\n'.join(attachment_contexts) @@ -279,7 +281,7 @@ def get_instruction( # Format issues string with both the issue body and attachments issues_str = issue.body if attachments_context: - issues_str += f"\n\nAttached Files:\n{attachments_context}" + issues_str += f'\n\nAttached Files:\n{attachments_context}' # Add attachments from closing issues if any if issue.closing_issues: @@ -289,7 +291,9 @@ def get_instruction( for closing_issue in issue.closing_issues: closing_attachments = self._get_attachments_context(closing_issue) if closing_attachments: - closing_issues_str += f"\n\nAttached Files from Referenced Issue:\n{closing_attachments}" + closing_issues_str += ( + f'\n\nAttached Files from Referenced Issue:\n{closing_attachments}' + ) issues_str += f'\n\n{closing_issues_str}' template = jinja2.Template(prompt_template)