Context
Merge commit fe4da192 brought in new files from a parallel development stream:
pkg/repohealth/ownership.go + tests - Repository file ownership auditing
scripts/install-hooks.js - Node.js-based git hook installer
scripts/repo-ownership-check.js - Node.js-based repo ownership checker
These exist in the codebase but are not wired into the local CI pipeline or package.json scripts.
Questions to Answer
- pkg/repohealth/: Does this duplicate existing functionality? Is it worth integrating into
eos debug or eos update --fix?
- scripts/install-hooks.js: Local uses
scripts/prompts-submodule.sh install-hook and scripts/install-git-hooks.sh. Should we consolidate or keep JS version?
- scripts/repo-ownership-check.js: Useful for detecting root-owned files in /opt/eos. Worth adding to
npm run ci:preflight?
Proposed Approach
- Evaluate each for unique value vs existing tooling
- Either integrate fully or remove to avoid dead code
- If integrating JS scripts, add to package.json and CI pipeline
Acceptance Criteria