Skip to content

fix(evi): have first responder reply directly instead of narrating a draft - #572

Merged
HugoRCD merged 1 commit into
mainfrom
fix/evi-first-responder-reply
Aug 14, 2026
Merged

fix(evi): have first responder reply directly instead of narrating a draft#572
HugoRCD merged 1 commit into
mainfrom
fix/evi-first-responder-reply

Conversation

@evlogai

@evlogai evlogai Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

What

Rewrites the first-responder instruction so the reply reads as a direct comment from Evi to the reporter, not as a meta "draft for the channel".

Why

On issue #571 the posted comment led with "Labels bug and adapter are applied. I can't post the comment directly on this autonomous turn, so here's the draft reply for the channel:" and a "Draft comment for #571" banner. The cause is in the instruction itself, which said: "Keep the reply to one comment, the draft response Hugo builds on… the channel posts the comment." That phrasing told the model it was producing a draft and narrating the process, so the channel then posted that narration verbatim as the comment.

The new text makes it explicit that the single reply the model writes is the comment the channel posts, that it is addressed to the reporter, and that it must never describe drafts, approval, or channel mechanics.

No authorization change

Label tools are already authorized for autonomous turns (addLabels: autonomousWrite, createLabel: createLabelPolicy in apps/evi/agent/extensions/github.ts); that is why bug + adapter applied correctly on #571. The first-responder eval still pins the design: the reply is posted by the channel and github__addIssueComment stays out of the autonomous toolset, so no permission or eval change is needed.

Checks

  • pnpm --filter evi run typecheck: exit 0
  • pnpm run lint: 22 tasks successful, exit 0
  • Change is confined to apps/evi (instruction text only), so no changeset is required.

@changeset-bot

changeset-bot Bot commented Aug 13, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 205fa93

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@evlogai
evlogai Bot requested a review from HugoRCD August 13, 2026 11:57
@vercel

vercel Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
evi Ready Ready Preview Aug 13, 2026 11:58am
4 Skipped Deployments
Project Deployment Actions Updated (UTC)
evlog-docs Skipped Skipped v0 Aug 13, 2026 11:58am
evlog-render-lab Skipped Skipped Aug 13, 2026 11:58am
evlog-telemetry Skipped Skipped Aug 13, 2026 11:58am
just-use-evlog Skipped Skipped Aug 13, 2026 11:58am

Request Review

@github-actions github-actions Bot added the bug Something isn't working label Aug 13, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for following the naming conventions! 🙏

@pkg-pr-new

pkg-pr-new Bot commented Aug 13, 2026

Copy link
Copy Markdown
npm i https://pkg.pr.new/@evlog/cli@572
npm i https://pkg.pr.new/evlog@572
npm i https://pkg.pr.new/@evlog/nuxthub@572
npm i https://pkg.pr.new/@evlog/telemetry@572

commit: 205fa93

@HugoRCD
HugoRCD merged commit e53f3c3 into main Aug 14, 2026
22 checks passed
@HugoRCD
HugoRCD deleted the fix/evi-first-responder-reply branch August 14, 2026 16:07
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