Conversation
Add AI agent guidelines adapted from coder/coder best practices: - AGENTS.md with architecture overview, Go patterns, streaming code guidelines, commit style, essential commands, and common pitfalls - .mux/skills/pull-requests/SKILL.md with PR lifecycle rules, CI follow-up workflow, and pre-push validation checklist - .gitignore entry for AGENTS.local.md (local overrides)
ssncferreira
left a comment
ssncferreira
left a comment
There was a problem hiding this comment.
LGTM 👍 even though we're planning to move aibridge into coder shortly, so this will be short-lived, but could still be useful in the meantime. We'll need to reconcile with coder's AGENTS.md when that happens.
| - Do not push to `origin/main` or `origin/master`. | ||
| - Do not skip local validation before pushing. | ||
| - Do not fabricate or embellish PR descriptions. | ||
| - Do not use `--no-verify` on git operations. |
There was a problem hiding this comment.
I don't think this repo has any git hooks, so this will be a no-op. Nevertheless, good to have if that changes.
| Use LSP tools (go to definition, find references, hover) **before** resorting to grep. | ||
| This codebase has 90+ Go files across multiple packages — LSP is faster and more accurate. | ||
|
|
||
| ## Architecture Overview |
There was a problem hiding this comment.
This section is already covered in the repo readme: https://github.com/coder/aibridge/blob/main/README.md#architecture
In order to avoid maintenance problems (updating both sections to reflect new changes), it might be better to reference the readme file for the full architecture diagram and request flow.
| # Project specific | ||
| example/aibridge.db | ||
|
|
||
| AGENTS.local.md |
|
Firstly, I do appreciate the review @ssncferreira! I'm catching up with the progress and considering the ongoing draft from @pawbana, it may be a little late to introduce this PR. Unless somebody thinks differently, I'm down to closing the PR. |
I don't think code move would change usefulness of this PR? I think there can be multiple such files in single repo? |
3 participants
Changes
AGENTS.md— main agent guidelines document with:.mux/skills/pull-requests/SKILL.md— PR workflow skill with:gh pr checks --watch,gh run view --log-failed).gitignore— addedAGENTS.local.mdfor local overridesContext
Adapted from:
Skipped items not applicable to this repo: frontend, database, git hooks, swagger, OAuth2.