Skip to content

fix: allow scheduled evi runs to open ready pull requests - #784

Merged
HugoRCD merged 1 commit into
mainfrom
fix/evi-scheduled-pr-approval
Oct 6, 2026
Merged

HugoRCD merged 1 commit into
mainfrom
fix/evi-scheduled-pr-approval

Conversation

@HugoRCD

@HugoRCD HugoRCD commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Allow scheduled Evi runs to create ready or draft PRs without approval in the configured repository.
  • Keep approval for other repositories and preserve the existing caller and write gates.
  • Add 14 regression tests and correct the schedule note.

Evidence

Scheduled call Before After
Ready PR in home repository Approval required No approval card
Draft PR outside home repository No approval card Approval required

Validation

  • Regression tests failed before the fix and pass afterwards (14 tests).
  • Full lint, tests, coverage, changed-file lint and Evi typecheck pass.
  • Full typecheck is blocked by the telemetry app’s missing vue-tsc executable.
  • No changeset: changes are confined to apps/evi.

Summary by CodeRabbit

  • New Features
    • Scheduled runs can now push feature branches and open draft or ready pull requests in the home repository without an approval card.
  • Bug Fixes
    • Pull requests targeting other repositories continue to require approval. Scheduled runs still cannot push to main or master, and other write approvals remain unchanged.

@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
evi Ready Ready Preview Oct 6, 2026 7:15am UTC
4 Skipped Deployments
Project Deployment Actions Updated
evlog-docs Skipped Skipped v0 Oct 6, 2026 7:15am UTC
evlog-render-lab Skipped Skipped Oct 6, 2026 7:15am UTC
evlog-telemetry Skipped Skipped Oct 6, 2026 7:15am UTC
just-use-evlog Skipped Skipped Oct 6, 2026 7:15am UTC

Request Review

@vercel
vercel Bot temporarily deployed to Preview – evlog-render-lab October 6, 2026 07:13 Inactive
@vercel
vercel Bot temporarily deployed to Preview – evlog-telemetry October 6, 2026 07:13 Inactive
@vercel
vercel Bot temporarily deployed to Preview – just-use-evlog October 6, 2026 07:13 Inactive
@vercel
vercel Bot temporarily deployed to Preview – evlog-docs October 6, 2026 07:13 Inactive
@github-actions github-actions Bot added the bug Something isn't working label Oct 6, 2026
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Thank you for following the naming conventions! 🙏

@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6466c9d8-2823-4aa5-b00f-a9696abc5c33
📥 Commits

Reviewing files that changed from the base of the PR and between 4113236 and e9f6ce6.

📒 Files selected for processing (4)
  • apps/evi/agent/extensions/github.ts
  • apps/evi/agent/lib/github/pull-request-approval.test.ts
  • apps/evi/agent/lib/github/pull-request-approval.ts
  • apps/evi/docs/notes.md
 _____________________________________________
< Preventing Skynet from becoming self-aware. >
 ---------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@pkg-pr-new

pkg-pr-new Bot commented Oct 6, 2026

Copy link
Copy Markdown
@evlog/cli

npm i https://pkg.pr.new/@evlog/cli@784

evlog

npm i https://pkg.pr.new/evlog@784

@evlog/nuxthub

npm i https://pkg.pr.new/@evlog/nuxthub@784

@evlog/signals

npm i https://pkg.pr.new/@evlog/signals@784

@evlog/telemetry

npm i https://pkg.pr.new/@evlog/telemetry@784

commit: e9f6ce6

@HugoRCD
HugoRCD merged commit 7427fce into main Oct 6, 2026
20 of 22 checks passed
@HugoRCD
HugoRCD deleted the fix/evi-scheduled-pr-approval branch October 6, 2026 07:15

This branch was successfully deployed

1 active and 4 inactive deployments
Preview – evi — e9f6ce63 Deployed Oct 6, 2026 by vercel[bot]
Preview – evlog-render-lab — e9f6ce63 Deployed Oct 6, 2026 by vercel[bot]
Preview – evlog-telemetry — e9f6ce63 Deployed Oct 6, 2026 by vercel[bot]
Preview – just-use-evlog — e9f6ce63 Deployed Oct 6, 2026 by vercel[bot]
Preview – evlog-docs — e9f6ce63 Deployed Oct 6, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant