Skip to content

Refactor response validation#1264

Open
JackWilb wants to merge 4 commits into
devfrom
codex/response-validation-cleanup
Open

Refactor response validation#1264
JackWilb wants to merge 4 commits into
devfrom
codex/response-validation-cleanup

Conversation

@JackWilb

@JackWilb JackWilb commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a pure validateResponse API as the single source of truth for response validity
  • make Mantine form validation and response issue summaries delegate to the pure validator
  • extract response-value aggregation helpers in ResponseBlock
  • add focused unit coverage for current validation behavior and ResponseBlock feedback/error guards

Verification

  • yarn unittest --run src/components/response/tests/utils.spec.ts src/components/response/tests/ResponseBlock.spec.tsx
  • yarn typecheck
  • yarn lint
  • git diff --check

@github-actions

github-actions Bot commented Jun 11, 2026 •
edited
Loading

Copy link
Copy Markdown

A preview of 21f0486 is uploaded and can be seen here:

✨ https://revisit.dev/study/PR1264 ✨

Changes may take a few minutes to propagate.

jaykim1213
jaykim1213 requested changes Jun 23, 2026
View reviewed changes
Comment thread .github/workflows/lint_build.yaml
Comment thread .github/workflows/lint_build.yaml
Comment thread src/components/response/utils.ts
Comment thread src/components/response/responseErrors.ts Outdated
@JackWilb JackWilb requested a review from jaykim1213 June 24, 2026 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jaykim1213 jaykim1213 jaykim1213 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JackWilb @jaykim1213