Skip to content

feat: fix agent harness — remove out-of-scope platform files#55

Merged
afonsoft merged 1 commit into
mainfrom
devin/1780076709-agent-harness-fix
May 29, 2026
Merged

feat: fix agent harness — remove out-of-scope platform files#55
afonsoft merged 1 commit into
mainfrom
devin/1780076709-agent-harness-fix

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented May 29, 2026

Copy link
Copy Markdown
Contributor

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?

Description

Per the Agent Harness Production-Ready playbook, scope is limited to Claude Code, Devin, and Windsurf.

Changes:

  • Removed: DEVIN.md — Devin reads AGENTS.md directly (no dedicated file needed)
  • Removed: GEMINI.md — Gemini CLI is out of playbook scope
  • Updated: AGENTS.md platform table — now shows only Base, Claude Code, Devin, Windsurf

Impact: No functional code changes. Only documentation/harness files affected. Devin now uses AGENTS.md as its single source of truth.

Link to Devin session: https://app.devin.ai/sessions/3b8be10535714264b5ab7946c1f110c3
Requested by: @afonsoft


Open in Devin Review

- Remove DEVIN.md (Devin reads AGENTS.md directly)
- Remove GEMINI.md (Gemini CLI out of scope)
- Update AGENTS.md platform table: keep only Claude Code, Devin, Windsurf
- Devin now references AGENTS.md directly instead of DEVIN.md

Scope: Claude Code, Devin, Windsurf only (no Cursor, Gemini, JetBrains)
Co-Authored-By: Afonso Dutra Nogueira Filho <afonsoft@gmail.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@sonarqubecloud

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread AGENTS.md
| Devin | `DEVIN.md` | `.agents/skills/` | `.agents/RULES.md` |
| Gemini CLI | `GEMINI.md` | `.agents/skills/` | `.agents/RULES.md` |
| Devin | `AGENTS.md` | `.agents/skills/` | `.agents/RULES.md` |
| Windsurf | `.windsurfignore` | `.agents/skills/` | `rules/*.instructions.md` |

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🚩 Windsurf config file referenced in table does not exist

The new Windsurf row in the platform table (AGENTS.md:61) lists .windsurfignore as the Config Principal, but no such file exists in the repository. While the deleted DEVIN.md and GEMINI.md previously existed when they were referenced, this new entry references a non-existent file. This may be intentional (documenting the expected convention for a future Windsurf setup), but it's worth confirming whether .windsurfignore or .windsurfrules should be created as part of this PR or a follow-up.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@afonsoft afonsoft merged commit 838028c into main May 29, 2026
15 of 17 checks passed
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.

1 participant