Skip to content

Teach Agora agents to document public contract changes - #1664

Open
kelos-bot[bot] wants to merge 1 commit into
mainfrom
agora-config-update-latest
Open

Teach Agora agents to document public contract changes#1664
kelos-bot[bot] wants to merge 1 commit into
mainfrom
agora-config-update-latest

Conversation

@kelos-bot

@kelos-bot kelos-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

Adds one shared Agora agent instruction requiring matching README, example, or skill documentation whenever an implementation changes a public contract.

This is motivated by Agora PR #7, where a generated implementation changed the required AGORA_URL behavior but initially omitted the corresponding README and skills/agora-reporting updates. The P2 review finding required documentation for the changed public contract, and the responder follow-up confirmed that both documentation surfaces were updated.

The August 26–September 2 review window contained no Agora PR activity, so the audit also covered the repository recent PR set: #1, #2, #3, #6, and #7. Only #7 had substantive review feedback, and no additional configuration rules were added.

Which issue(s) this PR is related to:

N/A

Special notes for your reviewer:

The shared rule lives in self-development/agora/agentconfig.yaml so all Agora implementation agents receive it. Shared skills remain in self-development/base-agent.yaml; this change does not duplicate them in the Agora role AgentConfig.

Validation: make verify

Does this PR introduce a user-facing change?

NONE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant