Skip to content

More review agent refinements#6158

Merged
jasnell merged 6 commits intomainfrom
jasnell/agent-refinements-3
Feb 25, 2026
Merged

More review agent refinements#6158
jasnell merged 6 commits intomainfrom
jasnell/agent-refinements-3

Conversation

@jasnell
Copy link
Collaborator

@jasnell jasnell commented Feb 24, 2026

  • Cover rust, js, and ts checklists
  • Look for copyright headers
  • Add a skill for identifying the current user for better handling in code review and find-owners
  • Flag dependency updates in code reviews

@jasnell jasnell requested review from a team as code owners February 24, 2026 20:08
@ask-bonk
Copy link
Contributor

ask-bonk bot commented Feb 24, 2026

The discoverRustCrate function uses substring matching (.includes()), which could match unintended crates. For example, searching for "rand" would match "rand_core", "rand_chacha", etc. This is a known trade-off for fuzzy matching and the function caps results at 3, so it's not a correctness bug.

After reviewing all the changes thoroughly, this PR is well-structured documentation and tooling for AI agent workflows. The changes are:

  • New skills for Rust review, TS style, and reviewer identification
  • New bazel-deps tool for dependency impact analysis
  • Updates to existing commands to support Rust and multi-language workflows
  • New commands (dep-impact, rdeps)

There are no runtime code changes, no security implications, no backward compatibility concerns. The code quality of the TypeScript tool is good. The minor issues I found (inconsistent trailing slash on src/pyodide, extra single quotes in bazelQuery) are not actionable in the sense of being bugs.

LGTM

github run

@jasnell jasnell force-pushed the jasnell/agent-refinements-3 branch from 8b63edc to ee9b4de Compare February 25, 2026 14:59
@jasnell jasnell enabled auto-merge February 25, 2026 15:07
@jasnell jasnell merged commit 3fe4e81 into main Feb 25, 2026
22 of 23 checks passed
@jasnell jasnell deleted the jasnell/agent-refinements-3 branch February 25, 2026 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants