Skip to content

Commit 30b0d72

Browse files
committed
build(eval-author)!: stop shipping the skills in nemo-platform[all]
The package has no entry points and no importable code, so bundling it into the platform distribution shipped files that nothing can discover. `nemo skills list` reads the `nemo.skills` registry and these skills are not registered there yet, so a customer installing nemo-platform[all] received two SKILL.md files reachable only by knowing a path inside the wheel. Installing them into service images through enabled-plugins had the same problem. Removes the [tool.bundle-package] entry, which is what generated the nemo-eval-author-plugin extra along with its membership in the plugins and all extras, and drops the package from enabled-plugins. Regenerating cleared every eval-author reference out of the published wrapper. The package stays a uv workspace member, so uv sync --all-packages still installs it for development and the contract test still runs. Bundle it again when the skills register under nemo.skills and a distribution has something to expose. Signed-off-by: Alec Khoury <akhoury@nvidia.com>
1 parent 2d5c272 commit 30b0d72

3 files changed

Lines changed: 5 additions & 27 deletions

File tree

packages/nemo_platform/pyproject.toml

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -291,12 +291,6 @@ nemo-deployments-plugin = [
291291
"pyyaml>=6.0",
292292
]
293293

294-
# Generated from [tool.bundle-package]; do not edit by hand.
295-
nemo-eval-author-plugin = [
296-
"nemo-insights-plugin",
297-
"pyyaml>=6.0.3",
298-
]
299-
300294
# Generated from [tool.bundle-package]; do not edit by hand.
301295
nemo-evaluator-plugin = [
302296
"cloudpickle>=3.1.1",
@@ -484,7 +478,6 @@ plugins = [
484478
"nemo-platform[nemo-auditor-plugin]",
485479
"nemo-platform[nemo-data-designer-plugin]",
486480
"nemo-platform[nemo-deployments-plugin]",
487-
"nemo-platform[nemo-eval-author-plugin]",
488481
"nemo-platform[nemo-evaluator-plugin]",
489482
"nemo-platform[nemo-experimentalist-plugin]",
490483
"nemo-platform[nemo-guardrails-plugin]",
@@ -723,12 +716,10 @@ nemo-anonymizer-plugin = { source = "../../plugins/nemo-anonymizer/src/nemo_anon
723716
nemo-auditor-plugin = { source = "../../plugins/nemo-auditor/src/nemo_auditor", module = "nemo_auditor", inherit = { "entry-points" = ["nemo.*"] } }
724717
nemo-data-designer-plugin = { source = "../../plugins/nemo-data-designer/src/nemo_data_designer_plugin", module = "nemo_data_designer_plugin", inherit = { "entry-points" = ["nemo.*"] } }
725718
nemo-deployments-plugin = { source = "../../plugins/nemo-deployments/src/nemo_deployments_plugin", module = "nemo_deployments_plugin", inherit = { "entry-points" = ["nemo.*"] } }
726-
# Eval Author ships skills and no code, so it declares no entry points and the inherit
727-
# below is a no-op kept for uniformity. It stays bundled because that is how the skills
728-
# reach a customer through `nemo-platform[all]`. The cycle this entry once collapsed
729-
# (Experimentalist importing EvalAuthor, Eval Author borrowing Experimentalist helpers)
730-
# is gone; only the shared Insights profile contract remains.
731-
nemo-eval-author-plugin = { source = "../../plugins/nemo-eval-author/src/nemo_eval_author_plugin", module = "nemo_eval_author_plugin", inherit = { "entry-points" = ["nemo.*"] } }
719+
# Eval Author is deliberately absent. It ships skills and no code, declares no entry
720+
# points, and has nothing a distribution can expose, so bundling it into
721+
# `nemo-platform[all]` would ship files no mechanism can find. Add it back when the
722+
# skills register under `nemo.skills`.
732723
nemo-evaluator-plugin = { source = "../../plugins/nemo-evaluator/src/nemo_evaluator", module = "nemo_evaluator", inherit = { "entry-points" = ["nemo.*"] } }
733724
nemo-experimentalist-plugin = { source = "../../plugins/nemo-experimentalist/src/nemo_experimentalist_plugin", module = "nemo_experimentalist_plugin", inherit = { "entry-points" = ["nemo.*"] } }
734725
nemo-insights-plugin = { source = "../../plugins/nemo-insights/src/nemo_insights_plugin", module = "nemo_insights_plugin", inherit = { "entry-points" = ["nemo.*"] } }

pyproject.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,6 @@ enabled-plugins = [
187187
"nemo-evaluator-plugin",
188188
"nemo-insights-plugin",
189189
"nemo-experimentalist-plugin ; python_full_version < '3.14'",
190-
"nemo-eval-author-plugin ; python_full_version < '3.14'",
191190
"nemo-guardrails-plugin",
192191
"nemo-auditor-plugin",
193192
"nemo-safe-synthesizer-plugin",

uv.lock

Lines changed: 1 addition & 13 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)