fix: close remaining d365fo-cli skill audit items (#10, #12)

[dynamics365ninja]

Closes the last two open items from the PR #127 audit. Nothing from that review is left after this.

#10 — non-ASCII in Install-D365FoCopilotSkills.ps1

I filed this as cosmetic. It isn't: the em dashes sit in .PARAMETER and .EXAMPLE blocks, and the file has no BOM, so Windows PowerShell 5.1 reads it as ANSI and Get-Help renders them broken.

contains — or will contain — the .github/ directory

Em dashes and box-drawing characters are now ASCII. A BOM would have been the other fix, but not for these two scripts — emit-skills.ps1 carries a #!/usr/bin/env pwsh shebang that a BOM breaks, and it's worth more to keep both scripts encoded the same way than to keep the typography.

While in there: the migration notice suggested Remove-Item -Recurse on a file, and printed hints for paths that don't exist. Each hint now appears only when its target is actually present, with -Recurse only on the directory.

#12 — skill activation is a judgement call, not a glob

The skill replaced 19 .instructions.md files that applied deterministically through applyTo. A skill exposes only name + description, and the agent decides relevance for itself. That's exactly what makes it cheap — but it's a real behaviour change and it was nowhere in the docs.

docs/SETUP.md now covers it at the point users hit it: how to force activation (use the d365fo-cli skill, or naming a references/ file), where each IDE shows which skills were applied, and that skills/copilot/*.instructions.md is still emitted for anyone who wants deterministic scoping back.

That last point contradicted the installer, which flatly called the legacy files safe to delete — it now says the skill supersedes and doesn't conflict with them, and points at SETUP.md.

Verified on Windows PowerShell 5.1

• Get-Help renders clean ASCII.
• Installer deploys SKILL.md + 19 references, prunes a planted stale one, and the legacy notice only offers commands for paths that exist.
• Emitters remain drift-free (git diff clean after emit-skills.py).

No code changes — scripts and docs only.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FfAAnxQDnPHRB1ne5H724n