Skip to content

fix: Resolve linting and type checking errors#117

Merged
soovittt merged 1 commit into
masterfrom
fix/linting-errors
Dec 14, 2025
Merged

fix: Resolve linting and type checking errors#117
soovittt merged 1 commit into
masterfrom
fix/linting-errors

Conversation

@soovittt

Copy link
Copy Markdown
Owner

Summary

Fixes all linting and type checking errors found in CI/CD pipeline.

Changes

  • Remove unused imports and variables across multiple components
  • Fix TypeScript any types with proper interfaces
  • Add missing Asset properties (assetTypeKey, description)
  • Fix ImportMeta type issues with vite/client reference directives
  • Remove unused functions and parameters

Files Changed

  • app/components/EnvironmentEditor.tsx - Removed unused imports/variables
  • app/components/CodeViewTab.tsx - Fixed type issues
  • app/components/CodeValidationPanel.tsx - Added proper types
  • app/components/CodeReviewPanel.tsx - Added proper types
  • app/components/BanditCanvas.tsx - Fixed EnvSpec type
  • app/lib/assetClient.ts - Added missing properties
  • app/app.tsx - Added vite/client reference
  • And more formatting fixes

Testing

  • All linting errors resolved
  • All type checking errors resolved
  • Code formatted with Prettier

- Remove unused imports (useAction, StudioLayout, StudioTopBar, etc.) from EnvironmentEditor
- Remove unused variables (selectedObjectId, sceneError, handleSave function)
- Fix unused parameter by prefixing with underscore (_envId)
- Fix CodeViewTab file_type type instead of using 'as any'
- Format code with prettier
@soovittt
soovittt merged commit 30dd2cd into master Dec 14, 2025
13 checks passed
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.

1 participant