Add review_before_push tool: independent second opinion before commits#97
Open
babyblueviper1 wants to merge 1 commit into
Open
Add review_before_push tool: independent second opinion before commits#97babyblueviper1 wants to merge 1 commit into
babyblueviper1 wants to merge 1 commit into
Conversation
New tool (autoagent/tools/invinoveritas_review.py), auto-discovered by the existing import_tools_recursively() mechanism in tools/__init__.py -- no other files changed. Fits naturally alongside get_diff/stage_files/push_changes in github_ops.py: gets the current diff (same git command get_diff() uses), sends it to invinoveritas (https://api.babyblueviper.com) -- an independent, model-agnostic reviewer -- and returns a structured verdict (approve / approve_with_concerns / reject) + ranked issues before the agent pushes. Free registration returns an api_key with trial calls; no crypto/payment setup needed to try it. Verified before submitting (real diff, real live API, not a mock): - Planted a genuine SQL-injection diff and confirmed the live /review endpoint catches it: verdict=reject, confidence=0.95, blocker issue with the exact vulnerable line named. - Ran the actual function file end-to-end (stubbed context_variables/ env.run_command so it doesn't need the full docker/local environment setup, but the review-calling logic itself is the real, unmodified code) and confirmed the output formatting, including the two edge cases: no diff to review, and INVINOVERITAS_API_KEY unset.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
New tool,
review_before_push(autoagent/tools/invinoveritas_review.py) — fits alongsideget_diff/stage_files/push_changesingithub_ops.py. Gets the current diff and sends it to invinoveritas, an independent, model-agnostic second-opinion reviewer, returning a structured verdict (approve/approve_with_concerns/reject) + ranked issues before the agent pushes.Auto-discovered by the existing
import_tools_recursively()intools/__init__.py— no other files changed.Free registration (
/register) returns anapi_keywith trial calls per tool; no crypto/payment setup needed to try it.Verified before submitting
Not a mock — planted a real SQL-injection diff and confirmed the live
/reviewendpoint catches it:Ran the actual submitted function end-to-end (stubbed
context_variables/env.run_commandso it doesn't need the full docker/local environment, but the review-calling logic itself is the real, unmodified code) — confirmed output formatting and the two edge cases (no diff to review,INVINOVERITAS_API_KEYunset).Disclosure
We (invinoveritas) already have a merged PR in
HKUDS/Vibe-Trading(#330 — an advisory/reviewprovider follow-up), so this isn't a cold intro — flagging the existing relationship for transparency.Happy to adjust naming/placement/style to match project conventions more closely.