fix(labels): gate Studio module on remote feature

[sfmskywalker]

Purpose

Close #1038 by making the Studio Labels module follow the optional backend feature contract.

Scope

• Add the canonical Labels remote-feature metadata.
• Gate the Labels menu with the existing fail-closed helper.
• No Labels UX or server persistence changes.

Changes

• Add [RemoteFeature(RemoteFeatureName)] to the Labels Feature.
• Define RemoteFeatureName as Elsa.Labels.ShellFeatures.Labels.
• Inject IRemoteFeatureProvider into LabelsMenu and gate it with IsEnabledOrDefaultAsync.
• Add menu failure-path, feature-metadata, and widget-initialization coverage.

Verification

• Focused Labels tests: 8 passed.
• Full Elsa.Studio.Administration.Tests: 44 passed.
• git diff --check: clean.
• Commit signature verified locally.

[sfmskywalker]

Round 1 — APPROVE (HIGH)

Tip: d2d1aee9b51691e39d34da92c43e7ef059460884 → main (#1038)

Code Review gate: APPROVE + HIGH
(Formal GitHub event COMMENT — cannot approve an own-authored PR.)

The Labels module now declares the exact Core shell-feature name, the menu uses the existing fail-closed helper, and the tests cover enabled/disabled plus authentication, cancellation, and prerender failures. The existing Administration placement assertion was updated with an enabled provider; widget registration remains covered. Root verification: exact four-file diff reviewed, signed commit verified, worker reported 8/8 focused and 44/44 full Administration tests, and git diff --check is clean.

Merge only after exact-head Copilot is clear, Greptile is 5/5 with zero unresolved threads, and hosted checks are terminal green.

[Copilot]

🟢 Approved

The focused change follows established remote-feature patterns and includes appropriate coverage.

Pull request overview

Gates the Labels Studio module on the optional backend Labels feature.

Changes:

• Adds canonical remote-feature metadata.
• Hides Labels navigation when unavailable.
• Adds metadata, initialization, and failure-path tests.

File summaries

File	Description
Module.cs	Declares the required remote feature.
LabelsMenu.cs	Adds fail-closed feature gating.
LabelsMenuTests.cs	Covers gating and feature initialization.
AdministrationNavigationTests.cs	Updates construction for the new dependency.

Review details

• Files reviewed: 4/4 changed files
• Comments generated: 0
• Review effort level: Balanced (auto)

Note

Copilot is running an experiment and ran this review at Balanced.

---

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

[greptile-apps]

Confidence Score: 5/5

No confirmed change prevents merging, though the focused .NET test run remains unavailable until a .NET SDK is installed.

What we checked:

• T-Rex ran an environment check and confirmed the .NET toolchain is not installed, with dotnet not found during the validation flow.
• T-Rex attempted the prescribed labels-validation tests, but each run failed due to dotnet: command not found, and the environment check reported dotnet --info exiting with 127.
• T-Rex uploaded and referenced the labels validation command script and the after-run logs to support validation review.

Summary

• Summary

• The Labels module now declares its backend feature identifier and exposes Labels navigation only when that feature is available.
• No product issue was confirmed in the reviewed changes.

• T-Rex validation blocked

• The focused Labels restore and test run could not start because no .NET SDK or dotnet executable is installed in the environment.

_{Reviews (1) · Last reviewed commit: "feat(labels): gate navigation by remote ..."}