microsoft / microsoft/Dataverse-skills

Refactor dv-connect MCP config per-tool branches into ### subsections

Open Beginner friendly
#87 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
No language data
Stars
226
Forks
61
Avg merge
3d 9h
Merged PRs (30d)
7

Description

## Problem

The `dv-connect` MCP configuration reference ([`references/mcp-configuration.md`](https://github.com/microsoft/Dataverse-skills/blob/main/.github/plugins/dataverse/skills/dv-connect/references/mcp-configuration.md)) marks per-tool branch boundaries with `**If TOOL_TYPE is \`x\`:**` (bold) in sections 0, 1, 2, 5, and 8.

This is the **same markdown construct** used for *within-block* step labels in those sections — e.g. `**Generate a unique server name**`, `**Update the configuration file:**`, `**Important notes:**`. So branch boundaries are not structurally distinct from the sub-steps inside a branch.

An agent parsing the file has no reliable signal for where one tool's instructions end and the next tool's begin, which risks instruction bleed across tools (e.g. applying a Cursor step during a Codex flow).

## Proposed fix

Promote each per-tool branch to a real `###` subsection so the boundary sits structurally above the `**bold**` sub-labels:

```markdown
## 5. Register the MCP server

Execute only the subsection matching your `TOOL_TYPE`; skip the others.

### 5a. Copilot
...
### 5b. Claude
...
### 5c. Cursor
...
### 5d. Codex
...
```

- Apply uniformly to every per-tool conditional section (1, 2, 5, 8).
- Add an explicit "execute only the subsection matching your `TOOL_TYPE`" routing line at the top of each.
- This is a Level-3 reference file (no token budget), so the added headings are free.

## Context

Surfaced during PR #84 review. Deferred to its own issue to keep that PR scoped to Codex enablement; this refactor touches all four tool flows and is best reviewed independently.

Contributor guide

Open the contributing guide

Research direction

Start with .github/plugins/dataverse/skills/dv-connect/references/mcp-configuration.md and inspect sections 1, 2, 5, and 8, including their per-tool branches. Promote each branch to a ### subsection and add the specified routing line at the top of each conditional section. Done means all four tool flows have distinct subsection boundaries and consistent routing instructions.

Written by the indexing model from the issue text.

Assessment

Tech stack
markdown
Domain
documentation
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
78/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.