Skip to content

fix: add explicit task boundaries to prevent devin followup changes#4887

Merged
skibitsky merged 2 commits intomainfrom
fix/devin-unity-prompt-no-additional-changes
Aug 25, 2025
Merged

fix: add explicit task boundaries to prevent devin followup changes#4887
skibitsky merged 2 commits intomainfrom
fix/devin-unity-prompt-no-additional-changes

Conversation

@bkrem
Copy link
Copy Markdown
Contributor

@bkrem bkrem commented Aug 21, 2025

Summary

• Added explicit TASK BOUNDARIES section to devin_unity_cdn_update.yml workflow
• Prevents Devin from making additional changes after PR creation

Problem

The Devin session triggered by this workflow had a tendency to make additional followup updates in PRs (e.g. to fix failing tests) when it should NEVER do any actions beyond the ones instructed. The target repo maintainer should have complete control over any additional adjustments needed.

Solution

Added a prominent TASK BOUNDARIES - CRITICAL section that explicitly:

  • Defines task completion as "PR created, work done"
  • Prohibits any additional changes/commits after PR creation
  • States that test failures, CI issues, etc. are NOT Devin's responsibility
  • Emphasizes maintainer has complete control over follow-up work

🤖 Generated with Claude Code

Add TASK BOUNDARIES section to devin_unity_cdn_update.yml workflow to explicitly prevent Devin from making additional changes after PR creation. This addresses the issue where Devin would make followup updates (e.g. fixing failing tests) when it should only perform the specified version bump task.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings August 21, 2025 15:35
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Aug 21, 2025

⚠️ No Changeset found

Latest commit: 5ac8105

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

@vercel
Copy link
Copy Markdown

vercel bot commented Aug 21, 2025

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

Project Deployment Preview Comments Updated (UTC)
appkit-basic-html Ready Ready Preview Comment Aug 25, 2025 7:48am
appkit-demo Ready Ready Preview Comment Aug 25, 2025 7:48am
appkit-gallery Ready Ready Preview Comment Aug 25, 2025 7:48am
appkit-laboratory Ready Ready Preview Comment Aug 25, 2025 7:48am
10 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
appkit-basic-example Ignored Ignored Aug 25, 2025 7:48am
appkit-basic-sign-client-example Ignored Ignored Aug 25, 2025 7:48am
appkit-basic-up-example Ignored Ignored Aug 25, 2025 7:48am
appkit-ethers5-bera Ignored Ignored Aug 25, 2025 7:48am
appkit-nansen-demo Ignored Ignored Aug 25, 2025 7:48am
appkit-vue-solana Ignored Ignored Aug 25, 2025 7:48am
appkit-wagmi-cdn-example Ignored Ignored Aug 25, 2025 7:48am
ethereum-provider-wagmi-example Ignored Ignored Aug 25, 2025 7:48am
next-wagmi-solana-bitcoin-example Ignored Ignored Aug 25, 2025 7:48am
vue-wagmi-example Ignored Ignored Aug 25, 2025 7:48am

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds explicit task boundaries to the Devin Unity CDN update workflow to prevent unwanted follow-up changes after PR creation. The change clarifies that Devin's responsibility ends immediately upon creating a PR, ensuring maintainers retain full control over any subsequent work.

  • Added a critical "TASK BOUNDARIES" section that explicitly prohibits any actions after PR creation
  • Updated reminder text to specify error reporting should only occur before PR creation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 21, 2025

Warnings
⚠️

Workflow file .github/workflows/devin_unity_cdn_update.yml has been modified

Generated by 🚫 dangerJS against 5ac8105

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 21, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 78.45% 34752 / 44296
🔵 Statements 78.45% 34752 / 44296
🔵 Functions 76.18% 3820 / 5014
🔵 Branches 86.74% 8198 / 9451
File CoverageNo changed files found.
Generated in workflow #14499 for commit 5ac8105 by the Vitest Coverage Report Action

@skibitsky skibitsky added this pull request to the merge queue Aug 25, 2025
Merged via the queue into main with commit bfe1a25 Aug 25, 2025
41 checks passed
@skibitsky skibitsky deleted the fix/devin-unity-prompt-no-additional-changes branch August 25, 2025 11:33
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.

4 participants