bug(devcheck): a warning-demoted step prints '✅ finished successfully' in the running log

[cyanheads]

mcp-ts-core version

0.11.3

Runtime

Bun

Runtime version

Bun 1.3.14

Transport

N/A (devcheck)

OS

macOS (Darwin 25.1)

Description

Skills Sync and Skill Versions intentionally demote selected non-zero checks to warnings, but the running log reports success because runCheck() normalizes the exit code before rendering the result.

Reproduction

1. Create Skills Sync or Skill Versions drift.
2. Run its standalone check; it exits non-zero.
3. Run bun run devcheck.

Actual behavior

The running log prints ✅ … finished successfully while the final summary prints a warning.

Expected behavior

The running log and final summary both identify a warning-demoted result as a warning.

Additional context

The ignore controls in devcheck.config.json remain the deliberate-exemption path; this repair only makes an existing demotion legible.

Acceptance criteria

• A result with warning renders a warning marker and message in the running log.
• Clean and non-demoted-failure rendering remains unchanged.
• Skills Sync and Skill Versions retain their current warning-demotion policy.

Scope

• Render warning-demoted devcheck results accurately.

Out of scope

• Changing warning-demotion policy for Skills Sync or Skill Versions.

Touchpoints

• scripts/devcheck.ts, devcheck output tests, changelog entry