fix: YAML parse errors silently discard pnpm workspace and produce missing-catalog warnings #342

Syncpack version

15.3.3

Operating system(s)

Mac

Steps to reproduce

A leading tab on a comment in pnpm-workspace.yaml causes the workspace file to be silently discarded, even though the catalog definition is still present.

This reproduction creates a temporary directory and writes an actual tab before the comment:

repro_dir=$(mktemp -d)
cd "$repro_dir"

cat > package.json <<'JSON'
{
  "name": "repro",
  "version": "1.0.0",
  "private": true,
  "dependencies": {
    "webpack": "catalog:tooling"
  }
}
JSON

cat > .syncpackrc <<'JSON'
{
  "versionGroups": [
    {
      "dependencies": ["webpack"],
      "policy": "catalog"
    }
  ]
}
JSON

printf 'catalogs:\n\t# Tooling dependencies\n    tooling:\n        webpack: 5.97.x\npackages:\n    - packages/*\n' > pnpm-workspace.yaml

npx --yes syncpack@15.3.3 lint
echo "Exit status: $?"

# Control: change only the comment's indentation from one tab to four spaces.
printf 'catalogs:\n    # Tooling dependencies\n    tooling:\n        webpack: 5.97.x\npackages:\n    - packages/*\n' > pnpm-workspace.yaml

npx --yes syncpack@15.3.3 lint
echo "Exit status: $?"

Expected behavior

Comment indentation should not produce a misleading missing-catalog warning. If the YAML parser rejects the file, Syncpack should report the parsing error with the filename and exit unsuccessfully, instead of proceeding as though the workspace file were absent.

Actual behavior

With the tab-indented comment:

• The workspace YAML is not loaded, so its catalog definitions and workspace package patterns are unavailable.
• The webpack consumer is reported as DependsOnMissingCatalogDefinition, although catalogs.tooling.webpack is defined in the file.
• No YAML parsing error is reported, and lint exits with status 0.

Replacing the tab with four spaces loads the workspace and resolves the catalog successfully.

Code of Conduct

• I agree to follow the Code of Conduct.