Skip to content

add opencode skill for testgrid#6035

Merged
xavpaice merged 5 commits into
mainfrom
agents
Jun 23, 2026
Merged

add opencode skill for testgrid#6035
xavpaice merged 5 commits into
mainfrom
agents

Conversation

@xavpaice

Copy link
Copy Markdown
Member

What this PR does / why we need it:

Add a skill for Opencode to read Testgrid results and start to analyze them

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

NONE

Does this PR require documentation?

NONE

@xavpaice xavpaice requested a review from a team as a code owner June 21, 2026 21:48
@greptile-apps

greptile-apps Bot commented Jun 21, 2026

Copy link
Copy Markdown

Comments Outside Diff (1)

  1. .gitignore, line 33-34 (link)

    P1 .opencode directory is still gitignored despite committed skill files

    The root .gitignore retains the .opencode entry (in the # ai/editor metadata block), but this PR adds tracked files under .opencode/. Because they were committed via git add -f, existing contributors can work with them fine, but any contributor who tries to add new skill files (e.g., a second skill) will need to use git add -f to bypass the ignore rule — which is non-obvious and easy to miss. The .opencode line should be removed from .gitignore (or narrowed to only ignore transient artifacts like node_modules) so the directory is treated as a normal part of the source tree.

Reviews (1): Last reviewed commit: "add opencode skill for testgrid" | Re-trigger Greptile

Comment thread AGENTS.md Outdated
Comment thread .opencode/.gitignore
Comment thread .envrc Outdated
@greptile-apps

greptile-apps Bot commented Jun 21, 2026

Copy link
Copy Markdown

Reviews (2): Last reviewed commit: "fix .gitignore: remove .opencode so skil..." | Re-trigger Greptile

@greptile-apps

greptile-apps Bot commented Jun 21, 2026

Copy link
Copy Markdown

Want your agent to iterate on Greptile's feedback? Try greploops.

Comment thread .opencode/skills/testgrid-failure-analysis/fetch.py
@greptile-apps

greptile-apps Bot commented Jun 21, 2026

Copy link
Copy Markdown

Reviews (3): Last reviewed commit: "catch URLError alongside HTTPError in te..." | Re-trigger Greptile

@xavpaice

Copy link
Copy Markdown
Member Author

I should document what is in .env

@greptile-apps

greptile-apps Bot commented Jun 23, 2026

Copy link
Copy Markdown

Reviews (4): Last reviewed commit: "add docs for .env" | Re-trigger Greptile

@xavpaice xavpaice merged commit c8759ad into main Jun 23, 2026
7 checks passed
@xavpaice xavpaice deleted the agents branch June 23, 2026 02:47
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.

2 participants