Skip to content

feat: updating LLM calls and other minor fixes - #116

Merged
TEJASNARAYANS merged 7 commits into
mainfrom
import-fixes
Aug 21, 2026
Merged

feat: updating LLM calls and other minor fixes#116
TEJASNARAYANS merged 7 commits into
mainfrom
import-fixes

Conversation

@TEJASNARAYANS

@TEJASNARAYANS TEJASNARAYANS commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

What Changed?

Briefly describe what this PR changes.

Why?

Explain the problem this solves and why this approach was chosen.

How to Test?

List clear steps for reviewers to verify the change.

Release Label

Select one semantic version bump intent for this PR:

  • major - breaking change, next release bumps major version
  • minor - backward-compatible feature, next release bumps minor version
  • fix - backward-compatible bug fix, next release bumps patch version
  • No label (defaults to patch release)

If you do not have permission to apply labels, mention the intended release label here and a maintainer will set it.

Checklist

  • I have read the CONTRIBUTING.md guide.
  • My code follows the project's style guidelines.
  • I have added tests that prove my fix is effective or my feature works.
  • I have updated documentation where needed.

@TEJASNARAYANS TEJASNARAYANS added the fix Fix version release label Aug 18, 2026
@greptile-apps

greptile-apps Bot commented Aug 18, 2026

Copy link
Copy Markdown

Greptile Summary

The PR updates LLM metric-call grouping and expected-call reporting, adjusts call-import lifecycle rollups, and adds invitation-aware authentication flows.

  • Groups metrics sharing an LLM configuration into one evaluation call.
  • Exposes expected LLM calls per evaluation row.
  • Updates parent import status handling and evaluation materialization.
  • Adds invitation preview, signup, login, and acceptance behavior.
  • Includes frontend, recording-provider, configuration, and test updates.

Confidence Score: 4/5

The PR is not yet safe to merge because parent call-import status can still be derived from an arbitrary sibling evaluation when creation timestamps tie.

The latest-evaluation query still lacks a deterministic tie-breaker, while supported sibling evaluations are inserted in one transaction and can later reach different terminal states; the parent rollup can therefore expose an inconsistent status.

Files Needing Attention: app/services/call_imports/bulk_ops.py

Important Files Changed

Filename Overview
app/workers/tasks/evaluate_call_import_row_core.py Adds configuration-key resolution and buckets prompt groups so metrics sharing provider, model, generation configuration, and credentials use one LLM call.
app/services/call_imports/bulk_ops.py Adds evaluation-driven parent status mapping, but the previously reported ambiguous sibling-status selection remains.
app/api/v1/routes/call_import_evaluations.py Exposes expected LLM-call counts and changes row materialization and parent processing-state behavior.
app/api/v1/routes/auth.py Adds invitation preview and acceptance flows and allows signup to attach local credentials to existing passwordless users.
app/services/testing/test_agent_bridge_service.py Centralizes Vapi recording URL extraction and conditionally authenticates non-presigned recording downloads.

Reviews (8): Last reviewed commit: "fix: updating URL fix" | Re-trigger Greptile

Comment thread app/workers/tasks/evaluate_call_import_row_core.py
Comment thread app/services/call_imports/bulk_ops.py
@TEJASNARAYANS
TEJASNARAYANS merged commit 92f039d into main Aug 21, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Fix version release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant