Consolidated changes and features that were developed during xmas time #1678

Oksamies · 2025-12-22T15:22:51Z

No description provided.

coderabbitai · 2025-12-22T15:23:01Z

Important

Review skipped

Too many files!

150 files out of 300 files are above the max files limit of 150.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

The Footer component in the cyberstorm-remix application has been updated to reflect a new copyright year. The copyright text has been modified from "© 2025 Thunderstore and contributors" to "© 2026 Thunderstore and contributors". No structural changes, logic modifications, or public entity alterations are present in this update.

Pre-merge checks and finishing touches

❌ Failed checks (2 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title is vague and generic, using non-descriptive terms like 'consolidated changes and features' that don't convey what actually changed in the pull request.	Make the title more specific. For example: 'Update copyright year to 2026 in footer' would clearly describe the actual change in this PR.
Description check	❓ Inconclusive	No pull request description was provided by the author.	Add a description explaining what was developed during xmas time and why these changes are being consolidated in this PR.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Oksamies · 2025-12-22T15:23:07Z

Consolidated changes and features that were developed during xmas time #1678 : 4 dependent PRs (#1677 , #1679 , #1680 and 1 other) 👈 (View in Graphite)
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

- Migrate `cyberstorm`, `cyberstorm-theme`, and `ts-uploader` to use Vite for building. - Update root `package.json` to exclude Vite-built packages from Preconstruct and add explicit build steps. - Switch `moduleResolution` to `bundler` across all packages. - Add root `tsconfig.json` with project references. - Update `types` fields in `package.json` for various packages. - Add `@babel/runtime` dependency to multiple packages.

packages/cyberstorm/vite.config.ts

+
+import pkg from "./package.json";
+
+const { dependencies, peerDependencies } = pkg as any;


- Changed frontend and backend URLs in README to use thunderstore.localhost - Updated entrypoint script to map thunderstore.localhost to backend nginx IP - Modified docker-compose to sync new-thunderstore-localhost.conf - Added new-thunderstore-localhost.conf for Nginx configuration

- Added testing instructions to README.md for Docker usage. - Updated package.json to include new test and coverage scripts for containerized testing. - Modified test files to utilize environment variables for API host configuration. - Introduced vite-env.d.ts to define environment variables for TypeScript. - Created run_test_container.mjs to manage test execution in Docker. - Updated Dockerfile.test and docker-compose.yml for test service configuration. - Added entrypoint script for test container setup. - Refactored CI scripts to use Docker for backend services.

- expose a clearInvalidSession helper from ts-api-react that handles storage, cookies, and stale flags with proper error reporting - update Dapper instantiations (root app, singleton, client loaders) to rely on the shared cleanup hook instead of duplicating logic

- add interaction-aware helpers to `useStrongForm` so inputs only go invalid after user focus/blur or submit, and expose ready-to-spread props for focus/blur/csModifiers/aria - refactor create-team, add-member, and add-service-account forms to use the new helper props instead of hand-rolled wiring - teach the theme’s TextInput component about invalid state colors so `csModifiers` visually reflect errors

Backend requires owner-level permissions so no point waste other member's time by letting them fiddle with the form. I considered creating entirely separate component for viewing the data, which would save us from having to set up strongForm for no reason, but given there's no reusable components, it would repeat a lot of markup and lead to poor maintainability.

Change the on-hover cursor when to indicate the input is disabled.

Remove pointer-events: none rule from disabled buttons so they show the default on-hover cursor for them.

…ed fields

- Added URL validation to the ProfileForm and ServiceAccounts forms. - Implemented isValueEmpty utility function to check for empty values. - Created validation logic for required and URL fields in StrongForm. - Added unit tests for StrongForm validation and utility functions. - Refactored useStrongForm to utilize new validation logic. - Updated ProfileForm to include donation link validation. - Modified ServiceAccounts form to handle submission via form element.

…ks, improving error handling and response validation

apps/cyberstorm-remix/app/p/tabs/Wiki/WikiContent.tsx

 import { Await } from "react-router";
+import { Markdown } from "~/commonComponents/Markdown/Markdown";
+
+import { Heading, NewButton, NewIcon, RelativeTime } from "@thunderstore/cyberstorm";


apps/cyberstorm-remix/app/p/tabs/Wiki/WikiPage.tsx

 } from "cyberstorm/security/publicEnvVariables";
-import { WikiContent } from "./WikiContent";
+import { Suspense } from "react";
+import { Await, useParams, type LoaderFunctionArgs } from "react-router";


…_configuration_for_thunderstore.localhost_integration Update README and configuration for thunderstore.localhost integration

Enhance testing setup with Docker integration and Vitest configuration

…nvalid_session_cleanup_and_wire_dapper_to_it Add centralized invalid session cleanup and wire Dapper to it

…m_validation_ux_and_text_input_styling Improve StrongForm validation UX and text input styling

…_enable_package_version_dependencies_and_source_checks_improving_error_handling_and_response_validation Refactor tests to enable package version dependencies and source checks, improving error handling and response validation

… and related packages - Changed imports in various files to use type-only imports for better clarity and performance. - Updated package.json files to point to the correct type definitions in the dist folder. - Added TypeScript declaration options in tsconfig.json for better type support. - Refactored useDnDFileInput to use type imports for DragEvent. - Adjusted multiple API endpoints in thunderstore-api to improve type safety and clarity.

…mports_to_use_type-only_imports_in_thunderstore-api_and_related_packages refactor: update imports to use type-only imports in thunderstore-api and related packages

If the css file is not imported as the first thing, the css layers will bork

…ot.tsx_import_sorting_making_styles_break Fix issue with root.tsx import sorting making styles break

Oksamies added 2 commits December 22, 2025 17:39

Consolidated changes and features that were developed during xmas time

9c1b704

github-advanced-security bot found potential problems Dec 22, 2025

View reviewed changes

packages/cyberstorm/vite.config.ts

import pkg from "./package.json";

const { dependencies, peerDependencies } = pkg as any;

Check failure

Code scanning / ESLint

Disallow the `any` type

Unexpected any. Specify a different type.

Oksamies and others added 11 commits December 22, 2025 17:40

Run pre-commit on all files

0dc2cd2

Disable hover highlight styles from disabled text inputs

50c65d2

Change the on-hover cursor when to indicate the input is disabled.

Disable hover highlight styles from disabled buttons

a4f75be

Remove pointer-events: none rule from disabled buttons so they show the default on-hover cursor for them.

Add RequiredIndicator component and update forms to use it for requir…

2547e81

…ed fields

Refactor tests to enable package version dependencies and source chec…

2cdada4

…ks, improving error handling and response validation

Oksamies force-pushed the xmas-features branch from 9badef7 to 0dc2cd2 Compare December 22, 2025 15:46

github-advanced-security bot found potential problems Dec 22, 2025

View reviewed changes

Oksamies and others added 6 commits December 22, 2025 17:48

Merge pull request #1677 from thunderstore-io/12-20-update_readme_and…

bc69ac6

…_configuration_for_thunderstore.localhost_integration Update README and configuration for thunderstore.localhost integration

Merge pull request #1675 from thunderstore-io/testing_setup

360a8a3

Enhance testing setup with Docker integration and Vitest configuration

Merge pull request #1660 from thunderstore-io/12-09-add_centralized_i…

4e40255

…nvalid_session_cleanup_and_wire_dapper_to_it Add centralized invalid session cleanup and wire Dapper to it

Merge pull request #1661 from thunderstore-io/12-09-improve_strongfor…

f05203b

…m_validation_ux_and_text_input_styling Improve StrongForm validation UX and text input styling

Merge pull request #1680 from thunderstore-io/12-22-refactor_update_i…

2ab2d80

…mports_to_use_type-only_imports_in_thunderstore-api_and_related_packages refactor: update imports to use type-only imports in thunderstore-api and related packages

Oksamies mentioned this pull request Dec 22, 2025

refactor: update imports to use type-only imports in thunderstore-api and related packages #1680

Merged

Fix issue with root.tsx import sorting making styles break

d38461e

If the css file is not imported as the first thing, the css layers will bork

Oksamies mentioned this pull request Dec 22, 2025

Fix issue with root.tsx import sorting making styles break #1681

Merged

Merge pull request #1681 from thunderstore-io/12-22-fix_issue_with_ro…

10e0a7f

…ot.tsx_import_sorting_making_styles_break Fix issue with root.tsx import sorting making styles break

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Consolidated changes and features that were developed during xmas time #1678

Consolidated changes and features that were developed during xmas time #1678

Uh oh!

Oksamies commented Dec 22, 2025

Uh oh!

coderabbitai bot commented Dec 22, 2025 •

edited

Loading

Review skipped

Uh oh!

Oksamies commented Dec 22, 2025 •

edited

Loading

Uh oh!

Check failure

Check failure

Check failure

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants


		import pkg from "./package.json";

		const { dependencies, peerDependencies } = pkg as any;

Consolidated changes and features that were developed during xmas time #1678

Are you sure you want to change the base?

Consolidated changes and features that were developed during xmas time #1678

Uh oh!

Conversation

Oksamies commented Dec 22, 2025

Uh oh!

coderabbitai bot commented Dec 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Walkthrough

Pre-merge checks and finishing touches

Uh oh!

Oksamies commented Dec 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Check failure

Check failure

Check failure

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

coderabbitai bot commented Dec 22, 2025 •

edited

Loading

Oksamies commented Dec 22, 2025 •

edited

Loading