[maxdubrinsky]

Summary

nemo_platform_plugin.jobs.result_manager imports filesets, but nemo-platform-plugin only bundles it at wheel-build time and never declares it as a dependency. A workspace sync scoped to one package (uv run --package nemo-deployments-plugin ...) therefore has no filesets module, and importing nemo_platform_plugin.discovery fails with ModuleNotFoundError. This declares the dependency.

Changes

• packages/nemo_platform_plugin/pyproject.toml: add filesets (workspace source) to dependencies.
• packages/nemo_platform/pyproject.toml: the generated nemo-platform-plugin extra picks up filesets (output of make vendor, which lint-sdk-vendored enforces).
• uv.lock: the resulting edges, locked with uv 0.9.14 via script/uv-lock.sh.

The path-filtered "Deployments OpenShell backend tests" job is the first CI job to sync a single package since the import landed in #1137, which is why main never surfaced it. It fails today on #1956 for this reason.

The built wheel is unchanged: the bundler rewrites the new requirement into the nemo-platform-plugin[nemo-platform-sdk] extra it already emits, and Requires-Dist metadata is byte-identical before and after.

Type of Change

• Code change (feature, bug fix, or refactor)
• Code change with documentation updates
• Documentation only
• Contributor tooling or automation
• CI, build, or test infrastructure

Quality Gates

• Tests added or updated for changed behavior
• Existing tests cover changed behavior — justification: make test-deployments-openshell is the failing check and now passes (test_openshell_profile_is_discoverable imports nemo_platform_plugin.discovery in the scoped env).
• Tests not applicable — justification:
• Documentation updated for user-visible behavior
• Documentation not applicable — justification: packaging metadata only, no behavior change for users.

Verification

• Pull request title follows the repository's Conventional Commit format
• Every commit includes an appropriate Signed-off-by: trailer
• uv run pre-commit run -a passes, or any blocked checks are identified below
• Targeted tests pass, or tests are marked not applicable above
• No secrets, API keys, or credentials are included

Targeted validation:

• make test-deployments-openshell: 122 passed (was 1 failed, 126 passed on the same job for fix(deployments): dial openshell sandbox services through the gateway by Host header #1956)
• uv build --package nemo-platform-plugin --wheel on this branch and on main: Requires-Dist lines identical
• bash script/uv-lock.sh with uv 0.9.14: lock diff is four lines, uv lock --check clean
• uv run pre-commit run -a: not run locally (full-repo hooks); CI covers it

Summary by CodeRabbit

• Chores
  • Added the filesets dependency and configured it as a workspace source.
  • Updated optional project configuration to make filesets available where supported.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: be220c22-8790-4298-ad97-71057dd9a5db

📥 Commits

Reviewing files that changed from the base of the PR and between 9e2fce5 and bb70b92.

⛔ Files ignored due to path filters (1)

• uv.lock is excluded by !**/*.lock

📒 Files selected for processing (1)

• packages/nemo_platform/pyproject.toml

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.

---

📝 Walkthrough

Walkthrough

The plugin now declares filesets as a runtime dependency, resolves it from the workspace, and includes it in the bundled dependency group.

Changes

Filesets dependency registration

Layer / File(s)	Summary
Register filesets dependency packages/nemo_platform_plugin/pyproject.toml, packages/nemo_platform/pyproject.toml	Adds filesets to runtime dependencies, configures workspace resolution, and includes it in the nemo-platform-plugin bundled dependency group.

Suggested reviewers: mckornfield

Priority: ➖ Normal

Change: Bug fix

Merge Risk: ⚪ Minimal · up to bb70b

Filesets is available in workspace installs and bundled wheel builds without a concrete merge-blocking issue.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: declaring the filesets dependency required by nemo-platform-plugin imports.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch platform-plugin-declare-filesets/mdubrinsky

---

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

[github-actions]

Suite	Lines Covered	Line Rate	Branch Rate
Unit Tests	44179/56157	78.7%	62.2%
Integration Tests	27517/53426	51.5%	22.7%

[github-actions]

This PR has had no activity for 30 days. It will be closed after another 7 days without activity. Please comment or push an update if you plan to continue. Maintainers can add keep-open for work that is intentionally paused or waiting on review.

[github-actions]

Closing this PR after 7 additional days without activity. Please reopen it when you're ready to continue.