Skip to content

fix: scope batch prompt context per repository - #333

Merged
404-Page-Found merged 6 commits into
mainfrom
fix/315-batch-repo-prompt-context
Sep 24, 2026
Merged

404-Page-Found merged 6 commits into
mainfrom
fix/315-batch-repo-prompt-context

Conversation

@404-Page-Found

@404-Page-Found 404-Page-Found commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • scope branch and previous-commit prompt variables to an explicit repository cwd
  • pass each batch repository path into suggestion generation
  • cover batch context isolation and streaming prompt context with regression tests

Closes #315

Review in cubic

@coderabbitai

coderabbitai Bot commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 1 minute.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

This review ran on the open-source allowance, not this organization's plan, because the pull request author doesn't have an assigned seat. Waiting won't change this — ask an organization admin to assign them a seat, or add seats in Billing if every seat is already assigned, then retry.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: ec6498c7-e036-4148-9457-add207f39ca2

📥 Commits

Reviewing files that changed from the base of the PR and between 65718e4 and 648d945.

📒 Files selected for processing (4)
  • src/commands/batch.ts
  • src/git/diff.ts
  • src/llm/client.ts
  • tests/batch-prompt-context.test.mjs

Comment @coderabbitai help to get the list of available commands.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 4 files

Shadow auto-approve: would not auto-approve because issues were found.

Re-trigger cubic

Comment thread tests/batch-prompt-context.test.mjs
@sonarqubecloud

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0 issues found across 1 file (changes from recent commits).

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Shadow auto-approve: would auto-approve. Fix batch context isolation by explicit scope branch and previous-commit prompt variables to an actionable repo cwd; defaults preserve prior behavior, batch path passes the repository path, and regression tests cover batch and streaming contexts

Re-trigger cubic

@404-Page-Found
404-Page-Found merged commit 131c914 into main Sep 24, 2026
7 checks passed
@404-Page-Found
404-Page-Found deleted the fix/315-batch-repo-prompt-context branch September 24, 2026 03:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Batch prompts use the caller's branch and previous commit instead of each repository

1 participant