Skip to content

Release: merge develop into main (#41–#43) #44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ZappoMan merged 3 commits into from
May 10, 2026
Merged

Release: merge develop into main (#41–#43) #44

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
853f54c
fix(security): resolve Dependabot alerts via npm overrides and lockfi…
ZappoMan May 10, 2026
fce2805
fix: GitHub Code Quality AI findings (DebugTools, test helpers) (#42)
ZappoMan May 10, 2026
74a51bc
more AI feedback (#43)
ZappoMan May 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 18 additions & 8 deletions apps/mobile/src/components/DebugTools.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import {
profileFormSchema,
type ProfileFormInput,
} from '@beakerstack/shared/validation/profileSchema';
import { ZodError } from 'zod';
import { ZodError, type ZodIssue } from 'zod';
import { TextInput } from 'react-native';

export function DebugTools() {
Expand Down Expand Up @@ -55,7 +55,7 @@ export function DebugTools() {
clearTimeout(clickTimeoutRef.current);
}
};
}, [clickCount]);
}, []);

if (!isVisible) {
return (
Expand Down Expand Up @@ -441,9 +441,14 @@ function ValidationTestFormMobile() {
>(null);

const handleFieldChange = (field: keyof ProfileFormInput, value: string) => {
setFormData(prev => ({ ...prev, [field]: value }));
setFormData(prev =>
prev[field] === value ? prev : { ...prev, [field]: value }
);
if (errors[field]) {
setErrors(prev => {
if (!prev[field]) {
return prev;
}
const newErrors = { ...prev };
delete newErrors[field];
return newErrors;
Expand All @@ -456,13 +461,18 @@ function ValidationTestFormMobile() {
try {
profileFormSchema.parse({ ...formData, [field]: value });
setErrors(prev => {
if (!prev[field]) {
return prev;
}
const newErrors = { ...prev };
delete newErrors[field];
return newErrors;
});
} catch (err) {
if (err instanceof ZodError) {
const fieldError = err.errors.find(e => e.path.includes(field));
const fieldError = err.issues.find((issue: ZodIssue) =>
issue.path.includes(field)
);
if (fieldError) {
setErrors(prev => ({ ...prev, [field]: fieldError.message }));
}
Expand All @@ -479,14 +489,14 @@ function ValidationTestFormMobile() {
} catch (err) {
if (err instanceof ZodError) {
const newErrors: Record<string, string> = {};
err.errors.forEach(error => {
const field = error.path[0] as string;
err.issues.forEach((issue: ZodIssue) => {
const field = issue.path[0] as string;
if (field) {
newErrors[field] = error.message;
newErrors[field] = issue.message;
}
});
setErrors(newErrors);
const errorCount = err.errors.length;
const errorCount = err.issues.length;
Alert.alert(
'Validation Failed',
`❌ Validation failed: ${errorCount} error(s)`
Expand Down
Loading
Loading