fix: use .replace() instead of .format() in infer_intent to avoid KeyError on JSON prompt

[pleasingsunlight]

infer_intent was crashing with KeyError: '"purpose"' because INTENT_PROMPT.format(diff=...) misinterpreted the JSON curly braces in the prompt template as Python format placeholders.

Changes

• agent/llm/prompts.py — Added missing INTENT_PROMPT with a {diff} placeholder and a JSON output format example
• agent/llm/groq_client.py — Added infer_intent function using .replace("{diff}", diff[:8000]) instead of .format()
• agent/llm/test_generator.py — Added intent: dict = None parameter, injected into prompt as ---INTENT--- block

# Before — crashes when INTENT_PROMPT contains {"purpose": ...}
prompt = INTENT_PROMPT.format(diff=diff[:8000])  # KeyError: '"purpose"'

# After — safely substitutes only the {diff} token
prompt = INTENT_PROMPT.replace("{diff}", diff[:8000])

[haddybhaiya]

fix : groq_client.py infer_intent function by using .replace instead of .format

[haddybhaiya]

@copilot check for which issue that the check is failing

[Copilot]

fix : groq_client.py infer_intent function by using .replace instead of .format

Fixed in commit ecacb68. Added infer_intent to groq_client.py using INTENT_PROMPT.replace("{diff}", diff[:8000]) instead of .format(diff=...). The .format() approach caused KeyError: '"purpose"' because INTENT_PROMPT contains JSON curly braces ({"purpose": ...}) that Python's format parser tries to interpret as placeholders. Using .replace() safely substitutes only the {diff} placeholder without touching the JSON syntax.

@copilot check for which issue that the check is failing

The CI was failing with KeyError: '"purpose"' at groq_client.py line 55 in infer_intent (run 24144564276). Root cause: INTENT_PROMPT.format(diff=diff[:8000]) was used, but INTENT_PROMPT contains raw JSON curly braces ({"purpose": "..."}) that Python's .format() misinterprets as format placeholders. Fixed in commit ecacb68 by:

• Switching to .replace("{diff}", diff[:8000]) in infer_intent
• Adding INTENT_PROMPT to prompts.py (it was missing from the branch)
• Updating test_generator.py to accept the intent parameter

[haddybhaiya]

apparently the changes proposed are the same changed which are in the branch test-branch-phase3
will see to it

[github-actions]

Automated Code Review Comment:

Bugs

• None

Improvements

• The generate_tests function is improved by adding a docstring to explain its purpose and parameters.
• Error handling is added when committing tests to handle potential exceptions.
• The code is refactored for better readability and structure.

Suggestions

• Consider adding validation for the diff, context, and intent parameters in the generate_tests function.
• The infer_intent function could be improved by adding more robust error handling and intent extraction logic.
• Code comments could be added to explain complex logic and improve code readability.
• The final_output variable could be formatted using a templating engine or f-strings for better readability.
• Consider adding type hints for the infer_intent function and its return type.

Code looks good overall, but could benefit from additional improvements for better maintainability and robustness. ✅

---

### Suggested Tests
```bash
pytest tests/test_generated.py
def test_empty_diff():
assert generate_tests(diff="") == ""

def test_empty_context():
assert generate_tests(diff="diff --git a/app.py b/app.py", context=[]) == ""

def test_intent_extraction_failure():
assert generate_tests(diff="diff --git a/app.py b/app.py", intent={"error": "intent extraction failed"}) == ""

def test_commit_tests_failure():
try:
commit_tests()
assert False
except Exception as e:
assert str(e) != ""

def test_generate_review_with_intent():
diff = "diff --git a/app.py b/app.py"
context = ["def safe_divide(a, b):"]
intent = {"purpose": "generate tests", "properties": ["a", "b"], "edge_cases": ["a=0", "b=0"]}
assert generate_tests(diff, context, intent) != ""
```

[pleasingsunlight]

Good work fixing the files @haddybhaiya , i shall merge the pr once the bot works as intended

[haddybhaiya]

we Ball!!