• Joined on 2026-03-29
sarman commented on pull request sarman/tftsr-devops_investigation#35 2026-04-12 22:44:08 +00:00
feat/pr-review-workflow

🤖 Automated PR Review:\n\nThis PR introduces a new GitHub Actions workflow for automated PR review using Ollama, and also bumps version numbers across package.json and Tauri config files. Below is a detailed analysis covering correctness, security, and best practices.

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 22:40:23 +00:00
144a4551f2 fix: revert to two-dot diff — three-dot requires merge base unavailable in shallow clone
47b2e824e0 fix: replace github.server_url with hardcoded gogs.tftsr.com for container access
82aae00858 fix: resolve AI review false positives and address high/medium issues
1a4c6df6c9 fix: harden pr-review workflow — URLs, DNS, correctness and reliability
2d0f95e9db fix: configure container DNS to resolve ollama-ui.tftsr.com
Compare 20 commits »
sarman commented on pull request sarman/tftsr-devops_investigation#35 2026-04-12 22:16:03 +00:00
feat/pr-review-workflow

🤖 Automated PR Review:\n\nThis PR introduces a new GitHub Actions workflow (though it's configured for Gitea/Gogs, not GitHub—see below) for automated PR reviews using Ollama, and also bumps version numbers across package.json, Cargo.toml, and tauri.conf.json to 0.2.50.

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 22:11:59 +00:00
ac568add3f fix: revert to two-dot diff — three-dot requires merge base unavailable in shallow clone
sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 22:08:24 +00:00
0b49899384 fix: replace github.server_url with hardcoded gogs.tftsr.com for container access
sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 22:02:42 +00:00
bf1f81c563 fix: resolve AI review false positives and address high/medium issues
sarman commented on pull request sarman/tftsr-devops_investigation#35 2026-04-12 21:53:57 +00:00
feat/pr-review-workflow

🤖 Automated PR Review:\n\nHere's a comprehensive security, correctness, and best-practices review of the provided changes:

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 21:50:15 +00:00
e1724d9631 fix: harden pr-review workflow — URLs, DNS, correctness and reliability
sarman commented on pull request sarman/tftsr-devops_investigation#35 2026-04-12 21:43:36 +00:00
feat/pr-review-workflow

🤖 Automated PR Review:\n\nHere's a detailed analysis of the provided code changes, focusing on correctness, security, and best practices—especially for the new CI/CD workflow file (.gitea/workflows/pr-review.yml). The package.json and Cargo.toml/tauri.conf.json version bumps are minor and correct (assuming intentional release), so they’re noted but not the main focus.

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 21:39:45 +00:00
7492f33a5c fix: configure container DNS to resolve ollama-ui.tftsr.com
sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 21:32:18 +00:00
8f9529fd77 fix: harden pr-review workflow and sync versions to 0.2.50
sarman commented on pull request sarman/tftsr-devops_investigation#35 2026-04-12 21:24:03 +00:00
feat/pr-review-workflow

🤖 Automated PR Review:\n\nThank you for the request — let's break this down carefully.

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 21:20:23 +00:00
02b40bb12d fix: restore migration 014, bump version to 0.2.50, harden pr-review workflow
sarman commented on pull request sarman/tftsr-devops_investigation#35 2026-04-12 21:11:43 +00:00
feat/pr-review-workflow

🤖 Automated PR Review:\n\nHere's a detailed analysis of the provided changes, focusing on correctness, security, and best practices:

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 21:07:43 +00:00
2f7be85e64 fix: use bash shell and remove bash-only substring expansion in pr-review
sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 21:03:32 +00:00
0105e321d2 fix: add diagnostics to identify empty Ollama response root cause
sarman commented on pull request sarman/tftsr-devops_investigation#35 2026-04-12 21:00:41 +00:00
feat/pr-review-workflow

🤖 Automated PR Review:\n\nHere's a detailed security, correctness, and best-practices review of the changes in the PR. The PR title "feat/pr-review-workflow" is misleading—it actually removes migration code and adds a GitHub Actions–style workflow (though targeting Gitea APIs), suggesting the name should be updated.

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-12 20:56:19 +00:00
dbd567972f fix: correct Ollama URL, API endpoint, and JSON construction in pr-review workflow
sarman commented on pull request sarman/tftsr-devops_investigation#35 2026-04-10 21:19:50 +00:00
feat/pr-review-workflow

🤖 Automated PR Review:

sarman pushed to feat/pr-review-workflow at sarman/tftsr-devops_investigation 2026-04-10 21:16:58 +00:00
313dcf7cdc fix: add debugging output for Ollamaresponse