fix(wiza): surface Wiza's nested error messages by TheodoreSpeaks · Pull Request #8532 · simstudioai/sim · GitHub
Skip to content

fix(wiza): surface Wiza's nested error messages - #8532

Merged
TheodoreSpeaks merged 3 commits into
stagingfrom
fix/wiza
Oct 1, 2026
Merged

fix(wiza): surface Wiza's nested error messages#8532
TheodoreSpeaks merged 3 commits into
stagingfrom
fix/wiza

Conversation

@TheodoreSpeaks

@TheodoreSpeaks TheodoreSpeaks commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Wiza returns errors as {status: {code, message}}, which none of the generic extractors read, so failed Wiza calls only showed Request failed with status 400
  • Added a wiza-errors extractor that reads status.message (falls back to top-level message) and wired it into all four Wiza tools

Type of Change

  • Bug fix

Testing

Added extractor unit tests. Lint, block-registry, and check:audits pass.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

🤖 Generated with Claude Code

https://claude.ai/code/session_01U3y2D5HRzFyhf2a22tgXUd

@vercel

vercel Bot commented Oct 1, 2026 •
edited
Loading

Copy link
Copy Markdown

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 1, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 6 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Oct 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Improves error message display for Wiza API calls.

The PR appears safe to merge; no outstanding findings remain.

Summary

The PR adds a Wiza-specific extractor for nested error messages, connects it to four Wiza tools, and applies it to Individual Reveal polling failures. It also retains plain-text errors and adds tests for the extractor’s fallbacks.

Reviews (3) · Last reviewed commit: "fix(wiza): only accept trimmed string me..."

greptile-apps[bot]
greptile-apps Bot reviewed Oct 1, 2026
View reviewed changes
Comment thread apps/sim/tools/error-extractors.ts Outdated
Comment thread apps/sim/tools/wiza/individual_reveal.ts
@TheodoreSpeaks

TheodoreSpeaks commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

@TheodoreSpeaks

TheodoreSpeaks commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@TheodoreSpeaks I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 1, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 6 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread apps/sim/tools/error-extractors.ts Outdated
@TheodoreSpeaks

TheodoreSpeaks commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

@TheodoreSpeaks

TheodoreSpeaks commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 1, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 6 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@TheodoreSpeaks
TheodoreSpeaks merged commit 7dad107 into staging Oct 1, 2026
32 checks passed
@TheodoreSpeaks
TheodoreSpeaks deleted the fix/wiza branch October 1, 2026 19:22
@waleedlatif1 waleedlatif1 mentioned this pull request Oct 1, 2026
Merged

This branch was previously deployed

1 inactive deployment
Preview — 82481039 Deployed Oct 1, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments
@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@TheodoreSpeaks