Skip to content

Improve orca.yaml invalid-state UI with repair guidance - #405

Merged
AmethystLiang merged 2 commits into
mainfrom
add-instruction-for-orca-yaml
Apr 9, 2026
Merged

Improve orca.yaml invalid-state UI with repair guidance#405
AmethystLiang merged 2 commits into
mainfrom
add-instruction-for-orca-yaml

Conversation

@AmethystLiang

Copy link
Copy Markdown
Contributor

Summary

  • Replaces the terse "fix the file format" message when orca.yaml is invalid with a structured warning card showing parse-error explanation, numbered repair steps, and the example template
  • Extracts ExampleTemplateCard component to reuse the template block across both the "no file" and "invalid file" states
  • Restores the structured return type on hooks.check preload API to preserve type safety at the IPC boundary
  • Removes a redundant setupConfig guard in AddWorktreeDialog (the resolved decision already accounts for missing config)

Test plan

  • Open Settings for a repo without orca.yaml — verify the example template card renders correctly
  • Add a malformed orca.yaml to a repo — verify the warning card with numbered fixes and template appears
  • Add a valid orca.yaml — verify the loaded state shows script preview as before
  • Create a new worktree for a repo with/without setup hooks — verify setup decision is passed correctly

@AmethystLiang
AmethystLiang merged commit 90d3fd3 into main Apr 9, 2026
1 check passed
@AmethystLiang
AmethystLiang deleted the add-instruction-for-orca-yaml branch April 9, 2026 04:38
thomaszdxsn pushed a commit to thomaszdxsn/orca that referenced this pull request Jun 15, 2026
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.

1 participant