chore(ai): add agent notes, Cursor rules, and skills - #2133
felipebergamin wants to merge 1 commit into
Conversation
Give coding agents a local map of commands, invariants, and review skills so they follow project conventions without copying onboarding. Signed-off-by: Felipe Bergamin <felipebergamin@profusion.mobi>
|
@alanpeixinho please feel free to suggest your own rules and skills that you find relevant |
|
#1719 also brings some skills on it, might be worth it reviewing. |
| @@ -0,0 +1,40 @@ | |||
| #!/usr/bin/env bash | |||
There was a problem hiding this comment.
what exatcly is this sript?
|
|
||
| ## Git | ||
|
|
||
| Conventional Commits with a scope, e.g. `fix(tree-compare): restore drawer navigation`. One concern per commit/PR. PR titles match; describe why, how, trade-offs, tests; link issues; screenshots/GIFs for UI. CI green before asking for review. Commit with "Signed-off-by". |
There was a problem hiding this comment.
we could include reminder to a github closes tags
| 4. Pass each subagent the same scoped diff/file context and ask it to return prioritized findings with file references and evidence. | ||
| 5. After both finish, synthesize the results with findings first, deduplicated across reviewers. Weight overlapping findings more heavily, resolve disagreements with your own judgment, and keep summaries brief. | ||
|
|
||
| If individual background summaries are already visible to the user, do not restate them wholesale. Surface the unified verdict, the highest-signal findings, and any remaining uncertainty. |
There was a problem hiding this comment.
maybe worth including a guard to ask for human intervention for review after it finishes running pre-review
| @@ -0,0 +1,16 @@ | |||
| --- | |||
There was a problem hiding this comment.
might be worth keeping a kci-dashboard folder, so we can know which skills are specific written for the project
| @@ -0,0 +1,68 @@ | |||
| --- | |||
There was a problem hiding this comment.
is it worth keeping a non development related skill?
alanpeixinho
left a comment
alanpeixinho
left a comment
There was a problem hiding this comment.
Left some comments, none are blockers
This branch has not been deployed
2 participants
Summary
AGENTS.mdwith where-to-look, commands, when/how to run the app, Postgres caveats, and project invariants for coding agents..agentsskills andskills-lock.json.Human onboarding stays in
docs/Onboarding.md/CONTRIBUTING.md; this is the short agent-facing map.Test plan
AGENTS.mdand confirm commands/URLs matchdocs/dev-environment.md(make dev, ports 9000/5173/8000)..cursor/rules/and.agents/skills/load in Cursor for this repo..pnpm-store/is not in the PR.