Skip to content

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

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

feat: fix agent harness — remove out-of-scope platform files#69
afonsoft merged 1 commit into
mainfrom
devin/1780076708-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 <[email protected]>
@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

@afonsoft

Copy link
Copy Markdown
Owner

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@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 2 potential issues.

Open in Devin Review

Comment thread AGENTS.md
Comment on lines +64 to +65
| 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.

🚩 Stale references to deleted DEVIN.md and GEMINI.md in .agents/ files

This PR deletes DEVIN.md and GEMINI.md but does not update the files that still reference them:

  • .agents/CONTEXT.md:16 lists DEVIN.md and GEMINI.md in the priority hierarchy: "Arquivo de plataforma (CLAUDE.md, DEVIN.md, GEMINI.md)"
  • .agents/README.md:8-9 lists both files in the diagram: "DEVIN.md ← Delta para Devin" and "GEMINI.md ← Delta para Gemini CLI"

An agent following the context hierarchy in .agents/CONTEXT.md would attempt to load these non-existent files. These should be updated to reflect the new state (Devin now uses AGENTS.md directly, Gemini row was removed).

Open in Devin Review

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

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.

📝 Info: Windsurf 'Config Principal' lists an ignore file rather than an instruction file

The new Windsurf row sets .windsurfignore as the "Config Principal", but this is an ignore/exclusion file, not a configuration or instruction file. Every other platform lists a markdown instruction file in this column (AGENTS.md, CLAUDE.md). While there is no WINDSURF.md or .windsurfrules in the repo so this may be the best available option, it's semantically inconsistent with the column's purpose. Consider whether AGENTS.md would be more appropriate here (since Windsurf would presumably also use it as its SSoT), or add a note clarifying that Windsurf has no dedicated config file.

Open in Devin Review

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

@afonsoft
afonsoft merged commit ec2a9ff into main May 29, 2026
16 of 18 checks passed
@afonsoft
afonsoft deleted the devin/1780076708-agent-harness-fix branch July 12, 2026 18:26
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