hubcio opened a new pull request, #4244: URL: https://github.com/apache/iggy/pull/4244
Review runs by hand today: a maintainer copies a diff into a local agent and pastes the findings back. The triage commands already turn a comment into an action, so a review can ride the same path. A committer comments `/skill <name>`. One workflow parses the command, gates the author, installs Rust with nextest, serena, rtk and the Simple English plugin, checks out the pull request head and runs the repo skill headlessly against DeepSeek, with each tool call streamed into the job log. A second, on workflow_run where the token can write, posts findings.json as one review with inline comments. The skill, the settings and the instruction files come from the base branch, never from the pull request under review, and the agent runs with --setting-sources user and --strict-mcp-config so project settings and foreign MCP servers never load. The first workflow carries no write token and nothing in it posts, because an issue_comment run of a fork receives a read-only token. The session system prompt is the one serena prints for Claude Code, and serena's session-start and reminder hooks are wired in, because an MCP tool list alone left the agent on grep and sed. The agent reads code rather than running tests. What CI did on the commit reaches it as a file, and it builds the binaries under core/integration first when a test is the only way to settle a finding. Each review opens with the skill's own verdict, the reason and a count per severity. The job log closes with the cost, priced at DeepSeek rates and at two Claude tiers. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
