)]}'
{".skillsaw.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"dd3447acb0cf5c2059f08e049edede43026d5a11","unresolved":true,"context_lines":[{"line_number":82,"context_line":"  hooks-dangerous:"},{"line_number":83,"context_line":"    enabled: auto"},{"line_number":84,"context_line":"    severity: error"},{"line_number":85,"context_line":"  hooks-prohibited:"},{"line_number":86,"context_line":"    enabled: true"},{"line_number":87,"context_line":"    severity: error"},{"line_number":88,"context_line":"    allowlist: []"},{"line_number":89,"context_line":"  mcp-prohibited:"},{"line_number":90,"context_line":"    enabled: true"},{"line_number":91,"context_line":"    severity: error"},{"line_number":92,"context_line":"    allowlist: []"},{"line_number":93,"context_line":"  settings-dangerous:"},{"line_number":94,"context_line":"    enabled: auto"},{"line_number":95,"context_line":"    severity: error"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f7af36bb_00daaa08","line":92,"range":{"start_line":85,"start_character":0,"end_line":92,"end_character":17},"updated":"2026-06-26 23:21:50.000000000","message":"i guess this may answer my prevsiou quesiton \n\ni think we really woudl want to start at least with blocking the addtion of all hooks or mcps to this repo.","commit_id":"c6b118a2eee2eaa0437b38b2eeed75cd7eedc036"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3d3f685d624a878096f05fcff4fb1b0aab05800d","unresolved":true,"context_lines":[{"line_number":82,"context_line":"  hooks-dangerous:"},{"line_number":83,"context_line":"    enabled: auto"},{"line_number":84,"context_line":"    severity: error"},{"line_number":85,"context_line":"  hooks-prohibited:"},{"line_number":86,"context_line":"    enabled: true"},{"line_number":87,"context_line":"    severity: error"},{"line_number":88,"context_line":"    allowlist: []"},{"line_number":89,"context_line":"  mcp-prohibited:"},{"line_number":90,"context_line":"    enabled: true"},{"line_number":91,"context_line":"    severity: error"},{"line_number":92,"context_line":"    allowlist: []"},{"line_number":93,"context_line":"  settings-dangerous:"},{"line_number":94,"context_line":"    enabled: auto"},{"line_number":95,"context_line":"    severity: error"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"375dfc65_97c50e87","line":92,"range":{"start_line":85,"start_character":0,"end_line":92,"end_character":17},"in_reply_to":"da4cb8be_89f50084","updated":"2026-06-27 10:41:45.000000000","message":"if we want to buidl mcp server in this repo we could but we need to consider how those would be packaged. mcp server are not bad in theory you just need to limit the number fo tools they provide and the number you ahve enabeld.\n\nideally you do not use them directly but rather programaticl to repseve context.\n\ni think until we figure out how to ship them and make them optional we should not add them at all but if we find it useful to build a specific one in the future we could.\n\ni would be more inclined to create a cli + skill to use it instead but that more a prefosnal bias as i like the unix style composablity that brings and the fact a human can also use a cli trivialy \n\ni wanted to play with rust so i built \n\nhttps://github.com/SeanMooney/grt#read-review-comments specifcly to provied a trivaly way for llm to get review comment form gerrit so that you can use that to directly plan how to adress review feedback.\n\nmcp serves exist for gerrit but i dont need it for every task i do and manging enabeld mcp service per session is a lot more effort then havign a cli just telling the agent ot use it or creating a skill for it\n\nhttps://github.com/SeanMooney/grt/blob/master/skills/grt/SKILL.md\ni currntly disbale the skill because the skill tend to make it do extra check that are not needed due to the Prerequisites block so it need a littel polishing but if you mention a cli by name in a promt most agent will run \u003ccli\u003e --help if its not in there trainign data and \"just work\"\n\n\nhooks are more problematic because there is no standard for them, they are less visabel and tend to only work for specific tools.. iwhile the core of a hook may be generic the hook poitns in a given tools differ form other tools","commit_id":"c6b118a2eee2eaa0437b38b2eeed75cd7eedc036"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"666aadcdb1dc1aee3cbdb292cb180506ef0c7b67","unresolved":true,"context_lines":[{"line_number":82,"context_line":"  hooks-dangerous:"},{"line_number":83,"context_line":"    enabled: auto"},{"line_number":84,"context_line":"    severity: error"},{"line_number":85,"context_line":"  hooks-prohibited:"},{"line_number":86,"context_line":"    enabled: true"},{"line_number":87,"context_line":"    severity: error"},{"line_number":88,"context_line":"    allowlist: []"},{"line_number":89,"context_line":"  mcp-prohibited:"},{"line_number":90,"context_line":"    enabled: true"},{"line_number":91,"context_line":"    severity: error"},{"line_number":92,"context_line":"    allowlist: []"},{"line_number":93,"context_line":"  settings-dangerous:"},{"line_number":94,"context_line":"    enabled: auto"},{"line_number":95,"context_line":"    severity: error"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"da4cb8be_89f50084","line":92,"range":{"start_line":85,"start_character":0,"end_line":92,"end_character":17},"in_reply_to":"f7af36bb_00daaa08","updated":"2026-06-27 07:13:22.000000000","message":"awesome; yes.. We could squash these commits. \n\nI recall your arguments for this at the PTG, and agree with the points made regarding insecure/malicious hooks/mcps - which made me dig this out of the plugins skillsaw supports..","commit_id":"c6b118a2eee2eaa0437b38b2eeed75cd7eedc036"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"3d3f685d624a878096f05fcff4fb1b0aab05800d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f193921f_cb29e2f4","updated":"2026-06-27 10:41:45.000000000","message":"if you want to squash thsese that would be fine but i think this is also ok","commit_id":"c6b118a2eee2eaa0437b38b2eeed75cd7eedc036"}]}