feat(sinks): align sinks tools configurations, add e2e tests

[freeznet]

No description provided.

[Copilot]

Pull request overview

This pull request refactors the Pulsar sinks tool to align configurations with the functions tool and adds end-to-end tests. The changes make tenant and namespace optional (defaulting to "public" and "default"), expand the tool's parameter set to support more sink configuration options, and introduce a new buildSinkConfig method that replaces the previous processArguments approach.

Changes:

• Refactored sink configuration building with a new buildSinkConfig method that supports loading configuration from YAML files
• Changed tenant and namespace from required to optional parameters with default values
• Added support for 20+ new sink configuration parameters (cpu, ram, disk, custom-serde-inputs, secrets, etc.)
• Added comprehensive e2e tests for create, update, get, and delete operations

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

File	Description
pkg/mcp/builders/pulsar/sinks_parity_test.go	New unit tests for sink configuration building, defaults, and validation
pkg/mcp/builders/pulsar/sinks.go	Major refactoring of sink configuration logic with new parameters and improved validation
docs/tools/pulsar_admin_sinks.md	Updated documentation to reflect optional tenant/namespace and new parameters
cmd/snmcp-e2e/main.go	Added e2e test workflow for sink lifecycle (create, update, get, delete)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.