Phase 0.9: Error messages & UX fixes (items 4-6)#78
Merged
Conversation
Pipe "y\n" to stdin for arg-mode CLI invocations (Anthropic, Google) so the --dangerously-skip-permissions Y/n warning doesn't block automated runs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Missing provider backend: suggests adding backend to run config - Missing API key: suggests exporting the env var or switching to CLI - Missing llm_provider: suggests adding a model_stylesheet rule - Unknown model ID: suggests checking spelling and format - Validation errors now include Fix hints when available Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
When a tool_command fails with "not found" or "No such file", check whether the referenced script exists in the source repo but not the worktree. If so, hint that the file needs to be committed to git. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The config validator catches empty/invalid backend values before the runtime check does. Add hint suggesting the fix. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
leegonzales
added a commit
to leegonzales/kilroy
that referenced
this pull request
Apr 3, 2026
engine.go — struct literal alignment drift from PR danshapiro#76 (decision logging) worktree_hint_test.go — comment alignment drift from PR danshapiro#78 (error UX) No logic changes. Pure whitespace. gofmt -l . → clean go vet ./... → clean go build ./... → clean 🤖 Servitor heartbeat fix — unblocks main CI Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
y\nto stdin for Anthropic/Google CLI invocations so--dangerously-skip-permissionsY/n warning doesn't block automated runshint:lines to common first-run errors (missing backend, missing API key, missing llm_provider, unknown model ID, invalid backend config)tool_commandfails with "not found", check if the script exists in the source repo but not the worktree and suggestgit add && git commitTest plan
TestRunWithConfig_FailsFastWhenProviderBackendMissingverifies hint presenceTestValidate_LLMProviderRequired_Metaspecverifies Fix field on diagnosticTestWorktreeNotFoundHint_*andTestExtractLeadingPathcover hint logic🤖 Generated with Claude Code