Skip to content

Comments

Improve client initialization with streamlined config and HTTP client support#8

Merged
AzeemMuzammil merged 4 commits intoballerina-platform:mainfrom
AzeemMuzammil:fb-client-info
Jun 24, 2025
Merged

Improve client initialization with streamlined config and HTTP client support#8
AzeemMuzammil merged 4 commits intoballerina-platform:mainfrom
AzeemMuzammil:fb-client-info

Conversation

@AzeemMuzammil
Copy link
Contributor

Purpose

$title.
Fixes: ballerina-platform/ballerina-library#8050

Examples

Checklist

  • Linked to an issue
  • Updated the specification
  • Updated the changelog
  • Added tests
  • Checked native-image compatibility

MohamedSabthar
MohamedSabthar previously approved these changes Jun 24, 2025
Comment on lines +30 to +31
# Whether to enforce strict capabilities compliance.
boolean enforceStrictCapabilities?;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this defined in the spec?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes,

Comment on lines +29 to +37
# Notification methods
public const NOTIFICATION_INITIALIZED = "notifications/initialized";
public const NOTIFICATION_CANCELLED = "notifications/cancelled";
public const NOTIFICATION_PROGRESS = "notifications/progress";
public const NOTIFICATION_RESOURCES_LIST_CHANGED = "notifications/resources/list_changed";
public const NOTIFICATION_RESOURCES_UPDATED = "notifications/resources/updated";
public const NOTIFICATION_PROMPTS_LIST_CHANGED = "notifications/prompts/list_changed";
public const NOTIFICATION_TOOLS_LIST_CHANGED = "notifications/tools/list_changed";
public const NOTIFICATION_MESSAGE = "notifications/message";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can also be an enum, but this is okay too.

Will need docs.

Nuvindu
Nuvindu previously approved these changes Jun 24, 2025
@AzeemMuzammil AzeemMuzammil merged commit 524f594 into ballerina-platform:main Jun 24, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MCP] Streamline client configuration and add HTTP client configuration support

4 participants