Skip to content

Conversation

@dr1rrb
Copy link
Member

@dr1rrb dr1rrb commented Oct 24, 2025

linked to https://github.com/unoplatform/uno-private/issues/1480

✨ Feature

Update texts to latest spec

What is the current behavior? πŸ€”

Temp dev texts

What is the new behavior? πŸš€

Follow the spec

PR Checklist βœ…

@Copilot Copilot AI review requested due to automatic review settings October 24, 2025 15:51
@dr1rrb dr1rrb changed the title Dev/dr/udei texts UDEI texts Oct 24, 2025
Copy link
Contributor

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 refactors the UDEI (Uno Development Environment Indicator) messaging system to follow a more structured specification. The changes consolidate message creation through well-known static properties and methods, improving maintainability and consistency across the codebase.

Key Changes

  • Removed extension methods in favor of static factory methods for UDEI messages
  • Split large message definition file into multiple partial class files for better organization
  • Updated all message references to use the new centralized factory methods

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
UnoDevelopmentEnvironmentIndicatorExtensions.cs Removed extension methods that created UDEI messages inline
IdeCommandHandler.cs Added case-insensitive command name matching
EntryPoint.cs Updated to use new static factory methods for UDEI messages
UpdateFileIdeMessage.cs Removed unused message type
DevelopmentEnvironmentStatusIdeMessage.cs Moved to UDEI subdirectory with partial class declaration
DevelopmentEnvironmentStatusIdeMessage.WellKnown.cs Added static factory methods and properties for common UDEI messages
DevelopmentEnvironmentStatus.cs Extracted enum to separate file in UDEI subdirectory
DevelopmentEnvironmentComponent.cs Extracted record to separate file in UDEI subdirectory
DevelopmentEnvironmentComponent.WellKnown.cs Moved well-known components to separate partial class file
DevelopmentEnvironmentStatusIdeMessage.cs (old location) Removed original file after splitting into multiple files
UnoDevEnvironmentService.cs Updated to use new static factory methods for UDEI messages

@Copilot Copilot AI review requested due to automatic review settings October 24, 2025 17:05
Copy link
Contributor

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

Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.

@unodevops
Copy link
Contributor

πŸ€– Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21675/docs/index.html

@Copilot Copilot AI review requested due to automatic review settings October 24, 2025 18:51
Copy link
Contributor

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

Copilot reviewed 13 out of 13 changed files in this pull request and generated no new comments.

@unodevops
Copy link
Contributor

⚠️⚠️ The build 179632 has failed on Uno.UI - docs.

@unodevops
Copy link
Contributor

⚠️⚠️ The build 179633 has failed on Uno.UI - CI.

@Copilot Copilot AI review requested due to automatic review settings October 24, 2025 20:42
@github-actions github-actions bot added the area/build Categorizes an issue or PR as relevant to build infrastructure label Oct 24, 2025
Copy link
Contributor

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

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

Comments suppressed due to low confidence (1)

src/Uno.UI.RemoteControl.Messaging/IDEChannel/UDEI/DevelopmentEnvironmentComponent.WellKnown.cs:1

  • Changed 'uno' to 'Uno' for proper capitalization of the platform name.
ο»Ώnamespace Uno.UI.RemoteControl.Messaging.IdeChannel;

@unodevops
Copy link
Contributor

⚠️⚠️ The build 179634 has failed on Uno.UI - docs.

@dr1rrb dr1rrb enabled auto-merge October 24, 2025 21:05
@unodevops
Copy link
Contributor

⚠️⚠️ The build 179635 has failed on Uno.UI - CI.

@unodevops
Copy link
Contributor

πŸ€– Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21675/docs/index.html

@Copilot Copilot AI review requested due to automatic review settings October 25, 2025 01:40
Copy link
Contributor

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

Copilot reviewed 17 out of 17 changed files in this pull request and generated no new comments.

@unodevops
Copy link
Contributor

πŸ€– Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-21675/docs/index.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build Categorizes an issue or PR as relevant to build infrastructure kind/documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants