wony617 commited on
Commit
d583cc9
Β·
1 Parent(s): 9247d25

log: always append JSONL log regardless of status

Browse files

Deduplicate logging and ensure every workflow run appends a JSON line to pr_success.log on the log_event branch.

Files changed (1) hide show
  1. agent/workflow.py +17 -33
agent/workflow.py CHANGED
@@ -203,25 +203,25 @@ def generate_github_pr(
203
  toctree_status = f"\nπŸ“‹ **Toctree Updated:** βœ… {toctree_result['message']}"
204
  else:
205
  toctree_status = f"\nπŸ“‹ **Toctree Update Failed:** ❌ {toctree_result['message']}"
206
-
207
- if result["status"] == "success":
208
- # Append full result JSON to GitHub log file on 'log_event' branch
209
  import json
210
  log_entry = json.dumps(result, ensure_ascii=False) + "\n"
211
- try:
212
- log_res = agent.append_to_log_file(
213
- owner=github_config["owner"],
214
- repo_name=github_config["repo_name"],
215
- branch_name="log_event",
216
- path="pr_success.log",
217
- log_entry=log_entry,
218
- # Ensure pure JSONL (no header)
219
- header_if_new="",
220
- )
221
- print(f"πŸ“ Log append result: {log_res}")
222
- except Exception as e:
223
- print(f"❌ Failed to append PR log via GitHub API: {e}")
224
-
225
  return f"""βœ… **GitHub PR Creation Successful!**
226
 
227
  πŸ”— **PR URL:** {result.get('pr_url', 'NO_PR_URL')}
@@ -231,22 +231,6 @@ def generate_github_pr(
231
  {result["message"]}"""
232
 
233
  elif result["status"] == "partial_success":
234
- # Append full result JSON to GitHub log file on 'log_event' branch
235
- import json
236
- log_entry = json.dumps(result, ensure_ascii=False) + "\n"
237
- try:
238
- log_res = agent.append_to_log_file(
239
- owner=github_config["owner"],
240
- repo_name=github_config["repo_name"],
241
- branch_name="log_event",
242
- path="pr_success.log",
243
- log_entry=log_entry,
244
- header_if_new="",
245
- )
246
- print(f"πŸ“ Log append result: {log_res}")
247
- except Exception as e:
248
- print(f"❌ Failed to append PR log via GitHub API: {e}")
249
-
250
  return f"""⚠️ **Partial Success**
251
 
252
  🌿 **Branch:** {result["branch"]}
 
203
  toctree_status = f"\nπŸ“‹ **Toctree Updated:** βœ… {toctree_result['message']}"
204
  else:
205
  toctree_status = f"\nπŸ“‹ **Toctree Update Failed:** ❌ {toctree_result['message']}"
206
+
207
+ # Append full result JSON to GitHub log file on 'log_event' branch (always)
208
+ try:
209
  import json
210
  log_entry = json.dumps(result, ensure_ascii=False) + "\n"
211
+ log_res = agent.append_to_log_file(
212
+ owner=github_config["owner"],
213
+ repo_name=github_config["repo_name"],
214
+ branch_name="log_event",
215
+ path="pr_success.log",
216
+ log_entry=log_entry,
217
+ # Ensure pure JSONL (no header)
218
+ header_if_new="",
219
+ )
220
+ print(f"πŸ“ Log append result: {log_res}")
221
+ except Exception as e:
222
+ print(f"❌ Failed to append PR log via GitHub API: {e}")
223
+
224
+ if result["status"] == "success":
225
  return f"""βœ… **GitHub PR Creation Successful!**
226
 
227
  πŸ”— **PR URL:** {result.get('pr_url', 'NO_PR_URL')}
 
231
  {result["message"]}"""
232
 
233
  elif result["status"] == "partial_success":
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
234
  return f"""⚠️ **Partial Success**
235
 
236
  🌿 **Branch:** {result["branch"]}