Save event creator

[HeshamHM28]

PR Type

Enhancement

---

Description

• Add git metadata to optimization payload

• Implement commit author retrieval utility

• Handle missing PR_NUMBER gracefully

---

Changes walkthrough 📝

Relevant files

Enhancement

aiservice.py Include git metadata in optimization payload                          codeflash/api/aiservice.py Imported git metadata utilities Retrieved repo owner and name Added current_username, repo_owner, repo_name to payload +5/-0      git_utils.py Add commit author retrieval utility                                            codeflash/code_utils/git_utils.py Added os import for environment checks Introduced get_last_commit_author_if_pr_exists function Handles missing PR_NUMBER and exceptions +18/-0

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

PR Reviewer Guide 🔍

(Review updated until commit 836c819)

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 Missing Imports The get_last_commit_author_if_pr_exists function references git.Repo, Repo, and logger, but the necessary imports (e.g., import git or from git import Repo, and logger) are not shown. Ensure these dependencies are imported and available. def get_last_commit_author_if_pr_exists(repo: Repo | None = None) -> str | None: """ Return the author's name of the last commit in the current branch if PR_NUMBER is set. Otherwise, return None. """ try: if "PR_NUMBER" not in os.environ: return None repository: Repo = repo if repo else git.Repo(search_parent_directories=True) last_commit = repository.head.commit return last_commit.author.name except Exception as e: logger.exception("Failed to get last commit author.") return None Null `current_username` The payload sets "current_username" to the result of get_last_commit_author_if_pr_exists, which can be None. Verify that downstream services accept null or consider providing a fallback (e.g., empty string) to avoid unexpected failures. "current_username" : get_last_commit_author_if_pr_exists(None), "repo_owner": git_repo_owner, "repo_name": git_repo_name, }

[github-actions]

PR Code Suggestions ✨

Latest suggestions up to 836c819
Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Add error handling around repo info Handle failures from get_repo_owner_and_name to avoid unhandled exceptions. Wrap the call in a try/except and use safe fallbacks if it raises an error. codeflash/api/aiservice.py [101] -git_repo_owner, git_repo_name = get_repo_owner_and_name() +try: + git_repo_owner, git_repo_name = get_repo_owner_and_name() +except Exception: + git_repo_owner, git_repo_name = None, None Suggestion importance[1-10]: 7 __ Why: Wrapping get_repo_owner_and_name in try/except improves robustness by preventing uncaught exceptions when repository info is unavailable.	Medium
General	Conditionally add optional payload fields Only include these optional fields when they are not None to prevent sending null values in the payload. codeflash/api/aiservice.py [110-112] -"current_username" : get_last_commit_author_if_pr_exists(None), -"repo_owner": git_repo_owner, -"repo_name": git_repo_name, +current_username = get_last_commit_author_if_pr_exists(None) +payload = { + "source_code": source_code, + "dependency_code": dependency_code, + "num_variants": num_candidates, + "trace_id": trace_id, + "python_version": platform.python_version(), + "experiment_metadata": experiment_metadata, + "codeflash_version": codeflash_version, +} +if current_username: + payload["current_username"] = current_username +if git_repo_owner and git_repo_name: + payload["repo_owner"] = git_repo_owner + payload["repo_name"] = git_repo_name Suggestion importance[1-10]: 5 __ Why: Avoiding null entries for current_username, repo_owner, and repo_name makes the payload cleaner, but the impact is limited to optional metadata handling.	Low
	Cache commit author lookup Cache the result of get_last_commit_author_if_pr_exists into a local variable before building the payload, to avoid calling it multiple times and improve readability. codeflash/api/aiservice.py [110] -"current_username" : get_last_commit_author_if_pr_exists(None), +current_username = get_last_commit_author_if_pr_exists(None) +... +"current_username": current_username, Suggestion importance[1-10]: 4 __ Why: Storing the result of get_last_commit_author_if_pr_exists in a variable improves readability, but this is a minor stylistic change.	Low

---

Previous suggestions

Suggestions up to commit 109ab3f

Category	Suggestion	Impact
Possible issue	Import and use Repo correctly The function refers to git.Repo and the Repo type without importing them, leading to a NameError. Import Repo from GitPython and use it directly. This ensures the code can instantiate a repository object correctly. codeflash/code_utils/git_utils.py [188] -repository: Repo = repo if repo else git.Repo(search_parent_directories=True) +from git import Repo +repository: Repo = repo if repo else Repo(search_parent_directories=True) Suggestion importance[1-10]: 10 __ Why: Without importing Repo or git, the call to git.Repo will raise a NameError, blocking functionality; importing Repo is critical to fix this bug.	High
General	Add error handling for repo lookup If the repository info lookup fails (e.g., not in a git directory), this call could raise an exception and crash the API call. Wrap the call in a try/except and provide fallback values or log an error to avoid unhandled exceptions. codeflash/api/aiservice.py [102] -git_repo_owner, git_repo_name = get_repo_owner_and_name() +try: + git_repo_owner, git_repo_name = get_repo_owner_and_name() +except Exception as e: + logger.warning(f"Unable to get repo owner/name: {e}") + git_repo_owner, git_repo_name = None, None Suggestion importance[1-10]: 6 __ Why: Wrapping get_repo_owner_and_name() in try/except improves robustness but isn't strictly required for correctness in most environments.	Low
	Call helper without redundant None Passing None explicitly into the helper is redundant since its default is already None. Remove the None argument for clarity and consistency with the function signature. codeflash/api/aiservice.py [111] -"current_username" : get_last_commit_author_if_pr_exists(None), +"current_username": get_last_commit_author_if_pr_exists(), Suggestion importance[1-10]: 3 __ Why: Removing the explicit None argument is a minor stylistic cleanup with limited impact on functionality.	Low

[Saga4]

Attached with https://github.com/codeflash-ai/codeflash-internal/pull/1596

[github-actions]

Persistent review updated to latest commit 836c819

[misrasaurabh1]

@HeshamHM28 whats the status here?

[HeshamHM28]

@HeshamHM28 whats the status here?

I will update it today with a small bug fix.

[Saga4]

LGTM