mfalland/.forgejo
1
0
Fork 0
Code Issues Pull requests 3 Projects Releases Packages Wiki Activity Actions

feat: Ollama Timeout konfigurierbar #3

Open
mfalland wants to merge 1 commit from test/review-endtest into main
pull from: test/review-endtest
merge into: mfalland:main
Conversation 1 Commits 1 Files changed 1 +3
mfalland commented 2026-05-30 15:44:33 +00:00
Owner
Copy link

Macht den Ollama-Timeout für langsame Modelle konfigurierbar via Environment Variable.

#security #quality

Macht den Ollama-Timeout für langsame Modelle konfigurierbar via Environment Variable. #security #quality
mfalland commented 2026-05-30 15:45:32 +00:00
Author
Owner
Copy link

🤖 AI Code Review — Ollama Timeout konfigurierbar

Gesamtbewertung: 🟢 Gut

🛡️ Sicherheit

Keine Sicherheits-Issues. Die Änderung setzt eine Environment-Variable mit sicherem Default (300s) und verwendet Bash-Parameter-Expansion (${VAR:-default}), was standardkonform und injection-sicher ist.

✨ Code-Qualität

  1. 🟢 Platzierung am EOF — Der Timeout-Default steht nach echo "All reviews completed.". Das ist semantisch korrekt (Default für nachfolgende ollama run-Aufrufe, kein Einfluss auf die bereits abgeschlossenen Reviews), aber es wäre lesbarer, ihn zu den anderen Konfigurations-Variablen am Dateianfang zu setzen, in review_with_agent() oder dort, wo ollama run aufgerufen wird.

  2. 🟢 Gut gemacht: OLLAMA_TIMEOUT folgt dem POSIX/Python `os.environ.get("OLLAMA_TIMEOUT", 300)"-Pattern — konsistent mit existierendem Konventionsstil im Skript.

  3. 🟢 Edge Cases abgedeckt: Der Default 300 ist ein vernünftiger 5-Minuten-Fallback. Der Code erlaubt Überschreibung via Environment (z.B. OLLAMA_TIMEOUT=600 ./review.sh), ohne dass eine Config-Datei nötig ist.

📋 Zusammenfassung

Solide, fokussierte Änderung. Die Timeout-Konfiguration wird sauber via Environment-Variable mit vernünftigem Default realisiert. Einzig die Platzierung am Dateiende (statt bei anderen Konfig- oder Aufrufstellen) könnte man bei Gelegenheit nachziehen — das ist aber reine Präferenz, kein Bug. ✅

Multi-Agent Review via OpenCode + oh-my-openagent (opencode/deepseek-v4-flash-free)
Agents: security quality | Critic-Pass: aktiv

## 🤖 AI Code Review — Ollama Timeout konfigurierbar ### Gesamtbewertung: 🟢 Gut ### 🛡️ Sicherheit Keine Sicherheits-Issues. Die Änderung setzt eine Environment-Variable mit sicherem Default (300s) und verwendet Bash-Parameter-Expansion (`${VAR:-default}`), was standardkonform und injection-sicher ist. ### ✨ Code-Qualität 1. 🟢 **Platzierung am EOF** — Der Timeout-Default steht nach `echo "All reviews completed."`. Das ist semantisch korrekt (Default für nachfolgende `ollama run`-Aufrufe, kein Einfluss auf die bereits abgeschlossenen Reviews), aber es wäre lesbarer, ihn zu den anderen Konfigurations-Variablen am Dateianfang zu setzen, in `review_with_agent()` oder dort, wo `ollama run` aufgerufen wird. 2. 🟢 **Gut gemacht:** `OLLAMA_TIMEOUT` folgt dem POSIX/Python `os.environ.get("OLLAMA_TIMEOUT", 300)"-Pattern — konsistent mit existierendem Konventionsstil im Skript. 3. 🟢 **Edge Cases abgedeckt:** Der Default `300` ist ein vernünftiger 5-Minuten-Fallback. Der Code erlaubt Überschreibung via Environment (z.B. `OLLAMA_TIMEOUT=600 ./review.sh`), ohne dass eine Config-Datei nötig ist. ### 📋 Zusammenfassung Solide, fokussierte Änderung. Die Timeout-Konfiguration wird sauber via Environment-Variable mit vernünftigem Default realisiert. Einzig die Platzierung am Dateiende (statt bei anderen Konfig- oder Aufrufstellen) könnte man bei Gelegenheit nachziehen — das ist aber reine Präferenz, kein Bug. ✅ --- *Multi-Agent Review via [OpenCode](https://opencode.ai) + oh-my-openagent (opencode/deepseek-v4-flash-free)* *Agents: security quality | Critic-Pass: aktiv*
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin test/review-endtest:test/review-endtest
git switch test/review-endtest

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff test/review-endtest
git switch test/review-endtest
git rebase main
git switch main
git merge --ff-only test/review-endtest
git switch test/review-endtest
git rebase main
git switch main
git merge --no-ff test/review-endtest
git switch main
git merge --squash test/review-endtest
git switch main
git merge --ff-only test/review-endtest
git switch main
git merge test/review-endtest
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
mfalland/.forgejo!3
No description provided.