Skip to content

[Web UI] Replace any types in ExecutionForm.tsx#312

Open
CommitHu502Craft wants to merge 1 commit intoAgent-Field:mainfrom
CommitHu502Craft:fix-105-execution-form-types
Open

[Web UI] Replace any types in ExecutionForm.tsx#312
CommitHu502Craft wants to merge 1 commit intoAgent-Field:mainfrom
CommitHu502Craft:fix-105-execution-form-types

Conversation

@CommitHu502Craft
Copy link
Copy Markdown
Contributor

Summary

  • replace the unsafe any usages in ExecutionForm.tsx with local typed helpers for refs, input props, and synthetic change events
  • introduce an ExecutionFormData shape so the form and its caller share a typed contract instead of SetStateAction<any>
  • tighten react-hook-form usage in the execution form and memoize the detail page loaders to satisfy hook dependency linting

Testing

  • npx tsc -b
  • npx eslint src/components/reasoners/ExecutionForm.tsx src/pages/ReasonerDetailPage.tsx

Notes

  • npm run lint in control-plane/web/client is still baseline-red on unrelated existing files across the repo; this PR does not add new full-repo lint failures.

Closes #105

@CommitHu502Craft CommitHu502Craft requested review from a team and AbirAbbas as code owners March 29, 2026 05:56
@AbirAbbas
Copy link
Copy Markdown
Contributor

merge conflicts

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.

[Web UI] Replace any types in ExecutionForm.tsx

2 participants