Skip to content

Parse command config with Effect Schema#26801

Merged
kitlangton merged 1 commit into
devfrom
cleanup/config-command-effect-parse
May 11, 2026
Merged

Parse command config with Effect Schema#26801
kitlangton merged 1 commit into
devfrom
cleanup/config-command-effect-parse

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented May 11, 2026

Summary

  • Replace command config Info.zod.safeParse with an Effect Schema decoder.
  • Preserve invalid-command errors while reporting Effect parse causes.
  • Remove the now-unused ConfigCommand.Info.zod compatibility static.

Testing

  • bun typecheck from packages/opencode
  • bun test --timeout 5000 test/config/config.test.ts -t "loads commands"
  • bunx prettier --check packages/opencode/src/config/command.ts
  • bunx oxlint packages/opencode/src/config/command.ts

Stack

  1. Validate prompt messages with Effect Schema #26796
  2. Parse command config with Effect Schema #26801 👈 current
  3. Define project update input with Effect Schema #26803

@kitlangton kitlangton force-pushed the cleanup/message-v2-effect-parse branch from 26466c5 to 0417d3b Compare May 11, 2026 02:45
@kitlangton kitlangton force-pushed the cleanup/config-command-effect-parse branch from f7cb012 to f8c1758 Compare May 11, 2026 02:45
@kitlangton kitlangton force-pushed the cleanup/message-v2-effect-parse branch from 0417d3b to 983c482 Compare May 11, 2026 02:50
@kitlangton kitlangton force-pushed the cleanup/config-command-effect-parse branch from f8c1758 to cbd8f75 Compare May 11, 2026 02:50
Base automatically changed from cleanup/message-v2-effect-parse to dev May 11, 2026 02:59
@kitlangton kitlangton force-pushed the cleanup/config-command-effect-parse branch from cbd8f75 to cc9c3d7 Compare May 11, 2026 13:52
@kitlangton kitlangton force-pushed the cleanup/config-command-effect-parse branch 2 times, most recently from cc9c3d7 to 7954897 Compare May 11, 2026 14:21
@kitlangton kitlangton merged commit 64c5042 into dev May 11, 2026
6 of 8 checks passed
@kitlangton kitlangton deleted the cleanup/config-command-effect-parse branch May 11, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant