Skip to main content

Frequently asked questions

Can I customize the prompt Shaka gives its local reviewer?​

Not the prompt text yet. Today, add your project's review criteria to AGENTS.md on the default branch. The agent passes that commit to shaka review run --criteria-ref, which adds the root AGENTS.md and any AGENTS.md in a directory the change touches. See invoke a reviewer locally.

A few parts of the prompt are protocol rather than policy. The closing REVIEWED line lets shaka review run confirm which commit was reviewed. The markers around the diff keep text in a contributor's change from reading as instructions.

Can a PR change Shaka's settings or rules for itself?​

Not until it merges. Settings and review criteria come from the default branch, so each PR is reviewed under the rules its maintainers already agreed on, and a PR from a fork cannot change how it is reviewed. Until it merges, Shaka reviews a settings change as part of the diff. See settings.

Where do I put project rules?​

WhatWhere
Commands, merge policy, and review jobsSettings in .agents/agent-workflow.yml
Project constraints, review criteria, and writing styleAGENTS.md
Changes to Shaka's workflowA fork of Shaka

Keep AGENTS.md a regular file with the instructions in it; Shaka reads it from Git at the trusted commit. Recent Claude Code versions can read AGENTS.md directly, but not in every session, and not by default when a CLAUDE.md exists. To give Claude the same instructions every time, add a CLAUDE.md containing @AGENTS.md rather than a symlink. See Claude Code's AGENTS.md support.

Why did the agent stop to ask about model and effort?​

Before it changes code, Shaka recommends a model and effort for the task and waits for you. To skip the wait, name the model and effort it would recommend, make sure they are active, and say “Go”. See give it an outcome.

Why didn't the agent merge my PR?​

Under Ask, the default, the agent labels the PR awaiting-merge-approval and names the commit to merge. Merge it on GitHub, or approve that commit and the agent merges it. Under Auto, a PR past the size limits, or one that changes trust, authentication, releases, or other consequential areas, still comes back to you. See choose a merge policy.

Does a green review job mean the PR was reviewed?​

No. A job can pass without producing a review. The agent reads the report and counts only one that covers the current commit. See review.ci_review_jobs.

Can the agent review its own work?​

It prefers a reviewer from a different provider, in the order set by review.local_review_agents. Next it uses a listed reviewer from the same provider. When no listed reviewer is available, a fresh session with the implementation model reviews the change without the implementation conversation.

Why did the agent ignore a PR comment?​

Shaka trusts the people who maintain the project: anyone with write access, plus the users, bots, and teams listed in .agents/trusted-github-actors.yml or your machine's allowlist. On a public repository, anyone else can comment, and that text could try to steer the agent. The agent withholds those comments and keeps their links for you to read. See configure trusted actors.

What does Shaka enforce, and what relies on the agent?​

Ruby checks settings, trusted comment authors, and the reviewed commit at merge. Test quality, screenshots, independent review, and privacy rely on the agent's judgment. See what is enforced.