Skip to content

linter and formatter - #36

Merged
ericgozzi merged 1 commit into
mainfrom
linter
Jul 3, 2026
Merged

ericgozzi merged 1 commit into
mainfrom
linter

Conversation

@ericgozzi

Copy link
Copy Markdown
Collaborator

This pull request introduces a set of improvements to the frontend/compas_threejs project, focusing on code quality, maintainability, and type safety. The changes include the addition of Prettier and ESLint for consistent code formatting and linting, refactoring of TypeScript types for stricter type checking, and cleanup of unused code. The update also features formatting improvements in Vue components for better readability.

Tooling and Code Quality Enhancements:

  • Added Prettier and ESLint configuration files (.prettierrc, .prettierignore, eslint.config.js) and updated package.json scripts and dependencies to support linting and formatting workflows. [1] [2] [3] [4] [5]

TypeScript Type Improvements:

  • Refactored function parameters throughout the codebase to use unknown or Record<string, unknown> instead of any for improved type safety, especially in communication and UI management modules (communication.ts, objectInfo.ts, sidebarStore.ts, websocket.ts). [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

Code Cleanup and Unused Code Removal:

  • Removed unused imports and variables, such as unpackMessage and SCENE_GEOMETRIES, to streamline the codebase. [1] [2] [3]

Formatting and Readability Improvements:

  • Reformatted Vue component templates (ObjectInfo.vue, Openbar.vue) for more concise markup and improved maintainability, and made minor CSS formatting adjustments. [1] [2] [3] [4] [5] [6] [7] [8]

Minor Logic Adjustments:

  • Added comments and minor logic handling for future extensibility in WebSocket message processing. [1] [2]

These changes collectively enhance the project's code quality, enforce consistent coding standards, and lay the groundwork for safer and more maintainable TypeScript development.

Signed-off-by: Eric <ericgozzi@cosmicgarage.ch>
@ericgozzi
ericgozzi merged commit 5ba0196 into main Jul 3, 2026
1 check passed
@ericgozzi
ericgozzi deleted the linter branch July 3, 2026 07:15
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