Skip to content

fix(web-ui): replace vcStatus any casts with shared VCStatusData#249

Open
anujboddu wants to merge 1 commit intoAgent-Field:mainfrom
anujboddu:fix/233-replace-vcstatus-any
Open

fix(web-ui): replace vcStatus any casts with shared VCStatusData#249
anujboddu wants to merge 1 commit intoAgent-Field:mainfrom
anujboddu:fix/233-replace-vcstatus-any

Conversation

@anujboddu
Copy link

Summary

Fixes #233 by removing vcStatus as any usage across execution components and replacing related TSX as any casts called out in the issue.

What changed

  • Added shared VCStatusData type:
    • control-plane/web/client/src/types/vc.ts
  • Updated execution components/pages/hooks to use VCStatusData instead of inline any-shaped objects:
    • CompactExecutionHeader.tsx
    • ExecutionHeader.tsx
    • EnhancedExecutionHeader.tsx
    • ExecutionIdentityPanel.tsx
    • ExecutionHero.tsx
    • ExecutionDetailPage.tsx
    • EnhancedExecutionDetailPage.tsx
    • RedesignedExecutionDetailPage.tsx
    • useVCVerification.ts
  • Updated VerifiableCredentialBadge to accept VCStatusData in vcData.
  • Removed remaining TSX as any patterns listed in [Web UI] Replace vcStatus as any pattern across execution components #233:
    • HoverDetailPanel.tsx (typed agent_name/task_name via WorkflowDAGLightweightNode)
    • ExecutionQueue.tsx (typed keyboard/mouse event handlers)
    • EnhancedJsonViewer.tsx (typed conversions instead of as any)
    • EnhancedModal.tsx (typed tab value guard)
    • ResponsiveGrid.tsx (typed forwarded ref)
  • Added optional agent_name and task_name to WorkflowDAGLightweightNode.

Testing

Checklist

  • I updated documentation where applicable.
  • I added or updated tests (or none were needed).
  • I updated CHANGELOG.md (or this change does not warrant a changelog entry).

Screenshots (if UI-related)

N/A (typing/refactor only).

Related issues

Fixes #233

@anujboddu anujboddu requested review from a team and AbirAbbas as code owners March 8, 2026 01:44
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@santoshkumarradha
Copy link
Member

Hey @anujboddu — thanks for the contribution! 🙌

Before we can merge this, could you please sign our Contributor License Agreement (CLA)? It's a quick one-time thing for first-time contributors.

Let us know if you have any questions!

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 vcStatus as any pattern across execution components

3 participants