Skip to content

Conversation

@agarny
Copy link
Contributor

@agarny agarny commented Jan 11, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 11, 2026 22:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves type safety in the codebase by exporting TypeScript interfaces and replacing unsafe type casts with proper type assertions. It also includes routine dependency updates and version bumps.

Changes:

  • Exported IUiJsonDiscreteInput and IUiJsonScalarInput interfaces for external use
  • Replaced as any type casts with specific type assertions in the Vue template
  • Updated package versions and dependencies (@types/node, systeminformation)

Reviewed changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/renderer/src/libopencor/locUiJsonApi.ts Made IUiJsonDiscreteInput and IUiJsonScalarInput interfaces public by adding export keywords
src/renderer/src/libopencor/locApi.ts Added exports for the newly public input type interfaces
src/renderer/src/components/views/SimulationExperimentView.vue Removed unsafe as any casts and replaced with specific type assertions for better type safety
src/renderer/package.json Bumped version to 0.20260112.0 and updated @types/node to 24.10.7
src/renderer/bun.lock Updated lock file to reflect @types/node version change
package.json Bumped version to 0.20260112.0, updated @types/node to 24.10.7 and systeminformation to 5.30.3
bun.lock Updated lock file to reflect dependency version changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@agarny agarny merged commit 76c1643 into opencor:main Jan 11, 2026
8 checks passed
@agarny agarny deleted the cleaning-up branch January 11, 2026 23:38
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