Problem
.pre-commit-config.yaml defines id: sync-manifest twice — line 125 (no file filter, runs every commit) and line 226 (filtered to manifest.toml). Causes redundant/confusing double execution.
Recommendation
Keep one definition (prefer the manifest.toml-filtered one), delete the other. Verify with pre-commit run sync-manifest --all-files.
Context
Found during the state-of-the-repo review of #670 (epic #625); flagged independently by two reviewers.
Refs: #625
Problem
.pre-commit-config.yamldefinesid: sync-manifesttwice — line 125 (no file filter, runs every commit) and line 226 (filtered tomanifest.toml). Causes redundant/confusing double execution.Recommendation
Keep one definition (prefer the
manifest.toml-filtered one), delete the other. Verify withpre-commit run sync-manifest --all-files.Context
Found during the state-of-the-repo review of #670 (epic #625); flagged independently by two reviewers.
Refs: #625