)]}'
{".agents/skills/commit-msg-refiner/evals/commit-message-evals.md":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"c32cb6f938321da6a259999996f775020230fad9","unresolved":true,"context_lines":[{"line_number":104,"context_line":"- Do not stage unrelated unstaged changes."},{"line_number":105,"context_line":"- Write the draft to `.tmp/commit-message.txt`."},{"line_number":106,"context_line":"- Wait for user approval or corrections before committing."},{"line_number":107,"context_line":"- Commit with DCO sign-off using `git commit -s -F .tmp/commit-message.txt`."},{"line_number":108,"context_line":"- Inspect the final commit message and amend before completion if needed."},{"line_number":109,"context_line":"- Generate an OpenStack-style message implicitly."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"cc3ffa64_6fdf526f","line":107,"updated":"2026-04-24 13:42:46.000000000","message":"An agent cannot be allowed to mark a commit as signed off on the human\u0027s behalf, because then the agent is potentially committing work which the human has not reviewed. This is because when agents get into plan mode, they can start to make additional changes without direct human visibility while believing the human has agreed to it. If a human wants to stack a bunch of patches, do `git diff HEAD~8` and then decide they are good with that content they can then do `git rebase HEAD~8 -signoff` to inject the signed-off-by.","commit_id":"8dd1a88051008605deea739e9e71092c11a478f7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"708f0d48f84d8ba441a924f65130cf816a5b8e2b","unresolved":true,"context_lines":[{"line_number":104,"context_line":"- Do not stage unrelated unstaged changes."},{"line_number":105,"context_line":"- Write the draft to `.tmp/commit-message.txt`."},{"line_number":106,"context_line":"- Wait for user approval or corrections before committing."},{"line_number":107,"context_line":"- Commit with DCO sign-off using `git commit -s -F .tmp/commit-message.txt`."},{"line_number":108,"context_line":"- Inspect the final commit message and amend before completion if needed."},{"line_number":109,"context_line":"- Generate an OpenStack-style message implicitly."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"66c43b34_5d1c1357","line":107,"in_reply_to":"5deed7df_96db1924","updated":"2026-04-24 20:10:45.000000000","message":"keeping this open because im still happy to refien this to remove -s and explicitly say it ok to flag it or suggest the user does rebase --signoff\nif that what we prefer.","commit_id":"8dd1a88051008605deea739e9e71092c11a478f7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"04717c51074f510c0a477e763be511ecf06fe5c5","unresolved":true,"context_lines":[{"line_number":104,"context_line":"- Do not stage unrelated unstaged changes."},{"line_number":105,"context_line":"- Write the draft to `.tmp/commit-message.txt`."},{"line_number":106,"context_line":"- Wait for user approval or corrections before committing."},{"line_number":107,"context_line":"- Commit with DCO sign-off using `git commit -s -F .tmp/commit-message.txt`."},{"line_number":108,"context_line":"- Inspect the final commit message and amend before completion if needed."},{"line_number":109,"context_line":"- Generate an OpenStack-style message implicitly."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"5deed7df_96db1924","line":107,"in_reply_to":"8032a280_47f3ec4f","updated":"2026-04-24 15:54:33.000000000","message":"i respond in the board discussion doc as well as the behaviour of this has been miss represented https://developercertificate.org/\n\nthe imporant disconenct between the two reading is this line \n\"\"\"\nBy making a contribution to this project, I certify that:\n\"\"\"\n\nyou do not make a contribution by committing locally or pushing to your fork\n\nyou only make a contribution when you publish it to the community code review system.","commit_id":"8dd1a88051008605deea739e9e71092c11a478f7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ccc15c1c0ed08bd5bef501e28c078570dd7f7445","unresolved":true,"context_lines":[{"line_number":104,"context_line":"- Do not stage unrelated unstaged changes."},{"line_number":105,"context_line":"- Write the draft to `.tmp/commit-message.txt`."},{"line_number":106,"context_line":"- Wait for user approval or corrections before committing."},{"line_number":107,"context_line":"- Commit with DCO sign-off using `git commit -s -F .tmp/commit-message.txt`."},{"line_number":108,"context_line":"- Inspect the final commit message and amend before completion if needed."},{"line_number":109,"context_line":"- Generate an OpenStack-style message implicitly."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"8032a280_47f3ec4f","line":107,"in_reply_to":"cc3ffa64_6fdf526f","updated":"2026-04-24 15:13:35.000000000","message":"so it depend on your interpeation of when the DCO takes effect.\nalmost all opensouce licnese only take effect of distibution\ni.e. when i push the commit for revewi not when i actully commit it.\n\nthe current guardrails for this skill also require the user to opt into the git operations.\ni.e. wont ever be used arbitrailly it will be used at teh request of the human which is an attestation of concent to add it.\n\nso like the gpl my intperation of the DCO is it takes effect at the point of distibution not when we commit at the same time the license we are contibuting the patch under takes effect.\n\n\ni cna remove this but i don think its correct to say the dco can never be added by a tool.","commit_id":"8dd1a88051008605deea739e9e71092c11a478f7"}],".agents/skills/commit-msg-refiner/references/commit-message.md":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"19613abf685c70467c97d8493bd5118b1b3f9f7c","unresolved":true,"context_lines":[{"line_number":20,"context_line":"- Explain why the change exists before implementation details."},{"line_number":21,"context_line":"- Make the message self-contained for future reviewers."},{"line_number":22,"context_line":"- Use normal prose in the body; imperative mood is for the subject."},{"line_number":23,"context_line":"- Avoid patch-set notes such as rebases or addressed comments."},{"line_number":24,"context_line":"- Avoid vague bodies such as \"Fix bug 12345\"."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"## Trailers"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"34228a15_5ca0177c","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":62},"updated":"2026-04-21 07:19:57.000000000","message":"this needs to be strghtened as https://review.opendev.org/c/openstack/agentic-workflows/+/985250/2//COMMIT_MSG contaisns some inter revisins wording that should not be there. i may use that as an example fo what not to do.\n\nspecfically \n```\nThe README now frames the shell as optional and points contributors to\nstandard validation documentation for required tools.\n```\n\nis a meta comment on how the current patch frames the nix dev shell vs the previous reviwion but the commit is intoducing it for the first tiem so how it was framed in a prire revison fo the review is irrelvent.","commit_id":"8dd1a88051008605deea739e9e71092c11a478f7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"19613abf685c70467c97d8493bd5118b1b3f9f7c","unresolved":true,"context_lines":[{"line_number":50,"context_line":"- Use `Generated-By:` when a commit is mostly machine-authored."},{"line_number":51,"context_line":"- Use `Assisted-By:` when AI materially assisted but a human directed and"},{"line_number":52,"context_line":"  revised the work."},{"line_number":53,"context_line":"- Never add `Made-with:`."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"## Commit File Workflow"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"ae5cc084_bf6bd042","line":53,"updated":"2026-04-21 07:19:57.000000000","message":"i should proalbly also say to never use Co-Authored-By: for agents. that is stictly for human collablerators.\nnot that it does this currently  but better safe than sorry.","commit_id":"8dd1a88051008605deea739e9e71092c11a478f7"}],"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"19613abf685c70467c97d8493bd5118b1b3f9f7c","unresolved":true,"context_lines":[{"line_number":26,"context_line":"overlay give agents a focused workflow for OpenStack-style commits. New"},{"line_number":27,"context_line":"reference snapshots keep commit-message, AI policy, and DCO context"},{"line_number":28,"context_line":"available without loading it into every prompt."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Generated-By: codex gpt-5.4"},{"line_number":31,"context_line":"Signed-off-by: Sean Mooney \u003cwork@seanmooney.info\u003e"},{"line_number":32,"context_line":"Change-Id: I6af4fc76bc2619a02fe383c42c2098cd4bb64f9b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"aac962a8_424e3103","line":29,"updated":"2026-04-21 07:19:57.000000000","message":"as you might expect this commit was created by this skill\n\nthis skill is basically https://github.com/SeanMooney/openstack-ai-style-guide/blob/master/docs/quick-rules.md + https://github.com/SeanMooney/openstack-ai-style-guide/blob/master/references/commit-message.md converted into a skill using the skill-stub/desigher skills plus dans request on the nova ai scfolding patch to make it exiclty that mutating git operations always require human approval.","commit_id":"8dd1a88051008605deea739e9e71092c11a478f7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7c15a41f4e508d1337d5be30f5d49b9effa3b178","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a230adba_df9982f4","updated":"2026-04-19 11:43:02.000000000","message":"recheck post-failure","commit_id":"7e0048e705955a685b5d6c5456bbeecff15d7298"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5dd6ab1c2bd3675ca5d685417023f86209bec6cf","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"19d2ff61_e2506154","updated":"2026-04-27 15:35:46.000000000","message":"by the way this skill beside beign directly useful and a port of soming i already used direclty demonstrate how i expect skill to be buidlt in this repo.\n\nfirst i selected som refernce matiral that is relevent to the skill\nit was help along with https://github.com/SeanMooney/openstack-ai-style-guide/blob/master/docs/quick-rules.md?plain\u003d1#L114-L135 as prt of the prompt \n\nThe skill-designer workflow skill was used with that refence content to produce the commit-massage topic doc and skill \n\nThe skill-deisgner was used to execute the evals a number of tiems in plan mode and i direct it to \n\nThe process of creating the skill was basicly use the skill-designer ot draft the skill, then use it to refine it, use the skill a bit to find ednge case, feed thos back adn refine again.\n\nthis v3 patch is actually about the 8 iteration fo this based on what i have beend usign in my quick rules file for hte past 8-10 months now in skill form.","commit_id":"b5ddc9d48b0443b3e62f1e6f2fb1cb3b6275022d"}]}