fix(plugin-client): URL encode parameters in plugin client

[BrianNewsom]

URL-encode path parameters when BaseNemoClient resolves endpoint
templates. This prevents spaces, slashes, and reserved characters
such as ? and # from corrupting generated request URLs.

Includes coverage for explicit path parameters and the default
workspace parameter.

Summary by CodeRabbit

• Bug Fixes

  • Improved request handling so path values are safely encoded before being sent, preventing malformed URLs when parameters contain spaces or special characters.
  • Fixed requests that rely on a configured workspace so encoded values are used consistently in the final request URL.

• Tests

  • Added coverage for encoded path parameters in client requests.
  • Added coverage for encoded workspace values in request paths.

[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: 547ffdd2-16e9-4e71-b9d2-4d472a23a609

📥 Commits

Reviewing files that changed from the base of the PR and between ae7e9fb and 0778b12.

📒 Files selected for processing (2)

• packages/nemo_platform_plugin/src/nemo_platform_plugin/client/client.py
• packages/nemo_platform_plugin/tests/client/test_client.py

---

📝 Walkthrough

Walkthrough

Client's _resolve_path now percent-encodes resolved path parameter values using urllib.parse.quote before formatting the request path template. New tests verify that path parameters and workspace values are properly encoded in outgoing HTTP request URLs.

Changes

Path Parameter Encoding

Layer / File(s)	Summary
Encode path params before URL formatting packages/nemo_platform_plugin/src/nemo_platform_plugin/client/client.py	Imports quote and encodes each resolved path parameter value before substituting into path_template in _resolve_path.
Encoding tests packages/nemo_platform_plugin/tests/client/test_client.py	Adds tests confirming percent-encoded path parameters and default workspace values appear in mocked outgoing request URLs.

🚥 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 summarizes the main change: URL-encoding parameters in the plugin client.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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 docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ase-90-plugin-client-path-encoding/brnewsom

---

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

[github-actions]

Suite	Lines Covered	Line Rate	Branch Rate
Unit Tests	23351/30531	76.5%	61.3%
Integration Tests	13618/29211	46.6%	19.8%