Skip to content

[CB] Configure AI functions and MCP in the administration part#4305

Open
sergeyteleshev wants to merge 3 commits intodevelfrom
8924-cb-configure-ai-functions-and-mcp-in-the-administration-part
Open

[CB] Configure AI functions and MCP in the administration part#4305
sergeyteleshev wants to merge 3 commits intodevelfrom
8924-cb-configure-ai-functions-and-mcp-in-the-administration-part

Conversation

@sergeyteleshev
Copy link
Copy Markdown
Contributor

closes dbeaver/pro#8924

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Apr 27, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 29 complexity

Metric Results
Complexity 29

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Comment on lines +9 to +18
export function headersToText(headers: Record<string, string> | undefined): string | undefined {
if (!headers || Object.keys(headers).length === 0) {
return undefined;
}
return Object.entries(headers)
.map(([key, value]) => `${key}=${value}`)
.join('\n');
}

export function textToHeaders(text: string | undefined): Record<string, string> | undefined {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I really don’t like the idea of having these methods. Please try to refactor the logic so they are no longer needed, or use a dedicated library (or native JavaScript features) that guarantees correct parsing

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.

2 participants