)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"359b38a108948d0e72964f7a5aa199fb97dc5d22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c8c69110_c0ff6f1a","updated":"2026-04-17 09:42:13.000000000","message":"merging it for now, but of course we can discuss the modifications in a new Gerrit patch.","commit_id":"849de5a2045ca7fbd2f42dd8d0821551c9858376"}],"AGENTS.md":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8ab0a5fdc73c086ce3784968ac504d061e44a4aa","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Skills live under `.agents/skills/` and follow the"},{"line_number":24,"context_line":"[Agent Skills specification](https://agentskills.io/specification)."},{"line_number":25,"context_line":"Knowledge files can also live directly in the OpenStack service repositories"},{"line_number":26,"context_line":"(e.g. `nova/knowledge/nova.md`) if the project community prefers."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"## Design Principles"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"a8c4e59d_86a51c21","line":26,"range":{"start_line":25,"start_character":1,"end_line":26,"end_character":65},"updated":"2026-04-16 20:09:03.000000000","message":"Hmm, not sure this is decided yet. Probably best to leave this out for now?","commit_id":"d95eb6eac1f976faea5ea5f8e19e1528c2c4bfa4"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"29571924bad08297f08c26e1e037bcd3187ecfef","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Skills live under `.agents/skills/` and follow the"},{"line_number":24,"context_line":"[Agent Skills specification](https://agentskills.io/specification)."},{"line_number":25,"context_line":"Knowledge files can also live directly in the OpenStack service repositories"},{"line_number":26,"context_line":"(e.g. `nova/knowledge/nova.md`) if the project community prefers."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"## Design Principles"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"8ffc875f_1f7bf3eb","line":26,"range":{"start_line":25,"start_character":1,"end_line":26,"end_character":65},"in_reply_to":"a8c4e59d_86a51c21","updated":"2026-04-16 20:32:34.000000000","message":"yeah, I\u0027m open to comments, I just wanted to write it super optional and left to be decided by project communities at their own will.","commit_id":"d95eb6eac1f976faea5ea5f8e19e1528c2c4bfa4"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8ab0a5fdc73c086ce3784968ac504d061e44a4aa","unresolved":true,"context_lines":[{"line_number":24,"context_line":"[Agent Skills specification](https://agentskills.io/specification)."},{"line_number":25,"context_line":"Knowledge files can also live directly in the OpenStack service repositories"},{"line_number":26,"context_line":"(e.g. `nova/knowledge/nova.md`) if the project community prefers."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"## Design Principles"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- **Do not duplicate deterministic checks.** If a linter or CI job already"},{"line_number":31,"context_line":"  enforces a rule, the skill should not re-check it."},{"line_number":32,"context_line":"- **Use in-tree docs as the source of truth.** Reference each project\u0027s"},{"line_number":33,"context_line":"  contributor documentation rather than forking rules into this repository."},{"line_number":34,"context_line":"- **Open format, zero duplication.** Skills, personas, and rules are authored"},{"line_number":35,"context_line":"  once in the agentskills format and discovered by any compatible AI client."},{"line_number":36,"context_line":"- **Project-agnostic skills.** Skills dynamically resolve project context at"},{"line_number":37,"context_line":"  invocation time. Adding support for a new project requires only new knowledge"},{"line_number":38,"context_line":"  and persona files — no skill changes."},{"line_number":39,"context_line":"- **Human decides, agent assists.** Skills provide analysis and draft comments,"},{"line_number":40,"context_line":"  but the human makes final decisions (e.g. Gerrit votes). Never automate"},{"line_number":41,"context_line":"  actions that require human judgement."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"## Skill Authoring Guidelines"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"All skills must follow the"},{"line_number":46,"context_line":"[Agent Skills specification](https://agentskills.io/specification). The"},{"line_number":47,"context_line":"guidelines below highlight the most important conventions for this repository."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"### Frontmatter"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Every `SKILL.md` starts with YAML frontmatter:"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"dc8ea16c_f0e70142","line":48,"range":{"start_line":27,"start_character":1,"end_line":48,"end_character":1},"updated":"2026-04-16 20:09:03.000000000","message":"This is duplicated from the `README.rst`. Can we drop it from one of the two?","commit_id":"d95eb6eac1f976faea5ea5f8e19e1528c2c4bfa4"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"30e46bab2e0d40dfd967e7aead68470702ce13cf","unresolved":false,"context_lines":[{"line_number":24,"context_line":"[Agent Skills specification](https://agentskills.io/specification)."},{"line_number":25,"context_line":"Knowledge files can also live directly in the OpenStack service repositories"},{"line_number":26,"context_line":"(e.g. `nova/knowledge/nova.md`) if the project community prefers."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"## Design Principles"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- **Do not duplicate deterministic checks.** If a linter or CI job already"},{"line_number":31,"context_line":"  enforces a rule, the skill should not re-check it."},{"line_number":32,"context_line":"- **Use in-tree docs as the source of truth.** Reference each project\u0027s"},{"line_number":33,"context_line":"  contributor documentation rather than forking rules into this repository."},{"line_number":34,"context_line":"- **Open format, zero duplication.** Skills, personas, and rules are authored"},{"line_number":35,"context_line":"  once in the agentskills format and discovered by any compatible AI client."},{"line_number":36,"context_line":"- **Project-agnostic skills.** Skills dynamically resolve project context at"},{"line_number":37,"context_line":"  invocation time. Adding support for a new project requires only new knowledge"},{"line_number":38,"context_line":"  and persona files — no skill changes."},{"line_number":39,"context_line":"- **Human decides, agent assists.** Skills provide analysis and draft comments,"},{"line_number":40,"context_line":"  but the human makes final decisions (e.g. Gerrit votes). Never automate"},{"line_number":41,"context_line":"  actions that require human judgement."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"## Skill Authoring Guidelines"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"All skills must follow the"},{"line_number":46,"context_line":"[Agent Skills specification](https://agentskills.io/specification). The"},{"line_number":47,"context_line":"guidelines below highlight the most important conventions for this repository."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"### Frontmatter"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Every `SKILL.md` starts with YAML frontmatter:"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"2264b2e0_9915cb83","line":48,"range":{"start_line":27,"start_character":1,"end_line":48,"end_character":1},"in_reply_to":"a8814684_2e92b4c2","updated":"2026-04-17 09:32:26.000000000","message":"Let\u0027s continue this discussion on the first patch.","commit_id":"d95eb6eac1f976faea5ea5f8e19e1528c2c4bfa4"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"29571924bad08297f08c26e1e037bcd3187ecfef","unresolved":true,"context_lines":[{"line_number":24,"context_line":"[Agent Skills specification](https://agentskills.io/specification)."},{"line_number":25,"context_line":"Knowledge files can also live directly in the OpenStack service repositories"},{"line_number":26,"context_line":"(e.g. `nova/knowledge/nova.md`) if the project community prefers."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"## Design Principles"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- **Do not duplicate deterministic checks.** If a linter or CI job already"},{"line_number":31,"context_line":"  enforces a rule, the skill should not re-check it."},{"line_number":32,"context_line":"- **Use in-tree docs as the source of truth.** Reference each project\u0027s"},{"line_number":33,"context_line":"  contributor documentation rather than forking rules into this repository."},{"line_number":34,"context_line":"- **Open format, zero duplication.** Skills, personas, and rules are authored"},{"line_number":35,"context_line":"  once in the agentskills format and discovered by any compatible AI client."},{"line_number":36,"context_line":"- **Project-agnostic skills.** Skills dynamically resolve project context at"},{"line_number":37,"context_line":"  invocation time. Adding support for a new project requires only new knowledge"},{"line_number":38,"context_line":"  and persona files — no skill changes."},{"line_number":39,"context_line":"- **Human decides, agent assists.** Skills provide analysis and draft comments,"},{"line_number":40,"context_line":"  but the human makes final decisions (e.g. Gerrit votes). Never automate"},{"line_number":41,"context_line":"  actions that require human judgement."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"## Skill Authoring Guidelines"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"All skills must follow the"},{"line_number":46,"context_line":"[Agent Skills specification](https://agentskills.io/specification). The"},{"line_number":47,"context_line":"guidelines below highlight the most important conventions for this repository."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"### Frontmatter"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"Every `SKILL.md` starts with YAML frontmatter:"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"a8814684_2e92b4c2","line":48,"range":{"start_line":27,"start_character":1,"end_line":48,"end_character":1},"in_reply_to":"dc8ea16c_f0e70142","updated":"2026-04-16 20:32:34.000000000","message":"see my comment on the first patch of the series, we unfortunately have a bit of redundancy between the two files as they\u0027re intended to be consumed by different categories : LLMs vs. humans.\n\nThat said, I\u0027m again, I\u0027m open to other comments.","commit_id":"d95eb6eac1f976faea5ea5f8e19e1528c2c4bfa4"}]}