feat(workflow-executor): include mcpServerId + mcpServerName in MCP step logs (PRD-418)

[christophebrun-forest]

What

Adds two fields to every MCP step execution log line so operators can unambiguously identify which MCP server a log entry refers to when several are configured:

• mcpServerId — DB id of the targeted MCP server (canonical).
• mcpServerName — human-readable name (the Record key from getMcpServerConfigs()).

How

• BaseStepExecutor — new getExtraLogContext() hook (default {}) merged into logCtx, so all step log lines (started/completed, errors, timeouts) pick up extra fields. Base class stays free of step-specific knowledge.
• McpStepExecutor — overrides getExtraLogContext() to return { mcpServerId, mcpServerName }.
• RemoteToolFetcher.fetch() — now resolves serverName from the scoped config Record key and returns { tools, serverName }; diagnostic log lines (not advertised…, failed to load tools) also carry mcpServerName.
• StepExecutorFactory — forwards serverName to the McpStepExecutor constructor.

mcpServerId is canonical; mcpServerName is the human-readable label (not guaranteed unique at the DB level — a user MCP can collide with a Forest connector). Logging both is what makes the entry unambiguous.

Tests

• remote-tool-fetcher: new { tools, serverName } shape, serverName resolution, mcpServerName in diagnostic logs.
• mcp-step-executor: Step execution started/completed lines include mcpServerId + mcpServerName; undefined when unresolved.
• runner: end-to-end forwarding of serverName to the executor.

Full suite: 836/836 green, lint clean, build OK.

fixes PRD-418

🤖 Generated with Claude Code

Note

Include mcpServerId and mcpServerName in MCP step executor log lines

• RemoteToolFetcher.fetch() now returns FetchRemoteToolsResult ({ tools, mcpServerName? }) instead of RemoteTool[], resolving mcpServerName from the scoped config record key.
• BaseStepExecutor adds a protected getExtraLogContext() hook so subclasses can inject type-specific fields into every log line.
• McpStepExecutor overrides getExtraLogContext() to include mcpServerId and mcpServerName, and StepExecutorFactory passes the resolved name through on construction.
• Behavioral Change: callers of fetchRemoteTools must now destructure { tools, mcpServerName } from the result instead of using it directly as an array.

^{Macroscope summarized 98bce7b.}

[linear]

PRD-418

[qltysh]

4 new issues

Tool	Category	Rule	Count
qlty	Structure	Function with many parameters (count = 5): create	3
qlty	Structure	Function with many returns (count = 8): create	1

[qltysh]

Coverage Impact

Unable to calculate total coverage change because base branch coverage was not found.

Modified Files with Diff Coverage (4)

Rating	File	% Diff	Uncovered Line #s
	packages/workflow-executor/src/executors/step-executor-factory.ts	100.0%
	packages/workflow-executor/src/remote-tool-fetcher.ts	100.0%
	packages/workflow-executor/src/executors/base-step-executor.ts	100.0%
	packages/workflow-executor/src/executors/mcp-step-executor.ts	100.0%
	Total	100.0%

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

[matthv]

LGTM