generated tests as multiple markdown blocks instead of a single block

[aseembits93]

PR Type

Enhancement

---

Description

• Output tests as separate Markdown code blocks

• Wrap concolic tests in Python fenced blocks

• Replace delimiter with Markdown formatting

---

Diagram Walkthrough

flowchart LR
  A["Generated tests list"] -- "iterate" --> B["Build Markdown string"]
  B -- "each test" --> C["Wrap in ```python fences"]
  C -- "append spacing" --> D["generated_tests_str"]
  E["concolic_test_str present?"] -- "yes" --> F["Wrap concolic in ```python fences"]
  F -- "append" --> D

Loading

File Walkthrough

	Relevant files
Enhancement	function_optimizer.py Generate Markdown-fenced blocks for tests                                codeflash/optimization/function_optimizer.py Replace single joined string with loop build. Wrap each test in python fenced blocks. Add blank lines between code blocks. Wrap concolic tests in fenced code block. +6/-4

---

[aseembits93]

will confirm on the backend if we are enclosing it in the block further in the streamline

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Formatting Consistency Ensure the new Markdown code blocks render correctly in all target environments (e.g., GitHub, Slack, docs site) and that the added trailing blank lines do not produce unintended spacing or parsing issues. generated_tests_str = "" for test in generated_tests.generated_tests: generated_tests_str += f"```python\n{test.generated_original_test_source}\n```" generated_tests_str += "\n\n" if concolic_test_str: generated_tests_str += f"```python\n{concolic_test_str}\n```\n\n" Missing Language Fence Safety Wrapping arbitrary test strings in python fences assumes they always contain valid Markdown-safe content; validate or escape backticks within tests to prevent fence breaking. for test in generated_tests.generated_tests: generated_tests_str += f"```python\n{test.generated_original_test_source}\n```" generated_tests_str += "\n\n" if concolic_test_str: generated_tests_str += f"```python\n{concolic_test_str}\n```\n\n"

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Use join instead of concatenation Avoid repeated string concatenation in a loop, which is O(n^2) and can degrade performance with many tests. Build a list of blocks and join once at the end for linear time complexity. codeflash/optimization/function_optimizer.py [1382-1385] -generated_tests_str = "" +blocks = [] for test in generated_tests.generated_tests: - generated_tests_str += f"```python\n{test.generated_original_test_source}\n```" - generated_tests_str += "\n\n" + blocks.append(f"```python\n{test.generated_original_test_source}\n```") + blocks.append("") # blank line separator +generated_tests_str = "\n".join(blocks) Suggestion importance[1-10]: 6 __ Why: Replacing repeated string concatenation with list join is correct and improves performance for many tests; it directly applies to the loop over generated_tests.generated_tests. It's a maintainability/performance enhancement, not a critical bug fix.	Low
	Normalize trailing newlines Ensure proper separation and avoid trailing double blank lines for consistent formatting. Append separators uniformly via a list and join, or strip the final newline to prevent extra spacing at the end. codeflash/optimization/function_optimizer.py [1387-1388] if concolic_test_str: - generated_tests_str += f"```python\n{concolic_test_str}\n```\n\n" + suffix = f"```python\n{concolic_test_str}\n```" + if generated_tests_str and not generated_tests_str.endswith("\n\n"): + generated_tests_str += "\n\n" + generated_tests_str += suffix Suggestion importance[1-10]: 5 __ Why: The suggestion correctly targets the handling of concolic_test_str formatting to avoid extra trailing blank lines. It's a minor formatting improvement and accurate, but low impact on functionality.	Low

[aseembits93]

[KRRT7]

what was this for? still needed?

[misrasaurabh1]

@aseembits93 dont let this get stale

[aseembits93]

@misrasaurabh1 this needs some changes on cf-api side, there could be unintended consequences on that end, will test it a bit more. aiservice works as expected

[aseembits93]

we can do everything on server side without changing the payload

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

---

Codeflash Bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}